PostPresenter::statusLabelClass() - Code Metrics - Inspection of "Slug for tags" - AsgardCms/Blog - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#38)

unknown

created 2016-05-24 06:23 UTC

PostPresenter::statusLabelClass() B

↳ Parent: PostPresenter

Complexity

Conditions	5
Paths	5

Size

Total Lines	20
Code Lines	17

Duplication

Lines	0
Ratio	0 %

Importance

Changes	3
Bugs	0	Features	0

Metric	Value
c	3
b	0
f	0
dl	0
loc	20
rs	8.8571
cc	5
eloc	17
nc	5
nop	0

<?php namespace Modules\Blog\Presenters;

use Laracasts\Presenter\Presenter;
use Modules\Blog\Entities\Status;

class PostPresenter extends Presenter
{
    /**
     * @var \Modules\Blog\Entities\Status
     */
    protected $status;
    /**
     * @var \Modules\Blog\Repositories\PostRepository
     */
    private $post;

    public function __construct($entity)
    {
        parent::__construct($entity);
        $this->post = app('Modules\Blog\Repositories\PostRepository');
        $this->status = app('Modules\Blog\Entities\Status');
    }

    /**
     * Get the previous post of the current post
     * @return object
     */
    public function previous()
    {
        return $this->post->getPreviousOf($this->entity);
    }

    /**
     * Get the next post of the current post
     * @return object
     */
    public function next()
    {
        return $this->post->getNextOf($this->entity);
    }

    /**
     * Get the post status
     * @return string
     */
    public function status()
    {
        return $this->status->get($this->entity->status);
    }

    /**
     * Getting the label class for the appropriate status
     * @return string
     */
    public function statusLabelClass()
    {
        switch ($this->entity->status) {
            case Status::DRAFT:
                return 'bg-red';
                break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
            case Status::PENDING:
                return 'bg-orange';
                break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
            case Status::PUBLISHED:
                return 'bg-green';
                break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
            case Status::UNPUBLISHED:
                return 'bg-purple';
                break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
            default:
                return 'bg-red';
                break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
        }
    }
}


1			<?php namespace Modules\Blog\Presenters;
2
3			use Laracasts\Presenter\Presenter;
4			use Modules\Blog\Entities\Status;
5
6			class PostPresenter extends Presenter
7			{
8			/**
9			* @var \Modules\Blog\Entities\Status
10			*/
11			protected $status;
12			/**
13			* @var \Modules\Blog\Repositories\PostRepository
14			*/
15			private $post;
16
17			public function __construct($entity)
18			{
19			parent::__construct($entity);
20			$this->post = app('Modules\Blog\Repositories\PostRepository');
21			$this->status = app('Modules\Blog\Entities\Status');
22			}
23
24			/**
25			* Get the previous post of the current post
26			* @return object
27			*/
28			public function previous()
29			{
30			return $this->post->getPreviousOf($this->entity);
31			}
32
33			/**
34			* Get the next post of the current post
35			* @return object
36			*/
37			public function next()
38			{
39			return $this->post->getNextOf($this->entity);
40			}
41
42			/**
43			* Get the post status
44			* @return string
45			*/
46			public function status()
47			{
48			return $this->status->get($this->entity->status);
49			}
50
51			/**
52			* Getting the label class for the appropriate status
53			* @return string
54			*/
55			public function statusLabelClass()
56			{
57			switch ($this->entity->status) {
58			case Status::DRAFT:
59			return 'bg-red';
60			break;
			0 ignored issues – show Unused Code introduced 2015-11-21 11:12 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
61			case Status::PENDING:
62			return 'bg-orange';
63			break;
			0 ignored issues – show Unused Code introduced 2015-11-21 11:12 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
64			case Status::PUBLISHED:
65			return 'bg-green';
66			break;
			0 ignored issues – show Unused Code introduced 2015-11-21 11:12 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
67			case Status::UNPUBLISHED:
68			return 'bg-purple';
69			break;
			0 ignored issues – show Unused Code introduced 2015-11-21 11:12 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
70			default:
71			return 'bg-red';
72			break;
			0 ignored issues – show Unused Code introduced 2015-11-21 11:12 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
73			}
74			}
75			}
76

AsgardCms / Blog

Pull Request — master (#38)

PostPresenter::statusLabelClass() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like