Adminetic\Website\Services\Statistic

Complexity

Total Complexity

10

Size/Duplication

Total Lines	83
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	passRegisterPerDay()	0	19	3
A	dayBook()	0	24	4
A	perDayPaymentTotal()	0	19	3

<?php

namespace Adminetic\Website\Services;

use Adminetic\Website\Models\Admin\Pass;

The type Adminetic\Website\Models\Admin\Pass was not found. Maybe you did not declare it correctly or list all dependencies?

use Adminetic\Website\Models\Admin\Payment;
use Carbon\Carbon;
use Carbon\CarbonPeriod;

class Statistic
{
    /*
    |--------------------------------------------------------------------------
    | Reports
    |--------------------------------------------------------------------------
    */
    public function dayBook(Carbon $start_date, Carbon $end_date, $event = null)
    {
        $data = null;
        $period = CarbonPeriod::create($start_date, $end_date);
        foreach ($period as $date) {
            if (! is_null($event)) {
                $payments = $event->payments->filter(function ($payment) use ($date) {
                    return $payment->created_at->format('Y-m-d') == $date->format('Y-m-d');

The method format() does not exist on null.

                });
                $passes = $event->passes->filter(function ($pass) use ($date) {
                    return $pass->created_at->format('Y-m-d') == $date->format('Y-m-d');
                });
            } else {
                $payments = Payment::whereDate('created_at', $date)->get();
                $passes = Pass::whereDate('created_at', $date);
            }
            $data[modeDate($date)] = [

It seems like $date can also be of type null; however, parameter $date of modeDate() does only seem to accept Carbon\Carbon, maybe add an additional type check?

                'total_payment' => $payments->sum('payment'),
                'total_pass' => $passes->count(),
                'total_transaction' => $payments->count(),
            ];
        }

        return ! is_null($data) ? array_reverse($data) : null;

The condition is_null($data) is always true.

    }

    /*
    |--------------------------------------------------------------------------
    | Pass Statistics
    |--------------------------------------------------------------------------
    */
    public function passRegisterPerDay($given_data = null, $limit = 30)
    {
        $data = $given_data ?? Pass::orderBy('position')->get();
        if ($data->count() > 0) {
            $data = $data->sortByDesc('created_at');
            $end_date = $data->first()->created_at;
            $start_date = $end_date->copy()->subDays($limit);
            $period = CarbonPeriod::create($start_date, $end_date);
            $total = [];
            foreach ($period as $date) {
                $total[modeDate($date)] = $data->filter(function ($p) use ($date) {

It seems like $date can also be of type null; however, parameter $date of modeDate() does only seem to accept Carbon\Carbon, maybe add an additional type check?

                    return $p->created_at->format('Y-m-d') == $date->format('Y-m-d');
                })->count();
            }

            return $total;
        }

        return null;
    }

    /*
    |--------------------------------------------------------------------------
    | Payment Statistics
    |--------------------------------------------------------------------------
    */
    public function perDayPaymentTotal($given_data = null, $limit = 30)
    {
        $data = $given_data ?? Payment::orderBy('position')->get();
        if ($data->count() > 0) {
            $data = $data->sortByDesc('created_at');
            $end_date = $data->first()->created_at;
            $start_date = $end_date->copy()->subDays($limit);
            $period = CarbonPeriod::create($start_date, $end_date);
            $total = [];
            foreach ($period as $date) {
                $total[modeDate($date)] = $data->filter(function ($p) use ($date) {

It seems like $date can also be of type null; however, parameter $date of modeDate() does only seem to accept Carbon\Carbon, maybe add an additional type check?

                    return $p->created_at->format('Y-m-d') == $date->format('Y-m-d');
                })->sum('payment');
            }

            return $total;
        }

        return  null;
    }
}