All theme devs short testing needed

[danyj]

We have a bit of a braking change in theme settings that needs more testing before the merge,
it will affect everyone so lets lend a hand and test properly please.

#3327

[ivandotv]

I can help test it. I have a lot of options both in customizer and on admin post pages.
Please provide me with the test scenario , and what to look for.

[danyj]

thnx bud , here
#3327 (comment)

basically just go to theme settings , save bunch of options/popups/inner multi and so on ,
than set max_input_vars to default ( 1000 ) , see if it saves without errors

[andronic90]

I tested with The Core theme and works fine.

[ivandotv]

I believe that max_input_vars is 1000 by default. ( at least it is in php 7.1 via docker containers )
I had no problems using default unyson plugin.
Should I lower it to 500 and test it then? Is it still a valid test?

[andronic90]

I tested with 50 initial with previous Unyson version and with the branch for this feature.

[danyj]

you guys can lower to anything , we just went for 1000 since it is default on most shared hosting

[danyj]

but it should not be an issue since form data on submit now is 1 where it used to go by number of inputs in the form , for me was gong well over 1000 and was crashing

[danyj]

Can you guys do some more testing please , so far we had 2 almost invisible but very significant bugs , looks like options with quotes/slashes/css and so on are not outputted properly. If we dont do this right and the commit is up many of our sites could brake. Lets help before that happens.