Plusieurs ameliorations:

- bugfix materiels/add : impossible changer l'utilisateur (responsable du matos) - bouton "de-valider" - diverses ameliorations graphiques - refactorisations de code

Plusieurs ameliorations:
- bugfix materiels/add : impossible changer l'utilisateur (responsable du matos) - bouton "de-valider" - diverses ameliorations graphiques - refactorisations de code
Etienne Pallier
1 parent 5b899c72
Showing 6 changed files with 73 additions and 65 deletions Show diff stats
README-LABINVENT.md
src/Template/Layout/default.ctp
src/Template/Materiels/add.ctp
src/Template/Materiels/edit.ctp
src/Template/Materiels/view.ctp
tests/TestCase/Controller/MaterielsControllerTest.php
@@ -47,10 +47,13 @@ Logiciel testé et validé sur les configurations suivantes :
 VERSION ACTUELLE
-Date: 23/08/2017
+Date: 24/08/2017
 Version: 2.7.5
-	- bugfix impossible changer le fournisseur d'un materiel (pb BD seulement sur mysql ancienne version) => changé les "MyISAM" en "InnoDB" pour que les contraintes d'integrité soient respectées et ne provoquent plus d'erreur "Action impossible"
-	- ATTENTION : Mise à jour de la base de données requise (cf database/update)
+	Plusieurs ameliorations:
+	- bugfix materiels/add : impossible changer l'utilisateur (responsable du matos)
+	- bouton "de-valider"
+	- diverses ameliorations graphiques
+	- refactorisations de code
 Version majeure en cours (2.7): https://projects.irap.omp.eu/versions/162
@@ -115,7 +115,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.7.5 (23/08/2017)</font>
+						<font color="black">VERSION 2.7.5 (24/08/2017)</font>
 						<br/>
 						<font color="black"><a href="<?php
@@ -46,7 +46,7 @@ if (isset($cpMateriel)) {
         <?=$this->Form->submit(__('Enregistrer'))?>
         <?=$this->Html->link(
-			__('<i class="icon-minus-sign"></i> Anuler'), 
+			__('<i class="icon-minus-sign"></i> Annuler'), 
 			['action' => 'index'], 
 			['escape' => false, 'onclick' => 'return true;', 'style' => 'margin-right: 10px']
 		)?>
@@ -191,29 +191,13 @@ if (isset($cpMateriel)) {
             'options' => $groupesMetiers,
             'default' => $Groupes_metier_id
         ]);
-        if ($USER_IS_RESPONSABLE_OR_MORE) {
-        /*
-        if (! (in_array($role, [
-            'Responsable',
-            'Administration',
-            'Administration Plus',
-            'Super Administrateur'
-        ]))) {
-        */
-            echo $this->Form->hidden('nom_responsable', [
-                'label' => 'Nom de l\'utilisateur',
-                'empty' => 'Choisir un utilisateur',
-                'default' => $username,
-                'options' => $utilisateurs
-            ]);
-        } else {
-            echo $this->Form->input('nom_responsable', [
-                'label' => 'Nom de l\'utilisateur',
-                'empty' => 'Choisir un utilisateur',
-                'default' => $username,
-                'options' => $utilisateurs
-            ]);
-        }
+        $fieldType = $USER_IS_UTILISATEUR ? 'hidden' : 'input';
+        echo $this->Form->$fieldType('nom_responsable', [
+            'label' => 'Nom de l\'utilisateur',
+            'empty' => 'Choisir un utilisateur',
+            'default' => $username,
+            'options' => $utilisateurs
+        ]);
         echo $this->Form->input('email_responsable', [
             'label' => 'Email de l\'utilisateur',
             'readonly' => true,
@@ -63,7 +63,16 @@ if ($IS_VALIDATED &amp;&amp; $materiel-&gt;numero_serie)
             echo '<br><br>';
         }
         ?>
+
+        <div id="boutons" class="actions" style="margin-bottom: 20px; width: 100%; float: none; padding: 10px 0;">
         <?=$this->Form->submit(__('Enregistrer'))?>
+        
+        <?=$this->Html->link(
+			__('<i class="icon-minus-sign"></i> Annuler'), 
+			['action' => 'index'], 
+			['escape' => false, 'onclick' => 'return true;', 'style' => 'margin-right: 10px']
+		)?>
+        </div>
         <?php
         echo $this->Form->input('designation', [
@@ -245,28 +254,15 @@ if ($IS_VALIDATED &amp;&amp; $materiel-&gt;numero_serie)
             'default' => 1
         ]);
-        if (! (in_array($role, [
-            'Responsable',
-            'Administration',
-            'Administration Plus',
-            'Super Administrateur'
-        ]))) {
-            echo $this->Form->hidden('nom_responsable', [
-                'label' => 'Nom de l\'utilisateur',
-                'disabled' => $isReadonlyField('nom_responsable', $myReadonlyFields),
-                'empty' => 'Choisir un utilisateur',
-                'default' => $username,
-                'options' => $utilisateurs
-            ]);
-        } else {
-            echo $this->Form->input('nom_responsable', [
-                'label' => 'Nom de l\'utilisateur',
-                'disabled' => $isReadonlyField('nom_responsable', $myReadonlyFields),
-                'empty' => 'Choisir un utilisateur',
-                'default' => $username,
-                'options' => $utilisateurs
-            ]);
-        }
+        $fieldType = $USER_IS_UTILISATEUR ? 'hidden' : 'input';
+        echo $this->Form->$fieldType('nom_responsable', [
+            'label' => 'Nom de l\'utilisateur',
+            'disabled' => $isReadonlyField('nom_responsable', $myReadonlyFields),
+            'empty' => 'Choisir un utilisateur',
+            'default' => $username,
+            'options' => $utilisateurs
+        ]);
+        
         if (isset($nom_ancien_responsable)) {
             echo $this->Form->input('nom_ancien_responsable', [
                 'label' => 'Nom ancien utilisateur',
@@ -343,7 +343,7 @@ $USER_IS_RESPONSABLE_AND_SAME_GROUP =
 				    ], 
 				    [
         					'title' => 'Valider le matériel',
-        					'style' => 'margin-right: 10px',
+        					'style' => 'margin-right: 10px; color: red',
         					'escape' => false
 				    ]
 				);
@@ -360,7 +360,7 @@ $USER_IS_RESPONSABLE_AND_SAME_GROUP =
             			    'view'
 				    ], 
 				    [
-        				    'title' => 'dé-valider le matériel (le repasser au statut CREATED)',
+        				    'title' => 'dé-valider le matériel (le repasser au statut Créé)',
         				    //'style' => 'margin-right: 10px; background: red; color: white',
         				    'style' => 'margin-right: 10px; color: red',
         				    'escape' => false
@@ -374,20 +374,21 @@ $USER_IS_RESPONSABLE_AND_SAME_GROUP =
         					'view'
         				], [
         					'title' => 'Demander la sortie de l\'inventaire',
-        					'style' => 'margin-right: 10px',
+        					'style' => 'margin-right: 10px; color: red',
         					'escape' => false
         				]);
 			    }
 			    else if ($status==$TOBEARCHIVED) {
         			    // Bouton ARCHIVER
-		            if ($role != 'Responsable' && $role != 'Super Administrateur') {
-        					echo $this->Html->link('<i class="icon-ok-sign"></i> Sortie inventaire', [
+		            //if ($role != 'Responsable' && $role != 'Super Administrateur') {
+	                if ($role != 'Responsable') {
+		                    echo $this->Html->link('<i class="icon-ok-sign"></i> Sortie inventaire', [
         						'action' => 'statusArchived',
         						$materiel->id,
         						'view'
         					], [
         						'title' => 'Sortir définitivement de l\'inventaire',
-        						'style' => 'margin-right: 10px',
+        					    'style' => 'margin-right: 10px; color: red',
         						'escape' => false,
         						'confirm' => 'Êtes-vous sur de bien vouloir archiver ' . $materiel->designation . ' ?'
         					]);
@@ -407,9 +407,29 @@ class MaterielsControllerTest extends IntegrationTestCase {
      *
      * @return void
      */
-    public function testEdit() {
-        $this->authSuperAdmin();
-        
+
+    public function testUpdate() { // ACTION 'edit'
+        foreach (array(1,2,3,4) as $role) {
+            switch ($role) {
+                case 1: $this->authUtilisateur();
+                    break;
+                case 2: $this->authResponsable();
+                   break;
+                case 3: $this->authAdmin();
+                   break;
+                case 4: $this->authAdminPlus();
+                   break;
+                case 5: $this->authSuperAdmin();
+                   break;
+            }
+            if ($role==1) continue;
+            $this->_testUpdates();
+        } // foreach
+    }
+
+    
+    private function _testUpdates() {
+           
         // 1) Test qu'on peut modifier un materiel CREATED
         // Toutes les donnees passees sont modifiees
         $data = [
@@ -831,20 +851,24 @@ class MaterielsControllerTest extends IntegrationTestCase {
      */
     public function testUpdateStatusSelectedMateriels() {
         $this->authSuperAdmin();
+        //$this->authAdmin();
         $this->post('/materiels/execActions', ['updateSelectedStatus' => 'true', 'what' => 'CREATED', 11 => '1', 12 => '1', 13 => '1']);
         $this->get('/materiels/view/11');
-        $this->assertResponseNotContains('CREATED', "La mise à jour de plusieurs statuts sur le materiel ne se fait pas correctement.");
-        $this->assertResponseNotContains('TOBEARCHIVED', "La mise à jour de plusieurs statuts sur le materiel ne se fait pas correctement.");
-        $this->assertResponseNotContains('ARCHIVED', "La mise à jour de plusieurs statuts sur le materiel ne se fait pas correctement.");
-
+        $this->assertResponseContains('VALIDATED', "(11) La mise à jour de plusieurs statuts sur le materiel ne se fait pas correctement.");
+        $this->assertResponseNotContains('CREATED', "(11) La mise à jour de plusieurs statuts sur le materiel ne se fait pas correctement.");
+        $this->assertResponseNotContains('TOBEARCHIVED', "(11) La mise à jour de plusieurs statuts sur le materiel ne se fait pas correctement.");
+        $this->assertResponseNotContains('ARCHIVED', "(11) La mise à jour de plusieurs statuts sur le materiel ne se fait pas correctement.");
+        
         $this->get('/materiels/view/12');
-        $this->assertResponseNotContains('CREATED', "La mise à jour de plusieurs statuts sur le materiel ne se fait pas correctement.");
+        $this->assertResponseContains('VALIDATED', "(12) La mise à jour de plusieurs statuts sur le materiel ne se fait pas correctement.");
+        $this->assertResponseNotContains('CREATED', "(12) La mise à jour de plusieurs statuts sur le materiel ne se fait pas correctement.");
         $this->assertResponseNotContains('TOBEARCHIVED', "La mise à jour de plusieurs statuts sur le materiel ne se fait pas correctement.");
         $this->assertResponseNotContains('ARCHIVED', "La mise à jour de plusieurs statuts sur le materiel ne se fait pas correctement.");
         $this->get('/materiels/view/13');
-        $this->assertResponseNotContains('CREATED', "La mise à jour de plusieurs statuts sur le materiel ne se fait pas correctement.");
+        $this->assertResponseContains('VALIDATED', "(13) La mise à jour de plusieurs statuts sur le materiel ne se fait pas correctement.");
+        $this->assertResponseNotContains('CREATED', "(13) La mise à jour de plusieurs statuts sur le materiel ne se fait pas correctement.");
         $this->assertResponseNotContains('TOBEARCHIVED', "La mise à jour de plusieurs statuts sur le materiel ne se fait pas correctement.");
         $this->assertResponseNotContains('ARCHIVED', "La mise à jour de plusieurs statuts sur le materiel ne se fait pas correctement.");
     }