[Dolibarr-dev] [bug #13934] décimales qui sautent dans les factures récurentes

[Christophe PEREZ]

Follow-up Comment #1, bug #13934 (project dolibarr):

Bon, j'ai fini par trouver, c'est le jeu avec les setlocale() qui provoque
tout ce méli-mélo.
Et j'ai vous que vous aviez prévu le coup avec une variable
MAIN_FORCE_SETLOCALE_LC_NUMERIC

Par contre, la fonction supprimer sur les factures récurrentes ne fontionne
pas. En effet, elle contrôle l'action sur un $_POST alors que c'est passé en
$_GET.

Pour corriger, fichier compta/facture/fiche-rec.php, changer la ligne 77, de
if ($_POST["action"] == 'delete' && $user->rights->facture->supprimer)
en
if ($_REQUEST["action"] == 'delete' && $user->rights->facture->supprimer) 

D'ailleurs, j'ai un peu de mal à comprendre pourquoi s'embêter autant avec ces
restrictions qui sont source d'erreur (la preuve) et qui n'apporte rien en
sécurité par rapport à un $_REQUEST.
surtout quand je lis des choses comme ça :
if (isset($_POST["type"]) || isset($_GET["type"])) {

Mais bon, vous devez avoir vos raisons que j'ignore ;-)


    _______________________________________________________

Reply to this item at:

  <http://savannah.nongnu.org/bugs/?func=detailitem&item_id=13934>

_______________________________________________
  Message posté via/par Savannah
  http://savannah.nongnu.org/