| @@ -18,7 +18,7 @@ discard block | ||
| 18 | 18 | } | 
| 19 | 19 | |
| 20 | 20 | /** | 
| 21 | - * @param $eventCode | |
| 21 | + * @param string $eventCode | |
| 22 | 22 | * @param null|int $start | 
| 23 | 23 | * @param null|int $maxResult | 
| 24 | 24 | * @return mixed | 
| @@ -138,7 +138,7 @@ discard block | ||
| 138 | 138 | |
| 139 | 139 | /** | 
| 140 | 140 | * @param array $appointments | 
| 141 | - * @param $timestamp | |
| 141 | + * @param string $timestamp | |
| 142 | 142 | * @return array | 
| 143 | 143 | */ | 
| 144 | 144 |      public static function insertDateTimeChanges(array $appointments,$timestamp){ | 
| @@ -23,14 +23,14 @@ discard block | ||
| 23 | 23 | * @param null|int $maxResult | 
| 24 | 24 | * @return mixed | 
| 25 | 25 | */ | 
| 26 | -    public function getCurrentAppoiments($eventCode,$start=null,$maxResult=null){ | |
| 27 | -        if(!$start){ | |
| 26 | +    public function getCurrentAppoiments($eventCode, $start = null, $maxResult = null) { | |
| 27 | +        if (!$start) { | |
| 28 | 28 | $start = 0; | 
| 29 | 29 | } | 
| 30 | -        if(!$maxResult){ | |
| 30 | +        if (!$maxResult) { | |
| 31 | 31 | $maxResult = 999999; | 
| 32 | 32 | } | 
| 33 | - return $this->certainAppointmentsList = $this->appointmentsCertain->get($eventCode,['start_index'=>$start,'max_results'=>$maxResult])->getResults()->appointments; | |
| 33 | + return $this->certainAppointmentsList = $this->appointmentsCertain->get($eventCode, ['start_index'=>$start, 'max_results'=>$maxResult])->getResults()->appointments; | |
| 34 | 34 | } | 
| 35 | 35 | |
| 36 | 36 | /** | 
| @@ -38,12 +38,12 @@ discard block | ||
| 38 | 38 | * @param array $appointmentsNew | 
| 39 | 39 | * @return bool | 
| 40 | 40 | */ | 
| 41 | -    public function hasChanged(array $appointmentsOld,array $appointmentsNew){ | |
| 41 | +    public function hasChanged(array $appointmentsOld, array $appointmentsNew) { | |
| 42 | 42 | $hasChanged = false; | 
| 43 | 43 | $appointmentsOld = self::recursiveArrayObjectToFullArray($appointmentsOld); | 
| 44 | 44 | $appointmentsNew = self::recursiveArrayObjectToFullArray($appointmentsNew); | 
| 45 | -        foreach ($appointmentsOld as $appointmentOld){ | |
| 46 | -            if(!in_array($appointmentOld,$appointmentsNew)){ | |
| 45 | +        foreach ($appointmentsOld as $appointmentOld) { | |
| 46 | +            if (!in_array($appointmentOld, $appointmentsNew)) { | |
| 47 | 47 | $hasChanged = true; | 
| 48 | 48 | break; | 
| 49 | 49 | } | 
| @@ -56,7 +56,7 @@ discard block | ||
| 56 | 56 | * @return array | 
| 57 | 57 | */ | 
| 58 | 58 |      public static  function objectToArray($object) { | 
| 59 | -        if(is_object($object)){ | |
| 59 | +        if (is_object($object)) { | |
| 60 | 60 | |
| 61 | 61 | return (array) $object; | 
| 62 | 62 | } | 
| @@ -67,7 +67,7 @@ discard block | ||
| 67 | 67 | * @param $appointments | 
| 68 | 68 | * @return array | 
| 69 | 69 | */ | 
| 70 | -    public static function recursiveArrayObjectToFullArray($appointments){ | |
| 70 | +    public static function recursiveArrayObjectToFullArray($appointments) { | |
| 71 | 71 | return json_decode(json_encode($appointments), true); | 
| 72 | 72 | } | 
| 73 | 73 | |
| @@ -78,8 +78,8 @@ discard block | ||
| 78 | 78 | */ | 
| 79 | 79 |      private function arrayRecursiveDiff(array $arrayOlds, array $arrayNews) { | 
| 80 | 80 | $difference = []; | 
| 81 | -        foreach($arrayOlds as $key => $arrayOld){ | |
| 82 | -            if(!in_array($arrayOld,$arrayNews)){ | |
| 81 | +        foreach ($arrayOlds as $key => $arrayOld) { | |
| 82 | +            if (!in_array($arrayOld, $arrayNews)) { | |
| 83 | 83 | $difference[$key] = $arrayOld; | 
| 84 | 84 | } | 
| 85 | 85 | } | 
| @@ -91,12 +91,12 @@ discard block | ||
| 91 | 91 | * @param array $appointmentsNew | 
| 92 | 92 | * @return array | 
| 93 | 93 | */ | 
| 94 | -    public function getListChangings(array $appointmentsOld,array $appointmentsNew){ | |
| 94 | +    public function getListChangings(array $appointmentsOld, array $appointmentsNew) { | |
| 95 | 95 | $appointmentsOld = self::recursiveArrayObjectToFullArray($appointmentsOld); | 
| 96 | 96 | $appointmentsNew = self::recursiveArrayObjectToFullArray($appointmentsNew); | 
| 97 | 97 | $changesList = []; | 
| 98 | -        if($this->hasChanged($appointmentsOld,$appointmentsNew)){ | |
| 99 | - $changesList = self::recursiveArrayObjectToFullArray($this->arrayRecursiveDiff($appointmentsOld,$appointmentsNew)); | |
| 98 | +        if ($this->hasChanged($appointmentsOld, $appointmentsNew)) { | |
| 99 | + $changesList = self::recursiveArrayObjectToFullArray($this->arrayRecursiveDiff($appointmentsOld, $appointmentsNew)); | |
| 100 | 100 | } | 
| 101 | 101 | return $changesList; | 
| 102 | 102 | } | 
| @@ -107,26 +107,26 @@ discard block | ||
| 107 | 107 | * @param array $changingsDetected | 
| 108 | 108 | * @return array ['deleted'=>[],'updated'=>[]] | 
| 109 | 109 | */ | 
| 110 | -    public function detectDeleteOrUpdated(array $currentAppointments,array $changingsDetected){ | |
| 110 | +    public function detectDeleteOrUpdated(array $currentAppointments, array $changingsDetected) { | |
| 111 | 111 | $delete = []; | 
| 112 | 112 | $update = []; | 
| 113 | 113 | //@Todo: Detect Fields has changed | 
| 114 | 114 | $appointmentsNew = self::recursiveArrayObjectToFullArray($currentAppointments); | 
| 115 | 115 | $changings = self::recursiveArrayObjectToFullArray($changingsDetected); | 
| 116 | -        foreach ($changings as $changing){ | |
| 116 | +        foreach ($changings as $changing) { | |
| 117 | 117 | $registration = $changing['registration']['regCode']; | 
| 118 | 118 | $registrationTarget = $changing['targetRegistration']['regCode']; | 
| 119 | -            foreach ($appointmentsNew as $currentAppointment){ | |
| 119 | +            foreach ($appointmentsNew as $currentAppointment) { | |
| 120 | 120 | $registrationCurrent = $currentAppointment['registration']['regCode']; | 
| 121 | 121 | $registrationTargetCurrent = $currentAppointment['targetRegistration']['regCode']; | 
| 122 | - if(in_array($registration,[$registrationCurrent,$registrationTargetCurrent]) | |
| 123 | - && in_array($registrationTarget,[$registrationCurrent,$registrationTargetCurrent]) | |
| 124 | -                    && !in_array($changing,$update) && !in_array($changing,$delete)) { | |
| 122 | + if (in_array($registration, [$registrationCurrent, $registrationTargetCurrent]) | |
| 123 | + && in_array($registrationTarget, [$registrationCurrent, $registrationTargetCurrent]) | |
| 124 | +                    && !in_array($changing, $update) && !in_array($changing, $delete)) { | |
| 125 | 125 | $update[] = $changing; | 
| 126 | 126 | break; | 
| 127 | 127 | } | 
| 128 | 128 | } | 
| 129 | -            if(!in_array($changing,$update) && !in_array($changing,$delete)){ | |
| 129 | +            if (!in_array($changing, $update) && !in_array($changing, $delete)) { | |
| 130 | 130 | $delete[] = $changing; | 
| 131 | 131 | } | 
| 132 | 132 | } | 
| @@ -141,8 +141,8 @@ discard block | ||
| 141 | 141 | * @param $timestamp | 
| 142 | 142 | * @return array | 
| 143 | 143 | */ | 
| 144 | -    public static function insertDateTimeChanges(array $appointments,$timestamp){ | |
| 145 | -        foreach ($appointments as $key => $appointment){ | |
| 144 | +    public static function insertDateTimeChanges(array $appointments, $timestamp) { | |
| 145 | +        foreach ($appointments as $key => $appointment) { | |
| 146 | 146 | $appointments[$key]['dateDetectChanges'] = $timestamp; | 
| 147 | 147 | } | 
| 148 | 148 | return $appointments; | 
| @@ -154,11 +154,11 @@ discard block | ||
| 154 | 154 | * @param string $timestamp | 
| 155 | 155 | * @return array ['deleted'=>[],'updated'=>[]] | 
| 156 | 156 | */ | 
| 157 | -    public function detectAppointmentsChangings(array $appointmentsOld,array $appointmentsNew,$timestamp){ | |
| 158 | - $changings = $this->getListChangings($appointmentsOld,$appointmentsNew); | |
| 159 | - $changesList = $this->detectDeleteOrUpdated($appointmentsNew,$changings); | |
| 160 | - $changesList['updated'] = self::insertDateTimeChanges($changesList['updated'],$timestamp); | |
| 161 | - $changesList['deleted'] = self::insertDateTimeChanges($changesList['deleted'],$timestamp); | |
| 157 | +    public function detectAppointmentsChangings(array $appointmentsOld, array $appointmentsNew, $timestamp) { | |
| 158 | + $changings = $this->getListChangings($appointmentsOld, $appointmentsNew); | |
| 159 | + $changesList = $this->detectDeleteOrUpdated($appointmentsNew, $changings); | |
| 160 | + $changesList['updated'] = self::insertDateTimeChanges($changesList['updated'], $timestamp); | |
| 161 | + $changesList['deleted'] = self::insertDateTimeChanges($changesList['deleted'], $timestamp); | |
| 162 | 162 | return $changesList; | 
| 163 | 163 | } | 
| 164 | 164 | |