contao / docs-archive

Contao Documentation
https://docs.contao.org/
Other
42 stars 76 forks source link

[4.2] add note about max_input_vars #391

Closed fritzmg closed 8 years ago

fritzmg commented 8 years ago

Added note for https://github.com/contao/core/issues/7938. French version is missing though.

lionel-m commented 8 years ago

Thanks ! The French translation:

En fonction de vos extensions installées, il peut être nécessaire d'augmenter la valeur de la variable PHP max_input_vars (par ex. 2000 ou supérieur). Sinon, les longs formulaires du back office ne peuvent pas être soumis entièrement.

lionel-m commented 8 years ago

I think it's better to place this paragraph as a note, because it is not directly related to the Contao core but if an extension displays a too long form. Correct?

So with > **Info** at the beginning of your paragraph. Example

fritzmg commented 8 years ago

I think it's better to place this paragraph as a note, because it is not directly related to the Contao core but if an extension displays a too long form. Correct?

So with > Info at the beginning of your paragraph. Example

Agreed. I'll update soon.

However, I would like to point out that you can run into this problem without extensions as well - it's just very unlikely (you would need to create a lot of news archives, FAQ archives, calendars etc. for that, around 255 entries in total I think).

fritzmg commented 8 years ago

I used > **Note** now. Or is > **Info** more fitting? I don't know ;)

lionel-m commented 8 years ago

I used > Note now. Or is > Info more fitting? I don't know ;)

As you wish :)

aschempp commented 8 years ago

Thanks for the PR! It is not necessary to create a pull request for each version of the manual. We simply merge changes from 3.5 into later releases. So can you please update issue #390 to contain all changes mentioned here? Then I can merge that into all manuals.

fritzmg commented 8 years ago

done, I've updated #390 I guess I can close this PR and delete the branch?

aschempp commented 8 years ago

Yes, but it looks like you forgot the latest commend from @lionel-m :)

fritzmg commented 8 years ago

right, done :)