App\Http\Controllers\HomeController

Complexity

Total Complexity

6

Size/Duplication

Total Lines	70
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	index()	0	5	1
A	getdata()	0	4	1
A	getreport()	0	4	1
B	pushdata()	0	25	2

<?php

namespace App\Http\Controllers;

use App\Model\helpdesk\Ticket\Tickets;

class HomeController extends Controller
{
    /*
      |--------------------------------------------------------------------------
      | Home Controller
      |--------------------------------------------------------------------------
      |
      | This controller renders your application's "dashboard" for users that
      | are authenticated. Of course, you are free to change or remove the
      | controller as you wish. It is just here to get your app started!
      |
     */

    /**
     * Create a new controller instance.
     *
     * @return void

Adding a @return annotation to constructors is generally not recommended as a constructor does not have a meaningful return value.

     */
    public function __construct()
    {
        // $this->middleware('auth');

75% of this comment could be valid code. Did you maybe forget this after debugging?

    }

    /**
     * Show the application dashboard to the user.
     *
     * @return Response
     */
    public function index()
    {
        // ksjdckjdsnc
        return view('themes/default1/admin/dashboard');
    }

    public function getdata()
    {
        return \View::make('emails/notifications/agent');
    }

    public function getreport()
    {
        return \View::make('test');
    }

    public function pushdata()
    {
        $date2 = strtotime(date('Y-m-d'));
        $date3 = date('Y-m-d');
        $format = 'Y-m-d';
        $date1 = strtotime(date($format, strtotime('-1 month'.$date3)));

        $return = '';
        $last = '';

$last is not used, you could remove the assignment.

        for ($i = $date1; $i <= $date2; $i = $i + 86400) {
            $thisDate = date('Y-m-d', $i);

            $created = \DB::table('tickets')->select('created_at')->where('created_at', 'LIKE', '%'.$thisDate.'%')->count();
            $closed = \DB::table('tickets')->select('closed_at')->where('closed_at', 'LIKE', '%'.$thisDate.'%')->count();
            $reopened = \DB::table('tickets')->select('reopened_at')->where('reopened_at', 'LIKE', '%'.$thisDate.'%')->count();

            $value = ['date' => $thisDate, 'open' => $created, 'closed' => $closed, 'reopened' => $reopened];
            $array = array_map('htmlentities', $value);
            $json = html_entity_decode(json_encode($array));
            $return .= $json.',';
        }
        $last = rtrim($return, ',');

        return '['.$last.']';
    }
}