[Dolibarr-bugtrack] [bug #28894] Lors de MAJ d'Action: Unknown column 'BuXU4iBh3e6vA' in 'where clause'

[Mario]

Follow-up Comment #6, bug #28894 (project dolibarr):

Je reviens également sur ce problème.  J'étais tout de même étonné que
la valeur de l'id dans un cookie soit important.

Après étude sommaire du code, je constates que '$_REQUEST' est utilisé au
lieu de '$_GET' ou '$_POST' seulement.
Effectivement, '$_REQUEST' comprends les valeurs de '$_GET', '$_POST' et
'$_COOKIE'.  C'est donc bien pour cela que nous avons ces problèmes.

En changeant la ligne:
        $_GET["id"]=$_REQUEST["id"];
 dans
        $_GET["id"]=$_POST["id"];

je résous déjà mon problème d'origine.
Toutefois, changer '$_GET' n'est pas très recommandable, mais bon.  Une
meilleure méthode me semble en partie engagée, une variable '$id' est
défini au début de fiche.php.  Il faudrait utiliser celle là AMA dans le
reste du script.

En tous cas, le '$_REQUEST' est à eviter (aussi pour socid).  Je penses
qu'il vaut mieux encore définir une fonction utilitaire:
function get_or_post($varname) {
  if(isset($_POST[$varname])) { 
     return $_POST[$varname];
  }
  return $_GET[varname];
}

et remplacer $_REQUEST par get_or_post .

Ce code pour 'socid' doit problablement aussi changer:
// If socid provided by ajax company selector
if (! empty($_REQUEST['socid_id']))
{
        $_GET['socid'] = $_GET['socid_id'];
        $_POST['socid'] = $_POST['socid_id'];
        $_REQUEST['socid'] = $_REQUEST['socid_id'];
}

dans

// If socid provided by ajax company selector
if (! empty($_REQUEST['socid_id']))
{
        $_GET['socid'] = $_REQUEST['socid_id'];
        $_POST['socid'] = $_REQUEST['socid_id'];
        $_REQUEST['socid'] = $_REQUEST['socid_id'];
}


Ceci dit, le problème de Fred est sur un autre URL et ne semble pas être un
problème de même nature.

    _______________________________________________________

Reply to this item at:

  <http://savannah.nongnu.org/bugs/?28894>

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