Page::packagePage() - Code Metrics - ddimaria/Laravel-REST-CMS - Measure and Improve Code Quality continuously with Scrutinizer

Page::packagePage() B
last analyzed 2016-01-27 22:58 UTC

↳ Parent: Page

Complexity

Conditions	2
Paths	2

Size

Total Lines	37
Code Lines	28

Duplication

Lines	0
Ratio	0 %

Importance

Changes	3
Bugs	0	Features	3

Metric	Value
c	3
b	0
f	3
dl	0
loc	37
rs	8.8571
cc	2
eloc	28
nc	2
nop	1

<?php namespace App\LaravelRestCms\Page;

use App\LaravelRestCms\BaseModel;
use App\LaravelRestCms\Page\PageDetail;
use App\LaravelRestCms\Seo\Seo;
use App\LaravelRestCms\Template\Template;

class Page extends BaseModel {

	public static $searchCols = ['nav_name', 'url', 'title'];

	/**
	 * The database table used by the model.
	 *
	 * @var string
	 */
	protected $table = 'pages';

	/**
	 * The attributes that are mass assignable.
	 *
	 * @var array
	 */
	protected $fillable = ['parent_id', 'template_id', 'nav_name', 'url', 'title', 'created_by', 'updated_by'];

	/**
	 * The attributes excluded from the model's JSON form.
	 *
	 * @var array
	 */
	protected $hidden = [];

	/**
	 * Rules to validate when creating a model
	 * 
	* @var array
	 */
	protected static $createRules = [
		'parent_id' => 'integer',
		'template_id' => 'integer',
		'nav_name' => 'required',
		'url' => 'required|unique:pages',
		'title' => 'required',
		'created_by' => 'integer',
		'updated_by' => 'integer',
	];

	/**
	 * Indicates if the model should be attributed with created_by and updated_by
	 * 
	* @var bool
	 */
	public $attirbution = true;

	/**
	 * Joins the page_detail table
	 * 
	 * @return \Illuminate\Database\Eloquent\Relations\HasMany
	 */
	public function detail()
    {
        return $this->hasMany(PageDetail::class, 'page_id', 'id');
    }

	/**
	 * Joins the templates table
	 * 
	 * @return \Illuminate\Database\Eloquent\Relations\hasOne
	 */
	public function template()
    {
        return $this->hasMany(Template::class, 'id', 'template_id');
    }

	/**
	 * Joins the seo table
	 * 
	 * @return \Illuminate\Database\Eloquent\Relations\hasOne
	 */
	public function seo()
    {
        return $this->hasMany(Seo::class, 'id', 'seo_id');
    }

	/**
	 * Joins the pages table
	 * 
	 * @return \Illuminate\Database\Eloquent\Relations\hasOne
	 */
	public function parent()
    {
        return $this->hasMany(Page::class, 'id', 'parent_id');
    }

    /**
     * Returns a page and associated detail and template data
     * 
     * @param  string $slug
     * @return array
     */
    public function showBySlug($slug)
    {        
        $data = Page::where(['url' => $slug])->with('template', 'detail.templateDetail')->firstOrFail();
        
        return $this->packagePage($data);
    }

    /**
     * Pacages a Page collection into an array for public consumption
     * 
     * @param  Page   $data 
     * @return array
     */
    protected function packagePage(Page $data)
    {
    	$seo = $data->seo->first();
<?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.");
        }
    }

}
    	$template = $data->template->first();
<?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.");
        }
    }

}
    	
    	$page = [
        	'nav_name' => $data->nav_name,
<?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.");
        }
    }

}
        	'url' => $data->url,
<?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.");
        }
    }

}
        	'title' => $data->title,
<?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.");
        }
    }

}
        	'vars' => [],
        	'template' => [
        		'name' => $template->name,
	            'class' => $template->class,
	            'method' => $template->method,
	            'params' => $template->params,
	            'template_name' => $template->template_name,
	            'layout' => $template->layout,
        	],
        	'seo' => [
        		'title' => $seo->title, 
	            'keywords' => $seo->keywords, 
	            'description' => $seo->description,
	            'og_title' => $seo->og_title,
	            'og_description' => $seo->og_description,
	            'og_image' => $seo->og_image,
	            'og_type' => $seo->og_type,
	            'fb_app_id' => $seo->fb_app_id,
	            'meta' => $seo->meta,
        	]
        ];

        foreach ($data->detail as $detail) {
<?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.");
        }
    }

}
        	$page['vars'][$detail->templateDetail->first()->var] = $detail->data;
        }

        return $page;
    }
}


1			<?php namespace App\LaravelRestCms\Page;
2
3			use App\LaravelRestCms\BaseModel;
4			use App\LaravelRestCms\Page\PageDetail;
5			use App\LaravelRestCms\Seo\Seo;
6			use App\LaravelRestCms\Template\Template;
7
8			class Page extends BaseModel {
9
10			public static $searchCols = ['nav_name', 'url', 'title'];
11
12			/**
13			* The database table used by the model.
14			*
15			* @var string
16			*/
17			protected $table = 'pages';
18
19			/**
20			* The attributes that are mass assignable.
21			*
22			* @var array
23			*/
24			protected $fillable = ['parent_id', 'template_id', 'nav_name', 'url', 'title', 'created_by', 'updated_by'];
25
26			/**
27			* The attributes excluded from the model's JSON form.
28			*
29			* @var array
30			*/
31			protected $hidden = [];
32
33			/**
34			* Rules to validate when creating a model
35			*
36			* @var array
37			*/
38			protected static $createRules = [
39			'parent_id' => 'integer',
40			'template_id' => 'integer',
41			'nav_name' => 'required',
42			'url' => 'required\|unique:pages',
43			'title' => 'required',
44			'created_by' => 'integer',
45			'updated_by' => 'integer',
46			];
47
48			/**
49			* Indicates if the model should be attributed with created_by and updated_by
50			*
51			* @var bool
52			*/
53			public $attirbution = true;
54
55			/**
56			* Joins the page_detail table
57			*
58			* @return \Illuminate\Database\Eloquent\Relations\HasMany
59			*/
60			public function detail()
61			{
62			return $this->hasMany(PageDetail::class, 'page_id', 'id');
63			}
64
65			/**
66			* Joins the templates table
67			*
68			* @return \Illuminate\Database\Eloquent\Relations\hasOne
69			*/
70			public function template()
71			{
72			return $this->hasMany(Template::class, 'id', 'template_id');
73			}
74
75			/**
76			* Joins the seo table
77			*
78			* @return \Illuminate\Database\Eloquent\Relations\hasOne
79			*/
80			public function seo()
81			{
82			return $this->hasMany(Seo::class, 'id', 'seo_id');
83			}
84
85			/**
86			* Joins the pages table
87			*
88			* @return \Illuminate\Database\Eloquent\Relations\hasOne
89			*/
90			public function parent()
91			{
92			return $this->hasMany(Page::class, 'id', 'parent_id');
93			}
94
95			/**
96			* Returns a page and associated detail and template data
97			*
98			* @param string $slug
99			* @return array
100			*/
101			public function showBySlug($slug)
102			{
103			$data = Page::where(['url' => $slug])->with('template', 'detail.templateDetail')->firstOrFail();
104
105			return $this->packagePage($data);
106			}
107
108			/**
109			* Pacages a Page collection into an array for public consumption
110			*
111			* @param Page $data
112			* @return array
113			*/
114			protected function packagePage(Page $data)
115			{
116			$seo = $data->seo->first();
			0 ignored issues – show Documentation introduced 2016-01-26 17:42 UTC by Report Bug Copy Issue Report The property `seo` does not exist on `object<App\LaravelRestCms\Page\Page>`. 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...
117			$template = $data->template->first();
			0 ignored issues – show Documentation introduced 2016-01-26 17:13 UTC by Report Bug Copy Issue Report The property `template` does not exist on `object<App\LaravelRestCms\Page\Page>`. 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...
118
119			$page = [
120			'nav_name' => $data->nav_name,
			0 ignored issues – show Documentation introduced 2016-01-26 17:13 UTC by Report Bug Copy Issue Report The property `nav_name` does not exist on `object<App\LaravelRestCms\Page\Page>`. 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...
121			'url' => $data->url,
			0 ignored issues – show Documentation introduced 2016-01-26 17:13 UTC by Report Bug Copy Issue Report The property `url` does not exist on `object<App\LaravelRestCms\Page\Page>`. 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			'title' => $data->title,
			0 ignored issues – show Documentation introduced 2016-01-26 17:13 UTC by Report Bug Copy Issue Report The property `title` does not exist on `object<App\LaravelRestCms\Page\Page>`. 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...
123			'vars' => [],
124			'template' => [
125			'name' => $template->name,
126			'class' => $template->class,
127			'method' => $template->method,
128			'params' => $template->params,
129			'template_name' => $template->template_name,
130			'layout' => $template->layout,
131			],
132			'seo' => [
133			'title' => $seo->title,
134			'keywords' => $seo->keywords,
135			'description' => $seo->description,
136			'og_title' => $seo->og_title,
137			'og_description' => $seo->og_description,
138			'og_image' => $seo->og_image,
139			'og_type' => $seo->og_type,
140			'fb_app_id' => $seo->fb_app_id,
141			'meta' => $seo->meta,
142			]
143			];
144
145			foreach ($data->detail as $detail) {
			0 ignored issues – show Documentation introduced 2016-01-26 17:13 UTC by Report Bug Copy Issue Report The property `detail` does not exist on `object<App\LaravelRestCms\Page\Page>`. 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...
146			$page['vars'][$detail->templateDetail->first()->var] = $detail->data;
147			}
148
149			return $page;
150			}
151			}
152

ddimaria / Laravel-REST-CMS

Page::packagePage() B last analyzed 2016-01-27 22:58 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

Page::packagePage() B
last analyzed 2016-01-27 22:58 UTC