Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork9.7k
[OptionsResolver] support array of instance validaion#15671
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
Closed
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
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.
It is very hard to read what happens whenif (!is_array()).
I would omit the single return and add aif (!is_array()) {return false;}
Member
stof commentedSep 2, 2015
I would like to see a profiling of the impact of this feature. Blackfire makes it easy by allowing to share a comparison graph of before vs after the change (and it is free). @webmozart do you have a test project for forms somewhere ? You used such projects in the past when working on the component. |
Contributor
OskarStark commentedSep 7, 2015
this PR looks interesting 👍 |
ContributorAuthor
EVODelavega commentedSep 8, 2015
Updated PR: exception message for complex types now format the complex value's type better (cf test case where allowed type is Oh, and@stof: I'll try to do some profiling if I can find the time. Do I compare the performance to 2.7 or current 2.8? |
Contributor
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.
Typo here ;) (Hi Elias!)
ContributorAuthor
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.
Typo fixed, (reading through commits in your own time, or not enough work back at NG? ;-P)
Contributor
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.
Saw your name come by last week, kept an eye out. ;) Hope you are well!
This PR was merged into the 3.0-dev branch.Discussion----------[Finder] removed obsolete code| Q | A| ------------- | ---| Bug fix? | no| New feature? | no| BC breaks? | no| Deprecations? | no| Tests pass? | yes| Fixed tickets | n/a| License | MIT| Doc PR | n/aCommits-------be729e3 [Finder] removed obsolete code
This PR was merged into the 3.0-dev branch.Discussion----------use try-finally when possible| Q | A| ------------- | ---| Bug fix? | no| New feature? | no| BC breaks? | no| Deprecations? | no| Tests pass? | I hope| Fixed tickets | n/a| License | MIT| Doc PR | n/aFound those with regex `catch \(\\Exception[^\}]+throw \$`Commits-------49edef2 use try-finally when possible
…urce() which was deprecated in 2.8 (mpdude)This PR was squashed before being merged into the 3.0-dev branch (closessymfony#15929).Discussion----------[3.0][Config] Remove ResourceInterface::getResource() which was deprecated in 2.8| Q | A| ------------- | ---| Bug fix? | no| New feature? | no| BC breaks? | yes| Deprecations? | no| Tests pass? | yes| Fixed tickets || License | MIT| Doc PR |Deprecated insymfony#15719.Commits-------7cef180 [3.0][Config] Remove ResourceInterface::getResource() which was deprecated in 2.8
Conflicts:composer.jsonsrc/Symfony/Bundle/FrameworkBundle/Command/RouterApacheDumperCommand.phpsrc/Symfony/Bundle/FrameworkBundle/Command/RouterDebugCommand.phpsrc/Symfony/Bundle/FrameworkBundle/Console/Descriptor/TextDescriptor.phpsrc/Symfony/Bundle/SecurityBundle/Resources/config/security.xmlsrc/Symfony/Bundle/SecurityBundle/Resources/config/security_listeners.xmlsrc/Symfony/Bundle/TwigBundle/Controller/ExceptionController.phpsrc/Symfony/Component/ClassLoader/ApcUniversalClassLoader.phpsrc/Symfony/Component/ClassLoader/DebugClassLoader.phpsrc/Symfony/Component/ClassLoader/UniversalClassLoader.phpsrc/Symfony/Component/Console/Input/StringInput.phpsrc/Symfony/Component/Debug/DebugClassLoader.phpsrc/Symfony/Component/DependencyInjection/Container.phpsrc/Symfony/Component/DependencyInjection/ContainerBuilder.phpsrc/Symfony/Component/DependencyInjection/ContainerInterface.phpsrc/Symfony/Component/DependencyInjection/Definition.phpsrc/Symfony/Component/DependencyInjection/DefinitionDecorator.phpsrc/Symfony/Component/DependencyInjection/Scope.phpsrc/Symfony/Component/DependencyInjection/ScopeInterface.phpsrc/Symfony/Component/DomCrawler/composer.jsonsrc/Symfony/Component/EventDispatcher/Event.phpsrc/Symfony/Component/HttpKernel/Kernel.phpsrc/Symfony/Component/HttpKernel/KernelInterface.phpsrc/Symfony/Component/HttpKernel/Log/LoggerInterface.phpsrc/Symfony/Component/HttpKernel/Log/NullLogger.phpsrc/Symfony/Component/Security/Core/composer.jsonsrc/Symfony/Component/Security/Resources/translations/security.tr.xlfsrc/Symfony/Component/Security/composer.jsonsrc/Symfony/Component/Translation/Translator.php
…(xabbuh)This PR was merged into the 3.0-dev branch.Discussion----------[Ldap] update branch alias for the master branch| Q | A| ------------- | ---| Bug fix? | yes| New feature? | no| BC breaks? | no| Deprecations? | no| Tests pass? | yes| Fixed tickets || License | MIT| Doc PR |Commits-------b4092a7 [Ldap] update branch alias for the master branch
* 2.8: Remove profiler storages deprecate finding deep items in request parameters [CssSelector] updated README [CssSelector] remove ConverterInterface [DependencyInjection] improved a comment for reading fluency [HttpKernel] change a class in tests to avoid depending on SQLite [FrameworkBundle] Fix tests [Bridge\Twig] Fix form lowest version [ci] Display fastest results first when running tests in parallel [Yaml] Improve newline handling in folded scalar blocks
* 2.8: fixed deprecation notices fixed typos [FrameworkBundle] Tag deprecated services [VarDumper] Dump PHP+Twig code excerpts in backtraces [Config] Fix ArrayNode extra keys "ignore" and "remove" behaviors
* 2.8: add dependency required by a replaced package Add a way to group toolbar info pieces Added general sf-toolbar-block-right class Bind input before executing the COMMAND event
…terJ)This PR was merged into the 3.0-dev branch.Discussion----------[3.0][Security] Remove deprecated features| Q | A| ------------- | ---| Bug fix? | no| New feature? | no| BC breaks? | yes| Deprecations? | no| Tests pass? | yes| Fixed tickets | -| License | MIT| Doc PR | -Commits-------c3c5989 Remove more tests6f9e897 Remove AbstractVoter#isGranted() method6b6de15 Removed supports{Attribute,Class}() methods
* 2.8: Set the redraw frequency at least to 1. Setting it to 0 would otherwise produce an error. [Process] Unset callback after stop to free memory Improve error message for undefined DIC aliases [VarDumper] fixed .sf-dump z-index [Validator] Updated Luxembourgish translations for 2.8 Disallow http-kernel 3.x,fixessymfony#16837. [DependencyInjection] Validate class names and factory methods ampq → amqp Fix typo Refactoring EntityUserProvider::__construct() to not do work, cause cache warm error [Form] Add context to FormFactory deprecations CS: remove unneeded parentheses around control statements [TwigBridge] Clean deps now that 2.8.0 is tagged
* 3.0: Set the redraw frequency at least to 1. Setting it to 0 would otherwise produce an error. [Form] document changes to form type interfaces [Process] Unset callback after stop to free memory Improve error message for undefined DIC aliases [VarDumper] fixed .sf-dump z-index [Validator] Updated Luxembourgish translations for 2.8 Disallow http-kernel 3.x,fixessymfony#16837. [Config] Throw an exception when using cannotBeEmpty() on numeric or boolean nodes [DependencyInjection] Validate class names and factory methods ampq → amqp Fix typo Refactoring EntityUserProvider::__construct() to not do work, cause cache warm error [Form] Add context to FormFactory deprecations Reapply the Yaml bugfix ofsymfony#16745 CS: remove unneeded parentheses around control statements [DomCrawler] add upgrade hint on interface changes [TwigBridge] Clean deps now that 2.8.0 is tagged
* 2.8: [Process] Fix stopping a process on Windows [PhpUnitBridge] Add weak-verbose mode and match against message instead of test name Added a note about the new requirement iconv. Improved error messages for Yaml Deprecations Added a test case for the Logger class. [Form] Fix choices defined as Traversable CS: general fixes Suggested Process dependency
* 3.0: [Process] Fix stopping a process on Windows [PhpUnitBridge] Add weak-verbose mode and match against message instead of test name Added a note about the new requirement iconv. Improved error messages for Yaml Deprecations Added a test case for the Logger class. [Form] Fix choices defined as Traversable CS: general fixes Suggested Process dependency
* 2.8: Fix the exception message expectation
* 3.0: Fix the exception message expectation
* 2.8: Fix DeprecationErrorHandler on PHP 5.3
* 2.8: [Process] Enhance compatiblity with --enable-sigchild [Validator] removes unused variable in LengthValidator class. [FrameworkBundle] [Bug] Fixes new InputStyle bugsymfony#16920 fix short array syntax for php 5.3 [Serializer] Fixed on array of objects in . [Process] Always call proc_close [Form] Add missing tests for StringUtil::fqcnToBlockPrefix() [Security] Fix a Polyfill import statement in StringUtils [Validator] Updated Luxembourgish translations for 2.8 Improved the code of the commands that use the new SymfonyStyle class disable server commands without Process component list all server command names in suggestion Suggested Process dependency disable server:run cmd without Process component Suggested Process dependency [FrameworkBundle] prevent cache:clear creating too long paths [FrameworkBundle] [Translation] Fixed translations not written when no translations directory in update commandConflicts:.travis.ymlsrc/Symfony/Bridge/Twig/Command/LintCommand.phpsrc/Symfony/Bundle/FrameworkBundle/Command/ConfigDebugCommand.phpsrc/Symfony/Bundle/FrameworkBundle/Command/ContainerDebugCommand.phpsrc/Symfony/Bundle/FrameworkBundle/Command/RouterApacheDumperCommand.phpsrc/Symfony/Bundle/FrameworkBundle/Command/RouterDebugCommand.phpsrc/Symfony/Bundle/FrameworkBundle/Command/ServerRunCommand.phpsrc/Symfony/Bundle/FrameworkBundle/Command/TranslationDebugCommand.phpsrc/Symfony/Bundle/FrameworkBundle/Command/YamlLintCommand.phpsrc/Symfony/Bundle/TwigBundle/Command/DebugCommand.phpsrc/Symfony/Component/Security/Core/Util/StringUtils.php
* 3.0: [Process] Enhance compatiblity with --enable-sigchild [Validator] removes unused variable in LengthValidator class. [FrameworkBundle] [Bug] Fixes new InputStyle bugsymfony#16920 fix short array syntax for php 5.3 [Serializer] Fixed on array of objects in . [Process] Always call proc_close [Form] Add missing tests for StringUtil::fqcnToBlockPrefix() [Security] Fix a Polyfill import statement in StringUtils [Validator] Updated Luxembourgish translations for 2.8 Improved the code of the commands that use the new SymfonyStyle class disable server commands without Process component list all server command names in suggestion Suggested Process dependency disable server:run cmd without Process component Suggested Process dependency Fix DeprecationErrorHandler on PHP 5.3 [FrameworkBundle] prevent cache:clear creating too long paths [FrameworkBundle] [Translation] Fixed translations not written when no translations directory in update command
in symfony 3.0 there is no ajax collector declared. Because of that debug toolbar do not display ajac calls
This PR was merged into the 3.0 branch.Discussion----------return ajax collector to collectors.php| Q | A| ------------- | ---| Bug fix? | yes| New feature? | no| BC breaks? | no| Deprecations? | no| Tests pass? | no| Fixed tickets |symfony#16880| License | MIT| Doc PR |in symfony 3.0 there is no ajax collector declared. Because of that debug toolbar do not display ajax callsCommits-------8a2c757 return ajax collector to collectors.php
…king (voronkovich)This PR was merged into the 3.0 branch.Discussion----------[HttpKernel] HttpCache: remove an ESI instance checking| Q | A| ------------- | ---| Bug fix? | no| New feature? | no| BC breaks? | no| Deprecations? | no| Tests pass? | yes| Fixed tickets | no| License | MIT| Doc PR | noThis checking was added by this commitsymfony@32d964b, but now it's not neededCommits-------7528134 Removed ESI instance check
…riggerDeprecation flag (ogizanagi)This PR was merged into the 3.0 branch.Discussion----------[FrameworkBundle][HttpKernel] Remove remaining $triggerDeprecation flag| Q | A| ------------- | ---| Fixed tickets | n/a| License | MITIntroduced insymfony#14665, this flag controlled the deprecation notice triggering when using the `ContainerAwareHttpKernel`, in order to avoid triggering it when using this class in the Symfony framework. This class no longer exists now, neither the flag.Commits-------ac15454 [FrameworkBundle][HttpKernel] Remove remaining $triggerDeprecation flag
ContributorAuthor
EVODelavega commentedDec 16, 2015
Status: Reviewed I'm marking this PR as reviewed, seeing as there haven't been any comments in some time |
Contributor
soullivaneuh commentedDec 16, 2015
Travis is failing. BTW, 2.8 and 3.0 are out now. I think this PR should target 3.1 now. |
ContributorAuthor
EVODelavega commentedDec 16, 2015
Created anew PR merging changes into 3.1 instead of 2.8 |
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.
A suggested approach to add support for typed-arrays as allowed types, using a simple recursive method