157 Commits

Author SHA1 Message Date
Nicolas Grekas
827fed07f7 minor #61371 chore: PHP CS Fixer - restore PHP / PHPUnit rulesets (keradus)
This PR was squashed before being merged into the 7.4 branch.

Discussion
----------

 chore: PHP CS Fixer - restore PHP / PHPUnit rulesets

| Q             | A
| ------------- | ---
| Branch?       | 7.4
| Bug fix?      | no
| New feature?  | no <!-- please update src/**/CHANGELOG.md files -->
| Deprecations? | no <!-- please update UPGRADE-*.md and src/**/CHANGELOG.md files -->
| Issues        | Fix CS <!-- prefix each issue number with "Fix #", no need to create an issue if none exists, explain below instead -->
| License       | MIT

relates to 5d4e211789 (r163638626) , showing:
- possible improvements to codebase that those ruleset can still offer (especially when targetting newest possible target version)
- and also preventing old way to still be in use, if someone forget to use new syntax/methods

with love by [PHP Coding Standards Fixer](https://cs.symfony.com/)

Commits
-------

92940de5e52  chore: PHP CS Fixer - restore PHP / PHPUnit rulesets
2025-08-12 22:21:03 +02:00
Dariusz Ruminski
8965ae6d73 chore: PHP CS Fixer - restore PHP / PHPUnit rulesets 2025-08-12 22:20:59 +02:00
Dariusz Ruminski
be639dea57 chore: heredoc indentation as of PHP 7.3
https://www.php.net/manual/en/language.types.string.php#language.types.string.syntax.heredoc
2025-08-10 00:28:14 +02:00
Nicolas Grekas
e1fbaac609 Remove some unneeded var annotations 2025-08-04 17:51:53 +02:00
Alexander M. Turek
4875a8487f Prefix all sprintf() calls 2024-06-20 17:52:34 +02:00
Alexander M. Turek
12901443fc [Filesystem] Add the readFile() method 2024-03-15 14:33:12 +01:00
Alexander M. Turek
427ab31004 [FrameworkBundle] Use CPP 2024-02-09 15:31:28 +01:00
Wouter de Jong
34bd487d2e Add void return types 2023-02-13 14:18:23 +01:00
Grégoire Pineau
fb7ade8fab [CS] Remove @inheritdoc PHPDoc 2022-08-25 17:27:04 +02:00
Thomas Calvet
2f62d5af2f Leverage non-capturing catches 2022-04-01 09:15:35 +02:00
Nicolas Grekas
8558b9ed9f Merge branch '6.0' into 6.1
* 6.0:
  [HttpClient] fix resetting DNS/etc when calling CurlHttpClient::reset()
  Fix invalid guess with enumType
  [HttpClient] Remove deprecated usage of GuzzleHttp\Promise\promise_for
  [HttpClient] Remove deprecated usage of `GuzzleHttp\Promise\queue`
  [PropertyAccess] Fix handling of uninitialized property of anonymous class
  [FrameworkBundle] Allow default cache pools to be overwritten by user
  [DependencyInjection] fix test
  ResolveBindingsPass remove loading of class iterable
  [Serializer] Fix AbstractObjectNormalizer TypeError on denormalization
  [FrameworkBundle] Avoid calling rtrim(null, '/') in AssetsInstallCommand
  Optimization of resolveEnvPlaceholders
  Fix incorrect format
  [DependencyInjection] Fix nested env var with resolve processor
  Allow OutputFormatter::escape() to be used for escaping URLs used in <href>
  allow a zero time-limit
  Use correct tag for ExpoTransportFactory service
  [DependencyInjection] Ignore argument type check in CheckTypeDeclarationsPass if it's a Definition with a factory
  [Validators] Add translations for Slovak #43735
2022-01-17 11:57:14 +01:00
Nicolas Grekas
f8edbc94d7 Merge branch '5.4' into 6.0
* 5.4:
  [HttpClient] Remove deprecated usage of `GuzzleHttp\Promise\queue`
  [PropertyAccess] Fix handling of uninitialized property of anonymous class
  [FrameworkBundle] Allow default cache pools to be overwritten by user
  [DependencyInjection] fix test
  ResolveBindingsPass remove loading of class iterable
  [FrameworkBundle] Avoid calling rtrim(null, '/') in AssetsInstallCommand
  Optimization of resolveEnvPlaceholders
  Fix incorrect format
  [DependencyInjection] Fix nested env var with resolve processor
  Allow OutputFormatter::escape() to be used for escaping URLs used in <href>
  allow a zero time-limit
  Use correct tag for ExpoTransportFactory service
  [DependencyInjection] Ignore argument type check in CheckTypeDeclarationsPass if it's a Definition with a factory
  [Validators] Add translations for Slovak #43735
2022-01-12 19:59:56 +01:00
Nicolas Grekas
2b25bceefd Merge branch '5.3' into 5.4
* 5.3:
  [HttpClient] Remove deprecated usage of `GuzzleHttp\Promise\queue`
  [PropertyAccess] Fix handling of uninitialized property of anonymous class
  [FrameworkBundle] Allow default cache pools to be overwritten by user
  [DependencyInjection] fix test
  ResolveBindingsPass remove loading of class iterable
  [FrameworkBundle] Avoid calling rtrim(null, '/') in AssetsInstallCommand
  [DependencyInjection] Fix nested env var with resolve processor
  Allow OutputFormatter::escape() to be used for escaping URLs used in <href>
  allow a zero time-limit
  [DependencyInjection] Ignore argument type check in CheckTypeDeclarationsPass if it's a Definition with a factory
  [Validators] Add translations for Slovak #43735
2022-01-12 19:55:10 +01:00
Nicolas Grekas
67f5d9a9e7 Merge branch '4.4' into 5.3
* 4.4:
  [HttpClient] Remove deprecated usage of `GuzzleHttp\Promise\queue`
  [PropertyAccess] Fix handling of uninitialized property of anonymous class
  [DependencyInjection] fix test
  ResolveBindingsPass remove loading of class iterable
  [FrameworkBundle] Avoid calling rtrim(null, '/') in AssetsInstallCommand
  [DependencyInjection] Fix nested env var with resolve processor
  Allow OutputFormatter::escape() to be used for escaping URLs used in <href>
  allow a zero time-limit
  [DependencyInjection] Ignore argument type check in CheckTypeDeclarationsPass if it's a Definition with a factory
  [Validators] Add translations for Slovak #43735
2022-01-12 19:37:36 +01:00
Pavol Tuka
5ae3655a69 [FrameworkBundle] Avoid calling rtrim(null, '/') in AssetsInstallCommand 2022-01-12 12:20:14 +01:00
Fabien Potencier
dd3bf10773 Revert "bug #44494 Remove FQCN type hints on properties (fabpot)"
This reverts commit 0a13eba7037c6636be7a35c194d7d2e86d6ea6ba, reversing
changes made to e4cfa82c31ffda491b73e181c8d3d4e0a307eee8.
2021-12-09 13:46:12 +01:00
Fabien Potencier
a871a08baa Remove FQCN type hints on properties 2021-12-08 16:13:44 +01:00
Alexander M. Turek
bf9d0d4969 [FrameworkBundle] Add types to private properties 2021-11-02 11:55:16 +01:00
Robin Chalas
c1ad9bd96a Merge branch '5.4' into 6.0
* 5.4:
  [Security][WIP] Add authenticators info to the profiler
  [Translation] Use symfony default locale when pulling translations from providers
  added missing translations for Bosnian (bs)
  Add the missing greek translations for security core and validator component
  [HttpKernel] Fix return types in `EventDataCollector`
  Do not call substr_count() if ip is null to avoid deprecation warning in PHP 8.1
  [Security][Validator] Add missing translations for Slovenian (sl)
  [Messenger] Add worker metadata inside logs
  [Messenger] Log when worker should stop and when `SIGTERM` is received
  cs fix
  [Security][Validator] Add missing translations for Finnish (fi)
  [VarDumper] returns a 500 when dd() is executed
  chore(VarDumper): declare that dd() never returns
  [MonologBridge] Deprecate the Swiftmailer handler
  [Cache] Commit items implicitly only when deferred keys are requested
  [MonologBridge] Deprecates ResetLoggersWorkerSubscriber
  Fix "can not" spelling
  [HttpClient] fix missing kernel.reset tag on TraceableHttpClient services
  [Form] Fix ChoiceType Extension to effectively set and use the translator
  Added new CssColor constraint
2021-10-11 18:08:10 +02:00
Michael Voříšek
2e213a04c4 Fix "can not" spelling 2021-10-06 09:48:14 +02:00
Nicolas Grekas
54788c67a5 Use #[AsCommand] to describe commands 2021-09-30 17:28:07 +02:00
Alexander M. Turek
95578328e3 Merge branch '5.2' into 5.3
* 5.2:
  Simplify some code with null coalesce operator
  Don't use deprecated TestLogger class
2021-07-18 18:30:56 +02:00
Alexander M. Turek
fa9d19d8d3 Merge branch '4.4' into 5.2
* 4.4:
  Simplify some code with null coalesce operator
2021-07-18 18:25:43 +02:00
Javier Eguiluz
c3557b6ddf Simplify some code with null coalesce operator 2021-07-18 18:24:15 +02:00
Fabien Potencier
3f8b570669 Merge branch '5.2' into 5.x
* 5.2:
  Fix #36973: Command description consistency
  Render email once
2021-03-06 09:05:19 +01:00
Fabien Potencier
a26f088761 Merge branch '4.4' into 5.2
* 4.4:
  Fix #36973: Command description consistency
  Render email once
2021-03-06 08:59:01 +01:00
Dane Powell
1d7c9a5590 Fix #36973: Command description consistency 2021-03-05 09:58:50 -08:00
Nicolas Grekas
a2ab2c0854 [Console] enable describing commands in ways that make the list command lazy 2021-01-20 12:45:16 +01:00
Nicolas Grekas
0296d1c99a Merge branch '4.4' into 5.1
* 4.4:
  Update .php_cs.dist
  Apply "visibility_required" CS rule to constants
2020-12-08 18:02:38 +01:00
Nicolas Grekas
2b9098892e Apply "visibility_required" CS rule to constants
php-cs-fixer fix --rules='{"visibility_required": ["property", "method", "const"]}'
2020-12-08 17:59:59 +01:00
Nicolas Grekas
1e1d4eb97c Merge branch '3.4' into 4.4
* 3.4:
  Enable "native_constant_invocation" CS rule
  Make AbstractPhpFileCacheWarmer public
2020-09-02 18:08:58 +02:00
Nicolas Grekas
5da40103c3 Enable "native_constant_invocation" CS rule 2020-09-02 18:06:40 +02:00
Nicolas Grekas
5f56e48331 Merge branch '4.4' into 5.0
* 4.4:
  [PhpUnitBridge] fix bad test
  [4.4] CS fixes
  [3.4] CS fixes
  Disable phpunit verbosity
  Queue name is a required parameter
  [FrameworkBundle] display actual target for error in AssetsInstallCommand
  Remove patches for Doctrine bugs and deprecations
  [Mime] fix bad method call on "EmailAddressContains"
  [DI][EventDispatcher] added contract for implementation
2020-05-08 14:34:39 +02:00
Nicolas Grekas
f7a2d40e6d Merge branch '3.4' into 4.4
* 3.4:
  [FrameworkBundle] display actual target for error in AssetsInstallCommand
  Remove patches for Doctrine bugs and deprecations
  [DI][EventDispatcher] added contract for implementation
2020-05-08 11:58:40 +02:00
Matthias Larisch
5e5c982d38 [FrameworkBundle] display actual target for error in AssetsInstallCommand
When assets:install fails because the target directory does not exist, it should display the actual directory it wanted to have instead of the configuration directive. In most cases, the target directory is retrieved from the kernel config and thus differs from the argument.
2020-05-08 11:56:06 +02:00
Alexander M. Turek
4426ee3231 Add types to constructors and private/final/internal methods (Batch III) 2019-10-07 13:33:25 +02:00
Nicolas Grekas
be0063ce5a Merge branch '4.4'
* 4.4:
  Add more return types after fixing a typo in my script
2019-08-23 14:18:57 +02:00
Nicolas Grekas
2d0aa18c81 Add more return types after fixing a typo in my script 2019-08-23 14:16:45 +02:00
Nicolas Grekas
4d82bb633f Merge branch '4.4'
* 4.4:
  [HttpKernel][FrameworkBundle] Add alternative convention for bundle directories
  [DI] deprecate support for non-object services
  [Translation] XliffLintCommand: allow .xliff file extension
  [Serializer] Encode empty objects as objects, not arrays
2019-08-13 15:43:55 +02:00
Yonel Ceruto
5d943040a7 [HttpKernel][FrameworkBundle] Add alternative convention for bundle directories 2019-08-13 15:29:02 +02:00
Christian Flothmann
5ca9675a12 remove deprecated features 2019-07-14 10:03:55 +02:00
Fabien Potencier
283ee81439 Merge branch '4.4'
* 4.4:
  fixed CS
  [Validator] Add a new constraint message when there is both min and max
  fixed CS
  [Bundles] Rename getPublicPath() as getPublicDir()
  Remove experimental notice from components
  [LDAP] add new option implemented in php 7.1
  Replace missing message parameter
2019-07-12 09:03:26 +03:00
Fabien Potencier
2b87f2f305 fixed CS 2019-07-12 07:47:23 +03:00
Javier Eguiluz
dee0fafc91 [Bundles] Rename getPublicPath() as getPublicDir() 2019-07-12 07:45:28 +03:00
Nicolas Grekas
920b084219 Merge branch '4.4'
* 4.4: (53 commits)
  Fix Twig 1.x compatibility
  Deprecating templateExists method
  [Translator] Improve farsi(persian) translations for Form
  [Validator] Fix Changelog for #31511
  [Lock][Console] bump lock requirement in console
  [Lock] minor: add missing alias for PersistenStoreInterface
  Improve fa translations
  Dynamic bundle assets
  [Lock] rename and deprecate Factory into LockFactory
  [Debug] Restoring back the state of the Debug component (1st step)
  Spell "triggering" properly
  [Lock] Fix tests
  Added tests to cover the possibility of having scalars as services.
  fixed CS
  [Lock] Split \"StoreInterface\" into multiple interfaces with less responsability
  [VarDumper] Let browsers trigger their own search on double CMD/CTRL + F hit
  [Validator] Allow to use property paths to get limits in range constraint
  Fix missing deprecations
  fixed tests on old PHP versions
  [FrameworkBundle] Inform the user when save_path will be ignored
  ...
2019-07-11 07:40:26 +02:00
Massimiliano Arione
7ecf5e931f Dynamic bundle assets 2019-07-09 08:12:54 +02:00
dFayet
73d51f2a88 Replace @return annotation by return type in final classes 2019-06-23 00:57:33 +02:00
Nicolas Grekas
0064e75afa [FrameworkBundle] remove deprecated code paths 2019-06-06 19:13:23 +02:00
Fabien Potencier
8a4c162db7 Merge branch '4.2' into short-array-master
* 4.2:
  fixed CS
  fixed CS
  fixed tests
  fixed CS
  fixed CS
  fixed CS
  fixed short array CS in comments
  fixed CS in ExpressionLanguage fixtures
  fixed CS in generated files
  fixed CS on generated container files
  fixed CS on Form PHP templates
  fixed CS on YAML fixtures
  fixed fixtures
  switched array() to []
2019-01-16 22:53:39 +01:00
Fabien Potencier
29063f297d Merge branch '4.1' into 4.2
* 4.1:
  fixed tests
  fixed CS
  fixed CS
  fixed CS
  fixed short array CS in comments
  fixed CS in ExpressionLanguage fixtures
  fixed CS in generated files
  fixed CS on generated container files
  fixed CS on Form PHP templates
  fixed CS on YAML fixtures
  fixed fixtures
  switched array() to []
2019-01-16 21:31:39 +01:00