Completed
Push — master ( c6be7b...ac3a6a )
by claudio
03:59
created
app/Http/Controllers/Companies/Groups/EmployeesController.php 2 patches
Unused Use Statements   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -3,8 +3,8 @@
 block discarded – undo
3 3
 namespace plunner\Http\Controllers\Companies\Groups;
4 4
 
5 5
 use Illuminate\Support\Facades\Response;
6
-use plunner\Group;
7 6
 use plunner\Employee;
7
+use plunner\Group;
8 8
 use plunner\Http\Controllers\Controller;
9 9
 use plunner\Http\Requests\Companies\Groups\EmployeeRequest;
10 10
 
Please login to merge, or discard this patch.
Braces   +3 added lines, -2 removed lines patch added patch discarded remove patch
@@ -71,8 +71,9 @@
 block discarded – undo
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
     }
Please login to merge, or discard this patch.
app/Employee.php 1 patch
Braces   +3 added lines, -2 removed lines patch added patch discarded remove patch
@@ -114,8 +114,9 @@
 block discarded – undo
114 114
     public function belongsToGroup(Group $group)
115 115
     {
116 116
         $group = $this->groups()->where('id', $group->id)->first();
117
-        if(is_object($group) && $group->exists)
118
-            return true;
117
+        if(is_object($group) && $group->exists) {
118
+                    return true;
119
+        }
119 120
         return false;
120 121
     }
121 122
 
Please login to merge, or discard this patch.