From 79f700c62249eff5c379645f3f6a361ea8716295 Mon Sep 17 00:00:00 2001
From: James Cole <thegrumpydictator@gmail.com>
Date: Tue, 19 Dec 2017 05:20:05 +0100
Subject: [PATCH] Catch query exception

---
 app/User.php | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/app/User.php b/app/User.php
index 1fa24e7a80..32325cf6cf 100644
--- a/app/User.php
+++ b/app/User.php
@@ -27,8 +27,10 @@ use FireflyIII\Models\CurrencyExchangeRate;
 use Illuminate\Database\Eloquent\Relations\BelongsToMany;
 use Illuminate\Database\Eloquent\Relations\HasMany;
 use Illuminate\Database\Eloquent\Relations\HasManyThrough;
+use Illuminate\Database\QueryException;
 use Illuminate\Foundation\Auth\User as Authenticatable;
 use Illuminate\Notifications\Notifiable;
+use Log;
 use Request;
 
 /**
@@ -84,8 +86,12 @@ class User extends Authenticatable
         if (is_array($role)) {
             $role = $role['id'];
         }
-
-        $this->roles()->attach($role);
+        try {
+            $this->roles()->attach($role);
+        } catch (QueryException $e) {
+            // don't care
+            Log::info(sprintf('Query exception when giving user a role: %s', $e->getMessage()));
+        }
     }
 
     /**