Completed
Pull Request — master (#5)
by
unknown
04:19 queued 02:08
created
src/Vacancy/VacancyManager.php 4 patches
Indentation   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -105,9 +105,9 @@
 block discarded – undo
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
         {
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -102,7 +102,7 @@
 block discarded – undo
102 102
             ];
103 103
 
104 104
         // If capacity is a slug, grab the humanresource
105
-        if(!is_numeric($statement['capacity']))
105
+        if (!is_numeric($statement['capacity']))
106 106
         {
107 107
             $vacancyValues['humanresource_id'] = $business->humanresources()
108 108
                                                           ->where('slug', $statement['capacity'])
Please login to merge, or discard this patch.
Braces   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -108,8 +108,7 @@
 block discarded – undo
108 108
                                                           ->where('slug', $statement['capacity'])
109 109
                                                           ->first()
110 110
                                                           ->id;
111
-        }
112
-        else
111
+        } else
113 112
         {
114 113
             $vacancyValues['capacity'] = intval($statement['capacity']);
115 114
         }
Please login to merge, or discard this patch.
Doc Comments   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -124,6 +124,9 @@  discard block
 block discarded – undo
124 124
         return $vacancy !== null;
125 125
     }
126 126
 
127
+    /**
128
+     * @param string $key
129
+     */
127 130
     protected function arrayGroupBy($key, $array)
128 131
     {
129 132
         $grouped = [];
@@ -156,7 +159,6 @@  discard block
 block discarded – undo
156 159
     /**
157 160
      * [generateAvailability description].
158 161
      *
159
-     * @param Illuminate\Database\Eloquent\Collection $vacancies
160 162
      * @param string                                  $startDate
161 163
      * @param int                                     $futureDays
162 164
      *
Please login to merge, or discard this patch.
src/Models/Appointment.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -440,11 +440,11 @@  discard block
 block discarded – undo
440 440
         $carbon = Carbon::parse('today midnight')->timezone('UTC');
441 441
 
442 442
         return $query
443
-            ->where(function ($query) use ($carbon) {
443
+            ->where(function($query) use ($carbon) {
444 444
 
445 445
                 $query->whereIn('status', [Self::STATUS_RESERVED, Self::STATUS_CONFIRMED])
446 446
                     ->where('start_at', '<=', $carbon)
447
-                    ->orWhere(function ($query) use ($carbon) {
447
+                    ->orWhere(function($query) use ($carbon) {
448 448
                         $query->where('start_at', '>=', $carbon);
449 449
                     });
450 450
             });
@@ -532,17 +532,17 @@  discard block
 block discarded – undo
532 532
         $finishAt->timezone('UTC');
533 533
 
534 534
         return $query
535
-            ->where(function ($query) use ($startAt, $finishAt) {
535
+            ->where(function($query) use ($startAt, $finishAt) {
536 536
 
537
-                $query->where(function ($query) use ($startAt, $finishAt) {
537
+                $query->where(function($query) use ($startAt, $finishAt) {
538 538
                     $query->where('finish_at', '>=', $finishAt)
539 539
                             ->where('start_at', '<', $startAt);
540 540
                 })
541
-                ->orWhere(function ($query) use ($startAt, $finishAt) {
541
+                ->orWhere(function($query) use ($startAt, $finishAt) {
542 542
                     $query->where('finish_at', '<=', $finishAt)
543 543
                             ->where('finish_at', '>', $startAt);
544 544
                 })
545
-                ->orWhere(function ($query) use ($startAt, $finishAt) {
545
+                ->orWhere(function($query) use ($startAt, $finishAt) {
546 546
                     $query->where('start_at', '>=', $startAt)
547 547
                             ->where('start_at', '<', $finishAt);
548 548
                 });
Please login to merge, or discard this patch.
src/Presenters/AppointmentPresenter.php 1 patch
Indentation   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -66,16 +66,16 @@
 block discarded – undo
66 66
         }
67 67
 
68 68
         $fromTime = $this->wrappedObject
69
-                         ->vacancy
70
-                         ->start_at
71
-                         ->timezone($this->timezone)
72
-                         ->format($timeFormat);
69
+                            ->vacancy
70
+                            ->start_at
71
+                            ->timezone($this->timezone)
72
+                            ->format($timeFormat);
73 73
 
74 74
         $toTime = $this->wrappedObject
75
-                       ->vacancy
76
-                       ->finish_at
77
-                       ->timezone($this->timezone)
78
-                       ->format($timeFormat);
75
+                        ->vacancy
76
+                        ->finish_at
77
+                        ->timezone($this->timezone)
78
+                        ->format($timeFormat);
79 79
 
80 80
         return ['from' => $fromTime, 'to' => $toTime];
81 81
     }
Please login to merge, or discard this patch.
migrations/2016_07_09_000001_add_humanresource_calendar_link.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -12,7 +12,7 @@  discard block
 block discarded – undo
12 12
      */
13 13
     public function up()
14 14
     {
15
-        Schema::table('humanresources', function (Blueprint $table) {
15
+        Schema::table('humanresources', function(Blueprint $table) {
16 16
             $table->string('calendar_link')->nullable();
17 17
         });
18 18
     }
@@ -24,7 +24,7 @@  discard block
 block discarded – undo
24 24
      */
25 25
     public function down()
26 26
     {
27
-        Schema::table('humanresources', function (Blueprint $table) {
27
+        Schema::table('humanresources', function(Blueprint $table) {
28 28
             $table->dropColumn('calendar_link');
29 29
         });
30 30
     }
Please login to merge, or discard this patch.
src/Models/Vacancy.php 1 patch
Indentation   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -266,11 +266,11 @@  discard block
 block discarded – undo
266 266
     public function hasRoomBetween(Carbon $startAt, Carbon $finishAt)
267 267
     {
268 268
         return $this->capacity > $this->business
269
-                                      ->bookings()
270
-                                      ->active()
271
-                                      ->affectingInterval($startAt, $finishAt)
272
-                                      ->affectingHumanresource($this->humanresource_id)
273
-                                      ->count() &&
269
+                                        ->bookings()
270
+                                        ->active()
271
+                                        ->affectingInterval($startAt, $finishAt)
272
+                                        ->affectingHumanresource($this->humanresource_id)
273
+                                        ->count() &&
274 274
             ($this->start_at <= $startAt && $this->finish_at >= $finishAt);
275 275
     }
276 276
 
@@ -286,11 +286,11 @@  discard block
 block discarded – undo
286 286
         }
287 287
 
288 288
         $count = $this->business
289
-                      ->bookings()
290
-                      ->active()
291
-                      ->affectingHumanresource($this->humanresource_id)
292
-                      ->affectingInterval($startAt, $finishAt)
293
-                      ->count();
289
+                        ->bookings()
290
+                        ->active()
291
+                        ->affectingHumanresource($this->humanresource_id)
292
+                        ->affectingInterval($startAt, $finishAt)
293
+                        ->count();
294 294
 
295 295
         return $this->capacity - intval($count);
296 296
     }
Please login to merge, or discard this patch.
src/Concierge.php 1 patch
Indentation   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -72,11 +72,11 @@  discard block
 block discarded – undo
72 72
         $comments = $request['comments'];
73 73
 
74 74
         $vacancies = $this->calendar()
75
-                          ->forService($service->id)
76
-                          ->withDuration($service->duration)
77
-                          ->forDate($request['date'])
78
-                          ->atTime($request['time'], $request['timezone'])
79
-                          ->find();
75
+                            ->forService($service->id)
76
+                            ->withDuration($service->duration)
77
+                            ->forDate($request['date'])
78
+                            ->atTime($request['time'], $request['timezone'])
79
+                            ->find();
80 80
 
81 81
         if ($vacancies->count() == 0) {
82 82
             // TODO: Log failure feedback message / raise exception
@@ -160,10 +160,10 @@  discard block
 block discarded – undo
160 160
         $fromDate = Carbon::parse($fromDate)->timezone($this->business->timezone);
161 161
 
162 162
         $count = $this->business
163
-                      ->vacancies()
164
-                      ->future($fromDate)
165
-                      ->until($fromDate->addDays($days))
166
-                      ->count();
163
+                        ->vacancies()
164
+                        ->future($fromDate)
165
+                        ->until($fromDate->addDays($days))
166
+                        ->count();
167 167
 
168 168
         return $count > 0;
169 169
     }
Please login to merge, or discard this patch.
src/Presenters/BusinessPresenter.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -23,7 +23,7 @@
 block discarded – undo
23 23
     {
24 24
         $url = parse_url($this->wrappedObject->social_facebook);
25 25
 
26
-        if(!$this->wrappedObject->social_facebook || !array_key_exists('path', $url)){
26
+        if (!$this->wrappedObject->social_facebook || !array_key_exists('path', $url)) {
27 27
             return "<img class=\"img-thumbnail\" src=\"//placehold.it/100x100\" height=\"100\" width=\"100\" alt=\"{$this->wrappedObject->name}\"/>";
28 28
         }
29 29
 
Please login to merge, or discard this patch.
src/Timetable/Strategies/TimetableTimeslotStrategy.php 1 patch
Indentation   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -19,11 +19,11 @@
 block discarded – undo
19 19
     protected function initTimetable($starting, $days)
20 20
     {
21 21
         $this->timetable
22
-             ->interval($this->interval)
23
-             ->format('date.service.time')
24
-             ->from($starting)
25
-             ->future($days)
26
-             ->init();
22
+                ->interval($this->interval)
23
+                ->format('date.service.time')
24
+                ->from($starting)
25
+                ->future($days)
26
+                ->init();
27 27
     }
28 28
 
29 29
     /**
Please login to merge, or discard this patch.