[orm] fix move & change slug

Author: Daria Prusova

library/orm/collection/BaseHierarchicCollection.php

@@ -215,6 +215,14 @@ public function move(
         IHierarchicObject $previousSibling = null
     )
     {
+        if (!$this
+            ->getObjectPersister()
+            ->getIsPersisted()
+        ) {
+            throw new RuntimeException($this->translate(
+                'Cannot move object. Not all objects are persisted. Commit transaction first.'
+            ));
+        }
 
         if (!$this->contains($object)) {
             throw new RuntimeException($this->translate(
@@ -294,6 +302,15 @@ public function move(
      */
     public function changeSlug(IHierarchicObject $object, $slug)
     {
+        if (!$this
+            ->getObjectPersister()
+            ->getIsPersisted()
+        ) {
+            throw new RuntimeException($this->translate(
+                'Cannot change slug for object. Not all objects are persisted. Commit transaction first.'
+            ));
+        }
+
         $branch = $object->getParent();
 
         $baseUri = $branch ? $branch->getURI() . '/' : '//';

tests/utest/orm/func/collection/commonhierarchy/CommonHierarchyChangeSlugTest.php

@@ -117,6 +117,22 @@ public function testChangeSlugPossibility()
             'Ожидается, что невозможно изменить uri объекта, если итоговый uri не уникальный'
         );
 
+        $post2->setValue('title', '1');
+        $e = null;
+        try {
+            $this->hierarchy->changeSlug($post2, 'new_slug');
+        } catch (\Exception $e) {
+        }
+        $this->assertInstanceOf(
+            'umi\orm\exception\RuntimeException',
+            $e,
+            'Ожидается, что невозможно изменить uri объекта, если есть модифицированные объекты'
+        );
+    }
+
+    public function testOutOfDateChangeSlug()
+    {
+        $post2 = $this->postsCollection->get($this->guid2);
         $post2->setVersion(10);
         $e = null;
         try {

tests/utest/orm/func/collection/commonhierarchy/CommonHierarchyMoveTest.php

@@ -193,6 +193,18 @@ public function testImpossibleMove()
             $e,
             'Ожидается, что невозможно переместить объект, если его новый итоговый урл совпадает с уже существующим'
         );
+
+        $this->blog4->setValue('title', 'new_title');
+        $e = null;
+        try {
+            $this->hierarchy->move($this->blog4, $this->blog1);
+        } catch (\Exception $e) {
+        }
+        $this->assertInstanceOf(
+            'umi\orm\exception\RuntimeException',
+            $e,
+            'Ожидается, что невозможно переместить объект и изменить объекты за одну транзакцию'
+        );
     }
 
     public function testMoveFirstWithoutSwitchingTheBranch()