dsd-meetme / backend

app/Http/Controllers/Employees/Planners/MeetingsController.php

Unused Use Statements +2 added lines, -5 removed lines

@@ -2,14 +2,11 @@
 
 namespace plunner\Http\Controllers\Employees\Planners;
 
-use Illuminate\Http\Request;
-
-use plunner\Http\Requests;
-use plunner\Http\Requests\Employees\MeetingRequest;
 use plunner\Http\Controllers\Controller;
+use plunner\Http\Requests\Employees\MeetingRequest;
 
-use plunner\Meeting;
 use plunner\Group;
+use plunner\Meeting;
 
 class MeetingsController extends Controller
 {

Spacing +5 added lines, -5 removed lines

@@ -18,8 +18,8 @@ 
      */
     public function __construct()
     {
-        config(['auth.model' => \plunner\Planner::class]);
-        config(['jwt.user' => \plunner\Planner::class]);
+        config([ 'auth.model' => \plunner\Planner::class ]);
+        config([ 'jwt.user' => \plunner\Planner::class ]);
         $this->middleware('jwt.authandrefresh:mode-en');
     }
 
@@ -60,7 +60,7 @@ 
     {
         return in_array($title, array_map(function($meeting)
         {
-            return $meeting['title'];
+            return $meeting[ 'title' ];
         },
         $group->meetings->toArray()));
     }
@@ -77,7 +77,7 @@ 
         $group = Group::findOrFail($groupId);
         $this->authorize($group);
         $input = $request->all();
-        if ($this->checkTitleAlreadyExists($input['title'], $group))//TODO create methods inside model is nto the correct way
+        if ($this->checkTitleAlreadyExists($input[ 'title' ], $group))//TODO create methods inside model is nto the correct way
         {
             abort(422); //TODO thsi is note the correct way
             //TODO This is not a contraint to do. but anyway if we want to do constraints we have to use requests
@@ -101,7 +101,7 @@ 
         $group = Group::findOrFail($groupId);
         $this->authorize($group);
         $input = $request->all();
-        if ($this->checkTitleAlreadyExists($input['title'], $group))
+        if ($this->checkTitleAlreadyExists($input[ 'title' ], $group))
         {
             abort(422);
         }

Braces +5 added lines, -2 removed lines

@@ -77,9 +77,12 @@
         $group = Group::findOrFail($groupId);
         $this->authorize($group);
         $input = $request->all();
-        if ($this->checkTitleAlreadyExists($input['title'], $group))//TODO create methods inside model is nto the correct way
+        if ($this->checkTitleAlreadyExists($input['title'], $group)) {
+            //TODO create methods inside model is nto the correct way
         {
-            abort(422); //TODO thsi is note the correct way
+            abort(422);
+        }
+        //TODO thsi is note the correct way
             //TODO This is not a contraint to do. but anyway if we want to do constraints we have to use requests
         }
         $meeting = $group->meetings()->create($input);

app/Http/Controllers/Employees/Planners/MeetingTimeslotsController.php

Unused Use Statements +1 added lines, -6 removed lines

@@ -2,13 +2,8 @@
 
 namespace plunner\Http\Controllers\Employees\Planners;
 
-use Illuminate\Http\Request;
-
-use plunner\Http\Requests;
-use plunner\Http\Requests\Employees\MeetingRequest;
 use plunner\Http\Controllers\Controller;
-
-use plunner\Meeting;
+use plunner\Http\Requests\Employees\MeetingRequest;
 
 class MeetingTimeslotsController extends Controller
 {

Spacing +2 added lines, -2 removed lines

@@ -17,8 +17,8 @@
      */
     public function __construct()
     {
-        config(['auth.model' => \plunner\Planner::class]);
-        config(['jwt.user' => \plunner\Planner::class]);
+        config([ 'auth.model' => \plunner\Planner::class ]);
+        config([ 'jwt.user' => \plunner\Planner::class ]);
         $this->middleware('jwt.authandrefresh:mode-en');
     }
 

app/Employee.php

Spacing +8 added lines, -8 removed lines

@@ -50,19 +50,19 @@ 
      *
      * @var array
      */
-    protected $fillable = ['name', 'email', 'password'];
+    protected $fillable = [ 'name', 'email', 'password' ];
 
     /**
      * The attributes excluded from the model's JSON form.
      *
      * @var array
      */
-    protected $hidden = ['password', 'remember_token', 'pivot'];
+    protected $hidden = [ 'password', 'remember_token', 'pivot' ];
 
     /**
      * @var array
      */
-    protected $appends = ['is_planner'];
+    protected $appends = [ 'is_planner' ];
 
     public function getIsPlannerAttribute()
     {
@@ -98,7 +98,7 @@ 
      * to get all meetings where the user can go user groups with meetings
      * @return \Illuminate\Database\Eloquent\Relations\BelongsToMany
      */
-    public function meetings(){
+    public function meetings() {
         //TODO durign the inserting chek if the meeting is of a group of the user
         return $this->belongsToMany(Meeting::class);
     }
@@ -114,16 +114,16 @@ 
     public function getEmailForPasswordReset()
     {
         list(, $caller) = debug_backtrace(false);
-        if(isset($caller['class']))
-            $caller = explode('\\', $caller['class']);
+        if (isset($caller[ 'class' ]))
+            $caller = explode('\\', $caller[ 'class' ]);
         else
             $caller = '';
 
         //check if this function is called by email sender
-        if ((count($caller) && $caller[count($caller) - 1] == 'PasswordBroker') || (defined('HHVM_VERSION') && $caller == ''))
+        if ((count($caller) && $caller[ count($caller) - 1 ] == 'PasswordBroker') || (defined('HHVM_VERSION') && $caller == ''))
             return $this->email;
         //return unique identify for token repository
-        return $this->email . $this->company->id;
+        return $this->email.$this->company->id;
     }
 
     /**

Braces +8 added lines, -6 removed lines

@@ -114,14 +114,16 @@
     public function getEmailForPasswordReset()
     {
         list(, $caller) = debug_backtrace(false);
-        if(isset($caller['class']))
-            $caller = explode('\\', $caller['class']);
-        else
-            $caller = '';
+        if(isset($caller['class'])) {
+                    $caller = explode('\\', $caller['class']);
+        } else {
+                    $caller = '';
+        }
 
         //check if this function is called by email sender
-        if ((count($caller) && $caller[count($caller) - 1] == 'PasswordBroker') || (defined('HHVM_VERSION') && $caller == ''))
-            return $this->email;
+        if ((count($caller) && $caller[count($caller) - 1] == 'PasswordBroker') || (defined('HHVM_VERSION') && $caller == '')) {
+                    return $this->email;
+        }
         //return unique identify for token repository
         return $this->email . $this->company->id;
     }

app/Http/Controllers/Employees/Groups/GroupsController.php

Spacing +4 added lines, -4 removed lines

@@ -16,8 +16,8 @@ 
      */
     public function __construct()
     {
-        config(['auth.model' => \plunner\Employee::class]);
-        config(['jwt.user' => \plunner\Employee::class]);
+        config([ 'auth.model' => \plunner\Employee::class ]);
+        config([ 'jwt.user' => \plunner\Employee::class ]);
         $this->middleware('jwt.authandrefresh:mode-en');
     }
 
@@ -29,9 +29,9 @@ 
     public function index()
     {
         $employee = \Auth::user();
-        return $employee->groups()->with(['meetings'=>function($query) {
+        return $employee->groups()->with([ 'meetings'=>function($query) {
             $query->where('start_time', '=', NULL);
-        }])->get();
+        } ])->get();
         //TODO get only current meetings
     }
 

app/Http/routes.php

Spacing +36 added lines, -36 removed lines

@@ -19,41 +19,41 @@ 
  * Companies
  */
 
-Route::group(['namespace' => 'Companies', 'prefix' => 'companies'], function() {
+Route::group([ 'namespace' => 'Companies', 'prefix' => 'companies' ], function() {
 
     //\Auth
 
-    Route::group(['namespace' => 'Auth'], function() {
-        Route::group(['prefix' => 'auth'], function () {
+    Route::group([ 'namespace' => 'Auth' ], function() {
+        Route::group([ 'prefix' => 'auth' ], function() {
             // Authentication routes...
-            Route::post('login', ['as' => 'companies.auth.login','uses'=>'AuthController@postLogin']);
+            Route::post('login', [ 'as' => 'companies.auth.login', 'uses'=>'AuthController@postLogin' ]);
 
             // Registration routes...
-            Route::post('register', ['as' => 'companies.auth.register', 'uses'=>'AuthController@postRegister']);
+            Route::post('register', [ 'as' => 'companies.auth.register', 'uses'=>'AuthController@postRegister' ]);
 
         });
 
-        Route::group(['prefix' => 'password'], function () {
+        Route::group([ 'prefix' => 'password' ], function() {
             // Password reset link request routes...
-            Route::post('email', ['as' => 'companies.auth.email', 'uses'=>'PasswordController@postEmail']);
+            Route::post('email', [ 'as' => 'companies.auth.email', 'uses'=>'PasswordController@postEmail' ]);
 
             // Password reset routes...
-            Route::post('reset', ['as' => 'companies.auth.reset', 'uses'=>'PasswordController@postReset']);
+            Route::post('reset', [ 'as' => 'companies.auth.reset', 'uses'=>'PasswordController@postReset' ]);
         });
     });
 
-    Route::group(['namespace' => 'Employees'], function() {
-        Route::resource('employees', 'EmployeesController', ['except' => ['create', 'edit']]);
+    Route::group([ 'namespace' => 'Employees' ], function() {
+        Route::resource('employees', 'EmployeesController', [ 'except' => [ 'create', 'edit' ] ]);
     });
 
-    Route::group(['namespace' => 'Groups'], function() {
-        Route::resource('groups', 'GroupsController', ['except' => ['create', 'edit']]);
-        Route::resource('groups.employees', 'EmployeesController', ['only' => ['store', 'index', 'destroy']]);
+    Route::group([ 'namespace' => 'Groups' ], function() {
+        Route::resource('groups', 'GroupsController', [ 'except' => [ 'create', 'edit' ] ]);
+        Route::resource('groups.employees', 'EmployeesController', [ 'only' => [ 'store', 'index', 'destroy' ] ]);
     });
 
-    Route::group(['namespace' => 'Company', 'prefix' => 'company'], function() {
-        Route::get('/', ['as' => 'companies.company.index', 'uses' => 'CompanyController@index']);
-        Route::put('/', ['as' => 'companies.company.update', 'uses' => 'CompanyController@update']);
+    Route::group([ 'namespace' => 'Company', 'prefix' => 'company' ], function() {
+        Route::get('/', [ 'as' => 'companies.company.index', 'uses' => 'CompanyController@index' ]);
+        Route::put('/', [ 'as' => 'companies.company.update', 'uses' => 'CompanyController@update' ]);
     });
 });
 
@@ -61,46 +61,46 @@ 
 /**
  * Employees
  */
-Route::group(['namespace' => 'Employees', 'prefix' => 'employees'], function() {
+Route::group([ 'namespace' => 'Employees', 'prefix' => 'employees' ], function() {
 
     //\Auth
-    Route::group(['namespace' => 'Auth'], function() {
-        Route::group(['prefix' => 'auth'], function () {
+    Route::group([ 'namespace' => 'Auth' ], function() {
+        Route::group([ 'prefix' => 'auth' ], function() {
             // Authentication routes...
-            Route::post('login', ['as' => 'companies.auth.login', 'uses'=>'AuthController@postLogin']);
+            Route::post('login', [ 'as' => 'companies.auth.login', 'uses'=>'AuthController@postLogin' ]);
 
             // Registration routes...
             //Route::post('register', ['as' => 'companies.auth.register', 'uses'=>'AuthController@postRegister']); //the registration is managed by the company
 
         });
 
-        Route::group(['prefix' => 'password'], function () {
+        Route::group([ 'prefix' => 'password' ], function() {
             // Password reset link request routes...
-            Route::post('email', ['as' => 'companies.auth.email', 'uses'=>'PasswordController@postEmail']);
+            Route::post('email', [ 'as' => 'companies.auth.email', 'uses'=>'PasswordController@postEmail' ]);
 
             // Password reset routes...
-            Route::post('reset', ['as' => 'companies.auth.reset', 'uses'=>'PasswordController@postReset']);
+            Route::post('reset', [ 'as' => 'companies.auth.reset', 'uses'=>'PasswordController@postReset' ]);
         });
     });
 
-    Route::group(['namespace' => 'Calendars'], function() {
-        Route::resource('calendars', 'CalendarsController', ['except' => ['create', 'edit']]);
-        Route::post('calendars/caldav', ['as' => 'employees.calendars.caldav','uses'=>'CalendarsController@storeCaldav']);
-        Route::post('calendars/calendars', ['as' => 'employees.calendars.calendars','uses'=>'CalendarsController@getCalendars']); //TODO it's correct post? or not
+    Route::group([ 'namespace' => 'Calendars' ], function() {
+        Route::resource('calendars', 'CalendarsController', [ 'except' => [ 'create', 'edit' ] ]);
+        Route::post('calendars/caldav', [ 'as' => 'employees.calendars.caldav', 'uses'=>'CalendarsController@storeCaldav' ]);
+        Route::post('calendars/calendars', [ 'as' => 'employees.calendars.calendars', 'uses'=>'CalendarsController@getCalendars' ]); //TODO it's correct post? or not
     });
 
-    Route::group(['namespace' => 'Planners', 'prefix' => 'planners'], function() {
-        Route::resource('groups', 'GroupsController', ['only' => ['index', 'show']]);
-        Route::resource('groups.meetings', 'MeetingsController', ['except' => ['create', 'edit']]);
-        Route::resource('groups.meetings.timeslots', 'MeetingTimeslotsController', ['except' => ['create', 'edit']]);
+    Route::group([ 'namespace' => 'Planners', 'prefix' => 'planners' ], function() {
+        Route::resource('groups', 'GroupsController', [ 'only' => [ 'index', 'show' ] ]);
+        Route::resource('groups.meetings', 'MeetingsController', [ 'except' => [ 'create', 'edit' ] ]);
+        Route::resource('groups.meetings.timeslots', 'MeetingTimeslotsController', [ 'except' => [ 'create', 'edit' ] ]);
     });
 
-    Route::group(['namespace' => 'Groups'], function() {
-        Route::resource('groups', 'GroupsController', ['only' => ['index', 'show']]);
+    Route::group([ 'namespace' => 'Groups' ], function() {
+        Route::resource('groups', 'GroupsController', [ 'only' => [ 'index', 'show' ] ]);
     });
 
-    Route::group(['namespace' => 'Employee', 'prefix' => 'employee'], function() {
-        Route::get('/', ['as' => 'employees.employee.index', 'uses' => 'EmployeeController@index']);
-        Route::put('/', ['as' => 'employees.employee.update', 'uses' => 'EmployeeController@update']);
+    Route::group([ 'namespace' => 'Employee', 'prefix' => 'employee' ], function() {
+        Route::get('/', [ 'as' => 'employees.employee.index', 'uses' => 'EmployeeController@index' ]);
+        Route::put('/', [ 'as' => 'employees.employee.update', 'uses' => 'EmployeeController@update' ]);
     });
 });
\ No newline at end of file