MaksimM\CompositePrimaryKeys\Http\Traits\OptionalBinaryTransformation

Complexity

Total Complexity

17

Size/Duplication

Total Lines	68
Duplicated Lines	0 %

Importance

Changes	3
Bugs	2	Features	0

9 Methods

Rating	Name	Duplication	Size	Complexity
A	hasGetMutator()	0	7	4
A	setMutatedAttributeValue()	0	10	2
A	mutateAttribute()	0	7	2
A	shouldProcessBinaryAttribute()	0	3	2
A	hasSetMutator()	0	7	3
A	disableBinaryMutators()	0	3	1
A	enableBinaryMutators()	0	3	1
A	hexBinaryColumns()	0	3	1
A	isAllowedToMutateBinaryAttributes()	0	3	1

<?php

namespace MaksimM\CompositePrimaryKeys\Http\Traits;

trait OptionalBinaryTransformation
{
    protected $hexBinaryColumns = false;

    private $shouldMutate = true;

    public function hexBinaryColumns()
    {
        return $this->hexBinaryColumns;
    }

    public function disableBinaryMutators()
    {
        $this->shouldMutate = false;
    }

    public function enableBinaryMutators()
    {
        $this->shouldMutate = true;
    }

    protected function isAllowedToMutateBinaryAttributes()
    {
        return $this->shouldMutate;
    }

    private function shouldProcessBinaryAttribute($key)
    {
        return $this->hexBinaryColumns() && in_array($key, $this->getBinaryColumns());

The method getBinaryColumns() does not exist on MaksimM\CompositePrimaryKeys\Http\Traits\OptionalBinaryTransformation. Did you maybe mean hexBinaryColumns()?

    }

    public function hasGetMutator($key)
    {
        if ($this->shouldProcessBinaryAttribute($key) && isset($this->{$key}) && $this->isAllowedToMutateBinaryAttributes()) {
            return true;
        }

        return parent::hasGetMutator($key);
    }

    public function mutateAttribute($key, $value)
    {
        if ($this->shouldProcessBinaryAttribute($key)) {
            return strtoupper(bin2hex($value));
        }

        return parent::mutateAttribute($key, $value);
    }

    public function hasSetMutator($key)
    {
        if ($this->shouldProcessBinaryAttribute($key) && $this->isAllowedToMutateBinaryAttributes()) {
            return true;
        }

        return parent::hasSetMutator($key);
    }

    public function setMutatedAttributeValue($key, $value)
    {
        if ($this->shouldProcessBinaryAttribute($key)) {
            $value = hex2bin($value);
            $this->attributes[$key] = $value;

The property attributes does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

            return $value;
        }

        return parent::setMutatedAttributeValue($key, $value);
    }
}