[orm] remove childCount field

Author: Daria Prusova

library/orm/collection/BaseHierarchicCollection.php

@@ -18,34 +18,12 @@
 use umi\orm\metadata\field\relation\BelongsToRelationField;
 use umi\orm\metadata\field\special\MaterializedPathField;
 use umi\orm\object\IHierarchicObject;
-use umi\orm\object\IObject;
-use umi\orm\object\property\calculable\ICounterProperty;
 
 /**
  * Базовый класс иерархической коллекции
  */
 abstract class BaseHierarchicCollection extends BaseCollection implements IHierarchicCollection
 {
-    /**
-     * {@inheritdoc}
-     */
-    public function delete(IObject $object)
-    {
-        parent::delete($object);
-        /**
-         * @var IHierarchicObject $object
-         */
-        if (null != ($parent = $object->getParent())) {
-            /**
-             * @var ICounterProperty $childCount
-             */
-            $childCount = $parent->getProperty(IHierarchicObject::FIELD_CHILD_COUNT);
-            $childCount->decrement();
-        }
-
-        return $this;
-    }
-
     /**
      * {@inheritdoc}
      */
@@ -92,14 +70,6 @@ public function getHierarchyLevelField()
         return $this->getRequiredField(IHierarchicObject::FIELD_HIERARCHY_LEVEL);
     }
 
-    /**
-     * {@inheritdoc}
-     */
-    public function getHierarchyChildCountField()
-    {
-        return $this->getRequiredField(IHierarchicObject::FIELD_CHILD_COUNT);
-    }
-
     /**
      * {@inheritdoc}
      */
@@ -336,12 +306,6 @@ function () use ($object, $branch, $previousSibling) {
                     ];
 
                     if ($object->getParent() !== $branch) {
-                        if (null != ($parent = $object->getParent())) {
-                            $builders[] = $this->buildUpdateChildCountQuery($parent, $this, -1);
-                        }
-                        if ($branch) {
-                            $builders[] = $this->buildUpdateChildCountQuery($branch, $this, 1);
-                        }
                         $builders[] = $this->buildUpdateHierarchicPropertiesQueryForMovedObject(
                             $object,
                             $this,
@@ -551,47 +515,6 @@ protected function buildUpdateHierarchicPropertiesQueryForMovedObjectChildren(
 
     }
 
-    /**
-     * Возвращает запрос на изменения количества детей.
-     * @param IHierarchicObject $object объект, у которого меняется количество детей
-     * @param IHierarchicCollection $collection коллекция, для которой формируется запрос
-     * @param int $childCountModifier число, на которое увеличивается или уменьшается количество детей
-     * @return IUpdateBuilder
-     */
-    protected function buildUpdateChildCountQuery(
-        IHierarchicObject $object,
-        IHierarchicCollection $collection,
-        $childCountModifier
-    )
-    {
-
-        $dataSource = $collection
-            ->getMetadata()
-            ->getCollectionDataSource();
-        $childCountField = $collection->getHierarchyChildCountField();
-        $idField = $collection->getIdentifyField();
-
-        /**
-         * @var $update IUpdateBuilder
-         */
-        $update = $dataSource->update();
-
-        $modifierExpression = $update
-                ->getConnection()
-                ->quoteIdentifier($childCountField->getColumnName()) . ' + (' . $childCountModifier . ')';
-        $update
-            ->set($childCountField->getColumnName())
-            ->bindExpression(':' . $childCountField->getColumnName(), $modifierExpression);
-
-        $update
-            ->where()
-            ->expr($idField->getColumnName(), '=', ':' . $idField->getName())
-            ->bindValue(':' . $idField->getName(), $object->getId(), $idField->getDataType());
-
-        return $update;
-
-    }
-
     /**
      * Возвращает запрос на изменение порядка в иерархии перемещаемого объекта
      * @param IHierarchicObject $object перемещаемый объект

library/orm/collection/CommonHierarchy.php

@@ -87,15 +87,6 @@ protected function persistMovedObjects(
         }
 
         if ($object->getParent() !== $branch) {
-
-            if (null != ($parent = $object->getParent())) {
-                $builders[] = $this->buildUpdateChildCountQuery($parent, $this, -1);
-                $builders[] = $this->buildUpdateChildCountQuery($parent, $parent->getCollection(), -1);
-            }
-            if ($branch) {
-                $builders[] = $this->buildUpdateChildCountQuery($branch, $this, 1);
-                $builders[] = $this->buildUpdateChildCountQuery($branch, $branch->getCollection(), 1);
-            }
             $builders[] = $this->buildUpdateHierarchicPropertiesQueryForMovedObject($object, $this, $branch);
             $builders[] = $this->buildUpdateHierarchicPropertiesQueryForMovedObject(
                 $object,

library/orm/collection/IHierarchicCollection.php

@@ -89,13 +89,6 @@ public function getHierarchyOrderField();
      */
     public function getHierarchyLevelField();
 
-    /**
-     * Возвращает поле, которое используется у базового типа коллекции для хранения информации о количестве детей
-     * @throws NonexistentEntityException если такого поля не существует
-     * @return IField
-     */
-    public function getHierarchyChildCountField();
-
     /**
      * Возвращает поле, которое используется у базового типа коллекции для хранения последней части ЧПУ
      * @throws NonexistentEntityException если такого поля не существует

library/orm/collection/SimpleHierarchicCollection.php

@@ -12,7 +12,6 @@
 use umi\orm\exception\NonexistentEntityException;
 use umi\orm\metadata\IObjectType;
 use umi\orm\object\IHierarchicObject;
-use umi\orm\object\property\calculable\ICounterProperty;
 
 /**
  * Простая коллекция иерархических объектов.
@@ -38,14 +37,10 @@ public function add($slug, $typeName = IObjectType::BASE, IHierarchicObject $bra
             ->registerNewObject($this, $this->metadata->getType($typeName), $guid);
         $object->getProperty(IHierarchicObject::FIELD_SLUG)
             ->setValue($slug);
+
         if ($branch) {
             $object->getProperty(IHierarchicObject::FIELD_PARENT)
                 ->setValue($branch);
-            /**
-             * @var ICounterProperty $childCountProperty
-             */
-            $childCountProperty = $branch->getProperty(IHierarchicObject::FIELD_CHILD_COUNT);
-            $childCountProperty->increment();
         }
 
         return $object;

library/orm/object/HierarchicObject.php

@@ -61,15 +61,6 @@ public function getLevel()
             ->getValue();
     }
 
-    /**
-     * {@inheritdoc}
-     */
-    public function getChildCount()
-    {
-        return $this->getProperty(self::FIELD_CHILD_COUNT)
-            ->getValue();
-    }
-
     /**
      * {@inheritdoc}
      */

library/orm/object/IHierarchicObject.php

@@ -31,10 +31,6 @@ interface IHierarchicObject extends IObject
      * Имя поля, используемого для хранения информации о иерархическом порядке
      */
     const FIELD_ORDER = 'order';
-    /**
-     * Имя поля, используемого для хранения количества непосредственных детей иерархических объектов
-     */
-    const FIELD_CHILD_COUNT = 'childCount';
     /**
      * Имя поля, используемого для хранения последней части ЧПУ
      */
@@ -68,12 +64,6 @@ public function getOrder();
      */
     public function getLevel();
 
-    /**
-     * Возвращает количество непостредственных детей
-     * @return int
-     */
-    public function getChildCount();
-
     /**
      * Возвращает URI
      * @return string

library/orm/object/Object.php

@@ -17,7 +17,6 @@
 use umi\orm\collection\ICollection;
 use umi\orm\exception\InvalidArgumentException;
 use umi\orm\exception\NonexistentEntityException;
-use umi\orm\exception\NotAllowedOperationException;
 use umi\orm\exception\ReadOnlyEntityException;
 use umi\orm\exception\RuntimeException;
 use umi\orm\manager\IObjectManagerAware;

library/validation/ValidatorCollection.php

@@ -9,8 +9,6 @@
 
 namespace umi\validation;
 
-use Traversable;
-
 /**
  * Класс коллекции валидаторов.
  * По очереди валидирует значение каждым валидатором.

tests/utest/form/unit/element/CheckboxElementTest.php

@@ -10,7 +10,6 @@
 namespace utest\form\unit\element;
 
 use umi\form\element\Checkbox;
-use umi\form\Form;
 
 /**
  * Тесты элемента формы - Флаг

tests/utest/monolog/unit/MonologToolsTest.php

@@ -10,7 +10,6 @@
 
 use Monolog\Handler\FirePHPHandler;
 use Monolog\Handler\StreamHandler;
-use Monolog\Handler\SwiftMailerHandler;
 use Monolog\Logger;
 use umi\extension\monolog\exception\RuntimeException;
 use umi\extension\monolog\toolbox\MonologTools;