bugfix: Node mixin remove does not accept path

Author: dantleech

CHANGELOG.md

@@ -1,6 +1,13 @@
 Changelog
 =========
 
+dev-master
+----------
+
+### Bug fixes
+
+- [node] Mixin remove command does not accept a path
+
 alpha-5
 -------
 

features/phpcr_node_mixin_remove.feature

@@ -10,7 +10,7 @@ Feature: Remove mixin to the current node
 
     Scenario: Remove a mixin to the current node
         Given the current node is "/tests_general_base"
-        And I execute the "node:mixin:remove mix:versionable --no-ansi" command
+        And I execute the "node:mixin:remove . mix:versionable --no-ansi" command
         And I save the session
         Then the command should not fail
         And the node at "/tests_general_base" should not have the mixin "mix:versionable"

src/PHPCR/Shell/Console/Command/Phpcr/NodeMixinRemoveCommand.php

@@ -13,6 +13,7 @@ protected function configure()
     {
         $this->setName('node:mixin:remove');
         $this->setDescription('Remove the named mixin to the current node');
+        $this->addArgument('path', InputArgument::REQUIRED, 'Path of node');
         $this->addArgument('mixinName', InputArgument::REQUIRED, 'The name of the mixin node type to be removeed');
         $this->setHelp(<<<HERE
 Removes the specified mixin node type from this node and removes
@@ -28,7 +29,8 @@ public function execute(InputInterface $input, OutputInterface $output)
     {
         $session = $this->getHelper('phpcr')->getSession();
         $mixinName = $input->getArgument('mixinName');
-        $currentNode = $session->getCurrentNode();
+        $path = $input->getArgument('path');
+        $currentNode = $session->getNodeByPathOrIdentifier($path);
         $currentNode->removeMixin($mixinName);
     }
 }