Bugfix, simplification, et amélioration fonction "Copier ce matériel"

Etienne Pallier
1 parent d27459ad
Showing 5 changed files with 166 additions and 43 deletions Show diff stats
README.md
src/Controller/MaterielsController.php
src/Template/Materiels/add.ctp
src/Template/Materiels/edit.ctp
src/Template/Materiels/view.ctp
@@ -53,10 +53,10 @@ Logiciel testé et validé sur les configurations suivantes :
 VERSION ACTUELLE
-Date: 13/05/2019
-Version: 2.12.10
+Date: 15/05/2019
+Version: 2.12.11
 Author: EP
-Commentaire: Ajout de l'option "Librairie pdf utilisée" pour choisir entre fpdf et dompdf 
+Commentaire: Bugfix, simplification, et amélioration de la fonctionnalité "Copier ce matériel" 
 IMPORTANT : 
 	- Pour connaitre la version actuelle, taper "./VERSION"
@@ -707,20 +707,56 @@ class MaterielsController extends AppController
      * @param $IS_ADD: True = add ; False = edit
      * @return \Cake\Network\Response|void Redirects on successful add/edit, renders view otherwise.
      */
-    public function add_or_edit($is_add, $id = null, $valeurs = null, $erreurs = null)
+    public function add_or_edit($is_add, $id=null, $valeurs=null, $erreurs=null)
     {
         $usersTable = TableRegistry::getTableLocator()->get('Users');
+        // Set $materiel
+        $materiel = NULL;
+
         // ADD
         if ($is_add) {
-            $materiel = $this->Materiels->newEntity();
+            // - COPIE de materiel (on a cliqué sur "Copier ce materiel") => id passé en argument 
+            if (isset($this->request->getAttribute('params')['pass'][0])) {
+                // 1) on crée un materiel vide
+                $materiel = $this->Materiels->newEntity();
+                // 2) on récupère le materiel à copier et on le copie dans $materiel
+                $materiel_to_copy = $this->Materiels->get($this->request->getAttribute('params')['pass'][0]);
+                $materiel_to_copy = $materiel_to_copy->toArray();
+                //var_dump($materiel_to_copy);
+                ///foreach ($materiel_to_copy as $key=>$value) $materiel->$key = $value;
+                // IMPORTANT: validate=False car sinon, les données sont validées avant la copie,
+                // et le numero_laboratoire est vu comme invalide car déjà utilisé et doit etre unique !!!
+                // et on a pour résultat : "le matériel n'a pas pu être ajouté" (sans savoir pourquoi !!!)
+                $materiel = $this->Materiels->patchEntity($materiel, $materiel_to_copy, ['validate' => false]);
+                // Du coup, on supprime le champ numero_laboratoire car il va être généré automatiquement
+                unset($materiel->numero_laboratoire);
+                // IMPORTANT: on ne doit pas laisser l'id égal à celui du matériel copié !!! il en faut un nouveau
+                $materiel->id = NULL;
+                //$materiel->id = False;
+                //unset($materiel->id);
+                
+                /*
+                $attribute="[original]";
+                $materiel->$attribute = [];
+                */
+                //$materiel->id = FALSE;
+                //$materiel->'[new]' => true,
+            }
+            // - NOUVEAU materiel (on a cliqué sur "Nouveau materiel")
+            else {
+                $materiel = $this->Materiels->newEntity();
+                // Set default values : "N/A"
+                $materiel->groupes_thematique_id = 1;
+                $materiel->groupes_metier_id = 1;
+                $materiel->site_id = 9;
+            }
         }
         // EDIT
-        else {
+        else
             $materiel = $this->Materiels->get($id, [
                 'contain' => []
             ]);
-        }
         /* SI POST
          * Les données ont été saisies et postées
@@ -796,6 +832,7 @@ class MaterielsController extends AppController
         //--- 1) INITIALISATION DE LISTES POUR ASSISTER LA SAISIE (listes de choix proposés)
+        // DOMAINES : LISTE COMPLETE
         $surCategories = $this->Materiels->SurCategories->find('list', [
             'keyField' => 'id',
             'valueField' => 'nom',
@@ -805,6 +842,35 @@ class MaterielsController extends AppController
                 'nom !=' => 'N/A'
             )
         ]);
+        
+        // CATEGORIES ET SOUS-CATEG (modes EDIT et ADD-by-COPIE only)
+        $categories = [];
+        $sousCategories = [];
+        if ( !$is_add or ($is_add and isset($this->request->getAttribute('params')['pass'][0])) ) {
+            // - Seulement les categories correspondant au domaine choisi
+            $categories = $this->Materiels->Categories->find('list', [
+                'conditions' => [
+                    'Categories.sur_categorie_id' => $materiel->sur_categorie_id
+                ],
+                //'recursive' => - 1,
+                'keyField' => 'id',
+                'valueField' => 'nom',
+                'order' => 'nom',
+                //'order' => 'Categories.nom',
+            ]);
+            // - Seulement les sous-categ correspondant à la categorie choisie
+            $sousCategories = $this->Materiels->SousCategories->find('list', [
+                'conditions' => [
+                    'SousCategories.categorie_id' => $materiel->categorie_id
+                ],
+                //'recursive' => - 1,
+                'keyField' => 'id',
+                'valueField' => 'nom',
+                'order' => 'nom'
+                //'order' => 'SousCategories.nom'
+            ]);
+        }
+        /* (EP) Totalement inutile !!!
         $categories = $this->Materiels->Categories->find('list', [
             'keyField' => 'id',
             'valueField' => 'nom',
@@ -815,6 +881,8 @@ class MaterielsController extends AppController
             'valueField' => 'nom',
             'order' => 'SousCategories.nom'
         ]);
+        */
+
         $groupesThematiques = $this->Materiels->GroupesThematiques->find('list', [
             'keyField' => 'id',
             'valueField' => 'nom',
@@ -958,17 +1026,21 @@ class MaterielsController extends AppController
             ])
             ->first()['email'];
+        /*
         // ADD only
-        if ( $is_add) {
+        if ($is_add) {
+            // Un id a été passé en argument => Copie de materiel (on a cliqué sur "Copier ce materiel")
             if (isset($this->request->getAttribute('params')['pass'][0])) {
                 $cpMateriel = $this->Materiels->get($this->request->getAttribute('params')['pass'][0]);
                 $this->set('cpMateriel', $cpMateriel);
             }
         }
-        
-        // EDIT only
         else {
+        */
+        // EDIT only
+        if (!$is_add) {
+                
             if (! empty($materiel->get('nom_responsable'))) {
                 //if (! in_array($materiel->get('nom_responsable'), $utilisateurs)) {
                 if (! in_array($materiel->get('nom_responsable'), $users_name)) {
@@ -1033,7 +1105,8 @@ class MaterielsController extends AppController
             'materiel',
             // not used
             //'utilisateurconnect',
-            'surCategories', 'categories', 'sousCategories',
+            'surCategories', 
+            'categories', 'sousCategories',
             'groupesThematiques', 'groupesMetiers',
             'organismes', 'sites',
             'mail_responsable',
@@ -1047,9 +1120,11 @@ class MaterielsController extends AppController
             'administrateurs'
         ));
+        /* (EP) INUTILE (sauf pour faire du json ou xml) !!!
         $this->set('_serialize', [
-            'materiel'
+            'materiel',
         ]);
+        */
     } //add_or_edit()
@@ -6,13 +6,20 @@
  */
 $users_option_list = $users_option_list;
 $users_login_and_email = $users_login_and_email;
+$materiel = $materiel;
 // TODO: yena d'autres... il faut les lister toutes ici, pour plus de clarté
+//debug($materiel);
 //debug($users_option_list);
 //debug($users_login_and_email);
+//debug($sousCategories);
 use Cake\ORM\TableRegistry;
+
+/*
+// COPIE de materiel (on a cliqué sur "Copier ce matériel")
 if (isset($cpMateriel)) {
+    //debug($cpMateriel);
     $Designation = $cpMateriel->designation;
     $Sur_categ_id = $cpMateriel->sur_categorie_id;
     $Categ_id = $cpMateriel->categorie_id;
@@ -30,6 +37,7 @@ if (isset($cpMateriel)) {
     $Lieu_detail = $cpMateriel->lieu_detail;
     $Site_id = $cpMateriel->site_id;
 } else {
+// NOUVEAU matériel (on a cliqué sur "Nouveau matériel")
     $Designation = NULL;
     if (!isset($Sur_categ_id)) $Sur_categ_id = NULL;
     $Categ_id = NULL;
@@ -47,6 +55,7 @@ if (isset($cpMateriel)) {
     $Lieu_detail = NULL;
     $Site_id = 9;
 }
+*/
 //debug($Sur_categ_id)
 ?>
@@ -67,34 +76,45 @@ if (isset($cpMateriel)) {
         <?php
         echo $this->Form->control('designation', [
             'label' => 'Désignation',
-            'default' => $Designation
+            //'default' => $Designation
+            'default' => $materiel->designation
         ]);
+        // SUR-CATEGORIES (DOMAINES)
+        //debug($surCategories);
         echo $this->Form->control('sur_categorie_id', [
             'label' => 'Domaine',
             'style' => 'width: 260px',
             'options' => $surCategories,
             'empty' => 'Choisir un domaine',
-            'default' => $Sur_categ_id
+            //'default' => $Sur_categ_id
+            'default' => $materiel->sur_categorie_id
         ]);
-        /* (EP 26/2/19) Inutile, je commente
-        // if a domaine is selected, reduce the categories list to this domaine
+        
+        // CATEGORIES
         $categs = $categories;
+        /*
+        // if a domain is selected, reduce the categories list to this domain
+        //debug($categories);
         if ($this->request->getData('sur_categorie_id') !== null && ($this->request->getData('sur_categorie_id') != '')) {
             $categs = $categs->where([
                 'sur_categorie_id =' => $this->request->getData('sur_categorie_id')
             ]);
         }
+        //$categs = [];
         */
-        $categs = [];
         echo $this->Form->control('categorie_id', [
             'label' => 'Catégorie',
             'style' => 'width: 380px',
             'options' => $categs,
             'empty' => 'Choisir une catégorie',
-            'default' => $Categ_id
+            //'default' => $Categ_id
+            'default' => $materiel->categorie_id
         ]);
-        // by default, list is empty
+        
+        // SOUS-CATEGORIES (by default, list is empty)
+        $souscategs = $sousCategories;
+        /*
         $souscategs = [];
         // if a categ is selected, update sous-categs list for this categ (only)
         if ($this->request->getData('categorie_id') !== null && ($this->request->getData('categorie_id') != '')) {
@@ -103,20 +123,25 @@ if (isset($cpMateriel)) {
                 'categorie_id' => $this->request->getData('categorie_id')
             ]);
         }
+        */
         echo $this->Form->control('sous_categorie_id', [
             'label' => 'Sous-catégorie',
             'style' => 'width: 380px',
             'options' => $souscategs,
             'empty' => 'Choisir une sous-catégorie',
-            'default' => $Sous_categ_id
+            //'default' => $Sous_categ_id
+            'default' => $materiel->sous_categorie_id
         ]);
+
         echo $this->Form->control('materiel_technique', [
             'label' => 'Technique',
-            'default' => $Mat_technique
+            //'default' => $Mat_technique
+            'default' => $materiel->materiel_technique
         ]);
         echo $this->Form->control('materiel_administratif', [
             'label' => 'Inventoriable (>' . $configuration->prix_inventaire_administratif . '€)',
-            'default' => $Mat_administratif
+            //'default' => $Mat_administratif
+            'default' => $materiel->materiel_administratif
         ]);
         if ($configuration->metrologie) {
             echo $this->Form->control('metrologie', [
@@ -126,7 +151,8 @@ if (isset($cpMateriel)) {
         }
         echo $this->Form->control('description', [
             'label' => 'Description',
-            'default' => $Description
+            //'default' => $Description
+            'default' => $materiel->description
         ]);
         if ($USER_IS_RESPONSABLE_OR_MORE) {
             /*
@@ -145,7 +171,8 @@ if (isset($cpMateriel)) {
         echo $this->Form->control('site_id', [
             'options' => $sites,
             'style' => 'width: 380px',
-            'default' => $Site_id
+            //'default' => $Site_id
+            'default' => $materiel->site_id
         ]);
         echo '<div class="input text"><label for="lieu_detail">Détail lieu de stockage</label><input list="lieu" id="lieu_detail" name="lieu_detail" type="text" placeholder="choisir/ajouter un lieu" autocomplete="off"><datalist id="lieu">';
         foreach ($lieu_detail as $e) {
@@ -157,7 +184,8 @@ if (isset($cpMateriel)) {
             'label' => 'Date de la commande',
             'class' => 'datepicker',
             'placeholder' => 'Cliquez pour sélectionner une date',
-            'default' => $Date_acquisition
+            //'default' => $Date_acquisition
+            'default' => $materiel->date_acquisition
         ]);
         echo $this->Form->control('date_reception', [
             'type' => 'text',
@@ -165,7 +193,8 @@ if (isset($cpMateriel)) {
             'class' => 'datepicker',
             'placeholder' => 'A éditer lors de la réception uniquement.',
             'empty' => true,
-            'default' => $Date_reception
+            //'default' => $Date_reception
+            'default' => $materiel->date_reception
         ]);
         echo '<table id="tableAlignementFrequence"><tr><td>';
         echo $this->Form->control('duree_garantie', [
@@ -203,25 +232,29 @@ if (isset($cpMateriel)) {
         echo $this->Form->control('groupes_thematique_id', [
             'label' => $configuration->nom_groupe_thematique,
             'options' => $groupesThematiques,
-            'default' => $Groupes_thematique_id
+            //'default' => $Groupes_thematique_id
+            'default' => $materiel->groupes_thematique_id
         ]);
         echo $this->Form->control('groupes_metier_id', [
             'label' => $configuration->nom_groupe_metier,
             'options' => $groupesMetiers,
-            'default' => $Groupes_metier_id
+            //'default' => $Groupes_metier_id
+            'default' => $materiel->groupes_metier_id
         ]);
         $fieldType = $USER_IS_UTILISATEUR ? 'hidden' : 'input';
         echo $this->Form->$fieldType('nom_responsable', [
             'label' => 'Nom de l\'utilisateur',
             'empty' => 'Choisir un utilisateur',
-            'default' => $username,
+            //'default' => $username,
+            'default' => $materiel->username,
             //'options' => $utilisateurs
             'options' => $users_option_list
         ]);
         echo $this->Form->control('email_responsable', [
             'label' => 'Email de l\'utilisateur',
             'readonly' => true,
-            'default' => $mail_responsable
+            //'default' => $mail_responsable
+            'default' => $materiel->mail_responsable,
         ]);
         /*
@@ -260,11 +293,13 @@ if (isset($cpMateriel)) {
             'options' => $organismes,
             'style' => 'width: 380px',
             'empty' => 'Choisir un organisme',
-            'default' => $Organisme_id
+            //'default' => $Organisme_id
+            'default' => $materiel->organisme_id,
         ]);
         echo $this->Form->control('prix_ht', [
             'label' => 'Prix HT (€)',
-            'default' => $Prix_ht
+            //'default' => $Prix_ht
+            'default' => $materiel->prix_ht,
         ]);
         if ($USER_IS_ADMIN_OR_MORE) {
             /*
@@ -11,7 +11,7 @@ $users_login_and_email = $users_login_and_email;
 //debug($users_option_list);
 //debug($users_login_and_email);
-
+//debug($materiel);
 use Cake\ORM\TableRegistry;
@@ -116,6 +116,8 @@ if ($IS_VALIDATED &amp;&amp; $materiel-&gt;numero_serie)
                 'label' => 'Appareil hors_service'
             ]);
         }
+        
+        // DOMAINES
         echo $this->Form->control('sur_categorie_id', [
             'label' => 'Domaine',
             'options' => $surCategories,
@@ -124,13 +126,17 @@ if ($IS_VALIDATED &amp;&amp; $materiel-&gt;numero_serie)
             'readonly' => $isReadonlyField('sur_categorie_id', $myReadonlyFields),
             'disabled' => $isReadonlyField('sur_categorie_id', $myReadonlyFields)
         ]);
+        
+        // CATEGORIES
         $categs = $categories;
+        /*
         // if a domain is selected, reduce the categories list to this domain
         //if ($this->request->getData('sur_categorie_id') !== null && ($this->request->getData('sur_categorie_id') != ''))
         if ($materiel->sur_categorie_id !== null && $materiel->sur_categorie_id != '')
             $categs = $categs->where([
             'sur_categorie_id =' => $materiel->sur_categorie_id
             ]);
+        */
         echo $this->Form->control('categorie_id', [
             'label' => 'Catégorie',
             'style' => 'width: 380px',
@@ -138,7 +144,10 @@ if ($IS_VALIDATED &amp;&amp; $materiel-&gt;numero_serie)
             'empty' => 'Choisir une catégorie',
             'disabled' => $isReadonlyField('categorie_id', $myReadonlyFields)
         ]);
+
         // SOUS-CATEGORIES
+        $souscategs = $sousCategories;
+        /*
         // by default, list is empty
         $souscategs = [];
         // if a categ is selected, update sous-categs list for this categ (only)
@@ -150,6 +159,7 @@ if ($IS_VALIDATED &amp;&amp; $materiel-&gt;numero_serie)
                 'categorie_id' => $materiel->categorie_id
             ]);
         }
+        */
         echo $this->Form->control('sous_categorie_id', [
             'label' => 'Sous-catégorie',
             'style' => 'width: 380px',
@@ -470,16 +470,19 @@ $panne = h($materiel-&gt;hors_service) ? &#39; (HORS SERVICE)&#39; : &#39;&#39;;
         } // switch
     } // BOUTON changement statut
-    // BOUTON Copier
-    echo $this->Html->link('<i class="icon-pencil"></i> Copier ce matériel', [
-        'controller' => 'materiels',
-        'action' => 'add',
-        $materiel->id
-    ], [
-        'title' => 'Copier ce matériel',
-        'style' => 'margin-right: 10px; margin-top: 10px; display: inline-block',
-        'escape' => false
-    ]);
+    // BOUTON Copier (seulement pour les materiels qui sont CREATED et pour les ADMINet+ ou USER owner)
+    if ( $IS_CREATED and ($USER_IS_ADMIN_OR_MORE || $USER_IS_UTILISATEUR_AND_CREATOR_OR_OWNER) )
+        echo $this->Html->link('<i class="icon-pencil"></i> Copier ce matériel', 
+        [
+            'controller' => 'materiels',
+            'action' => 'add',
+            $materiel->id
+        ], 
+        [
+            'title' => 'Copier ce matériel',
+            'style' => 'margin-right: 10px; margin-top: 10px; display: inline-block',
+            'escape' => false
+        ]);
     // BOUTON ETIQUETTE (si imprimante disponible)
     if ($IS_VALIDATED && $configuration->hasPrinter && in_array($role, [