HasTranslations::getTranslatedLocales() - Code Metrics - Inspection of "Add getAllTranslations()" - spatie/laravel-translatable - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#108)

unknown

created 2018-03-07 22:19 UTC

HasTranslations::getTranslatedLocales() A

↳ Parent: HasTranslations

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes

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

<?php

namespace Spatie\Translatable;

use Illuminate\Support\Str;
use Spatie\Translatable\Events\TranslationHasBeenSet;
use Spatie\Translatable\Exceptions\AttributeIsNotTranslatable;

trait HasTranslations
{
    /**
     * @param string $key
     *
     * @return mixed
     */
    public function getAttributeValue($key)
    {
        if (!$this->isTranslatableAttribute($key)) {
            return parent::getAttributeValue($key);
        }

        return $this->getTranslation($key, $this->getLocale());
    }

    /**
     * Set a given attribute on the model.
     *
     * @param string $key
     * @param mixed  $value
     *
     * @return $this
     */
    public function setAttribute($key, $value)
    {
        // pass arrays and untranslatable attributes to the parent method
        if (!$this->isTranslatableAttribute($key) || is_array($value)) {
            return parent::setAttribute($key, $value);
        }
        // if the attribute is translatable and not already translated (=array),
        // set a translation for the current app locale
        return $this->setTranslation($key, $this->getLocale(), $value);
    }

    /**
     * @param string $key
     * @param string $locale
     *
     * @return mixed
     */
    public function translate(string $key, string $locale = '')
    {
        return $this->getTranslation($key, $locale);
    }

    /***
     * @param string $key
     * @param string $locale
     * @param bool $useFallbackLocale
     *
     * @return mixed
     */
    public function getTranslation(string $key, string $locale, bool $useFallbackLocale = true)
    {
        $locale = $this->normalizeLocale($key, $locale, $useFallbackLocale);

        $translations = $this->getTranslations($key);

        $translation = $translations[$locale] ?? '';

        if ($this->hasGetMutator($key)) {
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            return $this->mutateAttribute($key, $translation);
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
        }

        return $translation;
    }

    public function getTranslationWithFallback(string $key, string $locale)
    {
        return $this->getTranslation($key, $locale, true);
    }

    public function getTranslationWithoutFallback(string $key, string $locale)
    {
        return $this->getTranslation($key, $locale, false);
    }

    public function getTranslations($key) : array
    {
        $this->guardAgainstUntranslatableAttribute($key);

        return json_decode($this->getAttributes()[$key] ?? '' ?: '{}', true) ?: [];
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
    }

    public function getAllTranslations() : array
    {
        $translations = [];

        foreach ($this->getTranslatableAttributes() as $attribute) {
            $this->guardAgainstUntranslatableAttribute($attribute);
            $translations[$attribute] = json_decode($this->getAttributes()[$attribute] ?? '' ?: '{}') ?: [];
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
        }

        return json_decode(json_encode($translations), true);
    }

    /**
     * @param string $key
     * @param string $locale
     * @param $value
     *
     * @return $this
     */
    public function setTranslation(string $key, string $locale, $value)
    {
        $this->guardAgainstUntranslatableAttribute($key);

        $translations = $this->getTranslations($key);

        $oldValue = $translations[$locale] ?? '';

        if ($this->hasSetMutator($key)) {
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            $method = 'set'.Str::studly($key).'Attribute';
            $this->{$method}($value, $locale);
            $value = $this->attributes[$key];
class MyClass { }

$x = new MyClass();
$x->foo = true;
        }

        $translations[$locale] = $value;

        $this->attributes[$key] = $this->asJson($translations);
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}

        event(new TranslationHasBeenSet($this, $key, $locale, $oldValue, $value));

        return $this;
    }

    /**
     * @param string $key
     * @param array  $translations
     *
     * @return $this
     */
    public function setTranslations(string $key, array $translations)
    {
        $this->guardAgainstUntranslatableAttribute($key);

        foreach ($translations as $locale => $translation) {
            $this->setTranslation($key, $locale, $translation);
        }

        return $this;
    }

    /**
     * @param string $key
     * @param string $locale
     *
     * @return $this
     */
    public function forgetTranslation(string $key, string $locale)
    {
        $translations = $this->getTranslations($key);

        unset($translations[$locale]);

        $this->setAttribute($key, $translations);

        return $this;
    }

    public function forgetAllTranslations(string $locale)
    {
        collect($this->getTranslatableAttributes())->each(function (string $attribute) use ($locale) {
            $this->forgetTranslation($attribute, $locale);
        });
    }

    public function getTranslatedLocales(string $key) : array
    {
        return array_keys($this->getTranslations($key));
    }

    public function isTranslatableAttribute(string $key) : bool
    {
        return in_array($key, $this->getTranslatableAttributes());
    }

    protected function guardAgainstUntranslatableAttribute(string $key)
    {
        if (!$this->isTranslatableAttribute($key)) {
            throw AttributeIsNotTranslatable::make($key, $this);
        }
    }

    protected function normalizeLocale(string $key, string $locale, bool $useFallbackLocale) : string
    {
        if (in_array($locale, $this->getTranslatedLocales($key))) {
            return $locale;
        }

        if (!$useFallbackLocale) {
            return $locale;
        }

        if (!is_null($fallbackLocale = config('translatable.fallback_locale'))) {
            return $fallbackLocale;
        }

        return $locale;
    }

    protected function getLocale() : string
    {
        return config('app.locale');
    }

    public function getTranslatableAttributes() : array
    {
        return is_array($this->translatable)
class MyClass { }

$x = new MyClass();
$x->foo = true;
            ? $this->translatable
            : [];
    }

    public function getCasts() : array
    {
        return array_merge(
            parent::getCasts(),
            array_fill_keys($this->getTranslatableAttributes(), 'array')
        );
    }
}


1			<?php
2
3			namespace Spatie\Translatable;
4
5			use Illuminate\Support\Str;
6			use Spatie\Translatable\Events\TranslationHasBeenSet;
7			use Spatie\Translatable\Exceptions\AttributeIsNotTranslatable;
8
9			trait HasTranslations
10			{
11			/**
12			* @param string $key
13			*
14			* @return mixed
15			*/
16			public function getAttributeValue($key)
17			{
18			if (!$this->isTranslatableAttribute($key)) {
19			return parent::getAttributeValue($key);
20			}
21
22			return $this->getTranslation($key, $this->getLocale());
23			}
24
25			/**
26			* Set a given attribute on the model.
27			*
28			* @param string $key
29			* @param mixed $value
30			*
31			* @return $this
32			*/
33			public function setAttribute($key, $value)
34			{
35			// pass arrays and untranslatable attributes to the parent method
36			if (!$this->isTranslatableAttribute($key) \|\| is_array($value)) {
37			return parent::setAttribute($key, $value);
38			}
39			// if the attribute is translatable and not already translated (=array),
40			// set a translation for the current app locale
41			return $this->setTranslation($key, $this->getLocale(), $value);
42			}
43
44			/**
45			* @param string $key
46			* @param string $locale
47			*
48			* @return mixed
49			*/
50			public function translate(string $key, string $locale = '')
51			{
52			return $this->getTranslation($key, $locale);
53			}
54
55			/***
56			* @param string $key
57			* @param string $locale
58			* @param bool $useFallbackLocale
59			*
60			* @return mixed
61			*/
62			public function getTranslation(string $key, string $locale, bool $useFallbackLocale = true)
63			{
64			$locale = $this->normalizeLocale($key, $locale, $useFallbackLocale);
65
66			$translations = $this->getTranslations($key);
67
68			$translation = $translations[$locale] ?? '';
69
70			if ($this->hasGetMutator($key)) {
			0 ignored issues – show Bug introduced 2016-04-11 14:45 UTC by Report Bug Copy Issue Report It seems like `hasGetMutator()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
71			return $this->mutateAttribute($key, $translation);
			0 ignored issues – show Bug introduced 2016-04-11 14:45 UTC by Report Bug Copy Issue Report It seems like `mutateAttribute()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
72			}
73
74			return $translation;
75			}
76
77			public function getTranslationWithFallback(string $key, string $locale)
78			{
79			return $this->getTranslation($key, $locale, true);
80			}
81
82			public function getTranslationWithoutFallback(string $key, string $locale)
83			{
84			return $this->getTranslation($key, $locale, false);
85			}
86
87			public function getTranslations($key) : array
88			{
89			$this->guardAgainstUntranslatableAttribute($key);
90
91			return json_decode($this->getAttributes()[$key] ?? '' ?: '{}', true) ?: [];
			0 ignored issues – show Bug introduced 2016-04-07 22:25 UTC by Report Bug Copy Issue Report It seems like `getAttributes()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
92			}
93
94			public function getAllTranslations() : array
95			{
96			$translations = [];
97
98			foreach ($this->getTranslatableAttributes() as $attribute) {
99			$this->guardAgainstUntranslatableAttribute($attribute);
100			$translations[$attribute] = json_decode($this->getAttributes()[$attribute] ?? '' ?: '{}') ?: [];
			0 ignored issues – show Bug introduced 2018-03-07 22:21 UTC by Report Bug Copy Issue Report It seems like `getAttributes()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
101			}
102
103			return json_decode(json_encode($translations), true);
104			}
105
106			/**
107			* @param string $key
108			* @param string $locale
109			* @param $value
110			*
111			* @return $this
112			*/
113			public function setTranslation(string $key, string $locale, $value)
114			{
115			$this->guardAgainstUntranslatableAttribute($key);
116
117			$translations = $this->getTranslations($key);
118
119			$oldValue = $translations[$locale] ?? '';
120
121			if ($this->hasSetMutator($key)) {
			0 ignored issues – show Bug introduced 2016-04-11 14:45 UTC by Report Bug Copy Issue Report It seems like `hasSetMutator()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
122			$method = 'set'.Str::studly($key).'Attribute';
123			$this->{$method}($value, $locale);
124			$value = $this->attributes[$key];
			0 ignored issues – show Bug introduced 2017-12-19 23:18 UTC by Report Bug Copy Issue Report The property `attributes` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history...
125			}
126
127			$translations[$locale] = $value;
128
129			$this->attributes[$key] = $this->asJson($translations);
			0 ignored issues – show Bug introduced 2016-04-11 14:45 UTC by Report Bug Copy Issue Report It seems like `asJson()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
130
131			event(new TranslationHasBeenSet($this, $key, $locale, $oldValue, $value));
132
133			return $this;
134			}
135
136			/**
137			* @param string $key
138			* @param array $translations
139			*
140			* @return $this
141			*/
142			public function setTranslations(string $key, array $translations)
143			{
144			$this->guardAgainstUntranslatableAttribute($key);
145
146			foreach ($translations as $locale => $translation) {
147			$this->setTranslation($key, $locale, $translation);
148			}
149
150			return $this;
151			}
152
153			/**
154			* @param string $key
155			* @param string $locale
156			*
157			* @return $this
158			*/
159			public function forgetTranslation(string $key, string $locale)
160			{
161			$translations = $this->getTranslations($key);
162
163			unset($translations[$locale]);
164
165			$this->setAttribute($key, $translations);
166
167			return $this;
168			}
169
170			public function forgetAllTranslations(string $locale)
171			{
172			collect($this->getTranslatableAttributes())->each(function (string $attribute) use ($locale) {
173			$this->forgetTranslation($attribute, $locale);
174			});
175			}
176
177			public function getTranslatedLocales(string $key) : array
178			{
179			return array_keys($this->getTranslations($key));
180			}
181
182			public function isTranslatableAttribute(string $key) : bool
183			{
184			return in_array($key, $this->getTranslatableAttributes());
185			}
186
187			protected function guardAgainstUntranslatableAttribute(string $key)
188			{
189			if (!$this->isTranslatableAttribute($key)) {
190			throw AttributeIsNotTranslatable::make($key, $this);
191			}
192			}
193
194			protected function normalizeLocale(string $key, string $locale, bool $useFallbackLocale) : string
195			{
196			if (in_array($locale, $this->getTranslatedLocales($key))) {
197			return $locale;
198			}
199
200			if (!$useFallbackLocale) {
201			return $locale;
202			}
203
204			if (!is_null($fallbackLocale = config('translatable.fallback_locale'))) {
205			return $fallbackLocale;
206			}
207
208			return $locale;
209			}
210
211			protected function getLocale() : string
212			{
213			return config('app.locale');
214			}
215
216			public function getTranslatableAttributes() : array
217			{
218			return is_array($this->translatable)
			0 ignored issues – show Bug introduced 2016-04-11 18:01 UTC by Report Bug Copy Issue Report The property `translatable` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history...
219			? $this->translatable
220			: [];
221			}
222
223			public function getCasts() : array
224			{
225			return array_merge(
226			parent::getCasts(),
227			array_fill_keys($this->getTranslatableAttributes(), 'array')
228			);
229			}
230			}
231

spatie / laravel-translatable

Pull Request — master (#108)

HasTranslations::getTranslatedLocales() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like