Isswp101\Persimmon\Relationship\HasManyRelationship

Complexity

Total Complexity

7

Size/Duplication

Total Lines	63
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	get()	0	7	1
A	find()	0	9	2
A	save()	0	10	3

<?php

namespace Isswp101\Persimmon\Relationship;

use Isswp101\Persimmon\ElasticsearchModel;
use Isswp101\Persimmon\QueryBuilder\Filters\ParentFilter;
use Isswp101\Persimmon\QueryBuilder\QueryBuilder;

class HasManyRelationship
{
    /**
     * @var ElasticsearchModel
     */
    protected $parent;

    /**
     * @var ElasticsearchModel
     */
    protected $childClassName;

    function __construct(ElasticsearchModel $parent, $childClassName)

It is generally recommended to explicitly declare the visibility for methods.

    {
        $this->parent = $parent;
        $this->childClassName = $childClassName;
    }

    /**
     * Find all children.
     *
     * @return ElasticsearchModel[]
     */
    public function get()
    {
        $child = $this->childClassName;
        $query = new QueryBuilder();
        $query->filter(new ParentFilter($this->parent->getId()));
        return $child::search($query);
    }

    /**
     * Find document by id.
     *
     * @param mixed $id
     * @return ElasticsearchModel
     */
    public function find($id)
    {
        $child = $this->childClassName;
        $model = $child::findWithParentId($id, $this->parent->getId());
        if ($model) {
            $model->setParent($this->parent);
        }
        return $model;
    }

    /**
     * Save children.
     *
     * @param ElasticsearchModel|ElasticsearchModel[] $child
     */
    public function save($child)
    {
        /** @var ElasticsearchModel[] $children */
        $children = !is_array($child) ? [$child] : $child;
        // @TODO: use bulk if count($children) > 1

39% of this comment could be valid code. Did you maybe forget this after debugging?

        foreach ($children as $child) {
            $child->setParent($this->parent);
            $child->save();
        }
    }
}