- 01 Oct, 2013 1 commit
-
-
Julio Montoya authored
-
- 19 Jun, 2013 1 commit
-
-
Julio Montoya authored
-
- 25 May, 2013 4 commits
-
-
Fabien Potencier authored
-
Fabien Potencier authored
This PR was submitted for the 1.0 branch but it was merged into the master branch instead (closes #705). Discussion ---------- Make dispatcher lazy, do not trigger its creation on middleware definition The dispatcher has quite a few dependencies due to all of the subscribers that are added to it. One of these is the logger, several other services are affected as well though. The listener shortcut methods like on(), before(), after(), error() all force creation of the dispatcher and thus all of its dependencies. This makes it impossible to have lazy configuration of those services. The specific issue that triggered this was lazy configuration of the logger, which simply does not work once you have before() or after(). By using extend(), all of those shortcut calls can delay the creation of the dispatcher and thus solve the issue. It will add a slight overhead, but it should be relatively small, since the results of creating a service are memoized through share(). Commits ------- efa0383 Add changelog entry for lazy dispatcher proxies 59b56b0 Make dispatcher lazy, do not trigger its creation on middleware definition
-
Igor Wiedler authored
-
Igor Wiedler authored
The dispatcher has quite a few dependencies due to all of the subscribers that are added to it. One of these is the logger, several other services are affected as well though. The listener shortcut methods like on(), before(), after(), error() all force creation of the dispatcher and thus all of its dependencies. This makes it impossible to have lazy configuration of those services. The specific issue that triggered this was lazy configuration of the logger, which simply does not work once you have before() or after(). By using extend(), all of those shortcut calls can delay the creation of the dispatcher and thus solve the issue. It will add a slight overhead, but it should be relatively small, since the results of creating a service are memoized through share().
-
- 04 May, 2013 1 commit
-
-
Fabien Potencier authored
This PR was merged into the master branch. Discussion ---------- [1.1] Drop support for symfony/* <2.3 I've left the ranges in, because it's likely that we will extend them in the future. Commits ------- 54be4f51 [1.1] Remove symfony <2.3 hacks 2b45f600 [1.1] Add minimum-stability of dev 8f8ba416 [1.1] Update all symfony version references in the docs to 2.3 7a14b209 [1.1] Update symfony versions in fat composer.json e9478ddd Drop support for symfony/* <2.3
-
- 03 May, 2013 16 commits
-
-
Igor Wiedler authored
-
Igor Wiedler authored
-
Igor Wiedler authored
-
Igor Wiedler authored
-
Igor Wiedler authored
-
Fabien Potencier authored
-
Fabien Potencier authored
-
Fabien Potencier authored
-
Fabien Potencier authored
This PR was merged into the master branch. Discussion ---------- Update install instructions for v1.0.0 Commits ------- 15edf844 Update install instructions for v1.0.0
-
Igor Wiedler authored
-
Fabien Potencier authored
This PR was merged into the master branch. Discussion ---------- Update Application::VERSION to 1.0.0 Commits ------- 72d9fea2 Update Application::VERSION to 1.0.0
-
Fabien Potencier authored
This PR was merged into the master branch. Discussion ---------- Add some of the missing changelog entries Commits ------- ba30e7be Add some of the missing changelog entries
-
Igor Wiedler authored
-
Igor Wiedler authored
-
Fabien Potencier authored
-
Fabien Potencier authored
This PR was merged into the master branch. Discussion ---------- [Doc] Add code-block for remove the red prompt Commits ------- 18159ea1 Add code-block for remove the red prompt
-
- 02 May, 2013 1 commit
-
-
Romain authored
-
- 01 May, 2013 2 commits
-
-
Fabien Potencier authored
This PR was merged into the master branch. Discussion ---------- Doc: trustProxyData() is now replaced by setTrustedProxies(). Although trustProxyData() is deprecated, the documentation still refers to it, causing deprecation warnings. This patch suggested an updated wording. Commits ------- d45ba3ed Doc: trustProxyData() is now replaced by setTrustedProxies().
-
Frederic G. MARAND authored
-
- 30 Apr, 2013 2 commits
-
-
Fabien Potencier authored
This PR was squashed before being merged into the master branch (closes #680). Discussion ---------- Adding "port" option in DoctrineServiceProvider In doctrine.rst and DoctrineServiceProvider.php Commits ------- a140a694 Adding "port" option in DoctrineServiceProvider
-
Julio Montoya authored
-
- 27 Apr, 2013 1 commit
-
-
Fabien Potencier authored
This PR was merged into the master branch. Discussion ---------- Validator 2.3 test fixes Commits ------- ac4c5a56 Pass constraints with keys for all symfony versions 87967798 Add alias method to conform with ConstraintValidatorInterface in 2.3
-
- 26 Apr, 2013 4 commits
-
-
Dave Marshall authored
-
Dave Marshall authored
-
Fabien Potencier authored
This PR was squashed before being merged into the master branch (closes #685). Discussion ---------- Bump upper bound for symfony/* from 2.2 to 2.3 Commits ------- 70eb564d Bump upper bound for symfony/* from 2.2 to 2.3
-
Igor Wiedler authored
-
- 24 Apr, 2013 1 commit
-
-
Fabien Potencier authored
This PR was merged into the master branch. Discussion ---------- Ported ConstraintValidatorFactory.php to Silex Included a modified version of `ConstraintValidatorFactory` to Silex for use with custom constraints. Since Silex comes with `ValidatorServiceProvider` built in, I guess it makes sense to include this, in case users might want dependencies with their custom validators. Usage would be: ```php // Custom.php public function validatedBy() { return 'my.validator.custom'; } // index.php $app['validator.custom'] = $app->share(function ($app) { return new CustomValidator(); }); $app->register(new ValidatorServiceProvider(), array( 'validator.validator_service_ids' => array( 'my.validator.custom' => 'validator.custom' ) )); ``` Commits ------- c4466a18 Changed constructor type hint to Pimple fd72c6ab Incorporated changes suggested by @Igorow. Separating $this->validators (instances) and $this->serviceNames. fa15389b Separating validator service names and instances 0c3234d2 Minor refactoring 83266af9 Changed mapping of 'validator.validator_service_ids' to use ternary operator 278c5e01 Backward compatibility for UserPasswordValidator 40734452 Changed deprecated namespaces 4d8956f3 Added custom constraint and validator to properly test the ConstraintValidatorFactory. b45b4fb2 SecurityServiceProvider now registers a UserPasswordValidator if ValidatorServiceProvider is registered. 70c1e365 Registering a ValidatorServiceProvider can take a validator.validator_service_ids parameter which is now an associative array of aliases as keys and service names as values so that ConstraintValidatorFactory can lazy load the validators. 77af56be Removed Symfony\Component\Validator\ConstraintValidatorFactory altogether from ValidatorServiceProvider. Custom validators are now lazy loaded. removed obsolete test. 423138aa - Type hint in Silex\ConstraintValidatorFactory constructor for container - Modifed Silex\Provider\ValidatorServiceProvider to accept an optional 'validator.validator_service_ids' parameter wich allows users to register custom validators. - Added test to verify that if users registered ValidatorServiceProvider using optional 'validator.validator_service_ids' parameter then the new Silex\ConstraintValidatorFactory will be used instead of the one provided in Symfony. eaf0af2a Ported ConstraintValidatorFactory.php to Silex/Pimple
-
- 23 Apr, 2013 2 commits
-
-
Fabien Potencier authored
This PR was merged into the master branch. Discussion ---------- Suggest using QSA in apache config to get access to query parameters A user had hard time finding why OAuth was not working on its Silex application (see https://github.com/GromNaN/FacebookServiceProvider/issues/1#issuecomment-16845793) For the Apache doc: http://httpd.apache.org/docs/current/rewrite/flags.html#flag_qsa Commits ------- 1cbb20e1 Suggest using QSA in apache config to get access to query parameters
-
Jerome TAMARELLE authored
-
- 22 Apr, 2013 1 commit
-
-
alexkappa authored
-
- 12 Apr, 2013 1 commit
-
-
alexkappa authored
Incorporated changes suggested by @Igorow. Separating $this->validators (instances) and $this->serviceNames.
-
- 11 Apr, 2013 2 commits
-
-
Fabien Potencier authored
This PR was merged into the master branch. Discussion ---------- Fix JSON parse error in Twig docs Commits ------- a7efc8f4 Fix JSON parse error in Twig docs
-
Dave Hulbert authored
-