Merge branch 'enhancement/workspace-import-uuid-behavior'

Author: dbu

src/PHPCR/Util/Console/Command/WorkspaceImportCommand.php

@@ -17,6 +17,13 @@
  */
 class WorkspaceImportCommand extends BaseCommand
 {
+    const UUID_BEHAVIOR = [
+        'new'     => ImportUUIDBehaviorInterface::IMPORT_UUID_CREATE_NEW,
+        'remove'  => ImportUUIDBehaviorInterface::IMPORT_UUID_COLLISION_REMOVE_EXISTING,
+        'replace' => ImportUUIDBehaviorInterface::IMPORT_UUID_COLLISION_REPLACE_EXISTING,
+        'throw'   => ImportUUIDBehaviorInterface::IMPORT_UUID_COLLISION_THROW,
+    ];
+
     /**
      * {@inheritdoc}
      */
@@ -28,6 +35,7 @@ protected function configure()
             ->setName('phpcr:workspace:import')
             ->addArgument('filename', null, 'The xml file to import')
             ->addOption('parentpath', 'p', InputOption::VALUE_OPTIONAL, 'Repository path to the parent where to import the file contents', '/')
+            ->addOption('uuid-behavior', null, InputOption::VALUE_REQUIRED, 'How to handle UUID collisions during the import', 'new')
             ->setDescription('Import xml data into the repository, either in JCR system view format or arbitrary xml')
             ->setHelp(<<<'EOF'
 The <info>import</info> command uses the PHPCR SessionInterface::importXml method
@@ -38,6 +46,17 @@ protected function configure()
 
 If the <info>parentpath</info> option is set, the document is imported to that
 path. Otherwise the document is imported at the repository root.
+
+The optional <info>uuid-behavior</info> option describes how UUIDs should be
+handled. The following options are available:
+
+* <info>new</info> recreate a new uuid for each imported node;
+* <info>remove</info> on collision, remove the old node from the repository and
+  put the imported data in the tree;
+* <info>replace</info> on collision, replace the existing node with the one being
+  imported. All children of the imported node also go to the new path;
+* <info>throw</info> throw an exception on uuid collision.
+
 EOF
             );
     }
@@ -58,11 +77,15 @@ protected function execute(InputInterface $input, OutputInterface $output)
             return 1;
         }
 
-        $session->importXML(
-            $parentPath,
-            $filename,
-            ImportUUIDBehaviorInterface::IMPORT_UUID_CREATE_NEW
-        );
+        $uuidBehavior = $input->getOption('uuid-behavior');
+        if (!array_key_exists($uuidBehavior, self::UUID_BEHAVIOR)) {
+            $output->writeln(sprintf('<error>UUID-Behavior "%s" is not supported</error>', $uuidBehavior));
+            $output->writeln(sprintf('Supported behaviors are %s', implode(', ', array_keys(self::UUID_BEHAVIOR))));
+
+            return 1;
+        }
+
+        $session->importXML($parentPath, $filename, self::UUID_BEHAVIOR[$uuidBehavior]);
         $session->save();
 
         $output->writeln(sprintf(

tests/PHPCR/Tests/Util/Console/Command/WorkspaceImportCommandTest.php

@@ -2,6 +2,7 @@
 
 namespace PHPCR\Tests\Util\Console\Command;
 
+use PHPCR\ImportUUIDBehaviorInterface;
 use PHPCR\RepositoryInterface;
 use PHPCR\Util\Console\Command\WorkspaceImportCommand;
 
@@ -14,7 +15,7 @@ public function setUp()
         $this->application->add(new WorkspaceImportCommand());
     }
 
-    public function testNodeTypeList()
+    public function testImport()
     {
         $this->session->expects($this->once())
             ->method('getRepository')
@@ -26,12 +27,36 @@ public function testNodeTypeList()
             ->will($this->returnValue(true));
 
         $this->session->expects($this->once())
-            ->method('importXml');
+            ->method('importXml')
+            ->with('/', 'test_import.xml', ImportUUIDBehaviorInterface::IMPORT_UUID_CREATE_NEW);
 
         $ct = $this->executeCommand('phpcr:workspace:import', [
             'filename' => 'test_import.xml',
         ]);
 
         $this->assertContains('Successfully imported', $ct->getDisplay());
     }
+
+    public function testImportUuidBehaviorThrow()
+    {
+        $this->session->expects($this->once())
+            ->method('getRepository')
+            ->will($this->returnValue($this->repository));
+
+        $this->repository->expects($this->once())
+            ->method('getDescriptor')
+            ->with(RepositoryInterface::OPTION_XML_IMPORT_SUPPORTED)
+            ->will($this->returnValue(true));
+
+        $this->session->expects($this->once())
+            ->method('importXml')
+            ->with('/', 'test_import.xml', ImportUUIDBehaviorInterface::IMPORT_UUID_COLLISION_THROW);
+
+        $ct = $this->executeCommand('phpcr:workspace:import', [
+            'filename'        => 'test_import.xml',
+            '--uuid-behavior' => 'throw',
+        ]);
+
+        $this->assertContains('Successfully imported', $ct->getDisplay());
+    }
 }