| @@ -29,7 +29,7 @@ | ||
| 29 | 29 |      { | 
| 30 | 30 |          if (isset($value)) { | 
| 31 | 31 | $this->preferences()->updateOrCreate(['key' => $key], ['value' => $this->cast($value, $type), | 
| 32 | - 'type' => $type, ]); | |
| 32 | + 'type' => $type, ]); | |
| 33 | 33 | |
| 34 | 34 | return $value; | 
| 35 | 35 | } | 
| @@ -19,7 +19,7 @@ | ||
| 19 | 19 | /** | 
| 20 | 20 | * Get or set preference value. | 
| 21 | 21 | * | 
| 22 | - * @param mixed $key | |
| 22 | + * @param string $key | |
| 23 | 23 | * @param mixed $value | 
| 24 | 24 | * @param string $type | 
| 25 | 25 | * | 
| @@ -16,6 +16,10 @@ | ||
| 16 | 16 | $this->timetable = $timetable; | 
| 17 | 17 | } | 
| 18 | 18 | |
| 19 | + /** | |
| 20 | + * @param string $starting | |
| 21 | + * @param integer $days | |
| 22 | + */ | |
| 19 | 23 | protected function initTimetable($starting, $days) | 
| 20 | 24 |      { | 
| 21 | 25 | $this->timetable | 
| @@ -95,11 +95,11 @@ | ||
| 95 | 95 | public function buildTimetable($vacancies, $starting = 'today', $days = 1) | 
| 96 | 96 |      { | 
| 97 | 97 | $this->timetable | 
| 98 | - ->interval($this->interval) | |
| 99 | -             ->format('date.service.time') | |
| 100 | - ->from($starting) | |
| 101 | - ->future($days) | |
| 102 | - ->init(); | |
| 98 | + ->interval($this->interval) | |
| 99 | +                ->format('date.service.time') | |
| 100 | + ->from($starting) | |
| 101 | + ->future($days) | |
| 102 | + ->init(); | |
| 103 | 103 | |
| 104 | 104 |          foreach ($vacancies as $vacancy) { | 
| 105 | 105 | $this->updateTimeslots($vacancy, $this->interval); | 
| @@ -121,7 +121,6 @@ discard block | ||
| 121 | 121 | /** | 
| 122 | 122 | * Setter for interval. | 
| 123 | 123 | * | 
| 124 | - * @param int $minutes | |
| 125 | 124 | * | 
| 126 | 125 | * @return $this | 
| 127 | 126 | */ | 
| @@ -280,8 +279,8 @@ discard block | ||
| 280 | 279 | * Helper method for Arr::set | 
| 281 | 280 | * | 
| 282 | 281 | * @param array &$array | 
| 283 | - * @param mixed $key | |
| 284 | - * @param mixed $value | |
| 282 | + * @param string $key | |
| 283 | + * @param integer $value | |
| 285 | 284 | * | 
| 286 | 285 | * @return mixed | 
| 287 | 286 | */ | 
| @@ -294,7 +293,7 @@ discard block | ||
| 294 | 293 | * Helper method for Arr::get | 
| 295 | 294 | * | 
| 296 | 295 | * @param array &$array | 
| 297 | - * @param mixed $key | |
| 296 | + * @param string $key | |
| 298 | 297 | * @param mixed $default | 
| 299 | 298 | * | 
| 300 | 299 | * @return mixed | 
| @@ -304,6 +303,9 @@ discard block | ||
| 304 | 303 | return Arr::get($array, $key, $default); | 
| 305 | 304 | } | 
| 306 | 305 | |
| 306 | + /** | |
| 307 | + * @param string $array1 | |
| 308 | + */ | |
| 307 | 309 | private function array_substitute(&$array1, $array2) | 
| 308 | 310 |      { | 
| 309 | 311 |          foreach ($array1 as $key => $value) { | 
| @@ -17,10 +17,10 @@ | ||
| 17 | 17 | protected function initTimetable($starting, $days) | 
| 18 | 18 |      { | 
| 19 | 19 | $this->timetable | 
| 20 | -             ->format('date.service.time') | |
| 21 | - ->from($starting) | |
| 22 | - ->future($days) | |
| 23 | - ->init(); | |
| 20 | +                ->format('date.service.time') | |
| 21 | + ->from($starting) | |
| 22 | + ->future($days) | |
| 23 | + ->init(); | |
| 24 | 24 | } | 
| 25 | 25 | |
| 26 | 26 | /** | 
| @@ -16,6 +16,10 @@ | ||
| 16 | 16 | $this->timetable = $timetable; | 
| 17 | 17 | } | 
| 18 | 18 | |
| 19 | + /** | |
| 20 | + * @param string $starting | |
| 21 | + * @param integer $days | |
| 22 | + */ | |
| 19 | 23 | protected function initTimetable($starting, $days) | 
| 20 | 24 |      { | 
| 21 | 25 | $this->timetable | 
| @@ -61,11 +61,11 @@ | ||
| 61 | 61 | $comments = $request['comments']; | 
| 62 | 62 | |
| 63 | 63 | $vacancies = $this->calendar() | 
| 64 | - ->forService($service->id) | |
| 65 | - ->withDuration($service->duration) | |
| 66 | - ->forDate($request['date']) | |
| 67 | - ->atTime($request['time']) | |
| 68 | - ->find(); | |
| 64 | + ->forService($service->id) | |
| 65 | + ->withDuration($service->duration) | |
| 66 | + ->forDate($request['date']) | |
| 67 | + ->atTime($request['time']) | |
| 68 | + ->find(); | |
| 69 | 69 | |
| 70 | 70 |          if ($vacancies->count() == 0) { | 
| 71 | 71 | // TODO: Log failure feedback message / raise exception | 
| @@ -97,6 +97,9 @@ | ||
| 97 | 97 | return $vacancy !== null; | 
| 98 | 98 | } | 
| 99 | 99 | |
| 100 | + /** | |
| 101 | + * @param string $key | |
| 102 | + */ | |
| 100 | 103 | protected function arrayGroupBy($key, $array) | 
| 101 | 104 |      { | 
| 102 | 105 | $grouped = []; | 
| @@ -105,9 +105,9 @@ | ||
| 105 | 105 | if(!is_numeric($statement['capacity'])) | 
| 106 | 106 |          { | 
| 107 | 107 | $vacancyValues['humanresource_id'] = $business->humanresources() | 
| 108 | -                                                          ->where('slug', $statement['capacity']) | |
| 109 | - ->first() | |
| 110 | - ->id; | |
| 108 | +                                                            ->where('slug', $statement['capacity']) | |
| 109 | + ->first() | |
| 110 | + ->id; | |
| 111 | 111 | } | 
| 112 | 112 | else | 
| 113 | 113 |          { | 
| @@ -54,16 +54,16 @@ | ||
| 54 | 54 | } | 
| 55 | 55 | |
| 56 | 56 | $fromTime = $this->wrappedObject | 
| 57 | - ->vacancy | |
| 58 | - ->start_at | |
| 59 | - ->timezone($this->wrappedObject->business->timezone) | |
| 60 | - ->format($timeFormat); | |
| 57 | + ->vacancy | |
| 58 | + ->start_at | |
| 59 | + ->timezone($this->wrappedObject->business->timezone) | |
| 60 | + ->format($timeFormat); | |
| 61 | 61 | |
| 62 | 62 | $toTime = $this->wrappedObject | 
| 63 | - ->vacancy | |
| 64 | - ->finish_at | |
| 65 | - ->timezone($this->wrappedObject->business->timezone) | |
| 66 | - ->format($timeFormat); | |
| 63 | + ->vacancy | |
| 64 | + ->finish_at | |
| 65 | + ->timezone($this->wrappedObject->business->timezone) | |
| 66 | + ->format($timeFormat); | |
| 67 | 67 | |
| 68 | 68 | return ['from' => $fromTime, 'to' => $toTime]; | 
| 69 | 69 | } | 
| @@ -33,6 +33,9 @@ | ||
| 33 | 33 |          $this->addStatement('  '.$startAt.' - '.$finishAt); | 
| 34 | 34 | } | 
| 35 | 35 | |
| 36 | + /** | |
| 37 | + * @param string $statement | |
| 38 | + */ | |
| 36 | 39 | protected function addStatement($statement) | 
| 37 | 40 |      { | 
| 38 | 41 | $this->statement[] = $statement; | 
| @@ -221,11 +221,11 @@ discard block | ||
| 221 | 221 | public function hasRoomBetween(Carbon $startAt, Carbon $finishAt) | 
| 222 | 222 |      { | 
| 223 | 223 | return $this->capacity > $this->business | 
| 224 | - ->bookings() | |
| 225 | - ->active() | |
| 226 | - ->affectingInterval($startAt, $finishAt) | |
| 227 | - ->affectingHumanresource($this->humanresource_id) | |
| 228 | - ->count() && | |
| 224 | + ->bookings() | |
| 225 | + ->active() | |
| 226 | + ->affectingInterval($startAt, $finishAt) | |
| 227 | + ->affectingHumanresource($this->humanresource_id) | |
| 228 | + ->count() && | |
| 229 | 229 | ($this->start_at <= $startAt && $this->finish_at >= $finishAt); | 
| 230 | 230 | } | 
| 231 | 231 | |
| @@ -241,12 +241,12 @@ discard block | ||
| 241 | 241 | } | 
| 242 | 242 | |
| 243 | 243 | $count = $this->business | 
| 244 | - ->bookings() | |
| 245 | -                      ->with('humanresource') | |
| 246 | - ->active() | |
| 247 | - ->affectingHumanresource($this->humanresource_id) | |
| 248 | - ->affectingInterval($startAt, $finishAt) | |
| 249 | - ->count(); | |
| 244 | + ->bookings() | |
| 245 | +                        ->with('humanresource') | |
| 246 | + ->active() | |
| 247 | + ->affectingHumanresource($this->humanresource_id) | |
| 248 | + ->affectingInterval($startAt, $finishAt) | |
| 249 | + ->count(); | |
| 250 | 250 | |
| 251 | 251 | return $this->capacity - intval($count); | 
| 252 | 252 | } |