GalleryImage - Code Metrics - gordonbanderson/ss3gallery - Measure and Improve Code Quality continuously with Scrutinizer

GalleryImage A
last analyzed 2016-04-18 12:42 UTC

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	74
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	2

Test Coverage

Coverage

100%

Metric	Value
wmc	5
lcom	0
cbo	2
dl	0
loc	74
ccs	16
cts	16
cp	1
rs	10

5 Methods

Rating	Name	Size	Complexity
A	getCMSFields()	10	1
A	getThumbnail()	4	1
A	getPortletTitle()	4	1
A	getPortletImage()	4	1
A	getPortletCaption()	4	1

<?php

class GalleryImage extends DataObject implements RenderableAsPortlet
namespace YourVendor;

class YourClass { }
{
    public static $db = array(
        'SortOrder' => 'Int',
        'Title' => 'Varchar',
        'Caption' => 'Text'
    );

    // One-to-one relationship with gallery page
    public static $has_one = array(
        'Image' => 'Image',
        'GalleryPage' => 'GalleryPage',
    );

    // tidy up the CMS by not showing these fields
    public function getCMSFields()
    {
        $fields = parent::getCMSFields();
        $fields->removeFieldFromTab('Root.Main', 'GalleryPageID');
        $fields->removeFieldFromTab('Root.Main', 'ExifRead');
        $fields->removeFieldFromTab('Root.Main', 'SortOrder');
        $fields->renameField('Title', _t('GalleryImage.TITLE', 'Title'));
        $fields->renameField('Image', _t('GalleryImage.IMAGE', 'Image'));
        return $fields;
    }

    // Tell the datagrid what fields to show in the table
    public static $summary_fields = array(
        'ID' => 'ID',
        'Title' => 'Title',
        'Thumbnail' => 'Thumbnail',
    );

    // this function creates the thumnail for the summary fields to use
    public function getThumbnail()
    {
        return $this->Image()->CMSThumbnail();
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
    }

    public function getPortletTitle()
    {
        return $this->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.");
        }
    }

}
    }

    /**
    * An accessor method for an image for a portlet.
    *
    * @example
    * <code>
    *  return $this->NewsItemImage;
    * </code>
    *
    * @return string
    */
    public function getPortletImage()
    {
        return $this->Image();
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
    }

    /**
    * An accessor for text associated with the portlet.
    *
    * @example
    * <code>
    * return $this->Summary
    * </code>
    *
    * @return string
    */
    public function getPortletCaption()
    {
        return '';
    }
}


1		<?php
2
3		class GalleryImage extends DataObject implements RenderableAsPortlet
		0 ignored issues – show Coding Style Compatibility introduced 2016-01-20 09:21 UTC by Report Bug Copy Issue Report PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions. You can fix this by adding a namespace to your class: namespace YourVendor; class YourClass { } When choosing a vendor namespace, try to pick something that is not too generic to avoid conflicts with other libraries. Loading history...
4		{
5		public static $db = array(
6		'SortOrder' => 'Int',
7		'Title' => 'Varchar',
8		'Caption' => 'Text'
9		);
10
11		// One-to-one relationship with gallery page
12		public static $has_one = array(
13		'Image' => 'Image',
14		'GalleryPage' => 'GalleryPage',
15		);
16
17		// tidy up the CMS by not showing these fields
18	1	public function getCMSFields()
19		{
20	1	$fields = parent::getCMSFields();
21	1	$fields->removeFieldFromTab('Root.Main', 'GalleryPageID');
22	1	$fields->removeFieldFromTab('Root.Main', 'ExifRead');
23	1	$fields->removeFieldFromTab('Root.Main', 'SortOrder');
24	1	$fields->renameField('Title', _t('GalleryImage.TITLE', 'Title'));
25	1	$fields->renameField('Image', _t('GalleryImage.IMAGE', 'Image'));
26	1	return $fields;
27		}
28
29		// Tell the datagrid what fields to show in the table
30		public static $summary_fields = array(
31		'ID' => 'ID',
32		'Title' => 'Title',
33		'Thumbnail' => 'Thumbnail',
34		);
35
36		// this function creates the thumnail for the summary fields to use
37	1	public function getThumbnail()
38		{
39	1	return $this->Image()->CMSThumbnail();
		0 ignored issues – show Documentation Bug introduced 2016-02-02 11:13 UTC by Report Bug Copy Issue Report The method `Image` does not exist on `object<GalleryImage>`? Since you implemented `__call`, maybe consider adding a @method annotation. If you implement `__call` and you know which methods are available, you can improve IDE auto-completion and static analysis by adding a @method annotation to the class. This is often the case, when `__call` is implemented by a parent class and only the child class knows which methods exist: class ParentClass { private $data = array(); public function __call($method, array $args) { if (0 === strpos($method, 'get')) { return $this->data[strtolower(substr($method, 3))]; } throw new \LogicException(sprintf('Unsupported method: %s', $method)); } } /** * If this class knows which fields exist, you can specify the methods here: * * @method string getName() */ class SomeClass extends ParentClass { } Loading history...
40		}
41
42	1	public function getPortletTitle()
43		{
44	1	return $this->Title;
		0 ignored issues – show Documentation introduced 2016-02-02 11:13 UTC by Report Bug Copy Issue Report The property `Title` does not exist on `object<GalleryImage>`. 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...
45		}
46
47		/**
48		* An accessor method for an image for a portlet.
49		*
50		* @example
51		* <code>
52		* return $this->NewsItemImage;
53		* </code>
54		*
55		* @return string
56		*/
57	1	public function getPortletImage()
58		{
59	1	return $this->Image();
		0 ignored issues – show Documentation Bug introduced 2016-02-02 11:13 UTC by Report Bug Copy Issue Report The method `Image` does not exist on `object<GalleryImage>`? Since you implemented `__call`, maybe consider adding a @method annotation. If you implement `__call` and you know which methods are available, you can improve IDE auto-completion and static analysis by adding a @method annotation to the class. This is often the case, when `__call` is implemented by a parent class and only the child class knows which methods exist: class ParentClass { private $data = array(); public function __call($method, array $args) { if (0 === strpos($method, 'get')) { return $this->data[strtolower(substr($method, 3))]; } throw new \LogicException(sprintf('Unsupported method: %s', $method)); } } /** * If this class knows which fields exist, you can specify the methods here: * * @method string getName() */ class SomeClass extends ParentClass { } Loading history...
60		}
61
62		/**
63		* An accessor for text associated with the portlet.
64		*
65		* @example
66		* <code>
67		* return $this->Summary
68		* </code>
69		*
70		* @return string
71		*/
72	1	public function getPortletCaption()
73		{
74	1	return '';
75		}
76		}
77

gordonbanderson / ss3gallery

GalleryImage A last analyzed 2016-04-18 12:42 UTC

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

5 Methods

Duplication Side-by-Side

Filter issues like

GalleryImage A
last analyzed 2016-04-18 12:42 UTC