From 549087fc4fc255da463c3594b6a8cd5fbf5428c7 Mon Sep 17 00:00:00 2001
From: Herpaderp Aldent <felix.a.huber@gmx.net>
Date: Fri, 9 Nov 2018 14:45:35 +0100
Subject: [PATCH 01/11] Dispatches jobs on `high` queue instead of default. *
 This is a community request and relevant for instances where the default
 queue is blocking its execution due to heavy load and slow computing time of
 all ESI requests.

---
 src/Commands/SeatGroupsUsersUpdate.php | 4 ++--
 src/Jobs/GroupDispatcher.php           | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/Commands/SeatGroupsUsersUpdate.php b/src/Commands/SeatGroupsUsersUpdate.php
index 184a9cd..64fc8ca 100644
--- a/src/Commands/SeatGroupsUsersUpdate.php
+++ b/src/Commands/SeatGroupsUsersUpdate.php
@@ -38,13 +38,13 @@ public function handle()
                     return $users_group->main_character_id != '0';
                 })
                 ->each(function ($group) {
-                    dispatch(new GroupSync($group));
+                    dispatch(new GroupSync($group))->onQueue('high');
                     $this->info(sprintf('A synchronization job has been queued in order to update %s (%s) roles.', $group->main_character->name,
                         $group->users->map(function ($user) { return $user->name; })->implode(', ')));
             });
 
         } else {
-            GroupDispatcher::dispatch();
+            GroupDispatcher::dispatch()->onQueue('high');
             $this->info('A synchronization job has been queued in order to update all SeAT Group roles.');
         }
 
diff --git a/src/Jobs/GroupDispatcher.php b/src/Jobs/GroupDispatcher.php
index b0627a0..ff7c5c2 100644
--- a/src/Jobs/GroupDispatcher.php
+++ b/src/Jobs/GroupDispatcher.php
@@ -31,7 +31,7 @@ public function handle()
             {
                $job = new GroupSync($users_group);
 
-               dispatch($job);
+               dispatch($job)->onQueue('high');
             });
 
         }, function ()

From f7c618f9e24ac63a317fb1d8a6438e6b49f78a88 Mon Sep 17 00:00:00 2001
From: Herpaderp Aldent <felix.a.huber@gmx.net>
Date: Sat, 10 Nov 2018 09:17:40 +0100
Subject: [PATCH 02/11] Empty the select box before appending options. * In
 order to avoid the avoid the reported issue:
 https://github.com/herpaderpaldent/seat-groups/issues/44

---
 .../views/partials/create-modal.blade.php         | 15 ++++++++++-----
 1 file changed, 10 insertions(+), 5 deletions(-)

diff --git a/src/resources/views/partials/create-modal.blade.php b/src/resources/views/partials/create-modal.blade.php
index 245e251..0e206a7 100644
--- a/src/resources/views/partials/create-modal.blade.php
+++ b/src/resources/views/partials/create-modal.blade.php
@@ -67,17 +67,22 @@
 
   <script type="text/javascript">
 
-    $(document).ready(function(){
+    $(document).ready(function () {
       $("#SeATGroupCreate").on('show.bs.modal', function () {
         $("#available_roles").select2({
           placeholder: "{{ trans('web::seat.select_item_add') }}"
         });
         $.ajax({
-          type: 'GET',
-          url: '{{ route('seatgroups.create') }}',
-          success: function(data){
+          type   : 'GET',
+          url    : '{{ route('seatgroups.create') }}',
+          success: function (data) {
+
+            var select = $('#available_roles');
+
+            select.empty();
+
             for (var i = 0; i < data.length; i++) {
-              $('#available_roles').append($('<option></option>').attr('value', data[i].id).text(data[i].title));
+              select.append($('<option></option>').attr('value', data[i].id).text(data[i].title));
             }
           },
           error  : function (xhr, textStatus, errorThrown) {

From 48e5523564b4df370d4a70f8bd09002645ae4624 Mon Sep 17 00:00:00 2001
From: Herpaderp Aldent <felix.a.huber@gmx.net>
Date: Sat, 10 Nov 2018 12:13:03 +0100
Subject: [PATCH 03/11] =?UTF-8?q?Respect=20not=20only=20role=20named=20?=
 =?UTF-8?q?=C2=B4superuser=C2=B4=20but=20every=20role=20with=20the=20permi?=
 =?UTF-8?q?ssion=20=C2=B4superuser=C2=B4?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 src/Jobs/GroupSync.php | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/src/Jobs/GroupSync.php b/src/Jobs/GroupSync.php
index a9ba19a..5b88854 100644
--- a/src/Jobs/GroupSync.php
+++ b/src/Jobs/GroupSync.php
@@ -76,11 +76,14 @@ public function handle()
                 $roles = collect();
                 $group = $this->group;
 
-                //Catch Superuser
+                //Catch superuser permissions
                 foreach ($group->roles as $role) {
-                    if ($role->title === 'Superuser') {
-                        $roles->push($role->id);
+                    foreach ($role->permissions as $permission) {
+                        if ($permission->title === 'superuser') {
+                            $roles->push($role->id);
+                        }
                     }
+
                 }
 
                 Seatgroup::all()->each(function ($seat_group) use ($roles, $group) {

From f732fef7e32d56d083c488f0e4c9a45178d27d1d Mon Sep 17 00:00:00 2001
From: Herpaderp Aldent <felix.a.huber@gmx.net>
Date: Sat, 10 Nov 2018 12:20:30 +0100
Subject: [PATCH 04/11] Optimized MissingRefreshTokenException * Added some
 explanatory text

---
 src/Exceptions/MissingRefreshTokenException.php | 8 ++++++++
 src/Jobs/GroupSync.php                          | 4 +++-
 2 files changed, 11 insertions(+), 1 deletion(-)

diff --git a/src/Exceptions/MissingRefreshTokenException.php b/src/Exceptions/MissingRefreshTokenException.php
index bce634c..3526f89 100644
--- a/src/Exceptions/MissingRefreshTokenException.php
+++ b/src/Exceptions/MissingRefreshTokenException.php
@@ -9,8 +9,16 @@
 namespace Herpaderpaldent\Seat\SeatGroups\Exceptions;
 
 use Exception;
+use Seat\Web\Models\User;
 
 class MissingRefreshTokenException extends Exception
 {
+    public function __construct(User $user)
+    {
+        $message = sprintf('The user group with ID %d is missing a refresh_token from %s (%d) ' .
+            'therefore the user group loses all its roles and permissions. To fix this: ask the user to login to SeAT again.', $user->group_id, $user->name, $user->id);
+
+        parent::__construct($message, 0, null);
+    }
 
 }
diff --git a/src/Jobs/GroupSync.php b/src/Jobs/GroupSync.php
index 5b88854..027c1fc 100644
--- a/src/Jobs/GroupSync.php
+++ b/src/Jobs/GroupSync.php
@@ -157,7 +157,7 @@ public function beforeStart()
                 ]);
 
                 // throw exception
-                throw new MissingRefreshTokenException();
+                $this->fail(new MissingRefreshTokenException($user));
             }
         }
     }
@@ -174,6 +174,8 @@ public function onFail($exception)
 
         ]);
 
+        $this->fail($exception);
+
     }
 
     public function onFinish()

From 97a496ae52630445e55e5ce8c450e6cf11034509 Mon Sep 17 00:00:00 2001
From: Herpaderp Aldent <felix.a.huber@gmx.net>
Date: Sat, 10 Nov 2018 12:28:29 +0100
Subject: [PATCH 05/11] Respect deactivated users * deactivated users in user
 groups are being ignored for RefreshTokenCheck * in case the user group only
 contains one deactivated user the whole sync job is being dismissed

---
 src/Jobs/GroupSync.php | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/src/Jobs/GroupSync.php b/src/Jobs/GroupSync.php
index 027c1fc..39c56f0 100644
--- a/src/Jobs/GroupSync.php
+++ b/src/Jobs/GroupSync.php
@@ -141,6 +141,10 @@ public function beforeStart()
 
         foreach ($this->group->users as $user) {
 
+            //If user is deactivated skip the refresh_token check
+            if (! $user->active)
+                continue;
+
             // If a RefreshToken is missing
             if (is_null($user->refresh_token)) {
                 // take away all roles
@@ -160,6 +164,10 @@ public function beforeStart()
                 $this->fail(new MissingRefreshTokenException($user));
             }
         }
+
+        // If deactivated user is alone in a group delete this job
+        if (! $this->group->users->first()->active && $this->group->users->count() === 1)
+            $this->delete();
     }
 
     public function onFail($exception)

From 7a81bfe7cc7b78f1953dccefeac5fffe1fa7e289 Mon Sep 17 00:00:00 2001
From: Herpaderp Aldent <felix.a.huber@gmx.net>
Date: Sat, 10 Nov 2018 12:49:54 +0100
Subject: [PATCH 06/11] Added some translations and fixed typos.

---
 src/resources/views/about.blade.php     | 4 ++--
 src/resources/views/logs/list.blade.php | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/resources/views/about.blade.php b/src/resources/views/about.blade.php
index cdff3a4..315dfe2 100644
--- a/src/resources/views/about.blade.php
+++ b/src/resources/views/about.blade.php
@@ -36,7 +36,7 @@
 
       <legend>Bugs and issues</legend>
 
-      <p>If you find something is not working as expectected, please don't hesitate and contact me. Either use SeAT-Slack or submit an <a href="https://github.com/herpaderpaldent/seat-groups/issues/new">issue on Github</a></p>
+      <p>If you find something is not working as expected, please don't hesitate and contact me. Either use SeAT-Slack or submit an <a href="https://github.com/herpaderpaldent/seat-groups/issues/new">issue on Github</a></p>
 
     </div>
   </div>
@@ -46,7 +46,7 @@
     <div class="box-header with-border">
       <i class="fa fa-archive"></i>
 
-      <h3 class="box-title">Last Event log</h3>
+      <h3 class="box-title">Last {{trans('seatgroups::seat.event')}} Log</h3>
 
       <span class="log-clear-button"></span>
 
diff --git a/src/resources/views/logs/list.blade.php b/src/resources/views/logs/list.blade.php
index 90083ac..0917c31 100644
--- a/src/resources/views/logs/list.blade.php
+++ b/src/resources/views/logs/list.blade.php
@@ -2,7 +2,7 @@
   <thead>
   <tr>
     <th>{{ trans('web::seat.date') }}</th>
-    <th>{{ trans('web::seat.category') }}</th>
+    <th>{{ trans('seatgroups::seat.event') }}</th>
     <th>{{ trans('web::seat.message') }}</th>
   </tr>
   </thead>

From 752b7038b2cf39667e649ce96faf246b566838ee Mon Sep 17 00:00:00 2001
From: Herpaderp Aldent <felix.a.huber@gmx.net>
Date: Sat, 10 Nov 2018 12:53:07 +0100
Subject: [PATCH 07/11] Only log attached and detached events.

---
 src/Jobs/GroupSync.php                        | 54 +++++++++++++------
 src/lang/en/seat.php                          |  3 ++
 .../views/logs/partials/event.blade.php       |  6 ++-
 3 files changed, 46 insertions(+), 17 deletions(-)

diff --git a/src/Jobs/GroupSync.php b/src/Jobs/GroupSync.php
index 39c56f0..595efa0 100644
--- a/src/Jobs/GroupSync.php
+++ b/src/Jobs/GroupSync.php
@@ -12,6 +12,7 @@
 use Herpaderpaldent\Seat\SeatGroups\Models\Seatgroup;
 use Herpaderpaldent\Seat\SeatGroups\Models\SeatgroupLog;
 use Illuminate\Support\Facades\Redis;
+use Seat\Web\Models\Acl\Role;
 use Seat\Web\Models\Group;
 
 class GroupSync extends SeatGroupsJobBase
@@ -115,9 +116,9 @@ public function handle()
                     }
                 });
 
-                $group->roles()->sync($roles->unique());
+                $sync = $group->roles()->sync($roles->unique());
 
-                $this->onFinish();
+                $this->onFinish($sync);
 
                 logger()->debug('Group has beend synced for ' . $this->main_character->name);
 
@@ -154,10 +155,10 @@ public function beforeStart()
                 });
 
                 SeatgroupLog::create([
-                    'event' => 'warning',
-                    'message' => sprintf('The RefreshToken of %s is missing, therefore user group of %s (%s) loses all permissions.',
-                        $user->name, $this->main_character->name, $this->group->users->map(function ($user) { return $user->name; })->implode(', ')),
-
+                    'event'   => 'error',
+                    'message' => sprintf('The RefreshToken of %s is missing, therefore user group of %s (%s) loses all permissions.' .
+                        'Ask the owner of this user group to login again with this user, in order to provide a new RefreshToken. ',
+                        $user->name, $this->main_character->name, $this->group->users->map(function ($user) {return $user->name;})->implode(', ')),
                 ]);
 
                 // throw exception
@@ -176,23 +177,46 @@ public function onFail($exception)
         report($exception);
 
         SeatgroupLog::create([
-            'event' => 'error',
+            'event'   => 'error',
             'message' => sprintf('An error occurred while syncing user group of %s (%s). Please check the logs.',
-                $this->main_character->name, $this->group->users->map(function ($user) { return $user->name; })->implode(', ')),
-
+                $this->main_character->name, $this->group->users->map(function ($user) {return $user->name;})->implode(', ')),
         ]);
 
         $this->fail($exception);
 
     }
 
-    public function onFinish()
+    public function onFinish($sync)
     {
-        SeatgroupLog::create([
-            'event' => 'success',
-            'message' => sprintf('The user group of %s (%s) has successfully been synced.',
-                $this->main_character->name, $this->group->users->map(function ($user) { return $user->name; })->implode(', ')),
 
-        ]);
+        if (! empty($sync['attached'])) {
+
+            SeatgroupLog::create([
+                'event'   => 'attached',
+                'message' => sprintf('The user group of %s (%s) has successfully been attached to the following roles: %s.',
+                    $this->main_character->name,
+                    $this->group->users->map(function ($user) {
+
+                        return $user->name;
+                    })->implode(', '),
+                    Role::whereIn('id', $sync['attached'])->pluck('title')->implode(', ')
+                ),
+            ]);
+        }
+
+        if (! empty($sync['detached'])) {
+
+            SeatgroupLog::create([
+                'event'   => 'detached',
+                'message' => sprintf('The user group of %s (%s) has been detached from the following roles: %s.',
+                    $this->main_character->name,
+                    $this->group->users->map(function ($user) {
+
+                        return $user->name;
+                    })->implode(', '),
+                    Role::whereIn('id', $sync['detached'])->pluck('title')->implode(', ')
+                ),
+            ]);
+        }
     }
 }
diff --git a/src/lang/en/seat.php b/src/lang/en/seat.php
index 62c0a82..61b0eb8 100644
--- a/src/lang/en/seat.php
+++ b/src/lang/en/seat.php
@@ -52,4 +52,7 @@
     'available_alliances' => 'Available Alliances',
     'add_alliance'        => 'Add Alliance',
 
+    //Event Log
+    'event' => 'Event',
+
 ];
diff --git a/src/resources/views/logs/partials/event.blade.php b/src/resources/views/logs/partials/event.blade.php
index ce51c9a..9be441c 100644
--- a/src/resources/views/logs/partials/event.blade.php
+++ b/src/resources/views/logs/partials/event.blade.php
@@ -1,10 +1,12 @@
 @switch($row->event)
-  @case('success')
+  @case('attached')
     <span class="label label-success"> {{$row->event}} </span>
     @break
-  @case('warning')
+  @case('detached')
     <span class="label label-warning"> {{$row->event}} </span>
     @break
   @case('error')
     <span class="label label-danger"> {{$row->event}} </span>
+  @default
+    <span class="label label-default"> {{$row->event}} </span>
 @endswitch
\ No newline at end of file

From d88ad93218f7bee60d976ee59d7aae3eb627b0e7 Mon Sep 17 00:00:00 2001
From: Herpaderp Aldent <felix.a.huber@gmx.net>
Date: Sat, 10 Nov 2018 12:56:20 +0100
Subject: [PATCH 08/11] Some code-format optimizations

---
 src/Jobs/GroupSync.php | 15 +++++++++------
 1 file changed, 9 insertions(+), 6 deletions(-)

diff --git a/src/Jobs/GroupSync.php b/src/Jobs/GroupSync.php
index 595efa0..9eb7cc4 100644
--- a/src/Jobs/GroupSync.php
+++ b/src/Jobs/GroupSync.php
@@ -44,6 +44,7 @@ class GroupSync extends SeatGroupsJobBase
      */
     public function __construct(Group $group)
     {
+
         $this->group = $group;
         $this->main_character = $group->main_character;
         if (is_null($group->main_character)) {
@@ -65,12 +66,13 @@ public function __construct(Group $group)
 
     public function handle()
     {
+
         // in case no main character has been set, throw an exception and abort the process
         if (is_null($this->main_character))
             throw new MissingMainCharacterException($this->group);
 
-        Redis::funnel('seat-groups:jobs.group_sync_' . $this->group->id)->limit(1)->then(function ()
-        {
+        Redis::funnel('seat-groups:jobs.group_sync_' . $this->group->id)->limit(1)->then(function () {
+
             $this->beforeStart();
 
             try {
@@ -95,7 +97,7 @@ public function handle()
                                 foreach ($seat_group->role as $role) {
                                     $roles->push($role->id);
                                 }
-                                if(! in_array($group->id, $seat_group->group->pluck('id')->toArray())){
+                                if (! in_array($group->id, $seat_group->group->pluck('id')->toArray())) {
                                     // add user_group to seat_group as member if no member yet.
                                     $seat_group->member()->attach($group->id);
                                 }
@@ -111,7 +113,7 @@ public function handle()
                                 }
                                 break;
                         }
-                    } elseif(in_array($group->id, $seat_group->group->pluck('id')->toArray())) {
+                    } elseif (in_array($group->id, $seat_group->group->pluck('id')->toArray())) {
                         $seat_group->member()->detach($group->id);
                     }
                 });
@@ -128,8 +130,8 @@ public function handle()
 
             }
 
-        }, function ()
-        {
+        }, function () {
+
             logger()->warning('A GroupSync job is already running for ' . $this->main_character->name . ' Removing the job from the queue.');
 
             $this->delete();
@@ -151,6 +153,7 @@ public function beforeStart()
                 // take away all roles
                 $this->group->roles()->sync([]);
                 Seatgroup::all()->each(function ($seatgroup) {
+
                     $seatgroup->member()->detach($this->group->id);
                 });
 

From b43cae27213a1ebcda878f52e96dba4898a08895 Mon Sep 17 00:00:00 2001
From: Herpaderp Aldent <felix.a.huber@gmx.net>
Date: Sat, 10 Nov 2018 14:44:19 +0100
Subject: [PATCH 09/11] Dispatch `seat-groups:users:update` from about-page.

---
 src/Http/Controllers/SeatGroupsController.php | 11 ++++++
 src/Http/routes.php                           |  4 +++
 src/resources/views/about.blade.php           | 35 ++++++++++++++-----
 3 files changed, 41 insertions(+), 9 deletions(-)

diff --git a/src/Http/Controllers/SeatGroupsController.php b/src/Http/Controllers/SeatGroupsController.php
index 4b204db..72b0962 100644
--- a/src/Http/Controllers/SeatGroupsController.php
+++ b/src/Http/Controllers/SeatGroupsController.php
@@ -11,6 +11,7 @@
 use Herpaderpaldent\Seat\SeatGroups\Http\Validation\DeleteSeatGroupRequest;
 use Herpaderpaldent\Seat\SeatGroups\Models\Seatgroup;
 use Illuminate\Http\Request;
+use Illuminate\Support\Facades\Artisan;
 use Seat\Services\Repositories\Character\Character;
 use Seat\Services\Repositories\Corporation\Corporation;
 use Seat\Web\Http\Controllers\Controller;
@@ -167,4 +168,14 @@ public function about(GetChangelog $action)
 
         return view('seatgroups::about', compact('changelog'));
     }
+
+    public function dispatchUpdate()
+    {
+
+        Artisan::queue('seat-groups:users:update')->onQueue('high');
+
+        return redirect()->back()
+            ->with('success', 'SeAT Group update scheduled. Check back in a few moments');
+
+    }
 }
diff --git a/src/Http/routes.php b/src/Http/routes.php
index 58489f4..96e3241 100644
--- a/src/Http/routes.php
+++ b/src/Http/routes.php
@@ -53,6 +53,10 @@
             'as'   => 'seatgroupuser.remove.manager',
             'uses' => 'SeatGroupUserController@removeManager',
         ]);
+        Route::get('/update', [
+            'as'   => 'seatgroup.user.update',
+            'uses' => 'SeatGroupsController@dispatchUpdate',
+        ]);
 
     });
 
diff --git a/src/resources/views/about.blade.php b/src/resources/views/about.blade.php
index 315dfe2..98cf10a 100644
--- a/src/resources/views/about.blade.php
+++ b/src/resources/views/about.blade.php
@@ -42,23 +42,40 @@
   </div>
 
   @if(auth()->user()->hasRole('seatgroups.create'))
-  <div class="box box-default">
-    <div class="box-header with-border">
-      <i class="fa fa-archive"></i>
+    <div class="box box-default">
+      <div class="box-header with-border">
+        <i class="fa fa-refresh"></i>
 
-      <h3 class="box-title">Last {{trans('seatgroups::seat.event')}} Log</h3>
+        <h3 class="box-title">{{ trans('web::seat.update') }} {{ trans('seatgroups::seat.seat_groups') }}</h3>
 
-      <span class="log-clear-button"></span>
 
+      </div>
+      <!-- /.box-header -->
+      <div class="box-body">
+        <a href="{{route('seatgroup.user.update')}}" class="btn btn-block btn-primary" role="button">{{ trans('seatgroups::seat.seat_groups') }} {{ trans_choice('web::seat.user',2) }} {{ trans('web::seat.update') }}</a>
+
+      </div>
+      <!-- /.box-body -->
     </div>
-    <!-- /.box-header -->
-    <div class="box-body">
+
+    <div class="box box-default">
+      <div class="box-header with-border">
+        <i class="fa fa-archive"></i>
+
+        <h3 class="box-title">Last {{trans('seatgroups::seat.event')}} Log</h3>
+
+        <span class="log-clear-button"></span>
+
+
+      </div>
+      <!-- /.box-header -->
+      <div class="box-body">
 
         @include('seatgroups::logs.list')
 
+      </div>
+      <!-- /.box-body -->
     </div>
-    <!-- /.box-body -->
-  </div>
   @endif
 
 @stop

From 90e16bd1783e234f70de30991be9033bdde05172 Mon Sep 17 00:00:00 2001
From: Herpaderp Aldent <felix.a.huber@gmx.net>
Date: Sat, 10 Nov 2018 15:07:08 +0100
Subject: [PATCH 10/11] Changelog and version bump

---
 CHANGELOG.md                     | 11 +++++++++++
 src/config/seatgroups.config.php |  2 +-
 2 files changed, 12 insertions(+), 1 deletion(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index c1a2cbb..d85bc85 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -1,3 +1,14 @@
+# Version 1.5.0
+Another major update on seat-groups. Containing many demanded features such as:
+* Dispatching update job via web interface.
+* Seat-Groups jobs are now dispatched on `high` queue (thanks @mmolitor87 for the suggestion).
+* Clicking multiple times on `Add new SeAT Group` does not append multiple sets of roles anymore (thank you @MoucceuWildfire for reporting).
+* SeAT-Groups now only logs attached and detached role events instead of every successful run (thank you @warlof for the idea).
+* Deactivated users don't raise `MissingRefreshTokenExceptions` anymore
+* `MissingRefreshTokenExceptions` are now more verbose.
+
+As you can see many of the implemented features are based from your valuable input. Please don't hesitate to contact me IG or via [slack](https://eveseat-slack.herokuapp.com/) (join plugin channel) with anything you'd like to discuss.
+
 # Version 1.4.2
 * Fixxed a bug causing to show all hidden-groups, instead of only the hidden-groups where a user is a member of. Thank you @jediefe for reporting this.
 * Removed raw:db statements from migrations (preparation for testing).
diff --git a/src/config/seatgroups.config.php b/src/config/seatgroups.config.php
index 9717d0e..c651c70 100644
--- a/src/config/seatgroups.config.php
+++ b/src/config/seatgroups.config.php
@@ -6,7 +6,7 @@
  * Time: 10:24.
  */
 return [
-    'version'   => '1.4.2',
+    'version'   => '1.5.0',
 ];
 
 //TODO: Update Version

From af274d7925acdd99125f5be3f3cbf76b4f1d1766 Mon Sep 17 00:00:00 2001
From: Herpaderp Aldent <felix.a.huber@gmx.net>
Date: Sat, 10 Nov 2018 15:13:01 +0100
Subject: [PATCH 11/11] Apply fixes from StyleCI (#46)

---
 src/Exceptions/MissingRefreshTokenException.php | 1 -
 src/Jobs/GroupSync.php                          | 4 ++--
 2 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/src/Exceptions/MissingRefreshTokenException.php b/src/Exceptions/MissingRefreshTokenException.php
index 3526f89..42bab1a 100644
--- a/src/Exceptions/MissingRefreshTokenException.php
+++ b/src/Exceptions/MissingRefreshTokenException.php
@@ -20,5 +20,4 @@ public function __construct(User $user)
 
         parent::__construct($message, 0, null);
     }
-
 }
diff --git a/src/Jobs/GroupSync.php b/src/Jobs/GroupSync.php
index 9eb7cc4..aad06b3 100644
--- a/src/Jobs/GroupSync.php
+++ b/src/Jobs/GroupSync.php
@@ -161,7 +161,7 @@ public function beforeStart()
                     'event'   => 'error',
                     'message' => sprintf('The RefreshToken of %s is missing, therefore user group of %s (%s) loses all permissions.' .
                         'Ask the owner of this user group to login again with this user, in order to provide a new RefreshToken. ',
-                        $user->name, $this->main_character->name, $this->group->users->map(function ($user) {return $user->name;})->implode(', ')),
+                        $user->name, $this->main_character->name, $this->group->users->map(function ($user) {return $user->name; })->implode(', ')),
                 ]);
 
                 // throw exception
@@ -182,7 +182,7 @@ public function onFail($exception)
         SeatgroupLog::create([
             'event'   => 'error',
             'message' => sprintf('An error occurred while syncing user group of %s (%s). Please check the logs.',
-                $this->main_character->name, $this->group->users->map(function ($user) {return $user->name;})->implode(', ')),
+                $this->main_character->name, $this->group->users->map(function ($user) {return $user->name; })->implode(', ')),
         ]);
 
         $this->fail($exception);