MomentPresenter::icon() - Code Metrics - Inspection of "fix problems with signup" - gitaminhq/Gitamin - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 37af5c...47524a )

by Phecho

created 2015-12-17 13:34 UTC

MomentPresenter::icon() B

↳ Parent: MomentPresenter

Complexity

Conditions	5
Paths	5

Size

Total Lines	14
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Importance

Changes	3
Bugs	2	Features	1

Metric	Value
c	3
b	2
f	1
dl	0
loc	14
rs	8.8571
cc	5
eloc	11
nc	5
nop	0

<?php

/*
 * This file is part of Gitamin.
 *
 * Copyright (C) 2015-2016 The Gitamin Team
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Gitamin\Presenters;

use Gitamin\Models\Comment;
use Gitamin\Models\Issue;
use Gitamin\Models\Moment;
use Gitamin\Models\Owner;
use Gitamin\Models\Project;
use Gitamin\Presenters\Traits\TimestampsTrait;
use GrahamCampbell\Markdown\Facades\Markdown;

class MomentPresenter extends AbstractPresenter
{
    use TimestampsTrait;

    /**
     * Renders the message from Markdown into HTML.
     *
     * @return string
     */
    public function formattedData()
    {
        return Markdown::convertToHtml($this->wrappedObject->data);
    }

    public function formattedTarget()
    {
        if ($this->wrappedObject->target instanceof Comment) {
            return Markdown::convertToHtml($this->wrappedObject->target->message);
<?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.");
        }
    }

}
        } elseif ($this->wrappedObject->target instanceof Issue) {
            return Markdown::convertToHtml($this->wrappedObject->target->description);
<?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.");
        }
    }

}
        } elseif ($this->wrappedObject->target instanceof Project) {
            return Markdown::convertToHtml($this->wrappedObject->target->description);
<?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 the moment action summary.
     *
     * @return string
     */
    protected function actionName()
    {
        switch ($this->wrappedObject->action) {
            case Moment::CREATED:
                return trans('gitamin.moments.created');
            case Moment::UPDATED:
                return trans('gitamin.moments.updated');
            case Moment::CLOSED:
                return trans('gitamin.moments.closed');
            case Moment::COMMENTED:
                return trans('gitamin.moments.commented');
            default:
                return 'Unknow';
        }
    }

    public function icon()
    {
        if ($this->wrappedObject->target instanceof Project) {
            return 'fa fa-cubes';
        } elseif ($this->wrappedObject->target instanceof Comment) {
            return 'fa fa-comments-o';
        } elseif ($this->wrappedObject->target instanceof Issue) {
            return 'fa fa-exclamation-circle';
        } elseif ($this->wrappedObject->target instanceof Owner) {
            return 'fa fa-user';
        } else {
            return 'fa fa-code-fork';
        }
    }

    /**
     * Convert presented moment to an array.
     *
     * @return array
     */
    public function toArray()
    {
        return [
            'summary' => $this->summary(),
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 { }
            'created_at_iso' => $this->created_at_iso(),
        ];
    }
}


1			<?php
2
3			/*
4			* This file is part of Gitamin.
5			*
6			* Copyright (C) 2015-2016 The Gitamin Team
7			*
8			* For the full copyright and license information, please view the LICENSE
9			* file that was distributed with this source code.
10			*/
11
12			namespace Gitamin\Presenters;
13
14			use Gitamin\Models\Comment;
15			use Gitamin\Models\Issue;
16			use Gitamin\Models\Moment;
17			use Gitamin\Models\Owner;
18			use Gitamin\Models\Project;
19			use Gitamin\Presenters\Traits\TimestampsTrait;
20			use GrahamCampbell\Markdown\Facades\Markdown;
21
22			class MomentPresenter extends AbstractPresenter
23			{
24			use TimestampsTrait;
25
26			/**
27			* Renders the message from Markdown into HTML.
28			*
29			* @return string
30			*/
31			public function formattedData()
32			{
33			return Markdown::convertToHtml($this->wrappedObject->data);
34			}
35
36			public function formattedTarget()
37			{
38			if ($this->wrappedObject->target instanceof Comment) {
39			return Markdown::convertToHtml($this->wrappedObject->target->message);
			0 ignored issues – show Documentation introduced 2015-12-10 13:31 UTC by Report Bug Copy Issue Report The property `message` does not exist on `object<Gitamin\Models\Comment>`. 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...
40			} elseif ($this->wrappedObject->target instanceof Issue) {
41			return Markdown::convertToHtml($this->wrappedObject->target->description);
			0 ignored issues – show Documentation introduced 2015-12-10 13:31 UTC by Report Bug Copy Issue Report The property `description` does not exist on `object<Gitamin\Models\Issue>`. 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...
42			} elseif ($this->wrappedObject->target instanceof Project) {
43			return Markdown::convertToHtml($this->wrappedObject->target->description);
			0 ignored issues – show Documentation introduced 2015-12-11 13:44 UTC by Report Bug Copy Issue Report The property `description` does not exist on `object<Gitamin\Models\Project>`. 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...
44			}
45			}
46
47			/**
48			* Get the moment action summary.
49			*
50			* @return string
51			*/
52			protected function actionName()
53			{
54			switch ($this->wrappedObject->action) {
55			case Moment::CREATED:
56			return trans('gitamin.moments.created');
57			case Moment::UPDATED:
58			return trans('gitamin.moments.updated');
59			case Moment::CLOSED:
60			return trans('gitamin.moments.closed');
61			case Moment::COMMENTED:
62			return trans('gitamin.moments.commented');
63			default:
64			return 'Unknow';
65			}
66			}
67
68			public function icon()
69			{
70			if ($this->wrappedObject->target instanceof Project) {
71			return 'fa fa-cubes';
72			} elseif ($this->wrappedObject->target instanceof Comment) {
73			return 'fa fa-comments-o';
74			} elseif ($this->wrappedObject->target instanceof Issue) {
75			return 'fa fa-exclamation-circle';
76			} elseif ($this->wrappedObject->target instanceof Owner) {
77			return 'fa fa-user';
78			} else {
79			return 'fa fa-code-fork';
80			}
81			}
82
83			/**
84			* Convert presented moment to an array.
85			*
86			* @return array
87			*/
88			public function toArray()
89			{
90			return [
91			'summary' => $this->summary(),
			0 ignored issues – show Documentation Bug introduced 2015-12-10 13:31 UTC by Report Bug Copy Issue Report The method `summary` does not exist on `object<Gitamin\Presenters\MomentPresenter>`? 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...
92			'created_at_iso' => $this->created_at_iso(),
93			];
94			}
95			}
96

gitaminhq / Gitamin

Push — master ( 37af5c...47524a )

MomentPresenter::icon() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like