@@ -1,21 +1,21 @@ |
||
1 | 1 | <?php |
2 | 2 | /** |
3 | - * Created by PhpStorm. |
|
4 | - * User: Claudio Cardinale <[email protected]> |
|
5 | - * Date: 24/11/15 |
|
6 | - * Time: 21.58 |
|
7 | - * This program is free software; you can redistribute it and/or |
|
8 | - * modify it under the terms of the GNU General Public License |
|
9 | - * as published by the Free Software Foundation; either version 2 |
|
10 | - * of the License, or (at your option) any later version. |
|
11 | - * This program is distributed in the hope that it will be useful, |
|
12 | - * but WITHOUT ANY WARRANTY; without even the implied warranty of |
|
13 | - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the |
|
14 | - * GNU General Public License for more details. |
|
15 | - * You should have received a copy of the GNU General Public License |
|
16 | - * along with this program; if not, write to the Free Software |
|
17 | - * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. |
|
18 | - */ |
|
3 | + * Created by PhpStorm. |
|
4 | + * User: Claudio Cardinale <[email protected]> |
|
5 | + * Date: 24/11/15 |
|
6 | + * Time: 21.58 |
|
7 | + * This program is free software; you can redistribute it and/or |
|
8 | + * modify it under the terms of the GNU General Public License |
|
9 | + * as published by the Free Software Foundation; either version 2 |
|
10 | + * of the License, or (at your option) any later version. |
|
11 | + * This program is distributed in the hope that it will be useful, |
|
12 | + * but WITHOUT ANY WARRANTY; without even the implied warranty of |
|
13 | + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the |
|
14 | + * GNU General Public License for more details. |
|
15 | + * You should have received a copy of the GNU General Public License |
|
16 | + * along with this program; if not, write to the Free Software |
|
17 | + * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. |
|
18 | + */ |
|
19 | 19 | |
20 | 20 | namespace plunner; |
21 | 21 |
@@ -51,8 +51,9 @@ |
||
51 | 51 | |
52 | 52 | public function getPlannerNameAttribute() |
53 | 53 | { |
54 | - if(is_object($this->planner) && $this->planner->exists) |
|
55 | - return $this->planner->name; |
|
54 | + if(is_object($this->planner) && $this->planner->exists) { |
|
55 | + return $this->planner->name; |
|
56 | + } |
|
56 | 57 | } |
57 | 58 | |
58 | 59 | /** |
@@ -71,8 +71,9 @@ |
||
71 | 71 | $employee = Employee::findOrFail($employeeId); |
72 | 72 | $this->authorize($employee); |
73 | 73 | $group = Group::findOrFail($groupId); |
74 | - if(!$employee->belongsToGroup($group)) |
|
75 | - return Response::json(['error' => 'employId <> groupId'],404); |
|
74 | + if(!$employee->belongsToGroup($group)) { |
|
75 | + return Response::json(['error' => 'employId <> groupId'],404); |
|
76 | + } |
|
76 | 77 | $employee->groups()->detach($groupId); |
77 | 78 | return $group->employees; |
78 | 79 | } |
@@ -53,8 +53,9 @@ discard block |
||
53 | 53 | // |
54 | 54 | $company = \Auth::user(); |
55 | 55 | $input = $request->all(); |
56 | - if(isset($input['password'])) |
|
57 | - $input['password'] = bcrypt($input['password']); |
|
56 | + if(isset($input['password'])) { |
|
57 | + $input['password'] = bcrypt($input['password']); |
|
58 | + } |
|
58 | 59 | $employee = $company->employees()->create($input); |
59 | 60 | return $employee; |
60 | 61 | } |
@@ -86,8 +87,9 @@ discard block |
||
86 | 87 | $employee = Employee::findOrFail($id); |
87 | 88 | $this->authorize($employee); |
88 | 89 | $input = $request->all(); |
89 | - if(isset($input['password'])) |
|
90 | - $input['password'] = bcrypt($input['password']); |
|
90 | + if(isset($input['password'])) { |
|
91 | + $input['password'] = bcrypt($input['password']); |
|
92 | + } |
|
91 | 93 | $employee->update($input); |
92 | 94 | return $employee; |
93 | 95 | } |
@@ -1,21 +1,21 @@ |
||
1 | 1 | <?php |
2 | 2 | /** |
3 | - * Created by PhpStorm. |
|
4 | - * User: Claudio Cardinale <[email protected]> |
|
5 | - * Date: 03/12/15 |
|
6 | - * Time: 2.33 |
|
7 | - * This program is free software; you can redistribute it and/or |
|
8 | - * modify it under the terms of the GNU General Public License |
|
9 | - * as published by the Free Software Foundation; either version 2 |
|
10 | - * of the License, or (at your option) any later version. |
|
11 | - * This program is distributed in the hope that it will be useful, |
|
12 | - * but WITHOUT ANY WARRANTY; without even the implied warranty of |
|
13 | - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the |
|
14 | - * GNU General Public License for more details. |
|
15 | - * You should have received a copy of the GNU General Public License |
|
16 | - * along with this program; if not, write to the Free Software |
|
17 | - * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. |
|
18 | - */ |
|
3 | + * Created by PhpStorm. |
|
4 | + * User: Claudio Cardinale <[email protected]> |
|
5 | + * Date: 03/12/15 |
|
6 | + * Time: 2.33 |
|
7 | + * This program is free software; you can redistribute it and/or |
|
8 | + * modify it under the terms of the GNU General Public License |
|
9 | + * as published by the Free Software Foundation; either version 2 |
|
10 | + * of the License, or (at your option) any later version. |
|
11 | + * This program is distributed in the hope that it will be useful, |
|
12 | + * but WITHOUT ANY WARRANTY; without even the implied warranty of |
|
13 | + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the |
|
14 | + * GNU General Public License for more details. |
|
15 | + * You should have received a copy of the GNU General Public License |
|
16 | + * along with this program; if not, write to the Free Software |
|
17 | + * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. |
|
18 | + */ |
|
19 | 19 | |
20 | 20 | namespace plunner\Console\Commands\SyncCaldav; |
21 | 21 |
@@ -91,11 +91,11 @@ discard block |
||
91 | 91 | try |
92 | 92 | { |
93 | 93 | $events = $this->getEvents(); |
94 | - }catch (\it\thecsea\caldav_client_adapter\CaldavException $e) |
|
94 | + } catch (\it\thecsea\caldav_client_adapter\CaldavException $e) |
|
95 | 95 | { |
96 | 96 | \Event::fire(new ErrorEvent($this->calendar, $e->getMessage())); |
97 | 97 | return ; |
98 | - }catch(\Illuminate\Contracts\Encryption\DecryptException $e){ |
|
98 | + } catch(\Illuminate\Contracts\Encryption\DecryptException $e){ |
|
99 | 99 | \Event::fire(new ErrorEvent($this->calendar, $e->getMessage())); |
100 | 100 | return ; |
101 | 101 | } |
@@ -110,10 +110,11 @@ discard block |
||
110 | 110 | |
111 | 111 | //insert new timeslots |
112 | 112 | foreach($events as $event){ |
113 | - if(!($event = $this->parseEvent($event))) |
|
114 | - \Event::fire(new ErrorEvent($this->calendar, 'problem during the parsing an event')); |
|
115 | - else |
|
116 | - $calendarMain->timeslots()->create($event); |
|
113 | + if(!($event = $this->parseEvent($event))) { |
|
114 | + \Event::fire(new ErrorEvent($this->calendar, 'problem during the parsing an event')); |
|
115 | + } else { |
|
116 | + $calendarMain->timeslots()->create($event); |
|
117 | + } |
|
117 | 118 | } |
118 | 119 | \Event::fire(new OkEvent($this->calendar)); |
119 | 120 | } |
@@ -126,17 +127,20 @@ discard block |
||
126 | 127 | { |
127 | 128 | $pattern = "/^((DTSTART;)|(DTEND;))(.*)\$/m"; |
128 | 129 | if(preg_match_all($pattern, $event->getData(), $matches)){ |
129 | - if(!isset($matches[4]) || count($matches[4]) != 2) |
|
130 | - return null; |
|
130 | + if(!isset($matches[4]) || count($matches[4]) != 2) { |
|
131 | + return null; |
|
132 | + } |
|
131 | 133 | $ret = []; |
132 | - if($tmp = $this->parseDate($matches[4][0])) |
|
133 | - $ret['time_start'] = $tmp; |
|
134 | - else |
|
135 | - return null; |
|
136 | - if($tmp = $this->parseDate($matches[4][1])) |
|
137 | - $ret['time_end'] = $tmp; |
|
138 | - else |
|
139 | - return null; |
|
134 | + if($tmp = $this->parseDate($matches[4][0])) { |
|
135 | + $ret['time_start'] = $tmp; |
|
136 | + } else { |
|
137 | + return null; |
|
138 | + } |
|
139 | + if($tmp = $this->parseDate($matches[4][1])) { |
|
140 | + $ret['time_end'] = $tmp; |
|
141 | + } else { |
|
142 | + return null; |
|
143 | + } |
|
140 | 144 | return $ret; |
141 | 145 | } |
142 | 146 | return null; |
@@ -153,7 +157,7 @@ discard block |
||
153 | 157 | if($matches[1][0] == 'TZID=') |
154 | 158 | { |
155 | 159 | return \DateTime::createFromFormat('Ymd\THis', $matches[5][0], new \DateTimeZone($matches[4][0])); |
156 | - }else if($matches[1][0] == 'VALUE=' && $matches[4][0] == 'DATE') |
|
160 | + } else if($matches[1][0] == 'VALUE=' && $matches[4][0] == 'DATE') |
|
157 | 161 | { |
158 | 162 | return \DateTime::createFromFormat('Ymd\THis', $matches[5][0].'T000000'); |
159 | 163 | } |
@@ -82,7 +82,7 @@ |
||
82 | 82 | { |
83 | 83 | $event = $this->schedule->command('sync:caldav '.$calendar->calendar_id)->withoutOverlapping(); |
84 | 84 | if($event->isDue($this->laravel)) |
85 | - $event->run($this->laravel); |
|
85 | + $event->run($this->laravel); |
|
86 | 86 | } |
87 | 87 | |
88 | 88 | /** |
@@ -53,10 +53,11 @@ discard block |
||
53 | 53 | { |
54 | 54 | // |
55 | 55 | $calendarId = $this->argument('calendarId'); |
56 | - if(is_numeric($calendarId)) |
|
57 | - $this->makeForeground(Caldav::findOrFail($calendarId)); |
|
58 | - else |
|
59 | - $this->syncAll(); |
|
56 | + if(is_numeric($calendarId)) { |
|
57 | + $this->makeForeground(Caldav::findOrFail($calendarId)); |
|
58 | + } else { |
|
59 | + $this->syncAll(); |
|
60 | + } |
|
60 | 61 | } |
61 | 62 | |
62 | 63 | private function syncAll() |
@@ -65,13 +66,15 @@ discard block |
||
65 | 66 | if($this->option('background')) { |
66 | 67 | \Log::debug(self::BACKGROUND_MOD_MEX); |
67 | 68 | $this->info(self::BACKGROUND_MOD_MEX); |
68 | - foreach ($calendars as $calendar) |
|
69 | - $this->makeBackground($calendar); |
|
69 | + foreach ($calendars as $calendar) { |
|
70 | + $this->makeBackground($calendar); |
|
71 | + } |
|
70 | 72 | \Log::debug(self::BACKGROUND_COMPLETED_MEX); |
71 | 73 | $this->info(self::BACKGROUND_COMPLETED_MEX); |
72 | - }else |
|
73 | - foreach($calendars as $calendar) |
|
74 | + } else { |
|
75 | + foreach($calendars as $calendar) |
|
74 | 76 | $this->makeForeground($calendar); |
77 | + } |
|
75 | 78 | } |
76 | 79 | |
77 | 80 | /** |
@@ -81,8 +84,9 @@ discard block |
||
81 | 84 | private function makeBackground(Caldav $calendar) |
82 | 85 | { |
83 | 86 | $event = $this->schedule->command('sync:caldav '.$calendar->calendar_id)->withoutOverlapping(); |
84 | - if($event->isDue($this->laravel)) |
|
85 | - $event->run($this->laravel); |
|
87 | + if($event->isDue($this->laravel)) { |
|
88 | + $event->run($this->laravel); |
|
89 | + } |
|
86 | 90 | } |
87 | 91 | |
88 | 92 | /** |
@@ -66,8 +66,9 @@ discard block |
||
66 | 66 | $employee = \Auth::user(); |
67 | 67 | $input = $request->all(); |
68 | 68 | $calendar = $employee->calendars()->create($input); |
69 | - if(isset($input['password'])) |
|
70 | - $input['password'] = \Crypt::encrypt($input['password']); |
|
69 | + if(isset($input['password'])) { |
|
70 | + $input['password'] = \Crypt::encrypt($input['password']); |
|
71 | + } |
|
71 | 72 | $calendar->caldav()->create($input); |
72 | 73 | //TODO test |
73 | 74 | //TODO validator |
@@ -107,15 +108,17 @@ discard block |
||
107 | 108 | if($caldav){ |
108 | 109 | $this->validateCaldav($request); |
109 | 110 | } |
110 | - if(isset($input['password'])) |
|
111 | - $input['password'] = \Crypt::encrypt($input['password']); |
|
111 | + if(isset($input['password'])) { |
|
112 | + $input['password'] = \Crypt::encrypt($input['password']); |
|
113 | + } |
|
112 | 114 | $calendar->update($input); |
113 | 115 | //TODO test |
114 | 116 | //TODO validator |
115 | 117 | //TODO check if caldav exists? |
116 | 118 | |
117 | - if($caldav) |
|
118 | - $caldav->update($input); |
|
119 | + if($caldav) { |
|
120 | + $caldav->update($input); |
|
121 | + } |
|
119 | 122 | return $calendar; |
120 | 123 | } |
121 | 124 | |
@@ -148,7 +151,7 @@ discard block |
||
148 | 151 | $caldavClient->connect($request->input('url'), $request->input('username'), $request->input('password')); |
149 | 152 | $calendars = $caldavClient->findCalendars(); |
150 | 153 | return array_keys($calendars); |
151 | - }catch (\it\thecsea\caldav_client_adapter\CaldavException $e) |
|
154 | + } catch (\it\thecsea\caldav_client_adapter\CaldavException $e) |
|
152 | 155 | { |
153 | 156 | return Response::json(['error' => $e->getMessage()],422); |
154 | 157 | } |
@@ -1,10 +1,10 @@ discard block |
||
1 | 1 | <?php |
2 | 2 | /** |
3 | - * Created by PhpStorm. |
|
4 | - * User: claudio |
|
5 | - * Date: 12/12/15 |
|
6 | - * Time: 15.41 |
|
7 | - */ |
|
3 | + * Created by PhpStorm. |
|
4 | + * User: claudio |
|
5 | + * Date: 12/12/15 |
|
6 | + * Time: 15.41 |
|
7 | + */ |
|
8 | 8 | |
9 | 9 | namespace plunner\Console\Commands\Optimise; |
10 | 10 | |
@@ -45,8 +45,8 @@ discard block |
||
45 | 45 | private $company; |
46 | 46 | |
47 | 47 | /** |
48 | - * @var Schedule laravel schedule object needed to perform command in background |
|
49 | - */ |
|
48 | + * @var Schedule laravel schedule object needed to perform command in background |
|
49 | + */ |
|
50 | 50 | private $schedule; |
51 | 51 | |
52 | 52 | /** |
@@ -413,7 +413,7 @@ discard block |
||
413 | 413 | $diff = explode(':',$diff->format('%R:%d:%h:%i:%s')); |
414 | 414 | $diff = $diff[1]*86400 + $diff[2]*3600 + $diff[3]*60 + $diff[4]; |
415 | 415 | //if($diff[0] != '-' && $diff != 0) |
416 | - // throw new OptimiseException('timeslot time <= startTime'); |
|
416 | + // throw new OptimiseException('timeslot time <= startTime'); |
|
417 | 417 | //TODO fix check |
418 | 418 | //TODO check if diff makes sense |
419 | 419 | //TODO check upper limit |
@@ -156,7 +156,7 @@ discard block |
||
156 | 156 | $solver = $this->setData($solver); |
157 | 157 | $solver = $solver->solve(); |
158 | 158 | $this->solver = $solver; |
159 | - }catch(\Exception $e) |
|
159 | + } catch(\Exception $e) |
|
160 | 160 | { |
161 | 161 | \Event::fire(new ErrorEvent($this->company, $e->getMessage())); |
162 | 162 | throw new OptimiseException('Optimising error', 0, $e); |
@@ -180,7 +180,7 @@ discard block |
||
180 | 180 | try { |
181 | 181 | $this->saveMeetings($this->solver); |
182 | 182 | $this->saveEmployeesMeetings($this->solver); |
183 | - }catch(\Exception $e) |
|
183 | + } catch(\Exception $e) |
|
184 | 184 | { |
185 | 185 | \Event::fire(new ErrorEvent($this->company, $e->getMessage())); |
186 | 186 | throw new OptimiseException('Optimising error', 0, $e); |
@@ -327,7 +327,7 @@ discard block |
||
327 | 327 | foreach($meetings as $meeting){ |
328 | 328 | if($usersMeetingsTmp->contains('meeting_id', $meeting)){ |
329 | 329 | $ret[$user][$meeting] = 1; |
330 | - }else{ |
|
330 | + } else{ |
|
331 | 331 | $ret[$user][$meeting] = 0; |
332 | 332 | } |
333 | 333 | } |
@@ -373,8 +373,9 @@ discard block |
||
373 | 373 | static private function fillTimeSlots(array $array, $id, \Illuminate\Support\Collection $timeSlots, $fill = '0') |
374 | 374 | { |
375 | 375 | foreach($timeSlots as $timeSlot) { |
376 | - if(!isset($array[$id])) |
|
377 | - $array[$id] = []; |
|
376 | + if(!isset($array[$id])) { |
|
377 | + $array[$id] = []; |
|
378 | + } |
|
378 | 379 | $array[$id] = self::arrayPadInterval($array[$id], $timeSlot->time_start, $timeSlot->time_end, $fill); |
379 | 380 | } |
380 | 381 | return $array; |
@@ -389,8 +390,9 @@ discard block |
||
389 | 390 | static private function fillRow(array $array, $id, $until, $fill = '0') |
390 | 391 | { |
391 | 392 | for($i = 1; $i <= $until; $i++){ |
392 | - if(!isset($array[$id][$i])) |
|
393 | - $array[$id][$i] = $fill; |
|
393 | + if(!isset($array[$id][$i])) { |
|
394 | + $array[$id][$i] = $fill; |
|
395 | + } |
|
394 | 396 | } |
395 | 397 | |
396 | 398 | return $array; |
@@ -405,8 +407,9 @@ discard block |
||
405 | 407 | */ |
406 | 408 | static private function arrayPadInterval(array $array, $from, $to, $pad = '0') |
407 | 409 | { |
408 | - for($i = $from; $i<$to; $i++) |
|
409 | - $array[$i] = $pad; |
|
410 | + for($i = $from; $i<$to; $i++) { |
|
411 | + $array[$i] = $pad; |
|
412 | + } |
|
410 | 413 | return $array; |
411 | 414 | } |
412 | 415 |
@@ -32,8 +32,8 @@ |
||
32 | 32 | protected $description = 'Optimise meetings'; |
33 | 33 | |
34 | 34 | /** |
35 | - * @var Schedule laravel schedule object needed to perform command in background |
|
36 | - */ |
|
35 | + * @var Schedule laravel schedule object needed to perform command in background |
|
36 | + */ |
|
37 | 37 | private $schedule; |
38 | 38 | |
39 | 39 | /** |
@@ -62,10 +62,11 @@ discard block |
||
62 | 62 | |
63 | 63 | //TODO try...catch with destruct |
64 | 64 | $companyId = $this->argument('companyId'); |
65 | - if(is_numeric($companyId)) |
|
66 | - $this->makeForeground(Company::findOrFail($companyId)); |
|
67 | - else |
|
68 | - $this->syncAll(); |
|
65 | + if(is_numeric($companyId)) { |
|
66 | + $this->makeForeground(Company::findOrFail($companyId)); |
|
67 | + } else { |
|
68 | + $this->syncAll(); |
|
69 | + } |
|
69 | 70 | } |
70 | 71 | |
71 | 72 | private function syncAll() |
@@ -74,13 +75,15 @@ discard block |
||
74 | 75 | if($this->option('background')) { |
75 | 76 | \Log::debug(self::BACKGROUND_MOD_MEX); |
76 | 77 | $this->info(self::BACKGROUND_MOD_MEX); |
77 | - foreach ($calendars as $calendar) |
|
78 | - $this->makeBackground($calendar); |
|
78 | + foreach ($calendars as $calendar) { |
|
79 | + $this->makeBackground($calendar); |
|
80 | + } |
|
79 | 81 | \Log::debug(self::BACKGROUND_COMPLETED_MEX); |
80 | 82 | $this->info(self::BACKGROUND_COMPLETED_MEX); |
81 | - }else |
|
82 | - foreach($calendars as $calendar) |
|
83 | + } else { |
|
84 | + foreach($calendars as $calendar) |
|
83 | 85 | $this->makeForeground($calendar); |
86 | + } |
|
84 | 87 | } |
85 | 88 | |
86 | 89 | /** |
@@ -90,8 +93,9 @@ discard block |
||
90 | 93 | private function makeBackground(Company $company) |
91 | 94 | { |
92 | 95 | $event = $this->schedule->command('optimise:meetings '.$company->id)->withoutOverlapping(); |
93 | - if($event->isDue($this->laravel)) |
|
94 | - $event->run($this->laravel); |
|
96 | + if($event->isDue($this->laravel)) { |
|
97 | + $event->run($this->laravel); |
|
98 | + } |
|
95 | 99 | } |
96 | 100 | |
97 | 101 | /** |