Workspace.removeItem() implementation. Refs midgardprojectgh-151

piotras · Jan 24, 2013 · 1182586 · 1182586
1 parent a10d551
commit 1182586
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 3 deletions.
diff --git a/src/Midgard/PHPCR/Item.php b/src/Midgard/PHPCR/Item.php
@@ -283,7 +283,7 @@ abstract protected function populateParent();
     public function getPath()
     {
         if ($this->is_removed) {
-            throw new \PHPCR\RepositoryException("Can not get path of removed item");
+            throw new \PHPCR\InvalidItemStateException("Can not get path of removed item");
         }
 
         return $this->getPathUnchecked();

diff --git a/src/Midgard/PHPCR/Property.php b/src/Midgard/PHPCR/Property.php
@@ -219,7 +219,7 @@ public function addValue($value)
     public function getValue()
     {
         if ($this->is_purged === true || $this->is_removed === true) {
-            throw new \PHPCR\RepositoryException("Can not get value of purged property.");
+            throw new \PHPCR\InvalidItemStateException("Can not get value of purged property.");
         }
 
         $type = $this->getType();

diff --git a/src/Midgard/PHPCR/Workspace.php b/src/Midgard/PHPCR/Workspace.php
@@ -335,7 +335,14 @@ public function deleteWorkspace($name)
 
     public function removeItem($absPath)
     {
-
+        $item = $this->session->getItem($absPath);
+        if ($item instanceof \PHPCR\PropertyInterface) {
+            $item->remove();
+            $item->removeMidgard2Property();
+        } else {
+            $item->removeMidgard2Node();
+            $item->remove();
+        }
     }
 
     public function getRepositoryManager()