8000 Fix URL validator failure with empty string by bocharsky-bw · Pull Request #15898 · symfony/symfony · GitHub
[go: up one dir, main page]

Skip to content

Fix URL validator failure with empty string #15898

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

Closed
wants to merge 1 commit into from

Conversation

bocharsky-bw
Copy link
Contributor
Q A
Bug fix? [yes]
New feature? [no]
BC breaks? [no]
Deprecations? [no]
Tests pass? [yes]
Fixed tickets ~
License MIT
Doc PR ~

Fix validator failure if method __toString of passed object return an empty string.

@stof
Copy link
Member
stof commented Sep 26, 2015

this needs a test to prevent regressions

@fabpot
Copy link
Member
fabpot commented Oct 6, 2015

Closing in favor of #16152 where I added a test

@fabpot fabpot closed this 88A5 Oct 6, 2015
@bocharsky-bw bocharsky-bw deleted the patch-1 branch October 6, 2015 17:27
fabpot added a commit that referenced this pull request Oct 6, 2015
…rsky-bw)

This PR was merged into the 2.3 branch.

Discussion
----------

Fix URL validator failure with empty string

| Q             | A
| ------------- | ---
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | #15898
| License       | MIT
| Doc PR        | ~

Fix validator failure if method `__toString` of passed object return an empty string.

Commits
-------

e0910d9 Fix URL validator failure with empty string
0f61859 [Validator] added a failing test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants
0