Jili voucher hack.REGISTER NOW GET FREE 888 PESOS REWARDS!

Resolved

(@fbgm)

Watch the video to see the problem:
https://www.youtube.com/watch?v=mKJK3P4xuYU

My code:

use Carbon_Fields\Container\Container;
use Carbon_Fields\Field\Field;

function abcdxyz_carbon_fields_setup() {
	Container::make('post_meta', 'Main')
	->show_on_post_type('post')
	->add_fields(array(

		Field::make('complex' , 'aaaaa', 'Aaaaaa')
		->add_fields('bbbb', array(

			Field::make('complex' , 'cccc', 'Ccccc')->set_width(50)
			->add_fields('oooo', array(
				Field::make('text', 'ccc1', 'Cccc1')->set_width(10),
				Field::make('text', 'ccc2', 'Cccc2')->set_width(90),
			)),

			Field::make('complex' , 'dddd', 'Ddddd')->set_width(50)
			->add_fields('oooo', array(
				Field::make('text', 'ddd1', 'Dddd1')->set_width(10),
				Field::make('text', 'ddd2', 'Dddd2')->set_width(90),
			)),

		))

	));

}
add_action('carbon_register_fields', 'abcdxyz_carbon_fields_setup');

https://www.ads-software.com/plugins/carbon-fields/

Viewing 11 replies - 1 through 11 (of 11 total)

Plugin Author htmlBurger
(@htmlburger)

8 years, 5 months ago

Hi George,

Thanks for the thorough report!

The issue has already been fixed in the development version. In case you need it right now, you can pull it from the GitHub repo of the plugin.

In case you need details about how we fixed it, head over to the PR page: https://github.com/htmlburger/carbon-fields/commit/e3c09b5dc99c1df860a9de93766f827dbaae3607

We’ll soon release a new version, containing this fix as well.

Thank you, and good luck with your project.

Thread Starter George Florea Banus
(@fbgm)

8 years, 5 months ago

How do i download the development version?

Is it the master branch?

Plugin Author htmlBurger
(@htmlburger)

8 years, 5 months ago

Yes, it is the master branch.
You can use GitHub to clone and pull it, or if you’re not familiar with Git, you can download it az a ZIP archive.

Thread Starter George Florea Banus
(@fbgm)

8 years, 5 months ago

I installed it but it doesn’t fix the problem.

Plugin Author htmlBurger
(@htmlburger)

8 years, 5 months ago

Thanks, we’ll have a look at the problem ASAP.

Thread Starter George Florea Banus
(@fbgm)

8 years, 5 months ago

Also when the plugin was released i played with the complex fields to see how many can be handled.
Was like this https://i.imgur.com/j9AzSiM.png
And after about 100 it wouldn’t save them anymore, just refresh the page.

Is that a wordpress limitation? Would it work if i’d allocate more resources to php/wordpress?

Plugin Author htmlBurger
(@htmlburger)

8 years, 5 months ago

This is a PHP limitation, actually. You’re probably hitting the max_input_vars limit of PHP, because of too many input fields being submitted in one request.

To clarify, such issues can occur if you have hundreds of variations in a product in WooCommerce, for example.

Fortunately, you can increase this limit in your php.ini: https://php.net/manual/en/info.configuration.php#ini.max-input-vars

Please, let us know if you have further issues.

Plugin Author htmlBurger
(@htmlburger)

8 years, 5 months ago

Hey George,

We were able to reproduce the issue with the complex fields.

We’ll work on fixing the issue the next few days.

Thank you for reporting.

Thread Starter George Florea Banus
(@fbgm)

8 years, 5 months ago

Thank you for the great plugin and amazing support.

Plugin Author htmlBurger
(@htmlburger)

8 years, 5 months ago

Hey George,

The issue has been fixed: https://github.com/htmlburger/carbon-fields/commit/bdfd68795d0b97df87cf13ac7842b842b09675af

Since a new version will not be released right away, feel free to use the latest version from GitHub.

Thank you for reporting the issue, and good luck with your project.

Thread Starter George Florea Banus
(@fbgm)

8 years, 5 months ago

Yes, it’s fine now.

Viewing 11 replies - 1 through 11 (of 11 total)

The topic ‘Problem with nested complex fields’ is closed to new replies.