BannerPageTypesManager::getPages() - Code Metrics - Inspection of "4.9" - imagecms/ImageCMS - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — development ( eb9524...db4517 )

by Andrij

created 2016-02-25 14:36 UTC

BannerPageTypesManager::getPages() B

↳ Parent: BannerPageTypesManager

Complexity

Conditions	5
Paths	4

Size

Total Lines	12
Code Lines	7

Duplication

Lines	12
Ratio	100 %

Metric	Value
dl	12
loc	12
rs	8.8571
cc	5
eloc	7
nc	4
nop	2

<?php
namespace xbanners\src\Managers;

class BannerPageTypesManager

{


    const PAGE_TYPE_MAIN = 'main';
    const PAGE_TYPE_SHOP_CATEGORY = 'shop_category';
    const PAGE_TYPE_PRODUCT = 'product';
    const PAGE_TYPE_CATEGORY = 'category';
    const PAGE_TYPE_PAGE = 'page';
    const PAGE_TYPE_BRAND = 'brand';
    const PAGE_TYPE_SEARCH = 'search';

    /**
     * @var BannerPageTypesManager instance
     */
    private static $instance = NULL;

    private function __construct() {

    }

    /**
     * Get BannerPageTypesManager instance
     * @return BannerPageTypesManager instance
     */
    public static function getInstance() {
        if (null === self::$instance) {
            self::$instance = new self();
        }
        return self::$instance;
    }

    /**
     * Get banner pages type|types: ['main', 'shop_category', 'product',...] on nothing to get all types array
     * @param string $type - banner type

     * @return array
     */
    public function getPagesTypes($type = NULL) {
        $pageTypes = [
            'main' => [
                'name' => lang('Main', 'xbanners'),
                'class' => NULL
            ],
            'shop_category' => [
                'name' => lang('Product category', 'xbanners'),
                'class' => '\xbanners\src\BannerPagesTypes\ShopCategory'
            ],
            'product' => [
                'name' => lang('Product', 'xbanners'),
                'class' => '\xbanners\src\BannerPagesTypes\Product'
            ],
            'category' => [
                'name' => lang('Page category', 'xbanners'),
                'class' => '\xbanners\src\BannerPagesTypes\Category'
            ],
            'page' => [
                'name' => lang('Page', 'xbanners'),
                'class' => '\xbanners\src\BannerPagesTypes\Page'
            ],
            'brand' => [
                'name' => lang('Product brand', 'xbanners'),
                'class' => '\xbanners\src\BannerPagesTypes\Brand'
            ],
            'search' => [
                'name' => lang('Search page', 'xbanners'),
                'class' => NULL
            ],
        ];

        return $type ? $pageTypes[$type] : $pageTypes;
    }

    /**
     * Get banner type view
     * @param string $type - banner page type name type: ['main', 'shop_category', 'product',...]
     * @param string $locale - locale name

     * @return array
     */
    public function getView($type, $locale = NULL) {
        $locale = $locale ? $locale : \MY_Controller::defaultLocale();
        $type = $this->getPagesTypes($type);

        if (!$type OR !$type['class'] OR !class_exists($type['class'])) {
            return NULL;
        }

        $typeObj = new $type['class']($locale);

        return $typeObj->getView();
    }

    /**
     * Get banner page type allowed pages list
     * @param string $type - banner page type name type: ['main', 'shop_category', 'product',...]
     * @param string $locale - locale name

     * @return array
     */
    public function getPages($type, $locale = NULL) {
        $locale = $locale ? $locale : \MY_Controller::defaultLocale();
        $type = $this->getPagesTypes($type);

        if (!$type OR !$type['class'] OR !class_exists($type['class'])) {
            return NULL;
        }

        $typeObj = new $type['class']($locale);

        return $typeObj->getPages();
    }
}

1		<?php
2		namespace xbanners\src\Managers;
3
4		class BannerPageTypesManager
		0 ignored issues – show Coding Style introduced 2015-11-28 17:39 UTC by Report Bug Copy Issue Report Since you have declared the constructor as private, maybe you should also declare the class as final. Loading history...
5		{
		0 ignored issues – show introduced 2016-02-25 15:02 UTC by Report Bug Copy Issue Report Opening brace of a class must be on the same line as the definition Loading history...
6
7		const PAGE_TYPE_MAIN = 'main';
8		const PAGE_TYPE_SHOP_CATEGORY = 'shop_category';
9		const PAGE_TYPE_PRODUCT = 'product';
10		const PAGE_TYPE_CATEGORY = 'category';
11		const PAGE_TYPE_PAGE = 'page';
12		const PAGE_TYPE_BRAND = 'brand';
13		const PAGE_TYPE_SEARCH = 'search';
14
15		/**
16		* @var BannerPageTypesManager instance
17		*/
18		private static $instance = NULL;
19
20		private function __construct() {
21
22		}
23
24		/**
25		* Get BannerPageTypesManager instance
26		* @return BannerPageTypesManager instance
27		*/
28		public static function getInstance() {
29		if (null === self::$instance) {
30		self::$instance = new self();
31		}
32		return self::$instance;
33		}
34
35		/**
36		* Get banner pages type\|types: ['main', 'shop_category', 'product',...] on nothing to get all types array
37		* @param string $type - banner type
		0 ignored issues – show Documentation introduced 2015-11-28 17:39 UTC by Report Bug Copy Issue Report Should the type for parameter `$type` not be `string\|null`? This check looks for `@param` annotations where the type inferred by our type inference engine differs from the declared type. It makes a suggestion as to what type it considers more descriptive. Most often this is a case of a parameter that can be null in addition to its declared types. Loading history...
38		* @return array
39		*/
40		public function getPagesTypes($type = NULL) {
41		$pageTypes = [
42		'main' => [
43		'name' => lang('Main', 'xbanners'),
44		'class' => NULL
45		],
46		'shop_category' => [
47		'name' => lang('Product category', 'xbanners'),
48		'class' => '\xbanners\src\BannerPagesTypes\ShopCategory'
49		],
50		'product' => [
51		'name' => lang('Product', 'xbanners'),
52		'class' => '\xbanners\src\BannerPagesTypes\Product'
53		],
54		'category' => [
55		'name' => lang('Page category', 'xbanners'),
56		'class' => '\xbanners\src\BannerPagesTypes\Category'
57		],
58		'page' => [
59		'name' => lang('Page', 'xbanners'),
60		'class' => '\xbanners\src\BannerPagesTypes\Page'
61		],
62		'brand' => [
63		'name' => lang('Product brand', 'xbanners'),
64		'class' => '\xbanners\src\BannerPagesTypes\Brand'
65		],
66		'search' => [
67		'name' => lang('Search page', 'xbanners'),
68		'class' => NULL
69		],
70		];
71
72		return $type ? $pageTypes[$type] : $pageTypes;
73		}
74
75		/**
76		* Get banner type view
77		* @param string $type - banner page type name type: ['main', 'shop_category', 'product',...]
78		* @param string $locale - locale name
		0 ignored issues – show Documentation introduced 2016-02-25 15:02 UTC by Report Bug Copy Issue Report Should the type for parameter `$locale` not be `string\|null`? This check looks for `@param` annotations where the type inferred by our type inference engine differs from the declared type. It makes a suggestion as to what type it considers more descriptive. Most often this is a case of a parameter that can be null in addition to its declared types. Loading history...
79		* @return array
80		*/
81	View Code Duplication	public function getView($type, $locale = NULL) {
82		$locale = $locale ? $locale : \MY_Controller::defaultLocale();
83		$type = $this->getPagesTypes($type);
84
85		if (!$type OR !$type['class'] OR !class_exists($type['class'])) {
86		return NULL;
87		}
88
89		$typeObj = new $type['class']($locale);
90
91		return $typeObj->getView();
92		}
93
94		/**
95		* Get banner page type allowed pages list
96		* @param string $type - banner page type name type: ['main', 'shop_category', 'product',...]
97		* @param string $locale - locale name
		0 ignored issues – show Documentation introduced 2016-02-25 15:02 UTC by Report Bug Copy Issue Report Should the type for parameter `$locale` not be `string\|null`? This check looks for `@param` annotations where the type inferred by our type inference engine differs from the declared type. It makes a suggestion as to what type it considers more descriptive. Most often this is a case of a parameter that can be null in addition to its declared types. Loading history...
98		* @return array
99		*/
100	View Code Duplication	public function getPages($type, $locale = NULL) {
101		$locale = $locale ? $locale : \MY_Controller::defaultLocale();
102		$type = $this->getPagesTypes($type);
103
104		if (!$type OR !$type['class'] OR !class_exists($type['class'])) {
105		return NULL;
106		}
107
108		$typeObj = new $type['class']($locale);
109
110		return $typeObj->getPages();
111		}
112		}

imagecms / ImageCMS

Push — development ( eb9524...db4517 )

BannerPageTypesManager::getPages() B

Complexity

Size

Duplication

Duplication Side-by-Side

Filter issues like