From 9f4232a994bf53f4f420551b728539427b897acb Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?C=C3=B4me=20Chilliet?= <come@opensides.be>
Date: Thu, 20 Sep 2018 15:13:37 +0200
Subject: [PATCH] :ambulance: fix(core) Cast userLocked to integer before
 calling hooks

Otherwise when false the hook gets an empty parameter.
Also this is consistent with previous behavior.

issue #3710
---
 plugins/personal/generic/class_user.inc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/plugins/personal/generic/class_user.inc b/plugins/personal/generic/class_user.inc
index c32c1c099..aaae7c5b7 100644
--- a/plugins/personal/generic/class_user.inc
+++ b/plugins/personal/generic/class_user.inc
@@ -393,7 +393,7 @@ class user extends simplePlugin
   function callHook($cmd, array $addAttrs = array(), &$returnOutput = array(), &$returnCode = NULL)
   {
     $addAttrs['passwordMethod'] = $this->attributesAccess['userPassword']->getMethod();
-    $addAttrs['userLocked']     = $this->attributesAccess['userPassword']->isLocked();
+    $addAttrs['userLocked']     = (int)($this->attributesAccess['userPassword']->isLocked());
     $addAttrs['passwordClear']  = $this->attributesAccess['userPassword']->getClear();
     return parent::callHook($cmd, $addAttrs, $returnOutput, $returnCode);
   }
-- 
GitLab