From 4cd79e5cbcd72dafd65f64e2c9e3b0d4db54b6d0 Mon Sep 17 00:00:00 2001
From: Damien <damien.burel@maarch.org>
Date: Wed, 28 Nov 2018 18:11:30 +0100
Subject: [PATCH] FEAT #8969 Do not save picture if empty

---
 src/app/user/models/UserModel.php | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/src/app/user/models/UserModel.php b/src/app/user/models/UserModel.php
index 7a34a52fd4..62b1413b80 100755
--- a/src/app/user/models/UserModel.php
+++ b/src/app/user/models/UserModel.php
@@ -103,13 +103,16 @@ class UserModel
         ValidatorModel::intVal($aArgs, ['id']);
         ValidatorModel::stringType($aArgs, ['firstname', 'lastname', 'picture']);
 
+        $set = [
+            'firstname' => $aArgs['firstname'],
+            'lastname'  => $aArgs['lastname']
+        ];
+        if (!empty($aArgs['picture'])) {
+            $set['picture'] = $aArgs['picture'];
+        }
         DatabaseModel::update([
             'table'     => 'users',
-            'set'       => [
-                'firstname' => $aArgs['firstname'],
-                'lastname'  => $aArgs['lastname'],
-                'picture'   => empty($aArgs['picture']) ? null : $aArgs['picture']
-            ],
+            'set'       => $set,
             'where'     => ['id = ?'],
             'data'      => [$aArgs['id']]
         ]);
-- 
GitLab