LaravelFreelancerNL\Aranguent\Eloquent\Concerns\HasAttributes

Complexity

Total Complexity

3

Size/Duplication

Total Lines	30
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	fromJson()	0	8	2
A	isJsonCastable()	0	3	1

<?php

declare(strict_types=1);

namespace LaravelFreelancerNL\Aranguent\Eloquent\Concerns;

trait HasAttributes
{
    /**
     * Determine whether a value is JSON castable for inbound manipulation.
     *
     * @param  string  $key
     * @return bool
     */
    protected function isJsonCastable($key)
    {
        return $this->hasCast($key, ['encrypted:array', 'encrypted:collection', 'encrypted:json', 'encrypted:object']);

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

    }

    /**
     * Decode the given JSON back into an array or object.
     *
     * @param  string  $value
     * @param  bool  $asObject
     * @return mixed
     *
     * @SuppressWarnings(PHPMD.BooleanArgumentFlag)
     */
    public function fromJson($value, $asObject = false)
    {
        // As data is stored as json in ArangoDB we don't have to decode it here.
        if ($asObject) {
            return (object) $value;
        }

        return (array) $value;
    }
}