Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP][Reference][Form Types] Add missing options for "form" type #3449

Merged
merged 3 commits into from
Feb 1, 2014
Merged

[WIP][Reference][Form Types] Add missing options for "form" type #3449

merged 3 commits into from
Feb 1, 2014

Conversation

bicpi
Copy link
Contributor

@bicpi bicpi commented Jan 9, 2014

Q A
Doc fix? no
New docs? yes
Applies to 2.3+
Fixed tickets #3410

Do we need to point out, that the pattern attribute is HTML5 only?

@xabbuh
Copy link
Member

xabbuh commented Jan 9, 2014

Do we need to point out, that the pattern attribute is HTML5 only?

I'd add it to the This validation is client-side only. caution.

**type**: ``string`` **default**: ``The uploaded file was too large. Please try to upload a smaller file.``

This is the validation error message that's used if submitted POST form data
exceeds php.ini's ``post_max_size``. The ``{{ max }}`` placeholder can be used
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

don't we use literals for php.ini?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we always use literals for files names, so imo we should use them too for php.ini.

@bicpi
Copy link
Contributor Author

bicpi commented Jan 11, 2014

I've done some improvements, fixes and a rebase.

.. caution:

The ``pattern`` attribute provides client-side validation for convenience
purposes only and cannot replace a reliable server-side validation.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe "[...] and must not be used as a replacement for reliable server-side validation."?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes thx, sounds better. And for the HTML5, I've added this above as it was done for the required option.

@weaverryan
Copy link
Member

@bicpi This is perfect. Can you rebase one more time? We've had a lot of PR's to this page lately :).

Thanks!

@bicpi
Copy link
Contributor Author

bicpi commented Jan 23, 2014

@weaverryan Rebase done.

weaverryan added a commit that referenced this pull request Feb 1, 2014
[WIP][Reference][Form Types] Add missing options for "form" type
@weaverryan weaverryan merged commit d3ee46b into symfony:2.3 Feb 1, 2014
@weaverryan
Copy link
Member

Beautiful - thanks and cheers!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants