Bugfix: when anonymous commands have same arguments, container fails …

…as their md5 hash is the same

src/Kdyby/Console/DI/ConsoleExtension.php

@@ -80,8 +80,8 @@ public function loadConfiguration()
 		}
 
 		Nette\Utils\Validators::assert($config, 'array');
-		foreach ($config['commands'] as $command) {
-			$def = $builder->addDefinition($this->prefix('command.' . md5(Nette\Utils\Json::encode($command))));
+		foreach ($config['commands'] as $i => $command) {
+			$def = $builder->addDefinition($this->prefix('command.' . $i));
 			list($def->factory) = Nette\DI\Compiler::filterArguments(array(
 				is_string($command) ? new Statement($command) : $command
 			));

tests/KdybyTests/Console/InputErrors.phpt

@@ -347,6 +347,52 @@ class TypoCommand extends Symfony\Component\Console\Command\Command
 
 
 
+	protected function execute(InputInterface $input, OutputInterface $output)
+	{
+		Assert::fail("This command shouldn't have been executed.");
+	}
+
+}
+
+class SameArgsCommandOne extends Symfony\Component\Console\Command\Command
+{
+
+	public function __construct(ArgCommand $argCommand, TypoCommand $typoCommand)
+	{
+		parent::__construct();
+	}
+
+
+	protected function configure()
+	{
+		$this->setName('sameArgsCommand:one');
+	}
+
+
+
+	protected function execute(InputInterface $input, OutputInterface $output)
+	{
+		Assert::fail("This command shouldn't have been executed.");
+	}
+
+}
+
+
+class SameArgsCommandTwo extends Symfony\Component\Console\Command\Command
+{
+
+	public function __construct(ArgCommand $argCommand, TypoCommand $typoCommand)
+	{
+		parent::__construct();
+	}
+
+	protected function configure()
+	{
+		$this->setName('sameArgsCommand:two');
+	}
+
+
+
 	protected function execute(InputInterface $input, OutputInterface $output)
 	{
 		Assert::fail("This command shouldn't have been executed.");

tests/KdybyTests/Console/config/input-errors.neon

@@ -6,3 +6,9 @@ console:
 		- KdybyTests\Console\NamespaceAmbiguousCommand1
 		- KdybyTests\Console\NamespaceAmbiguousCommand2
 		- KdybyTests\Console\ArgCommand
+		- KdybyTests\Console\SameArgsCommandOne(@service1, @service2)
+		- KdybyTests\Console\SameArgsCommandTwo(@service1, @service2)
+
+services:
+	service1: KdybyTests\Console\ArgCommand
+	service2: KdybyTests\Console\TypoCommand