@@ -11,8 +11,8 @@ discard block |
||
11 | 11 | { |
12 | 12 | public function __construct() |
13 | 13 | { |
14 | - config(['auth.model' => \plunner\Employee::class]); |
|
15 | - config(['jwt.user' => \plunner\Employee::class]); |
|
14 | + config([ 'auth.model' => \plunner\Employee::class ]); |
|
15 | + config([ 'jwt.user' => \plunner\Employee::class ]); |
|
16 | 16 | $this->middleware('jwt.authandrefresh:mode-en'); |
17 | 17 | } |
18 | 18 | |
@@ -27,10 +27,10 @@ discard block |
||
27 | 27 | $employee = \Auth::user(); |
28 | 28 | $meetings = $employee->meetings(); |
29 | 29 | if ($request->query('current')) |
30 | - $meetings->where(function ($query) { //parenthesis for conditions ...(C1 OR C2)... |
|
31 | - $query->where('start_time', '=', NULL);//to be planned |
|
30 | + $meetings->where(function($query) { //parenthesis for conditions ...(C1 OR C2)... |
|
31 | + $query->where('start_time', '=', NULL); //to be planned |
|
32 | 32 | //datetime to consider timezone, don't use mysql NOW() |
33 | - $query->orWhere('start_time', '>=', new \DateTime());//planned |
|
33 | + $query->orWhere('start_time', '>=', new \DateTime()); //planned |
|
34 | 34 | }); |
35 | 35 | return $meetings->get(); |
36 | 36 | } |
@@ -16,8 +16,8 @@ discard block |
||
16 | 16 | { |
17 | 17 | public function __construct() |
18 | 18 | { |
19 | - config(['auth.model' => \plunner\Planner::class]); |
|
20 | - config(['jwt.user' => \plunner\Planner::class]); |
|
19 | + config([ 'auth.model' => \plunner\Planner::class ]); |
|
20 | + config([ 'jwt.user' => \plunner\Planner::class ]); |
|
21 | 21 | $this->middleware('jwt.authandrefresh:mode-en'); |
22 | 22 | } |
23 | 23 | |
@@ -35,7 +35,7 @@ discard block |
||
35 | 35 | |
36 | 36 | if ($meeting->group_id == $groupId) |
37 | 37 | return $meeting->timeslots; |
38 | - return Response::json(['error' => 'meeting->group_id <> groupId'], 403); |
|
38 | + return Response::json([ 'error' => 'meeting->group_id <> groupId' ], 403); |
|
39 | 39 | } |
40 | 40 | |
41 | 41 | /** |
@@ -57,7 +57,7 @@ discard block |
||
57 | 57 | |
58 | 58 | if ($meeting->group_id == $groupId && $timeslot->meeting_id == $meetingId) |
59 | 59 | return $timeslot; |
60 | - return Response::json(['error' => 'meeting->group_id <> groupId || timeslot->meeting_id <> meetingId'], 403); |
|
60 | + return Response::json([ 'error' => 'meeting->group_id <> groupId || timeslot->meeting_id <> meetingId' ], 403); |
|
61 | 61 | } |
62 | 62 | |
63 | 63 | /** |
@@ -73,7 +73,7 @@ discard block |
||
73 | 73 | $meeting = Meeting::findOrFail($meetingId); |
74 | 74 | $this->authorize($meeting); |
75 | 75 | if ($meeting->start_time != NULL) |
76 | - return Response::json(['error' => 'the meeting is already planned'], 422); |
|
76 | + return Response::json([ 'error' => 'the meeting is already planned' ], 422); |
|
77 | 77 | |
78 | 78 | $input = $request->all(); |
79 | 79 | |
@@ -81,7 +81,7 @@ discard block |
||
81 | 81 | $timeslot = $meeting->timeslots()->create($input); |
82 | 82 | return $timeslot; |
83 | 83 | } |
84 | - return Response::json(['error' => 'meeting->group_id <> groupId'], 403); |
|
84 | + return Response::json([ 'error' => 'meeting->group_id <> groupId' ], 403); |
|
85 | 85 | } |
86 | 86 | |
87 | 87 | /** |
@@ -97,7 +97,7 @@ discard block |
||
97 | 97 | $meeting = Meeting::findOrFail($meetingId); |
98 | 98 | $this->authorize($meeting); |
99 | 99 | if ($meeting->start_time != NULL) |
100 | - return Response::json(['error' => 'the meeting is already planned'], 422); |
|
100 | + return Response::json([ 'error' => 'the meeting is already planned' ], 422); |
|
101 | 101 | $timeslot = MeetingTimeslot::findOrFail($timeslotId); |
102 | 102 | $this->authorize($timeslot); |
103 | 103 | |
@@ -107,7 +107,7 @@ discard block |
||
107 | 107 | $timeslot->update($input); |
108 | 108 | return $timeslot; |
109 | 109 | } |
110 | - return Response::json(['error' => 'meeting->group_id <> groupId || timeslot->meeting_id <> meetingId'], 403); |
|
110 | + return Response::json([ 'error' => 'meeting->group_id <> groupId || timeslot->meeting_id <> meetingId' ], 403); |
|
111 | 111 | } |
112 | 112 | |
113 | 113 | /** |
@@ -125,7 +125,7 @@ discard block |
||
125 | 125 | $meeting = Meeting::findOrFail($meetingId); |
126 | 126 | $this->authorize($meeting); |
127 | 127 | if ($meeting->start_time != NULL) |
128 | - return Response::json(['error' => 'the meeting is already planned'], 422); |
|
128 | + return Response::json([ 'error' => 'the meeting is already planned' ], 422); |
|
129 | 129 | $timeslot = MeetingTimeslot::findOrFail($timeslotId); |
130 | 130 | $this->authorize($timeslot); |
131 | 131 | |
@@ -133,6 +133,6 @@ discard block |
||
133 | 133 | $timeslot->delete(); |
134 | 134 | return $timeslot; |
135 | 135 | } |
136 | - return Response::json(['error' => 'meeting->group_id <> groupId || timeslot->meeting_id <> meetingId'], 403); |
|
136 | + return Response::json([ 'error' => 'meeting->group_id <> groupId || timeslot->meeting_id <> meetingId' ], 403); |
|
137 | 137 | } |
138 | 138 | } |
@@ -40,7 +40,7 @@ discard block |
||
40 | 40 | * cn = company normal |
41 | 41 | * @var array |
42 | 42 | */ |
43 | - protected $custom = ['mode' => 'cn']; |
|
43 | + protected $custom = [ 'mode' => 'cn' ]; |
|
44 | 44 | |
45 | 45 | /** |
46 | 46 | * Create a new authentication controller instance. |
@@ -48,19 +48,19 @@ discard block |
||
48 | 48 | */ |
49 | 49 | public function __construct() |
50 | 50 | { |
51 | - config(['auth.model' => \plunner\Company::class]); |
|
52 | - config(['jwt.user' => \plunner\Company::class]); |
|
51 | + config([ 'auth.model' => \plunner\Company::class ]); |
|
52 | + config([ 'jwt.user' => \plunner\Company::class ]); |
|
53 | 53 | } |
54 | 54 | |
55 | 55 | public function postLogin(Request $request) |
56 | 56 | { |
57 | 57 | //remember me |
58 | - $this->validate($request, ['remember' => 'required|boolean']); |
|
58 | + $this->validate($request, [ 'remember' => 'required|boolean' ]); |
|
59 | 59 | if ($request->input('remember', false)) { |
60 | - config(['jwt.ttl' => '43200']); //30 days |
|
61 | - $this->custom = array_merge($this->custom, ['remember' => 'true']); |
|
60 | + config([ 'jwt.ttl' => '43200' ]); //30 days |
|
61 | + $this->custom = array_merge($this->custom, [ 'remember' => 'true' ]); |
|
62 | 62 | } else |
63 | - $this->custom = array_merge($this->custom, ['remember' => 'false']); |
|
63 | + $this->custom = array_merge($this->custom, [ 'remember' => 'false' ]); |
|
64 | 64 | return $this->postLoginOriginal($request); |
65 | 65 | } |
66 | 66 | |
@@ -88,9 +88,9 @@ discard block |
||
88 | 88 | protected function create(array $data) |
89 | 89 | { |
90 | 90 | return Company::create([ |
91 | - 'name' => $data['name'], |
|
92 | - 'email' => $data['email'], |
|
93 | - 'password' => bcrypt($data['password']), |
|
91 | + 'name' => $data[ 'name' ], |
|
92 | + 'email' => $data[ 'email' ], |
|
93 | + 'password' => bcrypt($data[ 'password' ]), |
|
94 | 94 | ]); |
95 | 95 | } |
96 | 96 |
@@ -19,41 +19,41 @@ discard block |
||
19 | 19 | * Companies |
20 | 20 | */ |
21 | 21 | |
22 | -Route::group(['namespace' => 'Companies', 'prefix' => 'companies'], function () { |
|
22 | +Route::group([ 'namespace' => 'Companies', 'prefix' => 'companies' ], function() { |
|
23 | 23 | |
24 | 24 | //\Auth |
25 | 25 | |
26 | - Route::group(['namespace' => 'Auth'], function () { |
|
27 | - Route::group(['prefix' => 'auth'], function () { |
|
26 | + Route::group([ 'namespace' => 'Auth' ], function() { |
|
27 | + Route::group([ 'prefix' => 'auth' ], function() { |
|
28 | 28 | // Authentication routes... |
29 | - Route::post('login', ['as' => 'companies.auth.login', 'uses' => 'AuthController@postLogin']); |
|
29 | + Route::post('login', [ 'as' => 'companies.auth.login', 'uses' => 'AuthController@postLogin' ]); |
|
30 | 30 | |
31 | 31 | // Registration routes... |
32 | - Route::post('register', ['as' => 'companies.auth.register', 'uses' => 'AuthController@postRegister']); |
|
32 | + Route::post('register', [ 'as' => 'companies.auth.register', 'uses' => 'AuthController@postRegister' ]); |
|
33 | 33 | |
34 | 34 | }); |
35 | 35 | |
36 | - Route::group(['prefix' => 'password'], function () { |
|
36 | + Route::group([ 'prefix' => 'password' ], function() { |
|
37 | 37 | // Password reset link request routes... |
38 | - Route::post('email', ['as' => 'companies.auth.email', 'uses' => 'PasswordController@postEmail']); |
|
38 | + Route::post('email', [ 'as' => 'companies.auth.email', 'uses' => 'PasswordController@postEmail' ]); |
|
39 | 39 | |
40 | 40 | // Password reset routes... |
41 | - Route::post('reset', ['as' => 'companies.auth.reset', 'uses' => 'PasswordController@postReset']); |
|
41 | + Route::post('reset', [ 'as' => 'companies.auth.reset', 'uses' => 'PasswordController@postReset' ]); |
|
42 | 42 | }); |
43 | 43 | }); |
44 | 44 | |
45 | - Route::group(['namespace' => 'Employees'], function () { |
|
46 | - Route::resource('employees', 'EmployeesController', ['except' => ['create', 'edit']]); |
|
45 | + Route::group([ 'namespace' => 'Employees' ], function() { |
|
46 | + Route::resource('employees', 'EmployeesController', [ 'except' => [ 'create', 'edit' ] ]); |
|
47 | 47 | }); |
48 | 48 | |
49 | - Route::group(['namespace' => 'Groups'], function () { |
|
50 | - Route::resource('groups', 'GroupsController', ['except' => ['create', 'edit']]); |
|
51 | - Route::resource('groups.employees', 'EmployeesController', ['only' => ['store', 'index', 'destroy']]); |
|
49 | + Route::group([ 'namespace' => 'Groups' ], function() { |
|
50 | + Route::resource('groups', 'GroupsController', [ 'except' => [ 'create', 'edit' ] ]); |
|
51 | + Route::resource('groups.employees', 'EmployeesController', [ 'only' => [ 'store', 'index', 'destroy' ] ]); |
|
52 | 52 | }); |
53 | 53 | |
54 | - Route::group(['namespace' => 'Company', 'prefix' => 'company'], function () { |
|
55 | - Route::get('/', ['as' => 'companies.company.index', 'uses' => 'CompanyController@index']); |
|
56 | - Route::put('/', ['as' => 'companies.company.update', 'uses' => 'CompanyController@update']); |
|
54 | + Route::group([ 'namespace' => 'Company', 'prefix' => 'company' ], function() { |
|
55 | + Route::get('/', [ 'as' => 'companies.company.index', 'uses' => 'CompanyController@index' ]); |
|
56 | + Route::put('/', [ 'as' => 'companies.company.update', 'uses' => 'CompanyController@update' ]); |
|
57 | 57 | }); |
58 | 58 | }); |
59 | 59 | |
@@ -61,52 +61,52 @@ discard block |
||
61 | 61 | /** |
62 | 62 | * Employees |
63 | 63 | */ |
64 | -Route::group(['namespace' => 'Employees', 'prefix' => 'employees'], function () { |
|
64 | +Route::group([ 'namespace' => 'Employees', 'prefix' => 'employees' ], function() { |
|
65 | 65 | |
66 | 66 | //\Auth |
67 | - Route::group(['namespace' => 'Auth'], function () { |
|
68 | - Route::group(['prefix' => 'auth'], function () { |
|
67 | + Route::group([ 'namespace' => 'Auth' ], function() { |
|
68 | + Route::group([ 'prefix' => 'auth' ], function() { |
|
69 | 69 | // Authentication routes... |
70 | - Route::post('login', ['as' => 'companies.auth.login', 'uses' => 'AuthController@postLogin']); |
|
70 | + Route::post('login', [ 'as' => 'companies.auth.login', 'uses' => 'AuthController@postLogin' ]); |
|
71 | 71 | |
72 | 72 | // Registration routes... |
73 | 73 | //Route::post('register', ['as' => 'companies.auth.register', 'uses'=>'AuthController@postRegister']); //the registration is managed by the company |
74 | 74 | |
75 | 75 | }); |
76 | 76 | |
77 | - Route::group(['prefix' => 'password'], function () { |
|
77 | + Route::group([ 'prefix' => 'password' ], function() { |
|
78 | 78 | // Password reset link request routes... |
79 | - Route::post('email', ['as' => 'companies.auth.email', 'uses' => 'PasswordController@postEmail']); |
|
79 | + Route::post('email', [ 'as' => 'companies.auth.email', 'uses' => 'PasswordController@postEmail' ]); |
|
80 | 80 | |
81 | 81 | // Password reset routes... |
82 | - Route::post('reset', ['as' => 'companies.auth.reset', 'uses' => 'PasswordController@postReset']); |
|
82 | + Route::post('reset', [ 'as' => 'companies.auth.reset', 'uses' => 'PasswordController@postReset' ]); |
|
83 | 83 | }); |
84 | 84 | }); |
85 | 85 | |
86 | - Route::group(['namespace' => 'Calendars'], function () { |
|
87 | - Route::resource('calendars', 'CalendarsController', ['except' => ['create', 'edit']]); |
|
88 | - Route::post('calendars/caldav', ['as' => 'employees.calendars.caldav', 'uses' => 'CalendarsController@storeCaldav']); |
|
89 | - Route::post('calendars/calendars', ['as' => 'employees.calendars.calendars', 'uses' => 'CalendarsController@getCalendars']); |
|
90 | - Route::resource('calendars.timeslots', 'TimeslotsController', ['except' => ['create', 'edit']]); |
|
86 | + Route::group([ 'namespace' => 'Calendars' ], function() { |
|
87 | + Route::resource('calendars', 'CalendarsController', [ 'except' => [ 'create', 'edit' ] ]); |
|
88 | + Route::post('calendars/caldav', [ 'as' => 'employees.calendars.caldav', 'uses' => 'CalendarsController@storeCaldav' ]); |
|
89 | + Route::post('calendars/calendars', [ 'as' => 'employees.calendars.calendars', 'uses' => 'CalendarsController@getCalendars' ]); |
|
90 | + Route::resource('calendars.timeslots', 'TimeslotsController', [ 'except' => [ 'create', 'edit' ] ]); |
|
91 | 91 | }); |
92 | 92 | |
93 | - Route::group(['namespace' => 'Planners', 'prefix' => 'planners'], function () { |
|
94 | - Route::resource('groups', 'GroupsController', ['only' => ['index', 'show']]); |
|
95 | - Route::resource('groups.meetings', 'MeetingsController', ['except' => ['create', 'edit']]); |
|
96 | - Route::resource('groups.meetings.timeslots', 'MeetingTimeslotsController', ['except' => ['create', 'edit']]); |
|
93 | + Route::group([ 'namespace' => 'Planners', 'prefix' => 'planners' ], function() { |
|
94 | + Route::resource('groups', 'GroupsController', [ 'only' => [ 'index', 'show' ] ]); |
|
95 | + Route::resource('groups.meetings', 'MeetingsController', [ 'except' => [ 'create', 'edit' ] ]); |
|
96 | + Route::resource('groups.meetings.timeslots', 'MeetingTimeslotsController', [ 'except' => [ 'create', 'edit' ] ]); |
|
97 | 97 | }); |
98 | 98 | |
99 | - Route::group(['namespace' => 'Groups'], function () { |
|
100 | - Route::resource('groups', 'GroupsController', ['only' => ['index', 'show']]); |
|
99 | + Route::group([ 'namespace' => 'Groups' ], function() { |
|
100 | + Route::resource('groups', 'GroupsController', [ 'only' => [ 'index', 'show' ] ]); |
|
101 | 101 | }); |
102 | 102 | |
103 | 103 | |
104 | - Route::group(['namespace' => 'Meetings'], function () { |
|
105 | - Route::resource('meetings', 'MeetingsController', ['only' => ['index', 'show']]); |
|
104 | + Route::group([ 'namespace' => 'Meetings' ], function() { |
|
105 | + Route::resource('meetings', 'MeetingsController', [ 'only' => [ 'index', 'show' ] ]); |
|
106 | 106 | }); |
107 | 107 | |
108 | - Route::group(['namespace' => 'Employee', 'prefix' => 'employee'], function () { |
|
109 | - Route::get('/', ['as' => 'employees.employee.index', 'uses' => 'EmployeeController@index']); |
|
110 | - Route::put('/', ['as' => 'employees.employee.update', 'uses' => 'EmployeeController@update']); |
|
108 | + Route::group([ 'namespace' => 'Employee', 'prefix' => 'employee' ], function() { |
|
109 | + Route::get('/', [ 'as' => 'employees.employee.index', 'uses' => 'EmployeeController@index' ]); |
|
110 | + Route::put('/', [ 'as' => 'employees.employee.update', 'uses' => 'EmployeeController@update' ]); |
|
111 | 111 | }); |
112 | 112 | }); |
113 | 113 | \ No newline at end of file |
@@ -10,8 +10,8 @@ discard block |
||
10 | 10 | { |
11 | 11 | public function __construct() |
12 | 12 | { |
13 | - config(['auth.model' => \plunner\Company::class]); |
|
14 | - config(['jwt.user' => \plunner\Company::class]); |
|
13 | + config([ 'auth.model' => \plunner\Company::class ]); |
|
14 | + config([ 'jwt.user' => \plunner\Company::class ]); |
|
15 | 15 | $this->middleware('jwt.authandrefresh:mode-cn'); |
16 | 16 | } |
17 | 17 | |
@@ -36,9 +36,9 @@ discard block |
||
36 | 36 | public function update(CompanyRequest $request) |
37 | 37 | { |
38 | 38 | $company = \Auth::user(); |
39 | - $input = $request->only(['name', 'password']); |
|
40 | - if (isset($input['password'])) |
|
41 | - $input['password'] = bcrypt($input['password']); |
|
39 | + $input = $request->only([ 'name', 'password' ]); |
|
40 | + if (isset($input[ 'password' ])) |
|
41 | + $input[ 'password' ] = bcrypt($input[ 'password' ]); |
|
42 | 42 | $company->update($input); |
43 | 43 | return $company; |
44 | 44 | } |
@@ -10,8 +10,8 @@ discard block |
||
10 | 10 | { |
11 | 11 | public function __construct() |
12 | 12 | { |
13 | - config(['auth.model' => \plunner\Employee::class]); |
|
14 | - config(['jwt.user' => \plunner\Employee::class]); |
|
13 | + config([ 'auth.model' => \plunner\Employee::class ]); |
|
14 | + config([ 'jwt.user' => \plunner\Employee::class ]); |
|
15 | 15 | $this->middleware('jwt.authandrefresh:mode-en'); |
16 | 16 | } |
17 | 17 | |
@@ -36,9 +36,9 @@ discard block |
||
36 | 36 | public function update(EmployeeRequest $request) |
37 | 37 | { |
38 | 38 | $employee = \Auth::user(); |
39 | - $input = $request->only(['name', 'password']); |
|
40 | - if (isset($input['password'])) |
|
41 | - $input['password'] = bcrypt($input['password']); |
|
39 | + $input = $request->only([ 'name', 'password' ]); |
|
40 | + if (isset($input[ 'password' ])) |
|
41 | + $input[ 'password' ] = bcrypt($input[ 'password' ]); |
|
42 | 42 | $employee->update($input); |
43 | 43 | return $employee; |
44 | 44 | } |
@@ -36,7 +36,7 @@ |
||
36 | 36 | * @param Exception $previous |
37 | 37 | * @param bool $empty |
38 | 38 | */ |
39 | - public function __construct($message = "", $code = 0, Exception $previous = null, $empty = false) |
|
39 | + public function __construct($message = "", $code = 0, Exception $previous = null, $empty = false) |
|
40 | 40 | { |
41 | 41 | parent::__construct($message, $code, $previous); |
42 | 42 | $this->empty = $empty; |
@@ -70,17 +70,17 @@ discard block |
||
70 | 70 | */ |
71 | 71 | private function makeForeground(Company $company) |
72 | 72 | { |
73 | - $this->info('Optimisation company ' . $company->id . ' started'); |
|
73 | + $this->info('Optimisation company '.$company->id.' started'); |
|
74 | 74 | try { |
75 | 75 | (new Optimise($company, $this->schedule, $this->laravel))->optimise()->save(); |
76 | - $this->info('Optimisation ' . $company->id . ' completed'); |
|
77 | - }catch(OptimiseException $e) { |
|
76 | + $this->info('Optimisation '.$company->id.' completed'); |
|
77 | + } catch (OptimiseException $e) { |
|
78 | 78 | if ($e->isEmpty()) { |
79 | - $mex = 'Company ' . $company->id . ' doesn\'t have sufficient data'; |
|
79 | + $mex = 'Company '.$company->id.' doesn\'t have sufficient data'; |
|
80 | 80 | $this->warn($mex); |
81 | 81 | \Log::info($mex); |
82 | 82 | } else { |
83 | - $mex = 'Error during optimisation of company ' . $company->id . ': ' . $e->getMessage(); |
|
83 | + $mex = 'Error during optimisation of company '.$company->id.': '.$e->getMessage(); |
|
84 | 84 | $this->error($mex); |
85 | 85 | //TODO log cause, but don't send it to the user |
86 | 86 | //\Log::error($mex); //already logged in listener |
@@ -109,7 +109,7 @@ discard block |
||
109 | 109 | */ |
110 | 110 | private function makeBackground(Company $company) |
111 | 111 | { |
112 | - $event = $this->schedule->command('optimise:meetings ' . $company->id)->withoutOverlapping(); |
|
112 | + $event = $this->schedule->command('optimise:meetings '.$company->id)->withoutOverlapping(); |
|
113 | 113 | if ($event->isDue($this->laravel)) |
114 | 114 | $event->run($this->laravel); |
115 | 115 | } |
@@ -110,9 +110,9 @@ discard block |
||
110 | 110 | $caldavClient = new SimpleCaldavAdapter(); |
111 | 111 | $caldavClient->connect($this->calendar->url, $this->calendar->username, \Crypt::decrypt($this->calendar->password)); |
112 | 112 | $calendars = $caldavClient->findCalendars(); |
113 | - if(!isset($calendars[$this->calendar->calendar_name])) |
|
113 | + if (!isset($calendars[ $this->calendar->calendar_name ])) |
|
114 | 114 | throw new CaldavException("calendar inserted doesn't exist"); |
115 | - $caldavClient->setCalendar($calendars[$this->calendar->calendar_name]); |
|
115 | + $caldavClient->setCalendar($calendars[ $this->calendar->calendar_name ]); |
|
116 | 116 | /** |
117 | 117 | * 26 hours before to avoid tiemezone problems and dst problems |
118 | 118 | * 30 days after |
@@ -128,15 +128,15 @@ discard block |
||
128 | 128 | { |
129 | 129 | $pattern = "/^((DTSTART;)|(DTEND;))(.*)\$/m"; |
130 | 130 | if (preg_match_all($pattern, $event->getData(), $matches)) { |
131 | - if (!isset($matches[4]) || count($matches[4]) != 2) |
|
131 | + if (!isset($matches[ 4 ]) || count($matches[ 4 ]) != 2) |
|
132 | 132 | return null; |
133 | - $ret = []; |
|
134 | - if ($tmp = $this->parseDate($matches[4][0])) |
|
135 | - $ret['time_start'] = $tmp; |
|
133 | + $ret = [ ]; |
|
134 | + if ($tmp = $this->parseDate($matches[ 4 ][ 0 ])) |
|
135 | + $ret[ 'time_start' ] = $tmp; |
|
136 | 136 | else |
137 | 137 | return null; |
138 | - if ($tmp = $this->parseDate($matches[4][1])) |
|
139 | - $ret['time_end'] = $tmp; |
|
138 | + if ($tmp = $this->parseDate($matches[ 4 ][ 1 ])) |
|
139 | + $ret[ 'time_end' ] = $tmp; |
|
140 | 140 | else |
141 | 141 | return null; |
142 | 142 | return $ret; |
@@ -152,10 +152,10 @@ discard block |
||
152 | 152 | { |
153 | 153 | $pattern = "/^((TZID=)|(VALUE=))(.*):(.*)\$/m"; |
154 | 154 | if (preg_match_all($pattern, $date, $matches)) { |
155 | - if ($matches[1][0] == 'TZID=') { |
|
156 | - return \DateTime::createFromFormat('Ymd\THis', $matches[5][0], new \DateTimeZone($matches[4][0])); |
|
157 | - } else if ($matches[1][0] == 'VALUE=' && $matches[4][0] == 'DATE') { |
|
158 | - return \DateTime::createFromFormat('Ymd\THis', $matches[5][0] . 'T000000'); |
|
155 | + if ($matches[ 1 ][ 0 ] == 'TZID=') { |
|
156 | + return \DateTime::createFromFormat('Ymd\THis', $matches[ 5 ][ 0 ], new \DateTimeZone($matches[ 4 ][ 0 ])); |
|
157 | + } else if ($matches[ 1 ][ 0 ] == 'VALUE=' && $matches[ 4 ][ 0 ] == 'DATE') { |
|
158 | + return \DateTime::createFromFormat('Ymd\THis', $matches[ 5 ][ 0 ].'T000000'); |
|
159 | 159 | } |
160 | 160 | } |
161 | 161 | return null; |