JhFlexiTime\Service\RunningBalanceService

Complexity

Total Complexity

15

Size/Duplication

Total Lines	236
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	8

Importance

Changes

0

9 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	20	1
B	indexPreviousMonthBalance()	0	24	3
A	reIndexAllUsersRunningBalance()	0	21	2
A	reIndexIndividualUserRunningBalance()	0	19	1
A	setUserStartingBalance()	0	6	1
A	reIndexRunningBalance()	0	16	2
A	addMonthBalance()	0	9	1
A	calculateBalance()	0	7	1
A	getMonthRange()	0	16	3

<?php

namespace JhFlexiTime\Service;

use JhFlexiTime\Entity\RunningBalance;
use JhFlexiTime\Repository\BalanceRepositoryInterface;
use JhFlexiTime\Repository\BookingRepositoryInterface;
use JhUser\Repository\UserRepositoryInterface;
use JhFlexiTime\Repository\UserSettingsRepositoryInterface;
use Zend\EventManager\EventManagerAwareInterface;
use ZfcUser\Entity\UserInterface;
use Doctrine\Common\Persistence\ObjectManager;
use JhFlexiTime\DateTime\DateTime;
use Zend\EventManager\EventManagerAwareTrait;

/**
 * Class RunningBalanceService
 * @package JhFlexiTime\Service
 * @author Aydin Hassan <aydin@hotmail.co.uk>
 */
class RunningBalanceService implements EventManagerAwareInterface
{

    use EventManagerAwareTrait;

    /**
     * @var \DateTime
     */
    protected $date;

    /**
     * @var \JhUser\Repository\UserRepositoryInterface
     */
    protected $userRepository;

    /**
     * @var \JhFlexiTime\Repository\UserSettingRepositoryInterface
     */
    protected $userSettingsRepository;

    /**
     * @var \JhFlexiTime\Repository\BalanceRepositoryInterface
     */
    protected $balanceRepository;

    /**
     * @var \JhFlexiTime\Repository\BookingRepositoryInterface
     */
    protected $bookingRepository;

    /**
     * @var PeriodServiceInterface
     */
    protected $periodService;

    /**
     * @var \DateTime
     */
    protected $lastMonth;

    /**
     * @param UserRepositoryInterface $userRepository
     * @param UserSettingsRepositoryInterface $userSettingsRepository
     * @param BookingRepositoryInterface $bookingRepository
     * @param BalanceRepositoryInterface $balanceRepository
     * @param PeriodServiceInterface $periodService
     * @param ObjectManager $objectManager
     * @param DateTime $date
     */
    public function __construct(
        UserRepositoryInterface $userRepository,
        UserSettingsRepositoryInterface $userSettingsRepository,
        BookingRepositoryInterface $bookingRepository,
        BalanceRepositoryInterface $balanceRepository,
        PeriodServiceInterface $periodService,
        ObjectManager $objectManager,
        DateTime $date
    ) {
        $this->userRepository           = $userRepository;
        $this->userSettingsRepository   = $userSettingsRepository;

It seems like $userSettingsRepository of type object<JhFlexiTime\Repository\UserSettingsRepositoryInterface> is incompatible with the declared type object<JhFlexiTime\Repository\UserSettingRepositoryInterface> of property $userSettingsRepository.

        $this->bookingRepository        = $bookingRepository;
        $this->balanceRepository        = $balanceRepository;
        $this->periodService            = $periodService;
        $this->objectManager            = $objectManager;

The property objectManager does not exist. Did you maybe forget to declare it?

        $this->date                     = $date;

        $this->lastMonth = clone $this->date;
        $this->lastMonth->modify('first day of previous month 00:00');
    }

    /**
     * Calculate the previous month balance for all users
     * and add it to their running balance
     */
    public function indexPreviousMonthBalance()
    {
        foreach ($this->userRepository->findAll(false) as $user) {
            $runningBalance = $this->balanceRepository->findOneByUser($user);
            $userSettings   = $this->userSettingsRepository->findOneByUser($user);

            $date = $this->lastMonth;
            //if user started this month use that as from date instead
            if ($userSettings->getFlexStartDate() > $date) {
                $date = $userSettings->getFlexStartDate();
            }

            $this->addMonthBalance(
                $user,
                $runningBalance,
                [
                    'start' => $date,
                    'end' => $this->lastMonth->endOfMonth()

It seems like you code against a specific sub-type and not the parent class DateTime as the method endOfMonth() does only exist in the following sub-classes of DateTime: JhFlexiTime\DateTime\DateTime. Maybe you want to instanceof check for one of these explicitly?

                ]
            );
        }

        $this->objectManager->flush();
    }

    /**
     * Recalculate all user's running balance
     */
    public function reIndexAllUsersRunningBalance()
    {
        foreach ($this->userRepository->findAll(false) as $user) {
            $runningBalance = $this->balanceRepository->findOneByUser($user);
            $userSettings   = $this->userSettingsRepository->findOneByUser($user);

            $monthRanges = $this->getMonthRange(
                $userSettings->getFlexStartDate(),
                $this->lastMonth->endOfMonth()

It seems like you code against a specific sub-type and not the parent class DateTime as the method endOfMonth() does only exist in the following sub-classes of DateTime: JhFlexiTime\DateTime\DateTime. Maybe you want to instanceof check for one of these explicitly?

            );

            $this->reIndexRunningBalance(
                $user,
                $runningBalance,
                $monthRanges,
                $userSettings->getStartingBalance()
            );
        }

        $this->objectManager->flush();
    }

    /**
     * Recaulculate Individual user's running balance
     *
     * @param UserInterface $user
     */
    public function reIndexIndividualUserRunningBalance(UserInterface $user)
    {
        $runningBalance = $this->balanceRepository->findOneByUser($user);
        $userSettings   = $this->userSettingsRepository->findOneByUser($user);

        $monthRanges = $this->getMonthRange(
            $userSettings->getFlexStartDate(),
            $this->lastMonth->endOfMonth()

It seems like you code against a specific sub-type and not the parent class DateTime as the method endOfMonth() does only exist in the following sub-classes of DateTime: JhFlexiTime\DateTime\DateTime. Maybe you want to instanceof check for one of these explicitly?

        );

        $this->reIndexRunningBalance(
            $user,
            $runningBalance,
            $monthRanges,
            $userSettings->getStartingBalance()
        );

        $this->objectManager->flush();
    }

    /**
     * @param UserInterface $user
     * @param float $balance
     */
    public function setUserStartingBalance(UserInterface $user, $balance)
    {
        $settings = $this->userSettingsRepository->findOneByUser($user);
        $settings->setStartingBalance($balance);
        $this->objectManager->flush();
    }

    /**
     * @param UserInterface $user
     * @param RunningBalance $runningBalance
     * @param array $months
     * @param int $initialBalance
     */
    private function reIndexRunningBalance(
        UserInterface $user,
        RunningBalance $runningBalance,
        array $months,
        $initialBalance
    ) {
        $eventData = ['runningBalance' => $runningBalance];
        $this->getEventManager()->trigger('reIndexUserRunningBalance.pre', null, $eventData);

        $runningBalance->setBalance($initialBalance);
        foreach ($months as $month) {
            $this->addMonthBalance($user, $runningBalance, $month);
        }

        $this->getEventManager()->trigger('reIndexUserRunningBalance.post', null, $eventData);
    }

    /**
     * @param UserInterface $user
     * @param RunningBalance $runningBalance
     * @param $month
     */
    private function addMonthBalance(UserInterface $user, RunningBalance $runningBalance, array $month)
    {
        $eventData = ['runningBalance' => $runningBalance, 'month' => $month['end']];
        $this->getEventManager()->trigger('addMonthBalance.pre', null, $eventData);

        $runningBalance->addBalance($this->calculateBalance($user, $month['start'], $month['end']));

        $this->getEventManager()->trigger('addMonthBalance.post', null, $eventData);
    }

    /**
     * Calculate the balance between the two dates
     *
     * @param UserInterface $user
     * @param DateTime $start
     * @param DateTime $end
     * @return float
     */
    private function calculateBalance(UserInterface $user, DateTime $start, DateTime $end)
    {
        $monthTotalHours    = $this->periodService->getTotalHoursBetweenDates($start, $end);
        $totalBookedHours   = $this->bookingRepository->getTotalBookedBetweenByUser($user, $start, $end);
        $balance            = $totalBookedHours - $monthTotalHours;
        return $balance;
    }

    /**
     * @param DateTime $start
     * @param DateTime $end
     * @return array
     */
    private function getMonthRange(DateTime $start, DateTime $end)
    {
        $months = $start->getMonthsBetween($end);
        $monthRanges = [];
        foreach ($months as $key => $month) {
            $monthRanges[] = [
                'start' => $month->startOfMonth(),
                'end'   => $month->endOfMonth()
            ];

            if ($key === 0) {
                $monthRanges[$key]['start'] = $start;
            }
        }
        return $monthRanges;
    }
}