Merge pull request #112 from phpcr/fix_node_references_command

Fixed behavior of node:references

Author: dantleech

CHANGELOG.md

@@ -1,6 +1,11 @@
 Changelog
 =========
 
+dev-master
+----------
+
+- [node:references] Shows the referencing node paths instead of the referrered-to node path(s)
+
 alpha-6
 -------
 

features/phpcr_node_references.feature

@@ -13,23 +13,23 @@ Feature: Show node references
         Then the command should not fail
         And I should see a table containing the following rows:
             | Type | Property | Node Path                                                      |
-            | weak | ref2     | /tests_general_base/idExample/jcr:content/weakreference_target |
-            | weak | ref1     | /tests_general_base/idExample/jcr:content/weakreference_target |
+            | weak | ref2     | /tests_general_base/idExample/jcr:content/weakreference_source2 |
+            | weak | ref1     | /tests_general_base/idExample/jcr:content/weakreference_source1 |
 
     Scenario: List named weak references
         Given the current node is "/tests_general_base/idExample/jcr:content/weakreference_target"
         And I execute the "node:references . ref2 --no-ansi" command
         Then the command should not fail
         And I should see a table containing the following rows:
             | Type | Property | Node Path                                                      |
-            | weak | ref2     | /tests_general_base/idExample/jcr:content/weakreference_target |
+            | weak | ref2     | /tests_general_base/idExample/jcr:content/weakreference_source2 |
 
     Scenario: List strong references
         Given the current node is "/tests_general_base/idExample"
         And I execute the "node:references . --no-ansi" command
         Then the command should not fail
         And I should see a table containing the following rows:
             | Type   | Property | Node Path                              |
-            | strong | ref      | /tests_general_base/idExample          |
-            | strong | multiref | /tests_general_base/idExample          |
-            |        |          | /tests_general_base/multiValueProperty |
+            | strong | ref      | /tests_general_base/numberPropertyNode/jcr:content |
+            | strong | multiref | /tests_general_base/numberPropertyNode/jcr:content |
+

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

@@ -59,22 +59,12 @@ public function execute(InputInterface $input, OutputInterface $output)
 
         foreach ($references as $type => $typeReferences) {
             foreach ($typeReferences as $property) {
-                if ($property->isMultiple()) {
-                    $nodes = $property->getNode();
-                } else {
-                    $nodes = array($property->getNode());
-                }
-
-                $nodePaths = array();
-
-                foreach ($nodes as $node) {
-                    $nodePaths[] = $node->getPath();
-                }
+                $nodePath = $property->getParent()->getPath();
 
                 $table->addRow(array(
                     $type,
                     $property->getName(),
-                    implode("\n", $nodePaths),
+                    $nodePath
                 ));
             }
         }