Schedules\ShiftSchedule

Complexity

Total Complexity

20

Size/Duplication

Total Lines	87
Duplicated Lines	40.23 %

Coupling/Cohesion

Components	0
Dependencies	2

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	getRoleNameFromID()	0	23	4
A	shiftSort()	0	4	1
B	shiftTimeSort()	21	21	7
A	getStringBetween()	0	11	2
A	daySort()	0	6	1
A	groupSort()	14	14	5

<?php
namespace Schedules;

trait ShiftSchedule
{
    public function getRoleNameFromID($roleID)
    {
        static $roles = null;
        if($roles === null)
        {
            $dataTable = \DataSetFactory::getDataTableByNames('fvs', 'roles');
            $tmp = $dataTable->read();
            $roles = array();
            $count = count($tmp);
            for($i = 0; $i < $count; $i++)
            {
                if(isset($tmp[$i]['display_name']))
                {
                    $roles[$tmp[$i]['short_name']] = $tmp[$i]['display_name'];
                }
                else
                {
                    $roles[$tmp[$i]['short_name']] = $tmp[$i]['short_name'];
                }
            }
        }
        return $roles[$roleID];
    }

    public function shiftSort($a, $b)
    {
        return strcmp($this->getRoleNameFromID($a['roleID']), $this->getRoleNameFromID($b['roleID']));
    }

    public function shiftTimeSort($a, $b)

This method seems to be duplicated in your project.

    {
        $interval = $a['startTime']->diff($b['startTime']);
        if($interval->invert === 0)
        {
            if($interval->h || $interval->i)
            {
                return -1;
            }
            else
            {
                return 0;
            }
        }
        else if($interval->invert === 1 && ($interval->h || $interval->days))
        {
            return 1;
        }
        print_r($interval);
        die();
    }

    public function getStringBetween($string, $start, $end)
    {
        $index = strpos($string, $start);
        if($index === false)
        {
            return $string;
        }
        $index++;
        $len = strpos($string, $end, $index) - $index;
        return substr($string, $index, $len);
    }

    public function daySort($a, $b)
    {
        $a_date = $this->getStringBetween($a, '(', ')');
        $b_date = $this->getStringBetween($b, '(', ')'); 
        return strcasecmp($a_date, $b_date);
    }

    public function groupSort($a, $b)

This method seems to be duplicated in your project.

    {
        $aArr = explode(' ', $a);
        $bArr = explode(' ', $b);
        if($aArr[1] === 'PM' && $bArr[1] === 'AM')
        {
            return 1;
        }
        else if($aArr[1] === 'AM' && $bArr[1] === 'PM')
        {
            return -1;
        }
        return strcmp($a, $b);
    }
}