Amelioré la gestion des membres

* Renvoi uniquement de ce qui est nécessaire lors de l'ajout ou de la suppression * Animations

Amelioré la gestion des membres
* Renvoi uniquement de ce qui est nécessaire lors de l'ajout ou de la suppression * Animations
Geoffrey PREUD'HOMME
1 parent 9a18ee30
Showing 5 changed files with 50 additions and 35 deletions Show diff stats
app/routes/ApiRtes.js
app/services/MembresServ.js
public/css/style.css
public/js/controllers/MembreCtrl.js
public/views/membres.html
@@ -75,11 +75,8 @@ api.post(&#39;/membres&#39;, function (req, res) { // Ajout d&#39;un membre
         MembresServ.add(req.body, function (err, membre) {
             if (err)
                 res.send(err);
-            MembresServ.list(function (err, membres) {
-                if (err)
-                    res.send(err);
-                res.json(membres);
-            });
+            else
+                res.json(membre);
         });
     });
 });
@@ -89,11 +86,8 @@ api.delete(&#39;/membres/:membre_id&#39;, function (req, res) { // Supression d&#39;un membr
         MembresServ.remove(req.params.membre_id, function (err, membre) {
             if (err)
                 res.send(err);
-            MembresServ.list(function (err, membres) {
-                if (err)
-                    res.send(err);
-                res.json(membres);
-            });
+            else
+                res.json(null);
         });
     });
 });
@@ -2,36 +2,48 @@ var MembreModl = require(&#39;../models/MembreModl&#39;);
 var NomsServ = require('../services/NomsServ');
 var async = require('async');
-var membres = {};
+var MembresServ = {};
-membres.list = function (cb) {
+MembresServ.addData = function (membre, cb) {
+    NomsServ.get(membre.login, function (nom) {
+        if (nom) {
+            membre.nom = nom;
+        } else {
+            membre.nom = membre.login;
+        }
+        cb(null, membre);
+    });
+};
+
+MembresServ.get = function(id, cb) {
+    MembreModl.findById(id).lean().exec(function (err, membre) {
+        if (err)
+            cb(err);
+        else
+            MembresServ.addData(membre, cb);
+    });
+};
+
+MembresServ.list = function (cb) {
     MembreModl.find({}).lean().exec(function (err, membres) {
-        addNom = function (membre, cbA) {
-            NomsServ.get(membre.login, function (nom) {
-                if (nom) {
-                    membre.nom = nom;
-                } else {
-                    membre.nom = membre.login;
-                }
-                cbA(null, membre);
-            });
-        };
-        async.mapSeries(membres, addNom, cb);
+        async.mapSeries(membres, MembresServ.addData, cb);
     });
 };
-membres.add = function (data, cb) {
+MembresServ.add = function (data, cb) {
     MembreModl.create({
         login: data.login,
         role: data.role,
         section: data.section,
-    }, cb);
+    }, function(err, membre) {
+        MembresServ.get(membre._id, cb);
+    });
 };
-membres.remove = function (id, cb) {
+MembresServ.remove = function (id, cb) {
     MembreModl.remove({
         _id: id
     }, cb);
 };
-module.exports = membres;
+module.exports = MembresServ;
@@ -29,3 +29,13 @@ body {
 #membre-list {
     width: 90%;
 }
+
+#membre-list .ng-enter {
+    -webkit-animation: fadeInLeft 0.5s;
+    animation: fadeInLeft 0.5s;
+}
+
+#membre-list .ng-leave {
+    -webkit-animation: fadeOutLeft 0.5s;
+    animation: fadeOutLeft 0.5s;
+}
@@ -18,9 +18,9 @@ angular.module(&#39;MembreCtrl&#39;, [&#39;SessionsServ&#39;, &#39;NotifyServ&#39;]).controller(&#39;MembreC
         $scope.createMembre = function () {
             var not = NotifyServ.promise("Ajout du membre...");
             $http.post('/api/membres', $scope.formData)
-                .success(function (data) {
+                .success(function (membre) {
                     $scope.formData = {};
-                    $scope.membres = data;
+                    $scope.membres.push(membre);
                     not.success("Membre ajouté");
                 })
                 .error(function (data) {
@@ -28,11 +28,11 @@ angular.module(&#39;MembreCtrl&#39;, [&#39;SessionsServ&#39;, &#39;NotifyServ&#39;]).controller(&#39;MembreC
                 });
         };
-        $scope.deleteMembre = function (id) {
+        $scope.deleteMembre = function (index) {
             var not = NotifyServ.promise("Suppression du membre...");
-            $http.delete('/api/membres/' + id)
-                .success(function (data) {
-                    $scope.membres = data;
+            $http.delete('/api/membres/' + $scope.membres[index]._id)
+                .success(function (membre) {
+                    $scope.membres.splice(index, 1);
                     not.success("Membre supprimé");
                 })
                 .error(function (data) {
 <div class="container">
-    <!-- HEADER AND TODO COUNT -->
     <div class="jumbotron text-center">
         <h1>Il y a
             <span id="membre-qt" class="label label-info">{{ membres.length }}</span> personnes au Club Info !</h1>
@@ -20,7 +19,7 @@
                 <td>{{ membre.section }}</td>
                 <td>{{ membre.role }}</td>
                 <td ng-if="session.canDelMembre">
-                    <button type="button" class="btn btn-danger" aria-label="Expulser" ng-click="deleteMembre(membre._id)">
+                    <button type="button" class="btn btn-danger" aria-label="Expulser" ng-click="deleteMembre($index)">
                         <span class="glyphicon glyphicon-remove" aria-hidden="true"></span>
                     </button>
                 </td>