The "aggregate" command now doesn't require sub-command, when "--ignore-*" options are used

Author: Alexander Obuhovich

CHANGELOG.md

@@ -39,6 +39,7 @@ This project adheres to [Semantic Versioning](http://semver.org/).
 - Wrap list of associated revision next to conflicted paths to "4 per row" to avoid too wide table creation.
 - Name format of per-working copy config setting is changed, which will result in all data being lost unless migrated by hand in "~/.svn-buddy/config.json" file (old "path-settings.wc_url_hash.setting_name", new: "path-settings[wc_url].setting_name").
 - Major under the hood revision information storage changes.
+- The `aggregate` command no longer requires specifying `sub-command` argument, when `--ignore-*` options are used.
 
 ### Fixed
 - The Subversion repositories hosted on https://unfuddle.com/ were not usable from `log` and `merge` commands.

src/SVNBuddy/Command/AggregateCommand.php

@@ -37,7 +37,7 @@ protected function configure()
 			)
 			->addArgument(
 				'sub-command',
-				InputArgument::REQUIRED,
+				InputArgument::OPTIONAL,
 				'Command to execute on each found working copy'
 			)
 			->addArgument(
@@ -122,18 +122,22 @@ protected function getSubCommands()
 	 */
 	protected function execute(InputInterface $input, OutputInterface $output)
 	{
+		if ( $this->processIgnoreAdd() || $this->processIgnoreRemove() || $this->processIgnoreShow() ) {
+			return;
+		}
+
 		$sub_command = $this->io->getArgument('sub-command');
 
+		if ( $sub_command === null ) {
+			throw new \RuntimeException('Not enough arguments (missing: "sub-command").');
+		}
+
 		if ( !in_array($sub_command, $this->getSubCommands()) ) {
 			throw new \RuntimeException(
 				'The "' . $sub_command . '" sub-command is unknown or doesn\'t support aggregation.'
 			);
 		}
 
-		if ( $this->processIgnoreAdd() || $this->processIgnoreRemove() || $this->processIgnoreShow() ) {
-			return;
-		}
-
 		$this->runSubCommand($sub_command);
 	}