From b3feb6b095f8e189d7c2868d042f0e7c09d9a402 Mon Sep 17 00:00:00 2001
From: Laurent Destailleur <eldy@destailleur.fr>
Date: Wed, 6 Nov 2019 00:42:29 +0100
Subject: [PATCH] Fix lang not defined

---
 htdocs/contrat/class/contrat.class.php |  1 +
 htdocs/projet/class/task.class.php     |  4 +--
 htdocs/zapier/class/hook.class.php     | 47 ++++++++------------------
 3 files changed, 18 insertions(+), 34 deletions(-)

diff --git a/htdocs/contrat/class/contrat.class.php b/htdocs/contrat/class/contrat.class.php
index 7049ce77076..b85262586b9 100644
--- a/htdocs/contrat/class/contrat.class.php
+++ b/htdocs/contrat/class/contrat.class.php
@@ -1894,6 +1894,7 @@ class Contrat extends CommonObject
 	public function LibStatut($status, $mode)
 	{
         // phpcs:enable
+		global $langs;
 
 		if (empty($this->labelStatus) || empty($this->labelStatusShort))
 		{
diff --git a/htdocs/projet/class/task.class.php b/htdocs/projet/class/task.class.php
index dcc4c18c497..c10e9915c3b 100644
--- a/htdocs/projet/class/task.class.php
+++ b/htdocs/projet/class/task.class.php
@@ -1849,6 +1849,8 @@ class Task extends CommonObject
 	public function LibStatut($status, $mode = 0)
 	{
         // phpcs:enable
+		global $langs;
+
 		// list of Statut of the task
 		$this->statuts[0]='Draft';
 		$this->statuts[1]='ToDo';
@@ -1861,8 +1863,6 @@ class Task extends CommonObject
 		$this->statuts_short[3]='Completed';
 		$this->statuts_short[4]='Transfered';
 
-		global $langs;
-
 		if ($mode == 0)
 		{
 			return $langs->trans($this->statuts[$status]);
diff --git a/htdocs/zapier/class/hook.class.php b/htdocs/zapier/class/hook.class.php
index 5c79d68aefe..fdc7aba2bc9 100644
--- a/htdocs/zapier/class/hook.class.php
+++ b/htdocs/zapier/class/hook.class.php
@@ -633,39 +633,22 @@ class Hook extends CommonObject
     public function LibStatut($status, $mode = 0)
     {
         // phpcs:enable
-        if (empty($this->labelStatus)) {
-            global $langs;
-            //$langs->load("zapier@zapier");
-            $this->labelStatus[1] = $langs->trans('Enabled');
-            $this->labelStatus[0] = $langs->trans('Disabled');
-        }
+    	global $langs;
 
-        if ($mode == 0) {
-            return $this->labelStatus[$status];
-        } elseif ($mode == 1) {
-            return $this->labelStatus[$status];
-        } elseif ($mode == 2) {
-            if ($status == 1) {
-                return img_picto($this->labelStatus[$status], 'statut4', '', false, 0, 0, '', 'valignmiddle').' '.$this->labelStatus[$status];
-            } elseif ($status == 0) {
-                return img_picto($this->labelStatus[$status], 'statut5', '', false, 0, 0, '', 'valignmiddle').' '.$this->labelStatus[$status];
-            }
-        } elseif ($mode == 3) {
-            if ($status == 1) return img_picto($this->labelStatus[$status], 'statut4', '', false, 0, 0, '', 'valignmiddle');
-            elseif ($status == 0) return img_picto($this->labelStatus[$status], 'statut5', '', false, 0, 0, '', 'valignmiddle');
-        } elseif ($mode == 4) {
-            if ($status == 1) return img_picto($this->labelStatus[$status], 'statut4', '', false, 0, 0, '', 'valignmiddle').' '.$this->labelStatus[$status];
-            elseif ($status == 0) return img_picto($this->labelStatus[$status], 'statut5', '', false, 0, 0, '', 'valignmiddle').' '.$this->labelStatus[$status];
-        } elseif ($mode == 5) {
-            if ($status == 1) return $this->labelStatus[$status].' '.img_picto($this->labelStatus[$status], 'statut4', '', false, 0, 0, '', 'valignmiddle');
-            elseif ($status == 0) return $this->labelStatus[$status].' '.img_picto($this->labelStatus[$status], 'statut5', '', false, 0, 0, '', 'valignmiddle');
-        } elseif ($mode == 6) {
-            if ($status == 1) {
-                return $this->labelStatus[$status].' '.img_picto($this->labelStatus[$status], 'statut4', '', false, 0, 0, '', 'valignmiddle');
-            } elseif ($status == 0) {
-                return $this->labelStatus[$status].' '.img_picto($this->labelStatus[$status], 'statut5', '', false, 0, 0, '', 'valignmiddle');
-            }
-        }
+    	if (empty($this->labelStatus) || empty($this->labelStatusShort))
+    	{
+    		global $langs;
+    		//$langs->load("mymodule");
+    		$this->labelStatus[self::STATUS_DRAFT] = $langs->trans('Disabled');
+    		$this->labelStatus[self::STATUS_VALIDATED] = $langs->trans('Enabled');
+    		$this->labelStatusShort[self::STATUS_DRAFT] = $langs->trans('Disabled');
+    		$this->labelStatusShort[self::STATUS_VALIDATED] = $langs->trans('Enabled');
+    	}
+
+    	$statusType = 'status5';
+    	if ($status == self::STATUS_VALIDATED) $statusType = 'status4';
+
+    	return dolGetStatus($this->labelStatus[$status], $this->labelStatusShort[$status], '', $statusType, $mode);
     }
 
     /**