CompositeBelongsToMany::normalizeIds() - Code Metrics - Inspection of "additional fixes hex mode support for belongsToMan..." - maksimru/composite-primary-keys - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( c67049...162243 )

by Maksim

created 2019-08-29 19:44 UTC

CompositeBelongsToMany::normalizeIds() A

↳ Parent: CompositeBelongsToMany

Complexity

Conditions	3
Paths	4

Size

Total Lines	4
Code Lines	3

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
eloc	3
c	0
b	0
f	0
dl	0
loc	4
rs	10
cc	3
nc	4
nop	1

<?php

namespace MaksimM\CompositePrimaryKeys\Eloquent\Relationships;

use Illuminate\Database\Eloquent\Collection;
use Illuminate\Database\Eloquent\Model;
use Illuminate\Database\Eloquent\Relations\BelongsToMany;
use Illuminate\Support\Collection as BaseCollection;
use MaksimM\CompositePrimaryKeys\Http\Traits\CompositeRelationships;

class CompositeBelongsToMany extends BelongsToMany
{
    use CompositeRelationships;

    private function normalizeIds($ids){
        return $this->getQuery()->getModel()->hexBinaryColumns($this->relatedKey) ? (is_array($ids) ? array_map(function($id){
            return $this->getQuery()->getModel()->recoverBinaryKey($this->relatedKey, $id);
        }, $ids) : $this->getQuery()->getModel()->recoverBinaryKey($this->relatedKey, $ids)) : $ids;
    }

    /**
     * Get all of the IDs from the given mixed value.
     *
     * @param mixed $value
     *
     * @return array
     */
    protected function parseIds($value)
    {
        if ($value instanceof Model) {
            return [$this->executeWithinOptionalBinaryTransformation(function () use ($value) {
                return $value->{$this->relatedKey};
            }, $value)];
        }

        if ($value instanceof Collection) {
            return $this->normalizeIds($value->pluck($this->relatedKey)->all());
        }

        if ($value instanceof BaseCollection) {
            return $this->normalizeIds($value->toArray());
        }

        return ctype_xdigit($value) ? $this->normalizeIds([$value]) : [$value];
    }

    /**
     * Get the ID from the given mixed value.
     *
     * @param mixed $value
     *
     * @return mixed
     */
    protected function parseId($value)
    {
        return $value instanceof Model ? $this->executeWithinOptionalBinaryTransformation(function () use ($value) {
            $value->{$this->relatedKey};
        }, $value) : $value;
    }

    /**
     * Create a new query builder for the pivot table.
     *
     * @return \Illuminate\Database\Query\Builder
     */
    protected function newPivotQuery()
    {
        $query = $this->newPivotStatement();

        foreach ($this->pivotWheres as $arguments) {
            call_user_func_array([$query, 'where'], $arguments);
        }

        foreach ($this->pivotWhereIns as $arguments) {
            call_user_func_array([$query, 'whereIn'], $arguments);
        }

        return $this->executeWithinOptionalBinaryTransformation(function () use ($query) {
            return $query->where($this->foreignPivotKey, $this->parent->{$this->parentKey});
        }, $this->parent);
    }

    /**
     * Create a new pivot attachment record.
     *
     * @param  int   $id
     * @param  bool  $timed
     * @return array
     */
    protected function baseAttachRecord($id, $timed)
    {
        $record[$this->relatedPivotKey] = $id;


        $record[$this->foreignPivotKey] = $this->executeWithinOptionalBinaryTransformation(function () {
            return $this->parent->{$this->parentKey};
        }, $this->parent);

        // If the record needs to have creation and update timestamps, we will make
        // them by calling the parent model's "freshTimestamp" method which will
        // provide us with a fresh timestamp in this model's preferred format.
        if ($timed) {
            $record = $this->addTimestampsToAttachment($record);
        }

        foreach ($this->pivotValues as $value) {
            $record[$value['column']] = $value['value'];
        }

        return $record;
    }
}


1			<?php
2
3			namespace MaksimM\CompositePrimaryKeys\Eloquent\Relationships;
4
5			use Illuminate\Database\Eloquent\Collection;
6			use Illuminate\Database\Eloquent\Model;
7			use Illuminate\Database\Eloquent\Relations\BelongsToMany;
8			use Illuminate\Support\Collection as BaseCollection;
9			use MaksimM\CompositePrimaryKeys\Http\Traits\CompositeRelationships;
10
11			class CompositeBelongsToMany extends BelongsToMany
12			{
13			use CompositeRelationships;
14
15			private function normalizeIds($ids){
16			return $this->getQuery()->getModel()->hexBinaryColumns($this->relatedKey) ? (is_array($ids) ? array_map(function($id){
17			return $this->getQuery()->getModel()->recoverBinaryKey($this->relatedKey, $id);
18			}, $ids) : $this->getQuery()->getModel()->recoverBinaryKey($this->relatedKey, $ids)) : $ids;
19			}
20
21			/**
22			* Get all of the IDs from the given mixed value.
23			*
24			* @param mixed $value
25			*
26			* @return array
27			*/
28			protected function parseIds($value)
29			{
30			if ($value instanceof Model) {
31			return [$this->executeWithinOptionalBinaryTransformation(function () use ($value) {
32			return $value->{$this->relatedKey};
33			}, $value)];
34			}
35
36			if ($value instanceof Collection) {
37			return $this->normalizeIds($value->pluck($this->relatedKey)->all());
38			}
39
40			if ($value instanceof BaseCollection) {
41			return $this->normalizeIds($value->toArray());
42			}
43
44			return ctype_xdigit($value) ? $this->normalizeIds([$value]) : [$value];
45			}
46
47			/**
48			* Get the ID from the given mixed value.
49			*
50			* @param mixed $value
51			*
52			* @return mixed
53			*/
54			protected function parseId($value)
55			{
56			return $value instanceof Model ? $this->executeWithinOptionalBinaryTransformation(function () use ($value) {
57			$value->{$this->relatedKey};
58			}, $value) : $value;
59			}
60
61			/**
62			* Create a new query builder for the pivot table.
63			*
64			* @return \Illuminate\Database\Query\Builder
65			*/
66			protected function newPivotQuery()
67			{
68			$query = $this->newPivotStatement();
69
70			foreach ($this->pivotWheres as $arguments) {
71			call_user_func_array([$query, 'where'], $arguments);
72			}
73
74			foreach ($this->pivotWhereIns as $arguments) {
75			call_user_func_array([$query, 'whereIn'], $arguments);
76			}
77
78			return $this->executeWithinOptionalBinaryTransformation(function () use ($query) {
79			return $query->where($this->foreignPivotKey, $this->parent->{$this->parentKey});
80			}, $this->parent);
81			}
82
83			/**
84			* Create a new pivot attachment record.
85			*
86			* @param int $id
87			* @param bool $timed
88			* @return array
89			*/
90			protected function baseAttachRecord($id, $timed)
91			{
92			$record[$this->relatedPivotKey] = $id;
			0 ignored issues – show Comprehensibility Best Practice introduced 2019-08-29 18:39 UTC by Report Bug Copy Issue Report `$record` was never initialized. Although not strictly required by PHP, it is generally a good practice to add `$record = array();` before regardless. Loading history...
93
94			$record[$this->foreignPivotKey] = $this->executeWithinOptionalBinaryTransformation(function () {
95			return $this->parent->{$this->parentKey};
96			}, $this->parent);
97
98			// If the record needs to have creation and update timestamps, we will make
99			// them by calling the parent model's "freshTimestamp" method which will
100			// provide us with a fresh timestamp in this model's preferred format.
101			if ($timed) {
102			$record = $this->addTimestampsToAttachment($record);
103			}
104
105			foreach ($this->pivotValues as $value) {
106			$record[$value['column']] = $value['value'];
107			}
108
109			return $record;
110			}
111			}
112

maksimru / composite-primary-keys

Push — master ( c67049...162243 )

CompositeBelongsToMany::normalizeIds() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like