DashboardController::showNotifications() - Code Metrics - Inspection of "Merge branch 'dev-arch'" - gitaminhq/Gitamin - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 009dae...e9add0 )

by Phecho

created 2015-12-11 07:37 UTC

DashboardController::showNotifications() A

↳ Parent: DashboardController

Complexity

Conditions	1
Paths	1

Size

Total Lines	5
Code Lines	3

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	5
rs	9.4286
cc	1
eloc	3
nc	1
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\Http\Controllers;

use Gitamin\Facades\Setting;
use Gitamin\Models\Issue;
use Gitamin\Models\Project;
use Gitamin\Models\Subscriber;
use Illuminate\Support\Facades\View;
use Jenssegers\Date\Date;

class DashboardController extends Controller
{
    /**
     * Start date.
     *
     * @var \Jenssegers\Date\Date
     */
    protected $startDate;

    /**
     * The timezone the system is running in.
     *
     * @var string
     */
    protected $timeZone;

    /**
     * Creates a new dashboard controller.
     *
     * @return void

     */
    public function __construct()
    {
        $this->startDate = new Date();
        $this->dateTimeZone = Setting::get('app_timezone');

    }

    /**
     * Shows the dashboard view.
     *
     * @return \Illuminate\View\View
     */
    public function indexAction()
    {
        $projects = Project::get();
        $issues = $this->getIssues();
        $subscribers = $this->getSubscribers();

        return View::make('dashboard.index')
            ->withPageTitle(trans('dashboard.dashboard'))
            ->withProjects($projects)
            ->withIssues($issues)
            ->withSubscribers($subscribers);
    }

    /**
     * Shows the notifications view.
     *
     * @return \Illuminate\View\View
     */
    public function showNotifications()
    {
        return View::make('dashboard.notifications.index')
            ->withPageTitle(trans('dashboard.notifications.notifications').' '.trans('dashboard.dashboard'));
    }

    /**
     * Fetches all of the issues over the last 30 days.
     *
     * @return \Illuminate\Support\Collection
     */
    protected function getIssues()

    {
        $allIssues = Issue::whereBetween('created_at', [
            $this->startDate->copy()->subDays(30)->format('Y-m-d').' 00:00:00',
            $this->startDate->format('Y-m-d').' 23:59:59',
        ])->orderBy('created_at', 'desc')->get()->groupBy(function (Issue $issue) {
            return (new Date($issue->created_at))
<?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.");
        }
    }

}
                ->setTimezone($this->dateTimeZone)->toDateString();

        });

        // Add in days that have no issues
        foreach (range(0, 30) as $i) {
            $date = (new Date($this->startDate))->setTimezone($this->dateTimeZone)->subDays($i);


            if (!isset($allIssues[$date->toDateString()])) {
                $allIssues[$date->toDateString()] = [];
            }
        }

        // Sort the array so it takes into account the added days
        $allIssues = $allIssues->sortBy(function ($value, $key) {
            return strtotime($key);
        }, SORT_REGULAR, false);

        return $allIssues;
    }

    /**
     * Fetches all of the subscribers over the last 30 days.
     *
     * @return \Illuminate\Support\Collection
     */
    protected function getSubscribers()

    {
        $allSubscribers = Subscriber::whereBetween('created_at', [
            $this->startDate->copy()->subDays(30)->format('Y-m-d').' 00:00:00',
            $this->startDate->format('Y-m-d').' 23:59:59',
        ])->orderBy('created_at', 'desc')->get()->groupBy(function (Subscriber $issue) {
            return (new Date($issue->created_at))
<?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.");
        }
    }

}
                ->setTimezone($this->dateTimeZone)->toDateString();

        });

        // Add in days that have no issues
        foreach (range(0, 30) as $i) {
            $date = (new Date($this->startDate))->setTimezone($this->dateTimeZone)->subDays($i);


            if (!isset($allSubscribers[$date->toDateString()])) {
                $allSubscribers[$date->toDateString()] = [];
            }
        }

        // Sort the array so it takes into account the added days
        $allSubscribers = $allSubscribers->sortBy(function ($value, $key) {
            return strtotime($key);
        }, SORT_REGULAR, false);

        return $allSubscribers;
    }
}


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\Http\Controllers;
13
14		use Gitamin\Facades\Setting;
15		use Gitamin\Models\Issue;
16		use Gitamin\Models\Project;
17		use Gitamin\Models\Subscriber;
18		use Illuminate\Support\Facades\View;
19		use Jenssegers\Date\Date;
20
21		class DashboardController extends Controller
22		{
23		/**
24		* Start date.
25		*
26		* @var \Jenssegers\Date\Date
27		*/
28		protected $startDate;
29
30		/**
31		* The timezone the system is running in.
32		*
33		* @var string
34		*/
35		protected $timeZone;
36
37		/**
38		* Creates a new dashboard controller.
39		*
40		* @return void
		0 ignored issues – show Comprehensibility Best Practice introduced 2015-12-02 13:46 UTC by Report Bug Copy Issue Report Adding a `@return` annotation to constructors is generally not recommended as a constructor does not have a meaningful return value. Adding a `@return` annotation to a constructor is not recommended, since a constructor does not have a meaningful return value. Please refer to the PHP core documentation on constructors. Loading history...
41		*/
42		public function __construct()
43		{
44		$this->startDate = new Date();
45		$this->dateTimeZone = Setting::get('app_timezone');
		0 ignored issues – show Bug introduced 2015-12-02 13:46 UTC by Report Bug Copy Issue Report The property `dateTimeZone` does not seem to exist. Did you mean `timeZone`? An attempt at access to an undefined property has been detected. This may either be a typographical error or the property has been renamed but there are still references to its old name. If you really want to allow access to undefined properties, you can define magic methods to allow access. See the php core documentation on Overloading. Loading history...
46		}
47
48		/**
49		* Shows the dashboard view.
50		*
51		* @return \Illuminate\View\View
52		*/
53		public function indexAction()
54		{
55		$projects = Project::get();
56		$issues = $this->getIssues();
57		$subscribers = $this->getSubscribers();
58
59		return View::make('dashboard.index')
60		->withPageTitle(trans('dashboard.dashboard'))
61		->withProjects($projects)
62		->withIssues($issues)
63		->withSubscribers($subscribers);
64		}
65
66		/**
67		* Shows the notifications view.
68		*
69		* @return \Illuminate\View\View
70		*/
71		public function showNotifications()
72		{
73		return View::make('dashboard.notifications.index')
74		->withPageTitle(trans('dashboard.notifications.notifications').' '.trans('dashboard.dashboard'));
75		}
76
77		/**
78		* Fetches all of the issues over the last 30 days.
79		*
80		* @return \Illuminate\Support\Collection
81		*/
82	View Code Duplication	protected function getIssues()
		0 ignored issues – show Duplication introduced 2015-12-02 13:46 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
83		{
84		$allIssues = Issue::whereBetween('created_at', [
85		$this->startDate->copy()->subDays(30)->format('Y-m-d').' 00:00:00',
86		$this->startDate->format('Y-m-d').' 23:59:59',
87		])->orderBy('created_at', 'desc')->get()->groupBy(function (Issue $issue) {
88		return (new Date($issue->created_at))
		0 ignored issues – show Documentation introduced 2015-12-02 13:46 UTC by Report Bug Copy Issue Report The property `created_at` 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...
89		->setTimezone($this->dateTimeZone)->toDateString();
		0 ignored issues – show Bug introduced 2015-12-09 06:23 UTC by Report Bug Copy Issue Report The property `dateTimeZone` does not seem to exist. Did you mean `timeZone`? An attempt at access to an undefined property has been detected. This may either be a typographical error or the property has been renamed but there are still references to its old name. If you really want to allow access to undefined properties, you can define magic methods to allow access. See the php core documentation on Overloading. Loading history...
90		});
91
92		// Add in days that have no issues
93		foreach (range(0, 30) as $i) {
94		$date = (new Date($this->startDate))->setTimezone($this->dateTimeZone)->subDays($i);
		0 ignored issues – show Bug introduced 2015-12-09 06:23 UTC by Report Bug Copy Issue Report The property `dateTimeZone` does not seem to exist. Did you mean `timeZone`? An attempt at access to an undefined property has been detected. This may either be a typographical error or the property has been renamed but there are still references to its old name. If you really want to allow access to undefined properties, you can define magic methods to allow access. See the php core documentation on Overloading. Loading history...
95
96		if (!isset($allIssues[$date->toDateString()])) {
97		$allIssues[$date->toDateString()] = [];
98		}
99		}
100
101		// Sort the array so it takes into account the added days
102		$allIssues = $allIssues->sortBy(function ($value, $key) {
103		return strtotime($key);
104		}, SORT_REGULAR, false);
105
106		return $allIssues;
107		}
108
109		/**
110		* Fetches all of the subscribers over the last 30 days.
111		*
112		* @return \Illuminate\Support\Collection
113		*/
114	View Code Duplication	protected function getSubscribers()
		0 ignored issues – show Duplication introduced 2015-12-02 13:46 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
115		{
116		$allSubscribers = Subscriber::whereBetween('created_at', [
117		$this->startDate->copy()->subDays(30)->format('Y-m-d').' 00:00:00',
118		$this->startDate->format('Y-m-d').' 23:59:59',
119		])->orderBy('created_at', 'desc')->get()->groupBy(function (Subscriber $issue) {
120		return (new Date($issue->created_at))
		0 ignored issues – show Documentation introduced 2015-12-02 13:46 UTC by Report Bug Copy Issue Report The property `created_at` does not exist on `object<Gitamin\Models\Subscriber>`. 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...
121		->setTimezone($this->dateTimeZone)->toDateString();
		0 ignored issues – show Bug introduced 2015-12-09 06:23 UTC by Report Bug Copy Issue Report The property `dateTimeZone` does not seem to exist. Did you mean `timeZone`? An attempt at access to an undefined property has been detected. This may either be a typographical error or the property has been renamed but there are still references to its old name. If you really want to allow access to undefined properties, you can define magic methods to allow access. See the php core documentation on Overloading. Loading history...
122		});
123
124		// Add in days that have no issues
125		foreach (range(0, 30) as $i) {
126		$date = (new Date($this->startDate))->setTimezone($this->dateTimeZone)->subDays($i);
		0 ignored issues – show Bug introduced 2015-12-09 06:23 UTC by Report Bug Copy Issue Report The property `dateTimeZone` does not seem to exist. Did you mean `timeZone`? An attempt at access to an undefined property has been detected. This may either be a typographical error or the property has been renamed but there are still references to its old name. If you really want to allow access to undefined properties, you can define magic methods to allow access. See the php core documentation on Overloading. Loading history...
127
128		if (!isset($allSubscribers[$date->toDateString()])) {
129		$allSubscribers[$date->toDateString()] = [];
130		}
131		}
132
133		// Sort the array so it takes into account the added days
134		$allSubscribers = $allSubscribers->sortBy(function ($value, $key) {
135		return strtotime($key);
136		}, SORT_REGULAR, false);
137
138		return $allSubscribers;
139		}
140		}
141

gitaminhq / Gitamin

Push — master ( 009dae...e9add0 )

DashboardController::showNotifications() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like