From 4585f4e157014a201592ec41c06990cf4818d314 Mon Sep 17 00:00:00 2001 From: "alain.cisirika" Date: Tue, 9 Dec 2025 15:05:47 +0200 Subject: [PATCH] [FIX] Fixed sorting links in /product/stock/card.php by correcting print_liste_field_titre arguments (passing warehouse ID to the 5th argument instead of the 4th) and replacing & with & to prevent double-encoding. --- htdocs/product/stock/card.php | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/htdocs/product/stock/card.php b/htdocs/product/stock/card.php index 73effc43e1e..31f07a87e2d 100644 --- a/htdocs/product/stock/card.php +++ b/htdocs/product/stock/card.php @@ -649,36 +649,36 @@ if ($action == 'create') { $reshook = $hookmanager->executeHooks('printFieldPreListTitle', $parameters); // Note that $action and $object may have been modified by hook print $hookmanager->resPrint; - print_liste_field_titre("Products", "", "p.ref", "&id=".$id, "", "", $sortfield, $sortorder); - print_liste_field_titre("Label", "", "p.label", "&id=".$id, "", "", $sortfield, $sortorder); - print_liste_field_titre("NumberOfUnit", "", "ps.reel", "&id=".$id, "", '', $sortfield, $sortorder, 'right '); + print_liste_field_titre("Products", "", "p.ref", "", "&id=".$id, "", $sortfield, $sortorder); + print_liste_field_titre("Label", "", "p.label", "", "&id=".$id, "", $sortfield, $sortorder); + print_liste_field_titre("NumberOfUnit", "", "ps.reel", "", "&id=".$id, '', $sortfield, $sortorder, 'right '); $totalarray['nbfield'] += 3; $totalarray['pos'][$totalarray['nbfield']] = 'totalunit'; $totalarray['type'][$totalarray['nbfield']] = 'stock'; if (getDolGlobalString('PRODUCT_USE_UNITS')) { - print_liste_field_titre("Unit", "", "p.fk_unit", "&id=".$id, "", 'align="left"', $sortfield, $sortorder); + print_liste_field_titre("Unit", "", "p.fk_unit", "", "&id=".$id, 'align="left"', $sortfield, $sortorder); $totalarray['nbfield']++; $totalarray['pos'][$totalarray['nbfield']] = 'units'; $totalarray['type'][$totalarray['nbfield']] = 'string'; } $usercancreadsupplierprice = getDolGlobalString('MAIN_USE_ADVANCED_PERMS') ? $user->hasRight('product', 'product_advance', 'read_supplier_prices') : $user->hasRight('product', 'read'); if ($usercancreadsupplierprice) { - print_liste_field_titre($form->textwithpicto($langs->trans("AverageUnitPricePMPShort"), $langs->trans("AverageUnitPricePMPDesc")), "", "p.pmp", "&id=".$id, "", '', $sortfield, $sortorder, 'right '); + print_liste_field_titre($form->textwithpicto($langs->trans("AverageUnitPricePMPShort"), $langs->trans("AverageUnitPricePMPDesc")), "", "p.pmp", "", "&id=".$id, '', $sortfield, $sortorder, 'right '); $totalarray['nbfield']++; - print_liste_field_titre("EstimatedStockValueShort", "", "svalue", "&id=".$id, "", '', $sortfield, $sortorder, 'right '); + print_liste_field_titre("EstimatedStockValueShort", "", "svalue", "", "&id=".$id, '', $sortfield, $sortorder, 'right '); $totalarray['nbfield']++; $totalarray['pos'][$totalarray['nbfield']] = 'totalvalue'; $totalarray['type'][$totalarray['nbfield']] = ''; } if (!getDolGlobalString('PRODUIT_MULTIPRICES')) { - print_liste_field_titre("SellPriceMin", "", "p.price", "&id=".$id, "", '', $sortfield, $sortorder, 'right '); + print_liste_field_titre("SellPriceMin", "", "p.price", "", "&id=".$id, '', $sortfield, $sortorder, 'right '); $totalarray['nbfield']++; } if (!getDolGlobalString('PRODUIT_MULTIPRICES')) { - print_liste_field_titre("EstimatedStockValueSellShort", "", "", "&id=".$id, "", '', $sortfield, $sortorder, 'right '); + print_liste_field_titre("EstimatedStockValueSellShort", "", "", "", "&id=".$id, '', $sortfield, $sortorder, 'right '); $totalarray['nbfield']++; $totalarray['pos'][$totalarray['nbfield']] = 'totalvaluesell'; $totalarray['type'][$totalarray['nbfield']] = '';