Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork9.7k
[2.5] Test components using their lowest possible deps#12998
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Merged
Uh oh!
There was an error while loading.Please reload this page.
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters
…le validators, legacy validators were created
…rs for the different APIs
…utionContext (Tobion)This PR was merged into the 2.5 branch.Discussion----------[Validator] prevent unnecessary calls inside ExecutionContext| Q | A| ------------- | ---| Bug fix? | no| New feature? | no| BC breaks? | no| Deprecations? | no| Tests pass? | yes| Fixed tickets || License | MITSmall performance improvement by preventing calls to `PropertyPath::append($this->propertyPath, $subPath)` when not needed.Commits-------d6d462a [Validator] do not call getter inside ExecutionContext to prevent unnecessary calls
…gelog (JeroenDeDauw)This PR was submitted for the master branch but it was merged into the 2.5 branch instead (closessymfony#11507).Discussion----------Add point about ConsoleLogger to Console 2.5 changelogCommits-------8744826 Add point about ConsoleLogger to Console 2.5 changelog
* Added some translations in CA* Fixed some translations in ES
…ions (mmoreram)This PR was merged into the 2.4 branch.Discussion----------Issuesymfony#11489 Added some CA and ES translations* Added some translations in CA* Fixed some translations in ESCommits-------67a6b75 Issuesymfony#11489 Added some CA and ES translations
* 2.3: (22 commits) Fix incorrect romanian plural translations fix axes handling in Crawler::filterXPath() fix some docblocks Fixed self-reference in 'service_container' service breaks garbage collection (and clone). [Process] Fix tests when pcntl is not available. [DependencyInjection] Roll back changes made to generated files. [Console] Roll back changes made to fixture files. [Validator] Added more detailed inline documentation [Validator] Removed information from the violation output if the value is an array, object or resource partially reverted previous commit fixed CS properly handle null data when denormalizing [Validator] Renamed valueToString() to formatValue(); added missing formatValue() calls [Validator] Fixed CS [Validator] Fixed date-to-string conversion tests to match ICU 51 [Validator] Added "{{ value }}" parameters where they were missing [Validator] Simplified and explained the LuhnValidator [Validator] Simplified IssnValidator [Validator] Fixed and simplified IsbnValidator [Validator] Simplified IBAN validation algorithm ...Conflicts:src/Symfony/Component/Console/Helper/DescriptorHelper.phpsrc/Symfony/Component/DependencyInjection/Container.phpsrc/Symfony/Component/DomCrawler/Tests/CrawlerTest.phpsrc/Symfony/Component/HttpFoundation/File/UploadedFile.phpsrc/Symfony/Component/HttpKernel/Fragment/FragmentHandler.phpsrc/Symfony/Component/Validator/Constraints/AbstractComparisonValidator.phpsrc/Symfony/Component/Validator/Constraints/CollectionValidator.phpsrc/Symfony/Component/Validator/Tests/Constraints/EqualToValidatorTest.phpsrc/Symfony/Component/Validator/Tests/Constraints/GreaterThanValidatorTest.phpsrc/Symfony/Component/Validator/Tests/Constraints/IdenticalToValidatorTest.phpsrc/Symfony/Component/Validator/Tests/Constraints/LessThanOrEqualValidatorTest.phpsrc/Symfony/Component/Validator/Tests/Constraints/LessThanValidatorTest.phpsrc/Symfony/Component/Validator/Tests/Constraints/NotEqualToValidatorTest.phpsrc/Symfony/Component/Validator/Tests/Constraints/NotIdenticalToValidatorTest.php…alidator::validatePropertyValue()
…s name to validatePropertyValue() (webmozart)This PR was merged into the 2.5 branch.Discussion----------[Validator] Made it possible (again) to pass a class name to validatePropertyValue()| Q | A| ------------- | ---| Bug fix? | yes| New feature? | no| BC breaks? | no| Deprecations? | no| Tests pass? | yes| Fixed tickets |symfony#11139| License | MIT| Doc PR | -In the 2.4 API it was possible to do both:```php$validator->validatePropertyValue($object, 'propertyName', $myValue);$validator->validatePropertyValue('\Vendor\Namespace\ClassName', 'propertyName', $myValue);```In the 2.5 API, the second case was not supported anymore. This is fixed now.Together with the fix comes also a small change (also in the 2.4 API) which I'll demonstrate with a code snippet:```php$metadata->addPropertyConstraint('ClassName', 'propertyName', new Callback( function ($value, $context) { var_dump($context->getRoot()); var_dump($context->getPropertyPath()); }));$validator->validatePropertyValue('ClassName', 'propertyName', 'foobar');```Before this PR, the output would be:```string(9) "ClassName"string(12) "propertyName"```This doesn't make a lot of sense, because usually the following condition holds during validation:```php'' === $context->getPropertyPath() || $value === $propertyAccessor->getValue($context->getRoot(), $context->getPropertyPath())```which obviously cannot work if root is a class name. Thus I changed the root and property path to become:```string(6) "foobar"string(0) ""```With this change, the condition holds also in this case.Commits-------2bf1b37 [Validator] Fixed ExpressionValidator when the validation root is not an objectef6f5f5 [Validator] Fixed: Made it possible (again) to pass a class name to Validator::validatePropertyValue()
… API (webmozart)This PR was squashed before being merged into the 2.5 branch (closessymfony#11485).Discussion----------[Validator] Constraint validators now use the 2.5 API| Q | A| ------------- | ---| Bug fix? | no| New feature? | no| BC breaks? | no| Deprecations? | no| Tests pass? | yes| Fixed tickets | -| License | MIT| Doc PR | -See the comments insymfony#11049 for the origin of this PR.Currently, the 2.5 API needs to use `LegacyExecutionContextFactory` because the constraint validators rely on methods from the old `ExecutionContext` class (like `validate()`, `validateValue()`). Consequently it is impossible to switch to the pure 2.5 API and check whether all calls to deprecated methods were removed from application code. This is fixed now.This PR also introduces a complete test suite to test each constraint validator against all three APIs: 2.4, 2.5-BC and 2.5. Currently, some tests are not executed yet when running the complete test suite is run. I expect this to be fixed soon (ticket:sebastianbergmann/phpunit#529, pr:sebastianbergmann/phpunit#1327).Commits-------295e5bb [Validator] Fixed failing tests3bd6d80 [Validator] CS fixes870a41a [FrameworkBundle] Made ConstraintValidatorFactory aware of the legacy validators7504448 [Validator] Added extensive test coverage for the constraint validators for the different APIs8e461af [Validator] Constraint validators now use the 2.5 API. For incompatible validators, legacy validators were created
This PR was merged into the 2.4 branch.Discussion----------[Validator] Added Swedish translations| Q | A| ------------- | ---| Bug fix? | yes| New feature? | no| BC breaks? | no| Deprecations? | no| Tests pass? | yes| Fixed tickets | part ofsymfony#11489| License | MITAdded some missing Swedish translations.Should I make an additional PR to `master` for the `trans-unit id=78` ([ref](https://github.com/symfony/symfony/blob/master/src/Symfony/Component/Validator/Resources/translations/validators.en.xlf#L302))? Or should I add that in this PR?Commits-------a361f10 [Validator] Added Swedish translations
* 2.4: (24 commits) [Validator] Added Swedish translations Fix incorrect romanian plural translations fix axes handling in Crawler::filterXPath() fix some docblocks Fixed self-reference in 'service_container' service breaks garbage collection (and clone). [Process] Fix tests when pcntl is not available. [DependencyInjection] Roll back changes made to generated files. [Console] Roll back changes made to fixture files. Issuesymfony#11489 Added some CA and ES translations [Validator] Added more detailed inline documentation [Validator] Removed information from the violation output if the value is an array, object or resource partially reverted previous commit fixed CS properly handle null data when denormalizing [Validator] Renamed valueToString() to formatValue(); added missing formatValue() calls [Validator] Fixed CS [Validator] Fixed date-to-string conversion tests to match ICU 51 [Validator] Added "{{ value }}" parameters where they were missing [Validator] Simplified and explained the LuhnValidator [Validator] Simplified IssnValidator ...Conflicts:src/Symfony/Bridge/Propel1/Form/ChoiceList/ModelChoiceList.phpsrc/Symfony/Component/HttpKernel/Debug/TraceableEventDispatcher.phpsrc/Symfony/Component/Security/Core/Encoder/BasePasswordEncoder.phpsrc/Symfony/Component/Validator/Constraints/ChoiceValidator.phpsrc/Symfony/Component/Validator/Constraints/CollectionValidator.phpsrc/Symfony/Component/Validator/Constraints/FileValidator.phpsrc/Symfony/Component/Validator/Constraints/Isbn.phpsrc/Symfony/Component/Validator/Constraints/IsbnValidator.phpsrc/Symfony/Component/Validator/Constraints/LengthValidator.phpsrc/Symfony/Component/Validator/Tests/Constraints/AbstractComparisonValidatorTestCase.phpsrc/Symfony/Component/Validator/Tests/Constraints/BlankValidatorTest.phpsrc/Symfony/Component/Validator/Tests/Constraints/ChoiceValidatorTest.phpsrc/Symfony/Component/Validator/Tests/Constraints/CollectionValidatorTest.phpsrc/Symfony/Component/Validator/Tests/Constraints/DateTimeValidatorTest.phpsrc/Symfony/Component/Validator/Tests/Constraints/DateValidatorTest.phpsrc/Symfony/Component/Validator/Tests/Constraints/EmailValidatorTest.phpsrc/Symfony/Component/Validator/Tests/Constraints/FalseValidatorTest.phpsrc/Symfony/Component/Validator/Tests/Constraints/FileValidatorPathTest.phpsrc/Symfony/Component/Validator/Tests/Constraints/FileValidatorTest.phpsrc/Symfony/Component/Validator/Tests/Constraints/IbanValidatorTest.phpsrc/Symfony/Component/Validator/Tests/Constraints/IpValidatorTest.phpsrc/Symfony/Component/Validator/Tests/Constraints/IsbnValidatorTest.phpsrc/Symfony/Component/Validator/Tests/Constraints/LengthValidatorTest.phpsrc/Symfony/Component/Validator/Tests/Constraints/NullValidatorTest.phpsrc/Symfony/Component/Validator/Tests/Constraints/RegexValidatorTest.phpsrc/Symfony/Component/Validator/Tests/Constraints/TimeValidatorTest.phpsrc/Symfony/Component/Validator/Tests/Constraints/TrueValidatorTest.phpsrc/Symfony/Component/Validator/Tests/Constraints/UrlValidatorTest.php
…on-BC 2.5 Validation API (webmozart)This PR was merged into the 2.5 branch.Discussion----------[Form] Fixed FormValidator compatibility with the non-BC 2.5 Validation API| Q | A| ------------- | ---| Bug fix? | yes| New feature? | no| BC breaks? | no| Deprecations? | no| Tests pass? | yes| Fixed tickets |symfony#11568| License | MIT| Doc PR | -Commits-------6ac130e [Form] Fixed FormValidator compatibility with the non-BC 2.5 Validation API
* 2.3: Fixed relative redirects for ambiguous paths [BrowserKit] Fix browser kit redirect with ports [TwigBridge] [Form] Fixed some extra empty spaces Plural fix removed some .gitattributes that should have been removed a lot time ago [DependencyInjection] fixed missing 'factory-class' attribute in XmlDumper output fixed whitespace in Twig form template built-in server: exit when docroot does not existConflicts:src/Symfony/Bridge/Twig/Resources/views/Form/form_div_layout.html.twig
* 2.4: Fixed relative redirects for ambiguous paths [BrowserKit] Fix browser kit redirect with ports [TwigBridge] [Form] Fixed some extra empty spaces Plural fix removed some .gitattributes that should have been removed a lot time ago [DependencyInjection] fixed missing 'factory-class' attribute in XmlDumper output fixed whitespace in Twig form template built-in server: exit when docroot does not existConflicts:src/Symfony/Bundle/FrameworkBundle/Command/ServerRunCommand.phpsrc/Symfony/Component/DependencyInjection/Tests/Fixtures/xml/services9.xml
…s for the different Validation APIs (webmozart)This PR was merged into the 2.5 branch.Discussion----------[SecurityBundle] Added UserPasswordValidator tests for the different Validation APIs| Q | A| ------------- | ---| Bug fix? | yes| New feature? | no| BC breaks? | no| Deprecations? | no| Tests pass? | yes| Fixed tickets | -| License | MIT| Doc PR | -Commits-------0653426 [SecurityBundle] Added UserPasswordValidator tests for the different Validation APIs
…se the 2.5 Validation API (webmozart)This PR was merged into the 2.5 branch.Discussion----------[DoctrineBridge] Changed UniqueEntityValidator to use the 2.5 Validation API| Q | A| ------------- | ---| Bug fix? | yes| New feature? | no| BC breaks? | no| Deprecations? | no| Tests pass? | yes| Fixed tickets |symfony#11578| License | MIT| Doc PR | -Commits-------f45f1ab [DoctrineBridge] Changed UniqueEntityValidator to use the 2.5 Validation API
This PR was submitted for the master branch but it was merged into the 2.5 branch instead (closessymfony#12453).Discussion----------[Debug] Show only unique class candidates| Q | A| ------------- | ---| Bug fix? | yes| New feature? | no| BC breaks? | no| Deprecations? | no| Tests pass? | yes| Fixed tickets | -| License | MIT| Doc PR | -```PHPUnit 4.3.5 by Sebastian Bergmann.Configuration read from .../symfony/phpunit.xml.distS....S.............................FFFS.......Time: 2.29 seconds, Memory: 8.50MbThere were 3 failures:1) Symfony\Component\Debug\Tests\FatalErrorHandler\ClassNotFoundFatalErrorHandlerTest::testClassNotFound with data set#2 (array(1, 12, 'foo.php', 'Class \'UndefinedFunctionException\' not found'), 'Attempted to load class "UndefinedFunctionException" from the global namespace.Did you forget a "use" statement for "Symfony\\Component\\Debug\\Exception\\UndefinedFunctionException"?')Failed asserting that two strings are identical.--- Expected+++ Actual@@ @@ Attempted to load class "UndefinedFunctionException" from the global namespace.-Did you forget a "use" statement for "Symfony\Component\Debug\Exception\UndefinedFunctionException"?+Did you forget a "use" statement for e.g. "Symfony\Component\Debug\Exception\UndefinedFunctionException" or "Symfony\Component\Debug\Exception\UndefinedFunctionException"?.../symfony/src/Symfony/Component/Debug/Tests/FatalErrorHandler/ClassNotFoundFatalErrorHandlerTest.php:282) Symfony\Component\Debug\Tests\FatalErrorHandler\ClassNotFoundFatalErrorHandlerTest::testClassNotFound with data set#3 (array(1, 12, 'foo.php', 'Class \'PEARClass\' not found'), 'Attempted to load class "PEARClass" from the global namespace.Did you forget a "use" statement for "Symfony_Component_Debug_Tests_Fixtures_PEARClass"?')Failed asserting that two strings are identical.--- Expected+++ Actual@@ @@ Attempted to load class "PEARClass" from the global namespace.-Did you forget a "use" statement for "Symfony_Component_Debug_Tests_Fixtures_PEARClass"?+Did you forget a "use" statement for e.g. "Symfony_Component_Debug_Tests_Fixtures_PEARClass" or "Symfony_Component_Debug_Tests_Fixtures_PEARClass"?.../symfony/src/Symfony/Component/Debug/Tests/FatalErrorHandler/ClassNotFoundFatalErrorHandlerTest.php:283) Symfony\Component\Debug\Tests\FatalErrorHandler\ClassNotFoundFatalErrorHandlerTest::testClassNotFound with data set#4 (array(1, 12, 'foo.php', 'Class \'Foo\\Bar\\UndefinedFunctionException\' not found'), 'Attempted to load class "UndefinedFunctionException" from namespace "Foo\\Bar".Did you forget a "use" statement for "Symfony\\Component\\Debug\\Exception\\UndefinedFunctionException"?')Failed asserting that two strings are identical.--- Expected+++ Actual@@ @@ Attempted to load class "UndefinedFunctionException" from namespace "Foo\Bar".-Did you forget a "use" statement for "Symfony\Component\Debug\Exception\UndefinedFunctionException"?+Did you forget a "use" statement for e.g. "Symfony\Component\Debug\Exception\UndefinedFunctionException" or "Symfony\Component\Debug\Exception\UndefinedFunctionException"?.../symfony/src/Symfony/Component/Debug/Tests/FatalErrorHandler/ClassNotFoundFatalErrorHandlerTest.php:28```Commits-------db8a3ae [Debug] Show only unique class candidates
…igaminal)This PR was squashed before being merged into the 2.5 branch (closessymfony#12548).Discussion----------[Form] fixed a maxlength overring on a guessing| Q | A| ------------- | ---| Bug fix? | yes| New feature? | no| BC breaks? | no| Deprecations? | no| Tests pass? | yes| Fixed tickets |symfony#11527| License | MIT| Doc PR | -Commits-------7248680 [Form] fixed a maxlength overring on a guessing
* 2.3: [SecurityBundle] Firewall providers building - code cleaning [Filesystem] symlink use RealPath instead LinkTarget Fixed the AuthenticationProviderInterface alignment Fixed the proxy-manager version constraintConflicts:composer.jsonsrc/Symfony/Bridge/ProxyManager/composer.json
* 2.3: remove short array syntax fix session restart on PHP 5.3
Symfony 2.5 and higher requires ocramius/proxy-manager 0.4.0 orhigher.
…ekas)This PR was merged into the 2.5 branch.Discussion----------Fix failing tests after merge| Q | A| ------------- | ---| Bug fix? | yes| New feature? | no| BC breaks? | no| Deprecations? | no| Tests pass? | yes| Fixed tickets | -| License | MIT| Doc PR | -https://github.com/symfony/symfony/pull/12967/files?w=1Commits-------9b96373 Fix failing tests after merge6638535 fix ocramius/proxy-manager dependency version
* symfony/2.3: Test components using their lowest possible depsConflicts:src/Symfony/Bridge/Doctrine/composer.jsonsrc/Symfony/Bridge/Twig/composer.jsonsrc/Symfony/Bundle/FrameworkBundle/composer.jsonsrc/Symfony/Bundle/SecurityBundle/composer.jsonsrc/Symfony/Bundle/TwigBundle/composer.jsonsrc/Symfony/Component/DependencyInjection/composer.jsonsrc/Symfony/Component/EventDispatcher/composer.jsonsrc/Symfony/Component/Form/composer.jsonsrc/Symfony/Component/HttpKernel/composer.jsonsrc/Symfony/Component/Routing/composer.jsonsrc/Symfony/Component/Security/composer.jsonsrc/Symfony/Component/Validator/composer.json
01551af to16929a1CompareMember
fabpot commentedDec 16, 2014
Thank you@nicolas-grekas. |
fabpot added a commit that referenced this pull requestDec 16, 2014
…nicolas-grekas, fabpot)This PR was merged into the 2.5 branch.Discussion----------[2.5] Test components using their lowest possible deps| Q | A| ------------- | ---| Bug fix? | no| New feature? | no| BC breaks? | no| Deprecations? | no| Tests pass? | not yet| Fixed tickets | -| License | MIT| Doc PR | -Follow up of#12542Commits-------b1b5cca Test lowest versions of dependencies88d524e Merge remote-tracking branch 'symfony/2.3' into lowest-25206ebc7 minor#12542 Test components using their lowest possible deps (nicolas-grekas)25fef27 Test components using their lowest possible deps
nicolas-grekas added a commit that referenced this pull requestDec 16, 2014
…as, fabpot)This PR was merged into the 2.6 branch.Discussion----------[2.6] Test lowest versions of dependencies| Q | A| ------------- | ---| Bug fix? | no| New feature? | no| BC breaks? | no| Deprecations? | no| Tests pass? | yes| Fixed tickets | -| License | MIT| Doc PR | -Commits-------a29ffa8 [2.6] Test lowest versions of dependencies5e6959b Merge remote-tracking branch 'symfony/2.5' into lowest-26b2e0a80 minor#12998 [2.5] Test components using their lowest possible deps (nicolas-grekas, fabpot)b1b5cca Test lowest versions of dependencies88d524e Merge remote-tracking branch 'symfony/2.3' into lowest-25206ebc7 minor#12542 Test components using their lowest possible deps (nicolas-grekas)25fef27 Test components using their lowest possible deps
fabpot added a commit that referenced this pull requestFeb 18, 2022
…rves no purpose anymore (nicolas-grekas)This PR was merged into the 6.1 branch.Discussion----------[HttpKernel] Deprecate StreamedResponseListener, it serves no purpose anymore| Q | A| ------------- | ---| Branch? | 6.1| Bug fix? | no| New feature? | no| Deprecations? | yes| Tickets | -| License | MIT| Doc PR | -`StreamedResponseListener` has been introduced at the same time as `StreamedResponse` in#2935.Its purpose was to make catching exceptions easier by wrapping the call to `$response->send()` in the main try/catch of `HttpKernel`.Since#12998, we have `HttpKernel::terminateWithException()`, and we don't need that anymore, so we can just remove the listener.This will help [integrate Symfony into e.g. Swoole](php-runtime/runtime#115) /cc@alexander-schranz.Commits-------ee61774 [HttpKernel] Deprecate StreamedResponseListener, it serves no purpose anymore
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Follow up of#12542