diogo/symfony
Archived
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

bug #22159 [FrameworkBundle] Cache pool clear command requires at least 1 pool (ro0NL)

Browse Source 
This PR was submitted for the master branch but it was merged into the 3.2 branch instead (closes #22159).

Discussion
----------

[FrameworkBundle] Cache pool clear command requires at least 1 pool

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | no
| New feature?  | yes
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes/no
| Fixed tickets | #22047
| License       | MIT
| Doc PR        | symfony/symfony-docs#... <!--highly recommended for new features-->

The commands help description already states this should be required.

```
The cache:pool:clear command clears the given cache pools or cache pool clearers.

    bin/console cache:pool:clear <cache pool or clearer 1> [...<cache pool or clearer N>]
```

So this could also be qualified a bugfix.

Commits
-------

a61797f850 [FrameworkBundle] Cache pool clear command requires at least 1 pool
This commit is contained in:
Fabien Potencier 2017-03-25 08:48:16 -07:00
commit b1bfbcc04b
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/Symfony/Bundle/FrameworkBundle/Command/CachePoolClearCommand.php
View File

@ -33,7 +33,7 @@ final class CachePoolClearCommand extends ContainerAwareCommand
$this
->setName('cache:pool:clear')
->setDefinition(array(
new InputArgument('pools', InputArgument::IS_ARRAY, 'A list of cache pools or cache pool clearers'),
new InputArgument('pools', InputArgument::IS_ARRAY | InputArgument::REQUIRED, 'A list of cache pools or cache pool clearers'),
))
->setDescription('Clears cache pools')
->setHelp(<<<'EOF'