-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
3.0 FormHelper submit #2943
3.0 FormHelper submit #2943
Conversation
They have been systematically removed from the helper. They have been replaced everywhere with templates.
Templates should be used instead.
$options['name'] . '_x', $options['name'] . '_y' | ||
); | ||
$unlockFields = [ | ||
$options['name'] . '_x', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you remember what wa the purpose of this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I do! Image submits post 2 values instead of 1. They submit the x & y co-ordinates the user clicked on the image. When we're dealing with secured forms we need to hash the fields we'll get in the end.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Interesting, did not know that
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shows how long it's been since using image submits has gone out of fashion 😛
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Whatever, image submits 4 lyfe!
Part of #2267. When the tests for this method were re-enabled the method was not cleaned up and made consistent with other parts of the helper. These changes help address that and make the submit() method use templates like other parts of FormHelper.