Line - Code Metrics - Inspection of "Update Dimsav dependency to 7.*" - thinktomorrow/squanto - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 35ba31...f8a85f )

by Ben

created 2017-02-24 15:14 UTC

Line A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	145
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	7

Test Coverage

Coverage

63.16%

Importance

Changes	2
Bugs	0	Features	1

Metric	Value
wmc	18
c	2
b	0
f	1
lcom	1
cbo	7
dl	0
loc	145
ccs	24
cts	38
cp	0.6316
rs	10

13 Methods

Rating	Name	Size	Complexity
A	make()	13	1
A	findOrCreateByKey()	8	2
A	saveValue()	6	1
A	removeValue()	6	1
A	getValue()	4	1
A	findValue()	8	2
A	findByKey()	4	1
A	saveSuggestedType()	10	2
A	editInEditor()	4	1
A	editInTextarea()	4	1
A	editInTextinput()	4	2
A	areParagraphsAllowed()	4	1
A	getValuesByLocale()	14	2

<?php

namespace Thinktomorrow\Squanto\Domain;

use Illuminate\Database\Eloquent\Model;
use Dimsav\Translatable\Translatable as BaseTranslatable;
use Illuminate\Support\Collection;
use Illuminate\Support\Facades\DB;

/**
 * Class Line
 *
 * @package Thinktomorrow\Squanto\Domain
 */
class Line extends Model
{
    use BaseTranslatable, Translatable;

    public $table = 'squanto_lines';
    public $translatedAttributes = ['value'];

    /**
     * @param $key
     * @return Line
     */
    public static function make($key)
    {
        $linekey = new LineKey($key);

        $line = new self;
        $line->key = $linekey->get();
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
        $line->label = $linekey->getAsLabel();
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
        $line->page_id = Page::findOrCreateByKey($linekey->getPageKey())->id;
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}

        $line->save();

        return $line;
    }

    /**
     * @param $key
     * @return Line
     */
    public static function findOrCreateByKey($key)
    {
        if ($line = self::findByKey($key)) {
            return $line;
        }

        return self::make($key);
    }

    /**
     * Save a translated value
     *
     * @param $locale
     * @param $value
     * @return $this
     */
    public function saveValue($locale, $value)
    {
        $this->saveTranslation($locale, 'value', $value);

        return $this;
    }

    /**
     * Save a translated value
     *
     * @param $locale
     * @return $this
     */
    public function removeValue($locale)
    {
        $this->removeTranslation($locale);

        return $this;
    }

    /**
     * @param null $locale
     * @param bool $fallback
     * @return string
     */
    public function getValue($locale = null, $fallback = true)
    {
        return $this->getTranslationFor('value', $locale, $fallback);
    }

    public static function findValue($key, $locale)
    {
        if (!$line = self::findByKey($key)) {
            return null;
        }

        return $line->getValue($locale, false);
    }

    /**
     * @param $key
     * @return mixed
     */
    public static function findByKey($key)
    {
        return self::where('key', $key)->first();
    }

    public function saveSuggestedType()
    {
        // Based on first value found we will suggest a type
        if (!$value = $this->getValue()) {
            return;
        }

        $this->type = (new LineType($value))->suggest();
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
        $this->save();
    }

    public function editInEditor()
    {
        return $this->type == LineType::EDITOR;
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
    }

    public function editInTextarea()
    {
        return $this->type == LineType::TEXTAREA;
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
    }

    public function editInTextinput()
    {
        return (!$this->type || $this->type == LineType::TEXT);
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
    }

    public function areParagraphsAllowed()
    {
        return (false !== strpos($this->allowed_html,'<p>'));
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
    }

    /**
     * Get key - value pairs for all lines per locale
     *
     * @param $locale
     * @return mixed
     */
    public static function getValuesByLocale($locale)
    {
        // Since the dimsav translatable model trait injects its behaviour and overwrites our results
        // with the current locale, we will need to fetch results straight from the db instead.
        $lines = DB::table('squanto_lines')->join('squanto_line_translations', 'squanto_lines.id', '=', 'squanto_line_translations.line_id')
            ->select(['squanto_lines.*','squanto_line_translations.locale','squanto_line_translations.value'])
            ->where('squanto_line_translations.locale', $locale)
            ->get();

        // Assert we have a collection
        $lines = $lines instanceof Collection ? $lines : collect($lines);

        return $lines->pluck('value', 'key')->toArray();
    }
}


1		<?php
2
3		namespace Thinktomorrow\Squanto\Domain;
4
5		use Illuminate\Database\Eloquent\Model;
6		use Dimsav\Translatable\Translatable as BaseTranslatable;
7		use Illuminate\Support\Collection;
8		use Illuminate\Support\Facades\DB;
9
10		/**
11		* Class Line
12		*
13		* @package Thinktomorrow\Squanto\Domain
14		*/
15		class Line extends Model
16		{
17		use BaseTranslatable, Translatable;
18
19	36	public $table = 'squanto_lines';
20		public $translatedAttributes = ['value'];
21	36
22		/**
23	36	* @param $key
24	36	* @return Line
25	36	*/
26	36	public static function make($key)
27		{
28	36	$linekey = new LineKey($key);
29
30	36	$line = new self;
31		$line->key = $linekey->get();
		0 ignored issues – show Documentation introduced 2016-10-13 18:48 UTC by Report Bug Copy Issue Report The property `key` does not exist on `object<Thinktomorrow\Squanto\Domain\Line>`. Since you implemented `__set`, maybe consider adding a @property annotation. Since your code implements the magic setter `_set`, this function will be called for any write access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /** * @property int $x * @property int $y * @property string $text */ class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } Since the property has write access only, you can use the @property-write annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history...
32		$line->label = $linekey->getAsLabel();
		0 ignored issues – show Documentation introduced 2016-10-13 18:48 UTC by Report Bug Copy Issue Report The property `label` does not exist on `object<Thinktomorrow\Squanto\Domain\Line>`. Since you implemented `__set`, maybe consider adding a @property annotation. Since your code implements the magic setter `_set`, this function will be called for any write access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /** * @property int $x * @property int $y * @property string $text */ class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } Since the property has write access only, you can use the @property-write annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history...
33		$line->page_id = Page::findOrCreateByKey($linekey->getPageKey())->id;
		0 ignored issues – show Documentation introduced 2016-10-13 18:48 UTC by Report Bug Copy Issue Report The property `page_id` does not exist on `object<Thinktomorrow\Squanto\Domain\Line>`. Since you implemented `__set`, maybe consider adding a @property annotation. Since your code implements the magic setter `_set`, this function will be called for any write access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /** * @property int $x * @property int $y * @property string $text */ class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } Since the property has write access only, you can use the @property-write annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history...
34
35		$line->save();
36
37	12	return $line;
38		}
39	12
40	3	/**
41		* @param $key
42		* @return Line
43	12	*/
44		public static function findOrCreateByKey($key)
45		{
46		if ($line = self::findByKey($key)) {
47		return $line;
48		}
49
50		return self::make($key);
51		}
52
53	30	/**
54		* Save a translated value
55	30	*
56		* @param $locale
57	30	* @param $value
58		* @return $this
59		*/
60		public function saveValue($locale, $value)
61		{
62		$this->saveTranslation($locale, 'value', $value);
63
64		return $this;
65		}
66
67		/**
68		* Save a translated value
69		*
70		* @param $locale
71		* @return $this
72		*/
73		public function removeValue($locale)
74		{
75		$this->removeTranslation($locale);
76
77		return $this;
78	24	}
79
80	24	/**
81		* @param null $locale
82		* @param bool $fallback
83		* @return string
84		*/
85		public function getValue($locale = null, $fallback = true)
86		{
87		return $this->getTranslationFor('value', $locale, $fallback);
88		}
89
90		public static function findValue($key, $locale)
91		{
92		if (!$line = self::findByKey($key)) {
93		return null;
94		}
95
96	39	return $line->getValue($locale, false);
97		}
98	39
99		/**
100		* @param $key
101	12	* @return mixed
102		*/
103		public static function findByKey($key)
104	12	{
105		return self::where('key', $key)->first();
106		}
107
108	12	public function saveSuggestedType()
109	12	{
110	12	// Based on first value found we will suggest a type
111		if (!$value = $this->getValue()) {
112		return;
113		}
114
115		$this->type = (new LineType($value))->suggest();
		0 ignored issues – show Documentation introduced 2016-10-13 18:48 UTC by Report Bug Copy Issue Report The property `type` does not exist on `object<Thinktomorrow\Squanto\Domain\Line>`. Since you implemented `__set`, maybe consider adding a @property annotation. Since your code implements the magic setter `_set`, this function will be called for any write access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /** * @property int $x * @property int $y * @property string $text */ class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } Since the property has write access only, you can use the @property-write annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history...
116		$this->save();
117		}
118
119		public function editInEditor()
120		{
121		return $this->type == LineType::EDITOR;
		0 ignored issues – show Documentation introduced 2016-10-13 18:48 UTC by Report Bug Copy Issue Report The property `type` does not exist on `object<Thinktomorrow\Squanto\Domain\Line>`. Since you implemented `__get`, maybe consider adding a @property annotation. Since your code implements the magic getter `_get`, this function will be called for any read access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /** * @property int $x * @property int $y * @property string $text */ class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } If the property has read access only, you can use the @property-read annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history...
122		}
123
124		public function editInTextarea()
125		{
126		return $this->type == LineType::TEXTAREA;
		0 ignored issues – show Documentation introduced 2016-10-13 18:48 UTC by Report Bug Copy Issue Report The property `type` does not exist on `object<Thinktomorrow\Squanto\Domain\Line>`. Since you implemented `__get`, maybe consider adding a @property annotation. Since your code implements the magic getter `_get`, this function will be called for any read access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /** * @property int $x * @property int $y * @property string $text */ class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } If the property has read access only, you can use the @property-read annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history...
127		}
128
129		public function editInTextinput()
130		{
131		return (!$this->type \|\| $this->type == LineType::TEXT);
		0 ignored issues – show Documentation introduced 2016-10-13 18:48 UTC by Report Bug Copy Issue Report The property `type` does not exist on `object<Thinktomorrow\Squanto\Domain\Line>`. Since you implemented `__get`, maybe consider adding a @property annotation. Since your code implements the magic getter `_get`, this function will be called for any read access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /** * @property int $x * @property int $y * @property string $text */ class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } If the property has read access only, you can use the @property-read annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history...
132		}
133
134		public function areParagraphsAllowed()
135		{
136		return (false !== strpos($this->allowed_html,'<p>'));
		0 ignored issues – show Documentation introduced 2016-12-12 13:21 UTC by Report Bug Copy Issue Report The property `allowed_html` does not exist on `object<Thinktomorrow\Squanto\Domain\Line>`. Since you implemented `__get`, maybe consider adding a @property annotation. Since your code implements the magic getter `_get`, this function will be called for any read access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /** * @property int $x * @property int $y * @property string $text */ class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } If the property has read access only, you can use the @property-read annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history...
137		}
138
139		/**
140		* Get key - value pairs for all lines per locale
141		*
142		* @param $locale
143		* @return mixed
144		*/
145		public static function getValuesByLocale($locale)
146		{
147		// Since the dimsav translatable model trait injects its behaviour and overwrites our results
148		// with the current locale, we will need to fetch results straight from the db instead.
149		$lines = DB::table('squanto_lines')->join('squanto_line_translations', 'squanto_lines.id', '=', 'squanto_line_translations.line_id')
150		->select(['squanto_lines.*','squanto_line_translations.locale','squanto_line_translations.value'])
151		->where('squanto_line_translations.locale', $locale)
152		->get();
153
154		// Assert we have a collection
155		$lines = $lines instanceof Collection ? $lines : collect($lines);
156
157		return $lines->pluck('value', 'key')->toArray();
158		}
159		}
160

thinktomorrow / squanto

Push — master ( 35ba31...f8a85f )

Line A

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

13 Methods

Duplication Side-by-Side

Filter issues like