blast-project / CoreBundle

src/Admin/Traits/ManyToManyManager.php

<?php

/*
 * This file is part of the Blast Project package.
 *
 * Copyright (C) 2015-2017 Libre Informatique
 *
 * This file is licenced under the GNU LGPL v3.
 * For the full copyright and license information, please view the LICENSE.md
 * file that was distributed with this source code.
 */

namespace Blast\CoreBundle\Admin\Traits;

/**
 * This trait is used to delete many-to-many associated entities when the current
 * entity is not on the owning side.
 */
trait ManyToManyManager
{
    protected $manyToManyCollections = [];

    /**
     * @return array
     * */
    public function getManyToManyCollections()
    {
        return $this->manyToManyCollections;
    }

    /**
     * @param string|array $collections
     *
     * @return self
     */
    public function addManyToManyCollections($collections)
    {
        if (!is_array($collections)) {
            $collections = [$collections];
        }
        $this->manyToManyCollections = array_merge($this->manyToManyCollections, $collections);

        return $this;
    }

    /**
     * Delete many-to-many associated entities when the current entity is not on the owning side
     * This is called by the PreEvents::prePersistOrUpdate() method.
     *
     * @param object $object
     */
    protected function preUpdateManyToManyManager($object)
    {
        $this->configureManyToManyManager();

        $rc = new \ReflectionClass($this->getClass());

It seems like getClass() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        $remove_method = 'remove' . ucfirst($rc->getShortName());

        foreach ($this->manyToManyCollections as $fieldname) {
            $get_method = 'get' . ucfirst($fieldname);
            $orig_collection = $object->$get_method()->getSnapshot();
            $new_collection = $object->$get_method();
            foreach ($orig_collection as $entity) {
                if (!$new_collection->contains($entity)) {
                    $entity->$remove_method($object);
                    $this->getModelManager()->update($entity);

It seems like getModelManager() must be provided by classes using this trait. How about adding it as abstract method to this trait?

                }
            }
        }
    }

    private function configureManyToManyManager()

This method seems to be duplicated in your project.

    {
        $librinfo = $this->getConfigurationPool()->getContainer()->getParameter('blast');

It seems like getConfigurationPool() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        $key = 'manyToMany'; // name of the key in the librinfo.yml

        // merge configuration/parameters
        foreach ($this->getCurrentComposition() as $class) {

It seems like getCurrentComposition() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            if (isset($librinfo[$class])
          && isset($librinfo[$class]['manage'])
          && isset($librinfo[$class]['manage'][$key])) {
                if (!is_array($librinfo[$class]['manage'][$key])) {
                    $librinfo[$class]['manage'][$key] = [$librinfo[$class]['manage'][$key]];
                }
                $this->addManyToManyCollections($librinfo[$class]['manage'][$key]);
            }
        }

        return $this;
    }
}