From a029071c4ecddd9d1970874352d009ca87bfbd46 Mon Sep 17 00:00:00 2001
From: Juan Ferrer Toribio <juan@verdnatura.es>
Date: Tue, 2 May 2023 20:47:56 +0200
Subject: [PATCH] refs #5517 User image fixes

---
 front/salix/components/log/index.html | 13 +++----------
 front/salix/components/log/index.js   | 12 ++++++------
 front/salix/components/log/style.scss |  1 -
 3 files changed, 9 insertions(+), 17 deletions(-)

diff --git a/front/salix/components/log/index.html b/front/salix/components/log/index.html
index a51e07cb3..00ab23cc9 100644
--- a/front/salix/components/log/index.html
+++ b/front/salix/components/log/index.html
@@ -26,10 +26,9 @@
                     {{::log.user ? log.user.name.charAt(0).toUpperCase() : 'S'}}
                 </div>
                 <img
-                    ng-if="::log.user.worker"
-                    src="/api/Images/user/160x160/{{::log.user.worker.id}}/download?access_token={{::$ctrl.vnToken.token}}"
-                    ng-click="$ctrl.showWorkerDescriptor($event, log.user.worker.id)"
-                    onload="this.style.display = 'block'">
+                    ng-if="::log.user.image"
+                    ng-src="/api/Images/user/160x160/{{::log.userFk}}/download?access_token={{::$ctrl.vnToken.token}}"
+                    ng-click="$ctrl.showWorkerDescriptor($event, log)">
                 </img>
             </div>
             <div class="arrow bg-panel"></div>
@@ -115,12 +114,6 @@
     <form vn-vertical
         ng-model-options="{updateOn: 'change blur'}"
         class="vn-pa-md filter">
-        <h6
-            class="text-secondary vn-mb-md"
-            style="font-weight: normal;"
-            translate>
-            Filter
-        </h6>
         <vn-textfield
             label="Name"
             ng-model="filter.changedModelValue">
diff --git a/front/salix/components/log/index.js b/front/salix/components/log/index.js
index b3a27ca5e..46c6cd0f4 100644
--- a/front/salix/components/log/index.js
+++ b/front/salix/components/log/index.js
@@ -23,7 +23,7 @@ export default class Controller extends Section {
             include: [{
                 relation: 'user',
                 scope: {
-                    fields: ['nickname', 'name'],
+                    fields: ['nickname', 'name', 'image'],
                     include: {
                         relation: 'worker',
                         scope: {
@@ -120,11 +120,6 @@ export default class Controller extends Section {
         return this.dateFilter(date, `${format} HH:mm`);
     }
 
-    showWorkerDescriptor(event, workerId) {
-        if (!workerId) return;
-        this.$.workerDescriptor.show(event.target, workerId);
-    }
-
     resetFilter() {
         this.$.filter = {who: 'all'};
     }
@@ -201,6 +196,11 @@ export default class Controller extends Section {
             backgroundColor: '#'+ colors[hash % colors.length]
         };
     }
+
+    showWorkerDescriptor(event, log) {
+        if (log.user?.worker)
+            this.$.workerDescriptor.show(event.target, log.userFk);
+    }
 }
 
 ngModule.vnComponent('vnLog', {
diff --git a/front/salix/components/log/style.scss b/front/salix/components/log/style.scss
index 4bf6ca1a2..ca0c9473c 100644
--- a/front/salix/components/log/style.scss
+++ b/front/salix/components/log/style.scss
@@ -32,7 +32,6 @@ vn-log {
                     justify-content: center;
                 }
                 img {
-                    display: none;
                     cursor: pointer;
                     position: absolute;
                     top: 0;