View Trackers | Bugs | Export CSV

Date:
2014-04-10 18:40
Priority:
3
State:
Open
Submitted by:
Nicolas Haye (nhaye)
Assigned to:
Nicolas Haye (nhaye)
Version concernée:
4.1.0-rc6
Version cible:
à déterminer
Sévérité:
minor
Résolution:
Accepted As Bug
Summary:
Warning PHP lors d'une double validation du traitement j-5

Detailed description
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

No attached documents

Field Old Value Date By
Version cible4.2.0-rc12014-09-15 15:28Nicolas Haye
Version cibleà déterminer2014-08-04 17:58Nicolas Haye
Version cible4.1.02014-04-17 17:58Nicolas Haye
FEDER Powered By FusionForge Collaborative Development Environment Charte d'utilisation / Nous contacter / Mentions légales Haut de page