From d8a8df77502dfa03aeba06c52ff0c20472744f93 Mon Sep 17 00:00:00 2001
From: "florian.azizian" <florian.azizian@maarch.org>
Date: Wed, 26 Feb 2020 10:29:48 +0100
Subject: [PATCH] FIX #12091 TIME 0:15 session userbasket was always empty

---
 .../class/class_modules_tools_Abstract.php    |   4 +-
 modules/basket/lang/fr.php                    | 230 ++++++++++++------
 modules/basket/lang/nl.php                    |   3 -
 3 files changed, 155 insertions(+), 82 deletions(-)

diff --git a/modules/basket/class/class_modules_tools_Abstract.php b/modules/basket/class/class_modules_tools_Abstract.php
index 657bb6aded7..1f82e1d666d 100755
--- a/modules/basket/class/class_modules_tools_Abstract.php
+++ b/modules/basket/class/class_modules_tools_Abstract.php
@@ -103,7 +103,7 @@ abstract class basket_Abstract extends Database
 
         $user = \User\models\UserModel::getByLogin(['login' => $userData['UserId'], 'select' => ['id']]);
 
-        if (isset($userData['primarygroup']) && isset($userData['UserId'])) {
+        if (isset($userData['UserId'])) {
             $db = new Database();
             $stmt = $db->query("SELECT ubp.basket_id, ubp.group_serial_id FROM users_baskets_preferences ubp, baskets WHERE user_serial_id = ? AND ubp.display = TRUE AND ubp.basket_id = baskets.basket_id order by ubp.group_serial_id, baskets.basket_order, baskets.basket_name ", [$user['id']]);
             while ($res = $stmt->fetchObject()) {
@@ -113,7 +113,7 @@ abstract class basket_Abstract extends Database
             }
         }
 
-        if (isset($userData['primarygroup']) && isset($userData['UserId'])) {
+        if (isset($userData['UserId'])) {
             $absBasketsArr = $this->load_basket_abs($userData['UserId']);
             $_SESSION['user']['baskets'] = array_merge(
                $_SESSION['user']['baskets'],
diff --git a/modules/basket/lang/fr.php b/modules/basket/lang/fr.php
index c4a910ec499..c91f5962b38 100755
--- a/modules/basket/lang/fr.php
+++ b/modules/basket/lang/fr.php
@@ -20,159 +20,235 @@
  */
 
 /*********************** SERVICES ***********************************/
-if (!defined("_DIFFUSION_LIST"))
+if (!defined("_DIFFUSION_LIST")) {
     define("_DIFFUSION_LIST", "Liste de diffusion");
+}
 
 //class basket
-if (!defined("_BASKET"))
+if (!defined("_BASKET")) {
     define("_BASKET", "Bannette");
-if (!defined("_BASKETS_COMMENT"))
+}
+if (!defined("_BASKETS_COMMENT")) {
     define("_BASKETS_COMMENT", "Bannettes");
-if (!defined("_THE_ID"))
+}
+if (!defined("_THE_ID")) {
     define("_THE_ID", "L'identifiant ");
+}
 
 /************** Bannette : Liste + Formulaire**************/
-if (!defined("_ALL_BASKETS"))
+if (!defined("_ALL_BASKETS")) {
     define("_ALL_BASKETS", "Tout mon périmètre");
-if (!defined("_BASKET_LIST"))
+}
+if (!defined("_BASKET_LIST")) {
     define("_BASKET_LIST", "Liste des bannettes");
-if (!defined("_ADD_BASKET"))
+}
+if (!defined("_ADD_BASKET")) {
     define("_ADD_BASKET", "Ajouter une bannette");
-if (!defined("_BASKET_ADDITION"))
+}
+if (!defined("_BASKET_ADDITION")) {
     define("_BASKET_ADDITION", "Ajout d'une bannette");
-if (!defined("_BASKET_MODIFICATION"))
+}
+if (!defined("_BASKET_MODIFICATION")) {
     define("_BASKET_MODIFICATION", "Modification d'une bannette");
-if (!defined("_BASKET_VIEW"))
+}
+if (!defined("_BASKET_VIEW")) {
     define("_BASKET_VIEW", "Vue sur la table");
-if (!defined("_MODIFY_BASKET"))
+}
+if (!defined("_MODIFY_BASKET")) {
     define("_MODIFY_BASKET", "Modifier la bannette");
-if (!defined("_ADD_A_NEW_BASKET"))
+}
+if (!defined("_ADD_A_NEW_BASKET")) {
     define("_ADD_A_NEW_BASKET", "Créer une nouvelle bannette");
-if (!defined("_ADD_A_GROUP_TO_BASKET"))
+}
+if (!defined("_ADD_A_GROUP_TO_BASKET")) {
     define("_ADD_A_GROUP_TO_BASKET", "Associer un nouveau groupe à la bannette");
-if (!defined("_DEL_GROUPS"))
+}
+if (!defined("_DEL_GROUPS")) {
     define("_DEL_GROUPS", "Supprimer groupe(s)");
-if (!defined("_ASSOCIATED_GROUP"))
+}
+if (!defined("_ASSOCIATED_GROUP")) {
     define("_ASSOCIATED_GROUP", "Liste des groupes associés à la bannette");
-if (!defined("_TITLE_GROUP_BASKET"))
+}
+if (!defined("_TITLE_GROUP_BASKET")) {
     define("_TITLE_GROUP_BASKET", "Associer la bannette à un groupe");
-if (!defined("_ADD_TO_BASKET"))
+}
+if (!defined("_ADD_TO_BASKET")) {
     define("_ADD_TO_BASKET", "Associer la bannette");
-if (!defined("_TO_THE_GROUP"))
+}
+if (!defined("_TO_THE_GROUP")) {
     define("_TO_THE_GROUP", "à un groupe");
-if (!defined("_ALLOWED_ACTIONS"))
+}
+if (!defined("_ALLOWED_ACTIONS")) {
     define("_ALLOWED_ACTIONS", "Actions autorisées");
-if (!defined("_SERVICES_BASKETS"))
+}
+if (!defined("_SERVICES_BASKETS")) {
     define("_SERVICES_BASKETS", "Bannettes de services");
-if (!defined("_USERGROUPS_BASKETS"))
+}
+if (!defined("_USERGROUPS_BASKETS")) {
     define("_USERGROUPS_BASKETS", "Bannettes des groupes d'utilisateurs");
-if (!defined("_BASKET_RESULT_PAGE"))
+}
+if (!defined("_BASKET_RESULT_PAGE")) {
     define("_BASKET_RESULT_PAGE", "Liste de résultats");
-if (!defined("_ADD_THIS_GROUP"))
+}
+if (!defined("_ADD_THIS_GROUP")) {
     define("_ADD_THIS_GROUP", "Ajouter le groupe");
-if (!defined("_MODIFY_THIS_GROUP"))
+}
+if (!defined("_MODIFY_THIS_GROUP")) {
     define("_MODIFY_THIS_GROUP", "Modifier le groupe");
-if (!defined("_DEFAULT_ACTION_LIST"))
+}
+if (!defined("_DEFAULT_ACTION_LIST")) {
     define("_DEFAULT_ACTION_LIST", "Action par défaut sur la liste<br/><i>(Cliquez sur la ligne)");
-if (!defined("_NO_ACTION_DEFINED"))
+}
+if (!defined("_NO_ACTION_DEFINED")) {
     define("_NO_ACTION_DEFINED", "Aucune action définie");
+}
 
 //LIST
-if (!defined("_COPY_LIST"))
+if (!defined("_COPY_LIST")) {
     define("_COPY_LIST", "Liste des courriers en copie");
-if (!defined("_PROCESS_LIST"))
+}
+if (!defined("_PROCESS_LIST")) {
     define("_PROCESS_LIST", "Liste des courriers à traiter");
-if (!defined("_CLICK_LINE_TO_VIEW"))
+}
+if (!defined("_CLICK_LINE_TO_VIEW")) {
     define("_CLICK_LINE_TO_VIEW", "Cliquez sur une ligne pour visualiser");
-if (!defined("_CLICK_LINE_TO_PROCESS"))
+}
+if (!defined("_CLICK_LINE_TO_PROCESS")) {
     define("_CLICK_LINE_TO_PROCESS", "Cliquez sur une ligne pour traiter");
-if (!defined("_REDIRECT_TO_SENDER_ENTITY"))
+}
+if (!defined("_REDIRECT_TO_SENDER_ENTITY")) {
     define("_REDIRECT_TO_SENDER_ENTITY", "Redirection vers l'entité émetteur");
-if (!defined("_CHOOSE_DEPARTMENT"))
+}
+if (!defined("_CHOOSE_DEPARTMENT")) {
     define("_CHOOSE_DEPARTMENT", "Choisissez une entité");
-if (!defined("_ENTITY_UPDATE"))
+}
+if (!defined("_ENTITY_UPDATE")) {
     define("_ENTITY_UPDATE", "Entité mise à jour");
+}
 
 // USER ABS
-if (!defined("_MY_ABS"))
+if (!defined("_MY_ABS")) {
     define("_MY_ABS", "Gérer mes absences");
-if (!defined("_MY_ABS_TXT"))
+}
+if (!defined("_MY_ABS_TXT")) {
     define("_MY_ABS_TXT", "Permet de rediriger vos bannettes en cas de départ en congé.");
-if (!defined("_MY_ABS_REDIRECT"))
+}
+if (!defined("_MY_ABS_REDIRECT")) {
     define("_MY_ABS_REDIRECT", "Vos courriers sont actuellement redirigés vers");
-if (!defined("_MY_ABS_DEL"))
+}
+if (!defined("_MY_ABS_DEL")) {
     define("_MY_ABS_DEL", "Pour supprimer la redirection, cliquez ici pour stopper");
-if (!defined("_ADMIN_ABS"))
+}
+if (!defined("_ADMIN_ABS")) {
     define("_ADMIN_ABS", "Gérer les absences.");
-if (!defined("_ADMIN_ABS_TXT"))
+}
+if (!defined("_ADMIN_ABS_TXT")) {
     define("_ADMIN_ABS_TXT", "Permet de rediriger le courrier de l'utilisateur en attente en cas de départ en congé.");
-if (!defined("_ADMIN_ABS_REDIRECT"))
+}
+if (!defined("_ADMIN_ABS_REDIRECT")) {
     define("_ADMIN_ABS_REDIRECT", "Redirection d'absence en cours.");
-if (!defined("_ADMIN_ABS_FIRST_PART"))
+}
+if (!defined("_ADMIN_ABS_FIRST_PART")) {
     define("_ADMIN_ABS_FIRST_PART", "Les courrier de");
-if (!defined("_ADMIN_ABS_SECOND_PART"))
+}
+if (!defined("_ADMIN_ABS_SECOND_PART")) {
     define("_ADMIN_ABS_SECOND_PART", "sont actuellement redirigés vers ");
-if (!defined("_ADMIN_ABS_THIRD_PART"))
+}
+if (!defined("_ADMIN_ABS_THIRD_PART")) {
     define("_ADMIN_ABS_THIRD_PART", ". Cliquez ici pour supprimer la redirection.");
-if (!defined("_ACTIONS_DONE"))
+}
+if (!defined("_ACTIONS_DONE")) {
     define("_ACTIONS_DONE", "Actions effectuées le");
-if (!defined("_PROCESSED_MAIL"))
+}
+if (!defined("_PROCESSED_MAIL")) {
     define("_PROCESSED_MAIL", "Courriers traités");
-if (!defined("_INDEXED_MAIL"))
+}
+if (!defined("_INDEXED_MAIL")) {
     define("_INDEXED_MAIL", "Courriers indexés");
-if (!defined("_REDIRECTED_MAIL"))
+}
+if (!defined("_REDIRECTED_MAIL")) {
     define("_REDIRECTED_MAIL", "Courriers redirigés");
-if (!defined("_PROCESS_MAIL_OF"))
+}
+if (!defined("_PROCESS_MAIL_OF")) {
     define("_PROCESS_MAIL_OF", "Courrier à traiter de");
-if (!defined("_MISSING"))
+}
+if (!defined("_MISSING")) {
     define("_MISSING", "Absent");
-if (!defined("_BACK_FROM_VACATION"))
+}
+if (!defined("_BACK_FROM_VACATION")) {
     define("_BACK_FROM_VACATION", "de retour de son absence");
-if (!defined("_MISSING_ADVERT_TITLE"))
-    define("_MISSING_ADVERT_TITLE","Gestion des absences");
-if (!defined("_MISSING_ADVERT_01"))
-    define("_MISSING_ADVERT_01","Ce compte est actuellement défini en mode 'absent'.");
-if (!defined("_MISSING_ADVERT_02"))
-    define("_MISSING_ADVERT_02","Si vous desirez vous connecter avec ce compte, le mode 'absent' sera alors supprimé.<br/> La redirection des courriers arrivera à son terme et l'application sera réactivée");
-if (!defined("_MISSING_CHOOSE"))
-    define("_MISSING_CHOOSE","Souhaitez-vous continuer?");
-if (!defined("_CONFIG"))
+}
+if (!defined("_MISSING_ADVERT_TITLE")) {
+    define("_MISSING_ADVERT_TITLE", "Gestion des absences");
+}
+if (!defined("_MISSING_ADVERT_01")) {
+    define("_MISSING_ADVERT_01", "Ce compte est actuellement défini en mode 'absent'.");
+}
+if (!defined("_MISSING_ADVERT_02")) {
+    define("_MISSING_ADVERT_02", "Si vous desirez vous connecter avec ce compte, le mode 'absent' sera alors supprimé.<br/> La redirection des courriers arrivera à son terme et l'application sera réactivée");
+}
+if (!defined("_MISSING_CHOOSE")) {
+    define("_MISSING_CHOOSE", "Souhaitez-vous continuer?");
+}
+if (!defined("_CONFIG")) {
     define("_CONFIG", "(paramètrer)");
-if (!defined("_IN_ACTION"))
+}
+if (!defined("_IN_ACTION")) {
     define("_IN_ACTION", " dans l'action");
-if (!defined("_TO_ENTITIES"))
+}
+if (!defined("_TO_ENTITIES")) {
     define("_TO_ENTITIES", "Vers des entités");
-if (!defined("_TO_USERGROUPS"))
+}
+if (!defined("_TO_USERGROUPS")) {
     define("_TO_USERGROUPS", "Vers des groupes d'utilisateur");
-if (!defined("_USE_IN_MASS"))
+}
+if (!defined("_USE_IN_MASS")) {
     define("_USE_IN_MASS", "Action disponible dans la liste");
-if (!defined("_USE_ONE"))
+}
+if (!defined("_USE_ONE")) {
     define("_USE_ONE", "Action disponible dans la page d'action");
-if (!defined("_SAVE_CHANGES"))
+}
+if (!defined("_SAVE_CHANGES")) {
     define("_SAVE_CHANGES", "Enregistrer les modifications");
-if (!defined("_VIEW_BASKETS"))
+}
+if (!defined("_VIEW_BASKETS")) {
     define("_VIEW_BASKETS", "Mes bannettes");
-if (!defined("_MY_BASKETS"))
+}
+if (!defined("_MY_BASKETS")) {
     define("_MY_BASKETS", "Mes bannettes");
-if (!defined("_NAME"))
+}
+if (!defined("_NAME")) {
     define("_NAME", "Nom");
-if (!defined("_FORM_ERROR"))
+}
+if (!defined("_FORM_ERROR")) {
     define("_FORM_ERROR", "Erreur dans la transmission du formulaire...");
-if (!defined("_ABS_LOG_OUT"))
+}
+if (!defined("_ABS_LOG_OUT")) {
     define("_ABS_LOG_OUT", "si vous vous reconnectez, le mode absent sera annulé.");
-if (!defined("_ABS_USER"))
+}
+if (!defined("_ABS_USER")) {
     define("_ABS_USER", "Utilisateur absent");
-if (!defined("_ABSENCE"))
+}
+if (!defined("_ABSENCE")) {
     define("_ABSENCE", "Absence");
-if (!defined("_BASK_BACK"))
+}
+if (!defined("_BASK_BACK")) {
     define("_BASK_BACK", "Retour");
-if (!defined("_FILTER_BY"))
+}
+if (!defined("_FILTER_BY")) {
     define("_FILTER_BY", "Filtrer par");
-if (!defined("_VIEWED"))
+}
+if (!defined("_VIEWED")) {
     define("_VIEWED", "Vu?");
+}
 
 //NEW WF
-if (!defined("_WF"))
+if (!defined("_WF")) {
     define("_WF", "Workflow");
-if (!defined("_POSITION"))
+}
+if (!defined("_POSITION")) {
     define("_POSITION", "Position");
+}
+if (!defined("_SPREAD_SEARCH_TO_BASKETS")) {
+    define("_SPREAD_SEARCH_TO_BASKETS", "Etendre la recherche aux bannettes");
+}
diff --git a/modules/basket/lang/nl.php b/modules/basket/lang/nl.php
index e426afc0f8e..dc43e993f2e 100755
--- a/modules/basket/lang/nl.php
+++ b/modules/basket/lang/nl.php
@@ -570,6 +570,3 @@ if (!defined('_REDIRECT_TO_DEP_OK')) {
 if (!defined('_AUTHORISED_ENTITIES')) {
     define('_AUTHORISED_ENTITIES', 'Lijst van de toegelaten eenheden');
 }
-if (!defined('_SPREAD_SEARCH_TO_BASKETS')) {
-    define('_SPREAD_SEARCH_TO_BASKETS', 'Omvang van de opzoeking');
-}
-- 
GitLab