CDIFactoryDocumentation::__construct() - Code Metrics - mosbth/Anax-MVC - Measure and Improve Code Quality continuously with Scrutinizer

CDIFactoryDocumentation::__construct() A
last analyzed 2016-05-12 12:16 UTC

↳ Parent: CDIFactoryDocumentation

Complexity

Conditions	1
Paths	1

Size

Total Lines	13
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	0

Metric	Value
c	2
b	0
f	0
dl	0
loc	13
rs	9.4285
cc	1
eloc	8
nc	1
nop	0

<?php

namespace Anax\DI;

/**
 * Extended factory for Anax documentation.
 *
 */
class CDIFactoryDocumentation extends CDIFactoryDefault
{
   /**
     * Construct.
     *
     */
    public function __construct()
    {
        parent::__construct();

        $this->set('documentation', function () {
            $fc = new \Anax\Content\CFileContent();
            $fc->setBasePath(ANAX_INSTALL_PATH . 'docs/documentation');
            return $fc;
        });

        $this->theme->setVariable('style', "article {max-width: 650px;}");
<?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->theme->setBaseTitle(" - Anax documentation");
<?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.");
        }
    }

}
    }
}


1			<?php
2
3			namespace Anax\DI;
4
5			/**
6			* Extended factory for Anax documentation.
7			*
8			*/
9			class CDIFactoryDocumentation extends CDIFactoryDefault
10			{
11			/**
12			* Construct.
13			*
14			*/
15			public function __construct()
16			{
17			parent::__construct();
18
19			$this->set('documentation', function () {
20			$fc = new \Anax\Content\CFileContent();
21			$fc->setBasePath(ANAX_INSTALL_PATH . 'docs/documentation');
22			return $fc;
23			});
24
25			$this->theme->setVariable('style', "article {max-width: 650px;}");
			0 ignored issues – show Documentation introduced 2015-11-29 16:29 UTC by Report Bug Copy Issue Report The property `theme` does not exist on `object<Anax\DI\CDIFactoryDocumentation>`. 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...
26			$this->theme->setBaseTitle(" - Anax documentation");
			0 ignored issues – show Documentation introduced 2015-11-29 16:29 UTC by Report Bug Copy Issue Report The property `theme` does not exist on `object<Anax\DI\CDIFactoryDocumentation>`. 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...
27			}
28			}
29

mosbth / Anax-MVC

CDIFactoryDocumentation::__construct() A last analyzed 2016-05-12 12:16 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

CDIFactoryDocumentation::__construct() A
last analyzed 2016-05-12 12:16 UTC