Voir les traceurs | Bugs | Exporter au format CSV

Date :
10/04/2014 18:40
Priorité :
3
État :
Open
Proposé par :
Nicolas Haye (nhaye)
Confié à :
Nicolas Haye (nhaye)
Version concernée :
4.1.0-rc6
Version cible :
à déterminer
Sévérité :
minor
Résolution :
Accepted As Bug
Résumé :
Warning PHP lors d'une double validation du traitement j-5

Description détaillée
Ce bug n'est pas impactant pour l'utilisateur,
en revanche sa correction peut avoir des effets de bord et sera donc corrigé dans une prochaine release.
En attendant voici les diff :

Index: app/js/script.js
===================================================================
--- app/js/script.js (révision 1054)
+++ app/js/script.js (copie de travail)
@@ -254,7 +254,7 @@
// traitement
$.ajax({
type: "GET",
- url: url+"&action=traitement_"+link,
+ url: url+"&action=traitement_"+link+"&"+data,
cache: false,
success: function(html){
$("#traitement_"+link+"_status").empty();


Index: obj/traitement.j5.class.php
===================================================================
--- obj/traitement.j5.class.php (révision 1051)
+++ obj/traitement.j5.class.php (copie de travail)
@@ -42,19 +42,22 @@
$cinqjours = false;
$additions = array();
$mouvementatraiter = "";
- if (isset($_POST["mouvementatraiter"])) {
- $val = "";
- foreach($_POST["mouvementatraiter"] as $elem) {
- if ($elem == "Immediat") {
- $cinqjours = true;
- } else {
- array_push($additions, $elem);
- }
- $val .= $elem.";";
+ if(isset($_POST["mouvementatraiter"])) {
+ $mvmt = $_POST["mouvementatraiter"];
+ } elseif (isset($_GET["mouvementatraiter"])) {
+ $mvmt = $_GET["mouvementatraiter"];
+ }
+ $val = "";
+ foreach($mvmt as $elem) {
+ if ($elem == "Immediat") {
+ $cinqjours = true;
+ } else {
+ array_push($additions, $elem);
}
- $mouvementatraiter = $val;
- $this->form->setVal("mouvementatraiter", $val);
+ $val .= $elem.";";
}
+ $mouvementatraiter = $val;
+ $this->form->setVal("mouvementatraiter", $val);
$params = array("cinqjours" => $cinqjours,
"additions" => $additions,
"mouvementatraiter" => $mouvementatraiter);
@@ -166,17 +169,7 @@
}
$val .= $elem.";";
}
- } else {
- foreach($this->params["mouvementatraiter"] as $elem) {
- if ($elem == "Immediat") {
- $cinqjours = true;
- } else {
- array_push($additions, $elem);
- }
- $val .= $elem.";";
- }
}
-
//
require_once ("../obj/electeur.class.php");
include ("../sql/".$this->page->phptype."/trt_j5.inc");

No comments have been posted

Pas de documents joints

Champ Ancienne valeur Date Par
Version cible4.2.0-rc115/09/2014 15:28Nicolas Haye
Version cibleà déterminer04/08/2014 17:58Nicolas Haye
Version cible4.1.017/04/2014 17:58Nicolas Haye
FEDER Powered By FusionForge Collaborative Development Environment Charte d'utilisation / Nous contacter / Mentions légales Haut de page