JsFormValidatorBundle icon indicating copy to clipboard operation
JsFormValidatorBundle copied to clipboard

Preserve original submit button

Open richard-ejem opened this issue 8 years ago • 2 comments

Currently, the button clicked to submit the form is not preserved and therefore not included in the request sent to server, because this package calls event.preventDefault, then, if the form is valid, sends form.submit(). This PR fixes the problem.

richard-ejem avatar Aug 02 '17 13:08 richard-ejem

Pleas look on the tests they are falling on you realization.

66Ton99 avatar Sep 25 '20 15:09 66Ton99

Pleas look on the tests they are falling on you realization.

Thanks for the feedback, but honestly I did this 3 years ago, now I don't even do PHP much anymore and this PR is completely out of my interest now, so I'm not willing to spend any more time with it. I don't even know if the issue is still present in latest version. If you look at the test results, it seems the test setup is completely broken and it is obviously nothing that could be caused by the PR. It is very likely that if the PR is still valid, just rebasing on current master where tests work will fix it.

richard-ejem avatar Sep 25 '20 18:09 richard-ejem