--- /tmp/dsg/dolibarr/htdocs/opensurvey/wizard/github_19.0.3_choix_autre.php +++ /tmp/dsg/dolibarr/htdocs/opensurvey/wizard/client_choix_autre.php @@ -25 +24,0 @@ -// Load Dolibarr environment @@ -29 +28 @@ -require_once DOL_DOCUMENT_ROOT."/opensurvey/lib/opensurvey.lib.php"; +require_once DOL_DOCUMENT_ROOT."/opensurvey/fonctions.php"; @@ -32,3 +31 @@ -if (!$user->hasRight('opensurvey', 'write')) { - accessforbidden(); -} +if (!$user->rights->opensurvey->write) accessforbidden(); @@ -42,3 +38,0 @@ -$arrayofchoices = GETPOST('choix', 'array'); -$arrayoftypecolumn = GETPOST('typecolonne', 'array'); - @@ -48,2 +42,2 @@ - if (isset($arrayofchoices[$i])) { - $_SESSION["choix".$i] = $arrayofchoices[$i]; + if (isset($_POST["choix"][$i])) { + $_SESSION["choix$i"] = $_POST["choix"][$i]; @@ -51,2 +45,2 @@ - if (isset($arrayoftypecolumn[$i])) { - $_SESSION["typecolonne".$i] = $arrayoftypecolumn[$i]; + if (isset($_POST["typecolonne"][$i])) { + $_SESSION["typecolonne$i"] = $_POST["typecolonne"][$i]; @@ -59 +53,2 @@ -if (GETPOST("ajoutcases") || GETPOST("ajoutcases_x")) { +if (GETPOST("ajoutcases") || GETPOST("ajoutcases_x")) +{ @@ -64 +59,2 @@ -if (GETPOSTISSET("confirmecreation")) { +if (isset($_POST["confirmecreation"])) +{ @@ -67,2 +63,4 @@ - for ($i = 0; $i < $_SESSION["nbrecases"] + 1; $i++) { - if (!empty($arrayofchoices[$i])) { + for ($i = 0; $i < $_SESSION["nbrecases"] + 1; $i++) + { + if (!empty($_POST["choix"][$i])) + { @@ -70 +68 @@ - $toutchoix .= str_replace(array(",", "@"), " ", $arrayofchoices[$i]).(empty($arrayoftypecolumn[$i]) ? '' : '@'.$arrayoftypecolumn[$i]); + $toutchoix .= str_replace(array(",", "@"), " ", $_POST["choix"][$i]).(empty($_POST["typecolonne"][$i]) ? '' : '@'.$_POST["typecolonne"][$i]); @@ -79,2 +77,4 @@ - for ($i = 0; $i < $_SESSION["nbrecases"]; $i++) { - if (isset($arrayofchoices[$i])) { + for ($i = 0; $i < $_SESSION["nbrecases"]; $i++) + { + if (isset($_POST["choix"][$i])) + { @@ -96,0 +97,3 @@ + + + @@ -107 +110,2 @@ -if (empty($_SESSION['title'])) { +if (empty($_SESSION['titre'])) +{ --- /tmp/dsg/dolibarr/htdocs/opensurvey/wizard/github_19.0.3_choix_date.php +++ /tmp/dsg/dolibarr/htdocs/opensurvey/wizard/client_choix_date.php @@ -25 +24,0 @@ -// Load Dolibarr environment @@ -29 +28 @@ -require_once DOL_DOCUMENT_ROOT."/opensurvey/lib/opensurvey.lib.php"; +require_once DOL_DOCUMENT_ROOT."/opensurvey/fonctions.php"; @@ -32,3 +31 @@ -if (!$user->hasRight('opensurvey', 'write')) { - accessforbidden(); -} +if (!$user->rights->opensurvey->write) accessforbidden(); @@ -40 +36,0 @@ -$erreurNbchoice = 0; @@ -47 +43,2 @@ -if (GETPOST('confirmation')) { +if (GETPOST('confirmation')) +{ @@ -49,17 +46,30 @@ - if (issetAndNoEmpty('totalchoixjour', $_SESSION) === true && issetAndNoEmpty('nbrecaseshoraires', $_SESSION) === true) { - $nbofchoice = count($_SESSION["totalchoixjour"]); - $errheure = array(); - - if ($nbofchoice * $_SESSION["nbrecaseshoraires"] > 200) { - setEventMessages($langs->trans("ErrorFieldTooLong"), null, 'errors'); - $erreurNb++; - } else { - for ($i = 0; $i < $nbofchoice; $i++) { - // Show hours choices - for ($j = 0; $j < $_SESSION["nbrecaseshoraires"]; $j++) { - $horairesi = GETPOST("horaires".$i); - $_SESSION["horaires$i"][$j] = $horairesi[$j]; - - $tmphorairesi = GETPOST('horaires'.$i, 'array'); - - if (!is_array($tmphorairesi)) { + if (issetAndNoEmpty('totalchoixjour', $_SESSION) === true && issetAndNoEmpty('nbrecaseshoraires', $_SESSION) === true) + { + $nbofchoice = count($_SESSION["totalchoixjour"]); + + for ($i = 0; $i < $nbofchoice; $i++) + { + // Show hours choices + for ($j = 0; $j < $_SESSION["nbrecaseshoraires"]; $j++) + { + $_SESSION["horaires$i"][$j] = $_POST["horaires$i"][$j]; + + $case = $j + 1; + + if (isset($_POST['horaires'.$i]) === false || isset($_POST['horaires'.$i][$j]) === false) { + $errheure[$i][$j] = true; + $erreur = true; + continue; + } + + //si c'est un creneau type 8:00-11:00 + if (preg_match("/(\d{1,2}:\d{2})-(\d{1,2}:\d{2})/", $_POST["horaires$i"][$j], $creneaux)) { + //on recupere les deux parties du preg_match qu'on redécoupe autour des ":" + $debutcreneau = explode(":", $creneaux[1]); + $fincreneau = explode(":", $creneaux[2]); + + //comparaison des heures de fin et de debut + //si correctes, on entre les données dans la variables de session + if ($debutcreneau[0] < 24 && $fincreneau[0] < 24 && $debutcreneau[1] < 60 && $fincreneau[1] < 60 && ($debutcreneau[0] < $fincreneau[0] || ($debutcreneau[0] == $fincreneau[0] && $debutcreneau[1] < $fincreneau[1]))) { + $_SESSION["horaires$i"][$j] = $creneaux[1].'-'.$creneaux[2]; + } else { //sinon message d'erreur et nettoyage de la case @@ -68,67 +78,11 @@ - continue; - } - - // A range like 8:00-11:00 - $creneaux = array(); - $heures = array(); - if (preg_match("/(\d{1,2}:\d{2})-(\d{1,2}:\d{2})/", $tmphorairesi[$j], $creneaux)) { - //on recupere les deux parties du preg_match qu'on redécoupe autour des ":" - $debutcreneau = explode(":", $creneaux[1]); - $fincreneau = explode(":", $creneaux[2]); - - //comparaison des heures de fin et de debut - //si correctes, on entre les données dans la variables de session - if ($debutcreneau[0] < 24 && $fincreneau[0] < 24 && $debutcreneau[1] < 60 && $fincreneau[1] < 60 && ($debutcreneau[0] < $fincreneau[0] || ($debutcreneau[0] == $fincreneau[0] && $debutcreneau[1] < $fincreneau[1]))) { - $_SESSION["horaires$i"][$j] = $creneaux[1].'-'.$creneaux[2]; - } else { //sinon message d'erreur et nettoyage de la case - $errheure[$i][$j] = true; - $erreur = true; - } - } elseif (preg_match(";^(\d{1,2}h\d{0,2})-(\d{1,2}h\d{0,2})$;i", $tmphorairesi[$j], $creneaux)) { //si c'est un creneau type 8h00-11h00 - //on recupere les deux parties du preg_match qu'on redécoupe autour des "H" - $debutcreneau = preg_split("/h/i", $creneaux[1]); - $fincreneau = preg_split("/h/i", $creneaux[2]); - - //comparaison des heures de fin et de debut - //si correctes, on entre les données dans la variables de session - if ($debutcreneau[0] < 24 && $fincreneau[0] < 24 && $debutcreneau[1] < 60 && $fincreneau[1] < 60 && ($debutcreneau[0] < $fincreneau[0] || ($debutcreneau[0] == $fincreneau[0] && $debutcreneau[1] < $fincreneau[1]))) { - $_SESSION["horaires$i"][$j] = $creneaux[1].'-'.$creneaux[2]; - } else { //sinon message d'erreur et nettoyage de la case - $errheure[$i][$j] = true; - $erreur = true; - } - } elseif (preg_match(";^(\d{1,2}):(\d{2})$;", $tmphorairesi[$j], $heures)) { //si c'est une heure simple type 8:00 - //si valeures correctes, on entre les données dans la variables de session - if ($heures[1] < 24 && $heures[2] < 60) { - $_SESSION["horaires$i"][$j] = $heures[0]; - } else { //sinon message d'erreur et nettoyage de la case - $errheure[$i][$j] = true; - $erreur = true; - } - } elseif (preg_match(";^(\d{1,2})h(\d{0,2})$;i", $tmphorairesi[$j], $heures)) { //si c'est une heure encore plus simple type 8h - //si valeures correctes, on entre les données dans la variables de session - if ($heures[1] < 24 && $heures[2] < 60) { - $_SESSION["horaires$i"][$j] = $heures[0]; - } else { //sinon message d'erreur et nettoyage de la case - $errheure[$i][$j] = true; - $erreur = true; - } - } elseif (preg_match(";^(\d{1,2})-(\d{1,2})$;", $tmphorairesi[$j], $heures)) { //si c'est un creneau simple type 8-11 - //si valeures correctes, on entre les données dans la variables de session - if ($heures[1] < $heures[2] && $heures[1] < 24 && $heures[2] < 24) { - $_SESSION["horaires$i"][$j] = $heures[0]; - } else { //sinon message d'erreur et nettoyage de la case - $errheure[$i][$j] = true; - $erreur = true; - } - } elseif (preg_match(";^(\d{1,2})h-(\d{1,2})h$;", $tmphorairesi[$j], $heures)) { //si c'est un creneau H type 8h-11h - //si valeures correctes, on entre les données dans la variables de session - if ($heures[1] < $heures[2] && $heures[1] < 24 && $heures[2] < 24) { - $_SESSION["horaires$i"][$j] = $heures[0]; - } else { //sinon message d'erreur et nettoyage de la case - $errheure[$i][$j] = true; - $erreur = true; - } - } elseif ($tmphorairesi[$j] == "") { //Si la case est vide - unset($_SESSION["horaires$i"][$j]); - } else { //pour tout autre format, message d'erreur + } + } elseif (preg_match(";^(\d{1,2}h\d{0,2})-(\d{1,2}h\d{0,2})$;i", $_POST["horaires$i"][$j], $creneaux)) { //si c'est un creneau type 8h00-11h00 + //on recupere les deux parties du preg_match qu'on redécoupe autour des "H" + $debutcreneau = preg_split("/h/i", $creneaux[1]); + $fincreneau = preg_split("/h/i", $creneaux[2]); + + //comparaison des heures de fin et de debut + //si correctes, on entre les données dans la variables de session + if ($debutcreneau[0] < 24 && $fincreneau[0] < 24 && $debutcreneau[1] < 60 && $fincreneau[1] < 60 && ($debutcreneau[0] < $fincreneau[0] || ($debutcreneau[0] == $fincreneau[0] && $debutcreneau[1] < $fincreneau[1]))) { + $_SESSION["horaires$i"][$j] = $creneaux[1].'-'.$creneaux[2]; + } else { //sinon message d'erreur et nettoyage de la case @@ -138,30 +92,66 @@ - - if (issetAndNoEmpty('horaires'.$i, $_SESSION) === false || issetAndNoEmpty($j, $_SESSION['horaires'.$i]) === false) { - if (issetAndNoEmpty('horaires'.$i, $_SESSION) === true) { - $_SESSION["horaires$i"][$j] = ''; - } else { - $_SESSION["horaires$i"] = array(); - $_SESSION["horaires$i"][$j] = ''; - } - } - } - - if ($_SESSION["horaires$i"][0] == "" && $_SESSION["horaires$i"][1] == "" && $_SESSION["horaires$i"][2] == "" && $_SESSION["horaires$i"][3] == "" && $_SESSION["horaires$i"][4] == "") { - $choixdate .= ","; - $choixdate .= $_SESSION["totalchoixjour"][$i]; - } else { - for ($j = 0; $j < $_SESSION["nbrecaseshoraires"]; $j++) { - if ($_SESSION["horaires$i"][$j] != "") { - $choixdate .= ","; - $choixdate .= $_SESSION["totalchoixjour"][$i]; - $choixdate .= "@"; - // On remplace la virgule et l'arobase pour ne pas avoir de problème par la suite - $choixdate .= str_replace(array(',', '@'), array(',', '@'), $_SESSION["horaires$i"][$j]); - } - } - } - } - } - - - if (!empty($errheure)) { + } elseif (preg_match(";^(\d{1,2}):(\d{2})$;", $_POST["horaires$i"][$j], $heures)) { //si c'est une heure simple type 8:00 + //si valeures correctes, on entre les données dans la variables de session + if ($heures[1] < 24 && $heures[2] < 60) { + $_SESSION["horaires$i"][$j] = $heures[0]; + } else { //sinon message d'erreur et nettoyage de la case + $errheure[$i][$j] = true; + $erreur = true; + } + } elseif (preg_match(";^(\d{1,2})h(\d{0,2})$;i", $_POST["horaires$i"][$j], $heures)) { //si c'est une heure encore plus simple type 8h + //si valeures correctes, on entre les données dans la variables de session + if ($heures[1] < 24 && $heures[2] < 60) { + $_SESSION["horaires$i"][$j] = $heures[0]; + } else { //sinon message d'erreur et nettoyage de la case + $errheure[$i][$j] = true; + $erreur = true; + } + } elseif (preg_match(";^(\d{1,2})-(\d{1,2})$;", $_POST["horaires$i"][$j], $heures)) { //si c'est un creneau simple type 8-11 + //si valeures correctes, on entre les données dans la variables de session + if ($heures[1] < $heures[2] && $heures[1] < 24 && $heures[2] < 24) { + $_SESSION["horaires$i"][$j] = $heures[0]; + } else { //sinon message d'erreur et nettoyage de la case + $errheure[$i][$j] = true; + $erreur = true; + } + } elseif (preg_match(";^(\d{1,2})h-(\d{1,2})h$;", $_POST["horaires$i"][$j], $heures)) { //si c'est un creneau H type 8h-11h + //si valeures correctes, on entre les données dans la variables de session + if ($heures[1] < $heures[2] && $heures[1] < 24 && $heures[2] < 24) { + $_SESSION["horaires$i"][$j] = $heures[0]; + } else { //sinon message d'erreur et nettoyage de la case + $errheure[$i][$j] = true; + $erreur = true; + } + } elseif ($_POST["horaires$i"][$j] == "") { //Si la case est vide + unset($_SESSION["horaires$i"][$j]); + } else { //pour tout autre format, message d'erreur + $errheure[$i][$j] = true; + $erreur = true; + } + + if (issetAndNoEmpty('horaires'.$i, $_SESSION) === false || issetAndNoEmpty($j, $_SESSION['horaires'.$i]) === false) { + if (issetAndNoEmpty('horaires'.$i, $_SESSION) === true) { + $_SESSION["horaires$i"][$j] = ''; + } else { + $_SESSION["horaires$i"] = array(); + $_SESSION["horaires$i"][$j] = ''; + } + } + } + + if ($_SESSION["horaires$i"][0] == "" && $_SESSION["horaires$i"][1] == "" && $_SESSION["horaires$i"][2] == "" && $_SESSION["horaires$i"][3] == "" && $_SESSION["horaires$i"][4] == "") { + $choixdate .= ","; + $choixdate .= $_SESSION["totalchoixjour"][$i]; + } else { + for ($j = 0; $j < $_SESSION["nbrecaseshoraires"]; $j++) { + if ($_SESSION["horaires$i"][$j] != "") { + $choixdate .= ","; + $choixdate .= $_SESSION["totalchoixjour"][$i]; + $choixdate .= "@"; + // On remplace la virgule et l'arobase pour ne pas avoir de problème par la suite + $choixdate .= str_replace(array(',', '@'), array(',', '@'), $_SESSION["horaires$i"][$j]); + } + } + } + } + + if (isset($errheure)) { @@ -173,2 +163 @@ - $tmphoraires0 = GETPOST('horaires0', 'array'); - if (count($_SESSION["totalchoixjour"]) == "1" && $tmphoraires0[0] == "" && $tmphoraires0[1] == "" && $tmphoraires0[2] == "" && $tmphoraires0[3] == "" && $tmphoraires0[4] == "") { + if (count($_SESSION["totalchoixjour"]) == "1" && $_POST["horaires0"][0] == "" && $_POST["horaires0"][1] == "" && $_POST["horaires0"][2] == "" && $_POST["horaires0"][3] == "" && $_POST["horaires0"][4] == "") { @@ -180 +169,2 @@ - if (!$erreur && $erreurNb == 0) { + if (!$erreur) + { @@ -182 +171,0 @@ - unset($_SESSION["nbrecaseshoraires"]); @@ -189 +178,2 @@ -if (GETPOST('reset')) { +if (GETPOST('reset')) +{ @@ -191 +181,2 @@ - for ($i = 0; $i < $nbofchoice; $i++) { + for ($i = 0; $i < $nbofchoice; $i++) + { @@ -207 +198,2 @@ -if (!isset($_SESSION['description']) && !isset($_SESSION['mail'])) { +if (!isset($_SESSION['commentaires']) && !isset($_SESSION['mail'])) +{ @@ -217 +209,2 @@ -if (!isset($_SESSION["nbrecaseshoraires"])) { +if (!isset($_SESSION["nbrecaseshoraires"])) +{ @@ -219 +212,3 @@ -} elseif ((GETPOST('ajoutcases') || GETPOST("ajoutcases_y")) && $_SESSION["nbrecaseshoraires"] == 5) { +} +elseif (GETPOST('ajoutcases') && $_SESSION["nbrecaseshoraires"] == 5) +{ @@ -221,13 +216 @@ - //On sauvegarde les heures deja entrées - if (issetAndNoEmpty('totalchoixjour', $_SESSION) === true) { - $nbofchoice = count($_SESSION["totalchoixjour"]); - for ($i = 0; $i < $nbofchoice; $i++) { - //affichage des 5 cases horaires - for ($j = 0; $j < $_SESSION["nbrecaseshoraires"]; $j++) { - $horairesi = GETPOST("horaires".$i); - $_SESSION["horaires$i"][$j] = $horairesi[$j]; - } - } - } -} - +} @@ -241,9 +224,3 @@ -if (!isset($_SESSION['jour'])) { - $_SESSION['jour'] = date('j'); -} -if (!isset($_SESSION['mois'])) { - $_SESSION['mois'] = date('n'); -} -if (!isset($_SESSION['annee'])) { - $_SESSION['annee'] = date('Y'); -} +if (!isset($_SESSION['jour'])) $_SESSION['jour'] = date('j'); +if (!isset($_SESSION['mois'])) $_SESSION['mois'] = date('n'); +if (!isset($_SESSION['annee'])) $_SESSION['annee'] = date('Y'); @@ -268 +245,2 @@ - if (issetAndNoEmpty('totalchoixjour', $_SESSION) === true) { + if (issetAndNoEmpty('totalchoixjour', $_SESSION) === true) + { @@ -273,2 +251 @@ - $horairesi = GETPOST("horaires".$i); - $_SESSION["horaires$i"][$j] = $horairesi[$j]; + $_SESSION["horaires$i"][$j] = $_POST["horaires$i"][$j]; @@ -290 +267,20 @@ - if (issetAndNoEmpty('totalchoixjour', $_SESSION) === true) { + if (issetAndNoEmpty('totalchoixjour', $_SESSION) === true) + { + $nbofchoice = count($_SESSION["totalchoixjour"]); + for ($i = 0; $i < $nbofchoice; $i++) + { + //affichage des 5 cases horaires + for ($j = 0; $j < $_SESSION["nbrecaseshoraires"]; $j++) { + $_SESSION["horaires$i"][$j] = $_POST["horaires$i"][$j]; + } + } + } +} + +//mise a jour des valeurs de session si annee avant +if (issetAndNoEmpty('anneeavant_x') || issetAndNoEmpty('anneeavant')) { + $_SESSION["annee"] -= 1; + + //On sauvegarde les heures deja entrées + if (issetAndNoEmpty('totalchoixjour', $_SESSION) === true) + { @@ -295,10 +291,9 @@ - $horairesi = GETPOST("horaires".$i); - $_SESSION["horaires$i"][$j] = $horairesi[$j]; - } - } - } -} - -//mise a jour des valeurs de session si annee avant -if (issetAndNoEmpty('anneeavant_x') || issetAndNoEmpty('anneeavant')) { - $_SESSION["annee"] -= 1; + $_SESSION["horaires$i"][$j] = $_POST["horaires$i"][$j]; + } + } + } +} + +//mise a jour des valeurs de session si annee apres +if (issetAndNoEmpty('anneeapres_x') || issetAndNoEmpty('anneeapres')) { + $_SESSION["annee"] += 1; @@ -307 +302,2 @@ - if (issetAndNoEmpty('totalchoixjour', $_SESSION) === true) { + if (issetAndNoEmpty('totalchoixjour', $_SESSION) === true) + { @@ -312,19 +308 @@ - $horairesi = GETPOST("horaires".$i); - $_SESSION["horaires$i"][$j] = $horairesi[$j]; - } - } - } -} - -//mise a jour des valeurs de session si annee apres -if (issetAndNoEmpty('anneeapres_x') || issetAndNoEmpty('anneeapres')) { - $_SESSION["annee"] += 1; - - //On sauvegarde les heures deja entrées - if (issetAndNoEmpty('totalchoixjour', $_SESSION) === true) { - $nbofchoice = count($_SESSION["totalchoixjour"]); - for ($i = 0; $i < $nbofchoice; $i++) { - //affichage des 5 cases horaires - for ($j = 0; $j < $_SESSION["nbrecaseshoraires"]; $j++) { - $horairesi = GETPOST("horaires".$i); - $_SESSION["horaires$i"][$j] = $horairesi[$j]; + $_SESSION["horaires$i"][$j] = $_POST["horaires$i"][$j]; @@ -341 +319,2 @@ -if (is_integer($_SESSION["mois"]) && $_SESSION["mois"] > 0 && $_SESSION["mois"] < 13) { +if (is_integer($_SESSION["mois"]) && $_SESSION["mois"] > 0 && $_SESSION["mois"] < 13) +{ @@ -343 +322,3 @@ -} else { +} +else +{ @@ -363,7 +344,5 @@ -print ''; -print ''; -print ' '.$motmois.' '.$_SESSION["annee"].'
'; -print ''; -print ''; -print ''; -print ''; +print ''; +print ' '.$motmois.' '.$_SESSION["annee"].'
'; +print ''; +print ''; +print ''; @@ -374 +353 @@ -print '
'."\n"; +print '
'."\n"; @@ -379 +358,2 @@ -for ($i = 0; $i < 7; $i++) { +for ($i = 0; $i < 7; $i++) +{ @@ -393 +373,2 @@ - if (issetAndNoEmpty('totalchoixjour', $_SESSION) === true && issetAndNoEmpty('choixjourajout') === true) { + if (issetAndNoEmpty('totalchoixjour', $_SESSION) === true && issetAndNoEmpty('choixjourajout') === true) + { @@ -396,2 +377 @@ - $choixjourajout = GETPOST("choixjourajout"); - if ($_SESSION["totalchoixjour"][$i] == mktime(0, 0, 0, $_SESSION["mois"], $choixjourajout[0], $_SESSION["annee"])) { + if ($_SESSION["totalchoixjour"][$i] == mktime(0, 0, 0, $_SESSION["mois"], $_POST["choixjourajout"][0], $_SESSION["annee"])) { @@ -405,2 +385 @@ - $choixjourajout = GETPOST("choixjourajout"); - array_push($_SESSION["totalchoixjour"], dol_mktime(0, 0, 0, $_SESSION["mois"], $choixjourajout[0], $_SESSION["annee"])); + array_push($_SESSION["totalchoixjour"], dol_mktime(0, 0, 0, $_SESSION["mois"], $_POST["choixjourajout"][0], $_SESSION["annee"])); @@ -408 +387 @@ - $cle = array_search(dol_mktime(0, 0, 0, $_SESSION["mois"], $choixjourajout[0], $_SESSION["annee"]), $_SESSION["totalchoixjour"]); + $cle = array_search(dol_mktime(0, 0, 0, $_SESSION["mois"], $_POST["choixjourajout"][0], $_SESSION["annee"]), $_SESSION["totalchoixjour"]); @@ -412 +390,0 @@ - $horairesi = GETPOST("horaires".$i); @@ -415 +393 @@ - $_SESSION["horaires$i"][$j] = $horairesi[$j]; + $_SESSION["horaires$i"][$j] = $_POST["horaires$i"][$j]; @@ -425,2 +403 @@ - $horairesi = GETPOST("horaires".$i, 'array'); - $_SESSION["horaires$i"][$j] = $horairesi[$j]; + $_SESSION["horaires$k"][$j] = $_POST["horaires$i"][$j]; @@ -442,8 +419,8 @@ - $horairesi = GETPOST("horaires".$i); - $_SESSION["horaires$i"][$j] = $horairesi[$j]; - } - } - - for ($i = 0; $i < $nbofchoice; $i++) { - $choixjourretrait = GETPOST('choixjourretrait'); - if ($_SESSION["totalchoixjour"][$i] == mktime(0, 0, 0, $_SESSION["mois"], $choixjourretrait[0], $_SESSION["annee"])) { + $_SESSION["horaires$i"][$j] = $_POST["horaires$i"][$j]; + } + } + + for ($i = 0; $i < $nbofchoice; $i++) + { + if ($_SESSION["totalchoixjour"][$i] == mktime(0, 0, 0, $_SESSION["mois"], $_POST["choixjourretrait"][0], $_SESSION["annee"])) + { @@ -462 +439 @@ - $_SESSION["horaires0"] = GETPOST("horaires0"); + $_SESSION["horaires0"] = $_POST["horaires0"]; @@ -470 +447 @@ -//effacer les horaires dans toutes les cases +//report des horaires dans toutes les cases @@ -493 +470,2 @@ - if (issetAndNoEmpty('totalchoixjour', $_SESSION) === true) { + if (issetAndNoEmpty('totalchoixjour', $_SESSION) === true) + { @@ -498 +476 @@ - print ''."\n"; + print ''."\n"; @@ -508 +486 @@ - print ''."\n"; + print ''."\n"; @@ -510,2 +488,2 @@ - // grey button - print ''.$numerojour.''."\n"; + // grey button + print ''.$numerojour.''."\n"; @@ -525 +503,2 @@ -if (issetAndNoEmpty('totalchoixjour', $_SESSION) || $erreur) { +if (issetAndNoEmpty('totalchoixjour', $_SESSION) || $erreur) +{ @@ -548,2 +527 @@ - // Show list of selected days - + //affichage de la liste des jours choisis @@ -552 +530,2 @@ - for ($i = 0; $i < $nbofchoice; $i++) { + for ($i = 0; $i < $nbofchoice; $i++) + { @@ -554 +533 @@ - print ''.dol_print_date($_SESSION["totalchoixjour"][$i], 'daytext').' ('.dol_print_date($_SESSION["totalchoixjour"][$i], '%A').')'; + print ''.dol_print_date($_SESSION["totalchoixjour"][$i], 'daytext').' ('.dol_print_date($_SESSION["totalchoixjour"][$i], '%A').')'; @@ -574,3 +553 @@ - print ''; - print ''; - print ''."\n"; + print ''."\n"; --- /tmp/dsg/dolibarr/htdocs/opensurvey/wizard/github_19.0.3_create_survey.php +++ /tmp/dsg/dolibarr/htdocs/opensurvey/wizard/client_create_survey.php @@ -27 +26,0 @@ -// Load Dolibarr environment @@ -32 +31 @@ -require_once DOL_DOCUMENT_ROOT."/opensurvey/lib/opensurvey.lib.php"; +require_once DOL_DOCUMENT_ROOT."/opensurvey/fonctions.php"; @@ -35,3 +34 @@ -if (!$user->hasRight('opensurvey', 'write')) { - accessforbidden(); -} +if (!$user->rights->opensurvey->write) accessforbidden(); @@ -41,12 +38,13 @@ -$title = GETPOST('title'); -$description = GETPOST('description', 'restricthtml'); -$mailsonde = GETPOST('mailsonde'); -$creation_sondage_date = GETPOST('creation_sondage_date'); -$creation_sondage_autre = GETPOST('creation_sondage_autre'); - -// We init some session variable to avoir warning -$session_var = array('title', 'description', 'mailsonde'); -foreach ($session_var as $var) { - if (isset($_SESSION[$var])) { - $_SESSION[$var] = null; - } +// On teste toutes les variables pour supprimer l'ensemble des warnings PHP +// On transforme en entites html les données afin éviter les failles XSS +$post_var = array('titre', 'commentaires', 'mailsonde', 'creation_sondage_date', 'creation_sondage_autre'); +foreach ($post_var as $var) +{ + $$var = GETPOST($var); +} + +// On initialise egalement la session car sinon bonjour les warning :-) +$session_var = array('titre', 'commentaires', 'mailsonde'); +foreach ($session_var as $var) +{ + if (isset($_SESSION[$var])) $_SESSION[$var] = null; @@ -60,3 +58,4 @@ -if (!empty($creation_sondage_date) || !empty($creation_sondage_autre)) { - $_SESSION["title"] = $title; - $_SESSION["description"] = $description; +if (GETPOST("creation_sondage_date") || GETPOST("creation_sondage_autre")) +{ + $_SESSION["titre"] = $titre; + $_SESSION["commentaires"] = $commentaires; @@ -85 +84,2 @@ - if ($champdatefin && ($champdatefin > 0)) { // A date was provided + if ($champdatefin && ($champdatefin > 0)) // A date was provided + { @@ -87 +87,2 @@ - if ($champdatefin >= dol_now()) { + if ($champdatefin >= dol_now()) + { @@ -90 +91,3 @@ - } else { + } + else + { @@ -103,2 +106,4 @@ - if ($title && $testdate) { - if (!empty($creation_sondage_date)) { + if ($titre && $testdate) + { + if (!empty($creation_sondage_date)) + { @@ -109 +114,2 @@ - if (!empty($creation_sondage_autre)) { + if (!empty($creation_sondage_autre)) + { @@ -135 +141 @@ -print dol_get_fiche_head(); +dol_fiche_head(); @@ -140,2 +146,3 @@ -print ''.$langs->trans("PollTitle").''."\n"; -if (!$_SESSION["title"] && (GETPOST('creation_sondage_date') || GETPOST('creation_sondage_autre'))) { +print ''.$langs->trans("PollTitle").''."\n"; +if (!$_SESSION["titre"] && (GETPOST('creation_sondage_date') || GETPOST('creation_sondage_autre'))) +{ @@ -147 +154 @@ -$doleditor = new DolEditor('description', $_SESSION["description"], '', 120, 'dolibarr_notes', 'In', 1, 1, 1, ROWS_7, '90%'); +$doleditor = new DolEditor('commentaires', $_SESSION["commentaires"], '', 120, 'dolibarr_notes', 'In', 1, 1, 1, ROWS_7, '90%'); @@ -159 +166 @@ -print dol_get_fiche_end(); +dol_fiche_end(); @@ -163 +170 @@ -print 'document.formulaire.title.focus();'."\n"; +print 'document.formulaire.titre.focus();'."\n"; @@ -170,28 +177,16 @@ -if ($_SESSION["mailsonde"]) { - $cochemail = "checked"; -} - -print '
'."\n"; - -if ($_SESSION['allow_comments']) { - $allow_comments = 'checked'; -} -if (GETPOSTISSET('allow_comments')) { - $allow_comments = GETPOST('allow_comments') ? 'checked' : ''; -} -print '
'."\n"; - -if ($_SESSION['allow_spy']) { - $allow_spy = 'checked'; -} -if (GETPOSTISSET('allow_spy')) { - $allow_spy = GETPOST('allow_spy') ? 'checked' : ''; -} -print '
'."\n"; - -if (GETPOST('choix_sondage')) { - if (GETPOST('choix_sondage') == 'date') { - print ''; - } else { - print ''; - } +if ($_SESSION["mailsonde"]) $cochemail = "checked"; + +print ' '.$langs->trans("ToReceiveEMailForEachVote").'
'."\n"; + +if ($_SESSION['allow_comments']) $allow_comments = 'checked'; +if (isset($_POST['allow_comments'])) $allow_comments = GETPOST('allow_comments') ? 'checked' : ''; +print ' '.$langs->trans('CanComment').'
'."\n"; + +if ($_SESSION['allow_spy']) $allow_spy = 'checked'; +if (isset($_POST['allow_spy'])) $allow_spy = GETPOST('allow_spy') ? 'checked' : ''; +print ' '.$langs->trans('CanSeeOthersVote').'
'."\n"; + +if (GETPOST('choix_sondage')) +{ + if (GETPOST('choix_sondage') == 'date') print ''; + else print ''; @@ -200 +195,3 @@ -} else { +} +else +{