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

bug fix DefaultFormRenderer #220

Merged
merged 1 commit into from Apr 16, 2019
Merged

bug fix DefaultFormRenderer #220

merged 1 commit into from Apr 16, 2019

Conversation

@rydercz
Copy link
Contributor

rydercz commented Apr 16, 2019

  • bug fix
  • BC break? no
  • doc PR:

After update on new version with strict_type=1, DefaultFormRenderer started throw TypeError: preg_split() expects parameter 2 to be string, null given for forms with method GET if action URL does not contain query string parameters.

…to be string, null given" for GET form if action does not contain query parameters
@dg

This comment has been minimized.

Copy link
Member

dg commented Apr 16, 2019

Thanks!

@dg dg merged commit 85467ba into nette:master Apr 16, 2019
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage remained the same at 95.068%
Details
dg added a commit that referenced this pull request Jul 8, 2019
fixes "TypeError: preg_split() expects parameter 2 to be string, null given" for GET form if action does not contain query parameters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.