merge dev-IRAP

Etienne Pallier
2 parents 645f6179 704cd4c6
Showing 3 changed files with 110 additions and 111 deletions Show diff stats
README-LABINVENT.md
src/Template/Layout/default.ctp
src/Template/Materiels/edit.ctp
@@ -54,7 +54,7 @@ Logiciel testé et validé sur les configurations suivantes :
 VERSION ACTUELLE
  
 Date: 19/06/2017
-Version: 2.6.6
+Version: 2.6.7
 	-> Bugfixes add/edit dates materiels
 - ATTENTION : Mise à jour de la base de données requise (cf database/update)
  
@@ -101,7 +101,7 @@ $cakeDescription = &#39;Labinvent 2&#39;;
 					</i></td>
 					<td id="version">
 						<!-- VERSION M.m.f.b (version (M)ajeure, version (m)ineure, numero de nouvelle (f)onctionnalite, numero de (b)ugfix) -->
-						<font color="black">VERSION 2.6.6 (19/06/2017)</font>
+						<font color="black">VERSION 2.6.7 (19/06/2017)</font>
 						<br/>
 						<font color="black"><a href="<?php
  
@@ -66,15 +66,6 @@ if ($IS_VALIDATED &amp;&amp; $materiel-&gt;numero_serie)
         <?=$this->Form->submit(__('Enregistrer'))?>
  
         <?php
-        // Version LATMOS - Désignation
-        /*
-         * echo '<div class="input text required"><label for="designation">Désignation</label><input list="designa" disabled="' . $isReadonlyField('designation', $myReadonlyFields) . '" id="designation" name="designation" value="' . array_shift($designation_edit) . '" type="text" placeholder="choisir/ajouter une designation" autocomplete="off"><datalist id="designa">';
-         * foreach ($designation as $e) {
-         * echo "<option value=\"$e\">";
-         * }
-         * echo '</datalist> </div>';
-         */
-        // Version IRAP - Désignation
         echo $this->Form->input('designation', [
             'label' => 'Désignation',
             'disabled' => $isReadonlyField('designation', $myReadonlyFields)
@@ -407,47 +398,43 @@ echo $this-&gt;element(&#39;menu_form&#39;, [&#39;pluralHumanName&#39; =&gt; &#39;Matériels&#39;]);
  
 <script type="text/javascript">
  
-/**
- * Event DOMAINE change
- *
- * Sur sélection d'un DOMAINE ==> update CATEGORIES + empty SOUS-CATEGORIES
- *
- * Dans le détail :
- * - si select <> "choisir" ==> categ value = from domaine (select="choisir")
- * - si select = "choisir" ==> categ value = TOUTES (select="choisir")
- * - puis (dans les 2 cas) vider scateg
- *
- */
-
 $(document).ready(function () {
+
+	/**
+	 * Event DOMAINE change
+	 *
+	 * Sur sélection d'un DOMAINE ==> update CATEGORIES + empty SOUS-CATEGORIES
+	 *
+	 * Dans le détail :
+	 * - si select <> "choisir" ==> categ value = from domaine (select="choisir")
+	 * - si select = "choisir" ==> categ value = TOUTES (select="choisir")
+	 * - puis (dans les 2 cas) vider scateg
+	 *
+	 */
 	$("#sur-categorie-id").bind("change", function (event) {
-			var domaineId=$("#sur-categorie-id :selected").val();
-			var domaineresp= "<?=$domaineresp?>";
-			var role= "<?=$role?>";
-			if(domaineId== domaineresp){
-				if(role == "Responsable"){
-					$("#metro").css("display", "block");
-				}else
-					$("#metro").css("display", "none");
-			}
-			else
+		var domaineId=$("#sur-categorie-id :selected").val();
+		var domaineresp= "<?=$domaineresp?>";
+		var role= "<?=$role?>";
+		if(domaineId== domaineresp){
+			if(role == "Responsable"){
+				$("#metro").css("display", "block");
+			}else
 				$("#metro").css("display", "none");
+		}
+		else
+			$("#metro").css("display", "none");
  
-			if (domaineId=="")
-				updateSelectOptionsFromAnother("#categorie-id", "#sur-categorie-id", "Categories/getAll", "Choisir une catégorie");
-			else
-				updateSelectOptionsFromAnother("#categorie-id", "#sur-categorie-id", "Categories/getBySurCategorie", "Choisir une catégorie");
-			emptySelectOptions("#sous-categorie-id", "Choisir une sous-catégorie");
-			return false;
-		});
+		if (domaineId=="")
+			updateSelectOptionsFromAnother("#categorie-id", "#sur-categorie-id", "Categories/getAll", "Choisir une catégorie");
+		else
+			updateSelectOptionsFromAnother("#categorie-id", "#sur-categorie-id", "Categories/getBySurCategorie", "Choisir une catégorie");
+		emptySelectOptions("#sous-categorie-id", "Choisir une sous-catégorie");
+		return false;
 	});
  
-//Mettre un gestionnaire par défaut si c'est un admin qui modifie la fiche. Sinon l'utilisateur aura a le rentrer à la mano
- $(document).ready(function () {
+	/* Mettre un gestionnaire par défaut si c'est un admin qui modifie la fiche. Sinon l'utilisateur aura a le rentrer à la mano */
 	if ( <?=(! is_null($res)) ? "true" : "false"?>) $("#gestionnaire-id").val("<?=$_SESSION['Auth']['User']['sn'][0]?>");
- 	// Oui, c'est un batard entre du JavaScript et du PHP ... Mais au moins ca fonctionne en une seule ligne !
- });
-
+	/* Oui, c'est un batard entre du JavaScript et du PHP ... Mais au moins ca fonctionne en une seule ligne ! */
  
 	/**
 	 *
@@ -456,33 +443,29 @@ $(document).ready(function () {
 	 * - si select = "choisir" ==> vider scateg
 	 * - si select <> "choisir" ==> domaine value selected = celui de la categ, et scateg values = from categ
 	 */
-	$(document).ready(function() {
-		$("#categorie-id").bind("change", function(event) {
-			var categId = $("#categorie-id :selected").val();
-			var categLabel = $("#categorie-id :selected").text();
+	$("#categorie-id").bind("change", function(event) {
+		var categId = $("#categorie-id :selected").val();
+		var categLabel = $("#categorie-id :selected").text();
  
-			if (categId == "") emptySelectOptions("#sous-categorie-id", "Choisir une sous-catégorie");
-			else {
-				updateSelectOptionsFromAnother("#sous-categorie-id", "#categorie-id", "SousCategories/getByCategorie", "Choisir une sous-catégorie " + categLabel);
-				updateSelectOptionsFromAnother("#sur-categorie-id", "#categorie-id", "SurCategories/getFromCategorie", "");
-			}
-			return false;
-		});
+		if (categId == "") emptySelectOptions("#sous-categorie-id", "Choisir une sous-catégorie");
+		else {
+			updateSelectOptionsFromAnother("#sous-categorie-id", "#categorie-id", "SousCategories/getByCategorie", "Choisir une sous-catégorie " + categLabel);
+			updateSelectOptionsFromAnother("#sur-categorie-id", "#categorie-id", "SurCategories/getFromCategorie", "");
+		}
+		return false;
 	});
  
 	/**
 	 * Event Mail change
 	 */
-	$(document).ready(function() {
-		$("#nom-responsable").bind("change", function(event) {
-			var url = document.URL;
-			var reg = new RegExp("(materiels).*$", "g");
-			var emailUrl = url.replace(reg, "Users/getLdapEmail/");
-			$.ajax({
-				url: emailUrl + $("#nom-responsable").val()
-			}).done(function(data) {
-				$("#email-responsable").val(data)
-			});
+	$("#nom-responsable").bind("change", function(event) {
+		var url = document.URL;
+		var reg = new RegExp("(materiels).*$", "g");
+		var emailUrl = url.replace(reg, "Users/getLdapEmail/");
+		$.ajax({
+			url: emailUrl + $("#nom-responsable").val()
+		}).done(function(data) {
+			$("#email-responsable").val(data)
 		});
 	});
  
@@ -490,60 +473,76 @@ $(document).ready(function () {
 	/**
 	 * Event calcul date fin de garantie
 	 */
-	$(document).ready(function() {
-		$("#duree-garantie").bind("change", function(event) {
-			update_date();
-		});
+	/* On formatte COMME IL FAUT les dates des formulaires */
+	$("#date-acquisition")[0].value = $("#date-acquisition")[0].value.replace(/\//g, "-");
+	$("#date-reception")[0].value = $("#date-reception")[0].value.replace(/\//g, "-");
+	$("#date-fin-garantie")[0].value = $("#date-fin-garantie")[0].value.replace(/\//g, "-");
  
-		$("#unite-duree-garantie").bind("change", function(event) {
-			update_date();
-		});
+	/* Puis si nécessaire on update la date de fin de gurantie au chargement de la page */
+	if ($("#date-fin-garantie")[0].value != "") dateUpdate();
  
-		$("#date-reception").change( function(event) {
-			update_date();
-		});
-
-	});
-
-	/**
-	* Fonction de mise à jour de la date de fin de guarantie
-	*/
-	function update_date() {
-		if ($("#date-reception").val() != "" && $("#duree-garantie").val() != "") {
-			var url = document.URL;
-			var reg = new RegExp("(materiels).*$", "g");
-			var dateUrl = url.replace(reg, "Materiels/getDateGarantie/");
-			$.ajax({
-				url: dateUrl + $("#date-reception").val() + "/" + $("#duree-garantie").val() + "/" + $("#unite-duree-garantie").val()
-			}).done(function(data) {
-				$("#date-fin-garantie").val(data)
-			});
-		}
-	}
+	/* Et enfin les events purs */
+	$("#date-reception").change( function(event) { dateUpdate(); });
+	$("#duree-garantie").bind("change", function(event) { dateUpdate(); });
+	$("#unite-duree-garantie").bind("change", function(event) { dateUpdate(); });
+	$("#date-fin-garantie").bind("change", function(event) { $("#duree-garantie")[0].value = "" });
  
 	/**
 	 * Event - à la validation, on vérifie les dates
 	 */
-	$(document).ready(function(){
-		$( ".form" ).submit(function( event ) {
-			d1 = toDate($("#date-acquisition")[0].value); 	// Date de la commande
-			d2 = toDate($("#date-reception")[0].value); 		// Date de réception
-			if($("#date-reception")[0].value != undefined){
-				if(d1 > d2) {
-					alert("Les dates ne correspondent pas, la date de réception doit être supérieure à la date de commande.");
-					event.preventDefault();
-				}
+	$( ".form" ).submit(function( event ) {
+		s1 = $("#date-acquisition")[0].value;
+		s2 = $("#date-reception")[0].value;
+		s3 = $("#date-fin-garantie")[0].value;
+		if (!checkDate(s1) || !checkDate(s2) || !checkDate(s3)) {
+			alert("Les dates doivent être de la forme dd-mm-YYYY");
+			event.preventDefault();
+		}
+		d1 = toDate($("#date-acquisition")[0].value); 	/* Date de la commande */
+		d2 = toDate($("#date-reception")[0].value); 	/* Date de réception */
+		if($("#date-reception")[0].value != undefined){
+			if(d1 > d2) {
+				alert("Les dates ne correspondent pas, la date de réception doit être supérieure à la date de commande.");
+				event.preventDefault();
 			}
-		});
+		}
 	});
+});
+
+/**
+ * Fonction qui vérifie que la date en parametre soit bien au format dd-mm-YYYY ou d-m-YYYY
+ */
+function checkDate(str) {
+  return str.match(/^(\d{1,2})-(\d{1,2})-(\d{4})$/);
+}
  
-	function toDate(param) {
-		var DateString = param.split("-");
-		return new Date(parseInt(DateString[2]), parseInt(DateString[1]) + 1, parseInt(DateString[0]), 12, 00, 00);
-		// new Date(YYYY, mm+1, dd, 12, 00, 00)
-		// Le +1 du mois est nécessaire car les mois commencent à 0 en JS
-		// les 3 derniers chiffres ne sont pas nécessaires, mais si on les retire ça bug uniquement sous firefox (il est plus strict que chrome paar exemple)
-		// et la date devient du grand n'importe quoi
+/**
+* Fonction de mise à jour de la date de fin de guarantie
+*/
+function dateUpdate() {
+	if ($("#date-reception").val() != "" && $("#duree-garantie").val() != "") {
+		var url = document.URL;
+		var reg = new RegExp("(materiels).*$", "g");
+		var dateUrl = url.replace(reg, "Materiels/getDateGarantie/");
+		$.ajax({
+			url: dateUrl + $("#date-reception").val() + "/" + $("#duree-garantie").val() + "/" + $("#unite-duree-garantie").val()
+		}).done(function(data) {
+			$("#date-fin-garantie").val(data)
+		});
 	}
+}
+
+/**
+* Fonction qui transforme la chaine de format dd-mm-YYYY en date js pour pouvoir être comparée par la suite
+*/
+function toDate(str) {
+	var DateString = str.split("-");
+	return new Date(parseInt(DateString[2]), parseInt(DateString[1]) + 1, parseInt(DateString[0]), 12, 00, 00);
+	/* 	new Date(YYYY, mm+1, dd, 12, 00, 00)
+		Le +1 du mois est nécessaire car les mois commencent à 0 en JS
+		les 3 derniers chiffres ne sont pas nécessaires, mais si on les retire ça bug uniquement sous firefox (il est plus strict que chrome paar exemple)
+		et la date devient du grand n'importe quoi 
+	*/
+}
  
 </script>