Simplification des permissions

Vous pouvez mainteant éditer VOS messages. Fix #12

Simplification des permissions
Vous pouvez mainteant éditer VOS messages. Fix #12
Geoffrey PREUD'HOMME
1 parent ba3a9e89
Showing 7 changed files with 127 additions and 65 deletions Show diff stats
app/routes/ApiRtes.js
app/services/MembresServ.js
public/js/controllers/ForumConvCtrl.js
public/js/controllers/ForumDirCtrl.js
public/js/services/ApiServ.js
public/views/forumConv.html
public/views/forumDir.html
@@ -2,8 +2,11 @@ var MembresServ = require(&#39;../services/MembresServ&#39;);
 var PolyUserServ = require('../services/PolyUserServ');
 var DecryptServ = require('../services/DecryptServ');
 var DosssServ = require('../services/DosssServ');
+var DossModl = require('../models/DossModl');
 var ConvsServ = require('../services/ConvsServ');
+var ConvModl = require('../models/ConvModl');
 var MessServ = require('../services/MessServ');
+var MessModl = require('../models/MessModl'); // TODO Unfier ce bazar / supprimer Serv
 var fs = require('fs');
 var mongoose = require('mongoose');
 var express = require('express');
@@ -44,9 +47,21 @@ reqVerified = function (verify) { // Assert mais pour les droits (d&#39;où le 403)
     };
 };
  
-reqPerm = function (perm) {
+reqOwn = function (objName) {
     return reqVerified(function (req, res, cb) {
-        cb(null, req.session.data[perm]);
+        cb(null, req.session.data.bureau || req[objName].login == req.session.data.login);
+    });
+};
+
+reqMembre = function () {
+    return reqVerified(function (req, res, cb) {
+        cb(null, req.session.data.membre);
+    });
+};
+
+reqBureau = function () {
+    return reqVerified(function (req, res, cb) {
+        cb(null, req.session.data.bureau);
     });
 };
  
@@ -86,19 +101,12 @@ sessionData = function (session, cb) {
         // Nom
         session.nom = nom.nom;
         session.section = nom.section;
-        MembresServ.estBureau(session.login, function (bureau) {
-            session.bureau = bureau;
-            // Permissions
-            session.canAddMembre = session.bureau;
-            session.canDelMembre = session.bureau;
-            session.canAddDoss = session.bureau;
-            session.canDelDoss = session.bureau;
-            session.canAddConv = true;
-            session.canDelConv = session.bureau;
-            session.canAddMess = true;
-            session.canEditMess = session.bureau;
-            session.canDelMess = session.bureau;
-            cb(session);
+        MembresServ.estMembre(session.login, function (membre) { // TODO Asyc
+            session.membre = membre;
+            MembresServ.estBureau(session.login, function (bureau) {
+                session.bureau = bureau;
+                cb(session);
+            });
         });
     });
 };
@@ -133,7 +141,7 @@ api.post(&#39;/session&#39;, decrypt(), assert(function (req, res, cb) {
                     req.session.data = session;
                     req.session.save(function (err) {
                         if (err) {
-                            res.status(500).end("Sauvegarde session");
+                            res.status(500).end(err);
                         } else {
                             res.status(201).send(session);
                         }
@@ -142,7 +150,7 @@ api.post(&#39;/session&#39;, decrypt(), assert(function (req, res, cb) {
             } else {
                 req.session.destroy(function (err) {
                     if (err) {
-                        res.status(500).end("Suppression de la session");
+                        res.status(500).end(err);
                     } else {
                         res.status(401).end();
                     }
@@ -170,7 +178,7 @@ api.get(&#39;/membres&#39;, function (req, res) { // Liste des membres
  
 api.post('/membres', assert(function (req, res, cb) {
     cb(null, typeof req.body.login == 'string' && req.body.login !== '');
-}), reqPerm('canAddMembre'), function (req, res) { // Ajout d'un membre
+}), reqBureau(), function (req, res) { // Ajout d'un membre
     MembresServ.add(req.body, function (err, membre) {
         if (err)
             res.status(500).send(err);
@@ -179,7 +187,7 @@ api.post(&#39;/membres&#39;, assert(function (req, res, cb) {
     });
 });
  
-api.delete('/membres/:membre_id', reqPerm('canDelMembre'), function (req, res) { // Supression d'un membre
+api.delete('/membres/:membre_id', reqBureau(), function (req, res) { // Supression d'un membre
     MembresServ.remove(req.params.membre_id, function (err, membre) {
         if (err)
             res.status(500).send(err);
@@ -189,14 +197,14 @@ api.delete(&#39;/membres/:membre_id&#39;, reqPerm(&#39;canDelMembre&#39;), function (req, res) {
 });
  
 // Dossiers
-api.get('/dosss/:doss_id', function (req, res) { // Un doss
+api.get('/dosss/:doss_id', reqAuth(), function (req, res) { // Un doss
     // TODO Assertion 404 existe, transformer req.body.id avec la vraie id (ou redirect)
     // TODO Requêtes séparées ?
     DosssServ.get(req.params.doss_id, function (err, doss) { // TODO Async
         if (err) {
             res.status(500).send(err);
         } else if (!doss) {
-            res.status(404);
+            res.status(404).end();
         } else {
             DosssServ.children(doss._id, function (err, dosss) {
                 if (err) {
@@ -217,7 +225,7 @@ api.get(&#39;/dosss/:doss_id&#39;, function (req, res) { // Un doss
     });
 });
  
-api.post('/dosss', reqPerm('canAddDoss'), function (req, res) { // Ajout d'un doss
+api.post('/dosss', reqMembre(), function (req, res) { // Ajout d'un doss
     // TODO Assertion 404 existe, transformer req.body.id avec la vraie id (ou redirect)
     DosssServ.getId(req.body.parent, function (parent) { // TODO Async
         req.body.parent = parent;
@@ -230,7 +238,7 @@ api.post(&#39;/dosss&#39;, reqPerm(&#39;canAddDoss&#39;), function (req, res) { // Ajout d&#39;un do
     });
 });
  
-api.delete('/dosss/:doss_id', reqPerm('canDelDoss'), function (req, res) { // Supression d'un doss
+api.delete('/dosss/:doss_id', reqBureau(), function (req, res) { // Supression d'un doss
     DosssServ.remove(req.params.doss_id, function (err, doss) {
         if (err)
             res.status(500).send(err);
@@ -240,16 +248,27 @@ api.delete(&#39;/dosss/:doss_id&#39;, reqPerm(&#39;canDelDoss&#39;), function (req, res) { // Su
 });
  
 // Conversations
-api.get('/convs/:conv_id', function (req, res) { // Une conv
-    ConvsServ.get(req.params.conv_id, function (err, conv) {
-        if (err)
-            res.status(500).send(err);
-        else
-            res.json(conv);
+
+getConv = function (req, res, next) {
+    ConvModl.findById(req.params.conv_id, function (err, conv) {
+        if (err) {
+            res.status(500).json(err);
+        } else {
+            if (conv) {
+                req.conv = conv;
+                next();
+            } else {
+                res.status(404).end();
+            }
+        }
     });
+};
+
+api.get('/convs/:conv_id', reqAuth(), getConv, function (req, res) { // Une conv
+    res.json(req.conv);
 });
  
-api.post('/convs', reqPerm('canAddConv'), function (req, res) { // Ajout d'un conv
+api.post('/convs', reqMembre(), function (req, res) { // Ajout d'un conv
     // TODO Assertion 404 existe, transformer req.body.id avec la vraie id (ou redirect)
     DosssServ.getId(req.body.parent, function (parent) { // TODO Async
         req.body.parent = parent;
@@ -262,12 +281,12 @@ api.post(&#39;/convs&#39;, reqPerm(&#39;canAddConv&#39;), function (req, res) { // Ajout d&#39;un co
     });
 });
  
-api.delete('/convs/:conv_id', reqPerm('canDelConv'), function (req, res) { // Supression d'un conv
-    ConvsServ.remove(req.params.conv_id, function (err, conv) {
-        if (err)
+api.delete('/convs/:conv_id', reqBureau(), getConv, function (req, res) { // Supression d'un conv
+    req.conv.remove(function (err) {
+        if (err) // TODO Fonction propre
             res.status(500).send(err);
         else
-            res.json(null);
+            res.status(205).end();
     });
 });
  
@@ -281,16 +300,16 @@ api.get(&#39;/messs/:conv_id&#39;, reqAuth(), function (req, res) { // Liste des messs
     });
 });
  
-api.get('/messs/:mess_id', reqAuth(), function (req, res) { // Une mess
-    MessServ.get(req.params.mess_id, function (err, mess) {
-        if (err)
-            res.status(500).send(err);
-        else
-            res.json(mess);
-    });
-});
+// api.get('/messs/:mess_id', reqAuth(), function (req, res) { // Une mess
+//     MessServ.get(req.params.mess_id, function (err, mess) {
+//         if (err)
+//             res.status(500).send(err);
+//         else
+//             res.json(mess);
+//     });
+// });
  
-api.post('/messs', reqPerm('canAddMess'), function (req, res) { // Ajout d'un mess
+api.post('/messs', reqMembre(), function (req, res) { // Ajout d'un mess
     data = req.body;
     data.login = req.session.data.login;
     MessServ.add(data, function (err, mess) {
@@ -301,27 +320,50 @@ api.post(&#39;/messs&#39;, reqPerm(&#39;canAddMess&#39;), function (req, res) { // Ajout d&#39;un me
     });
 });
  
-api.put('/messs', reqPerm('canEditMess'), function (req, res) { // Édition d'un message
-    MessServ.edit(req.body, function (err, mess) {
-        console.log('CALLED', err, mess);
+api.put('/messs', reqMembre(), function (req, res, next) { // Édition d'un mess
+    MessModl.findById(req.body._id, function (err, mess) { // TODO Fonction propre
+        // TODO Utiliser req.params
         if (err) {
-            if (err == 'notfound') {
-                res.status(404).end();
+            res.status(500).json(err);
+        } else {
+            if (mess) {
+                req.mess = mess;
+                next();
             } else {
-                res.status(500).json(err);
+                res.status(404).end();
             }
-        } else {
-            res.status(201).json(mess);
         }
     });
+}, reqOwn('mess'), function (req, res) {
+    req.mess.content = req.body.content;
+    // TODO Edit date
+    req.mess.save(function (err, mess) {
+        if (err) // TODO Fonction propre
+            res.status(500).send(err);
+        else
+            res.json(mess);
+    });
 });
  
-api.delete('/messs/:mess_id', reqPerm('canDelMess'), function (req, res) { // Supression d'un mess
-    MessServ.remove(req.params.mess_id, function (err, mess) {
-        if (err)
+api.delete('/messs/:mess_id', reqMembre(), function (req, res, next) { // Supression d'un mess
+    MessModl.findById(req.params.mess_id, function (err, mess) { // TODO Fonction propre
+        if (err) {
+            res.status(500).json(err);
+        } else {
+            if (mess) {
+                req.mess = mess;
+                next();
+            } else {
+                res.status(404).end();
+            }
+        }
+    });
+}, reqOwn('mess'), function (req, res) {
+    req.mess.remove(function (err) {
+        if (err) // TODO Fonction propre
             res.status(500).send(err);
         else
-            res.json(null);
+            res.status(205).end();
     });
 });
  
@@ -46,6 +46,18 @@ MembresServ.remove = function (id, cb) {
     }, cb);
 };
  
+MembresServ.estMembre = function (login, cb) {
+    MembreModl.findOne({
+        login: login
+    }, function (err, data) {
+        if (!err && data) {
+            cb(true);
+        } else {
+            cb(false);
+        }
+    });
+};
+
 MembresServ.estBureau = function (login, cb) {
     MembreModl.findOne({
         login: login
@@ -57,4 +57,8 @@ angular.module(&#39;ForumConvCtrl&#39;, [&#39;SessionsServ&#39;, &#39;ApiServ&#39;, &#39;MessEditDrct&#39;])
             }
         };
  
+        $scope.mine = function (mess) {
+            return $scope.session.bureau || mess.login == $scope.session.login;
+        };
+
     });
@@ -59,4 +59,8 @@ angular.module(&#39;ForumDirCtrl&#39;, [&#39;SessionsServ&#39;, &#39;ApiServ&#39;])
             });
         };
  
+        $scope.mine = function (obj) {
+            return $scope.session.bureau;
+        };
+
     });
@@ -13,7 +13,7 @@ angular.module(&#39;ApiServ&#39;, [&#39;NotifyServ&#39;])
                 };
             }
  
-            if (method == 'get' || method == 'delete') {
+            if (method == 'get' || method == 'delete') { // TODO url & data en même temps
                 for (arg = 3; arg < arglen; arg++) {
                     link += '/' + arguments[arg];
                 }
@@ -28,16 +28,16 @@
                 <td mess-edit></td>
                 <td>
                     <mess-viewsource></mess-viewsource>
-                    <button type="button" class="btn btn-primary" ng-show="session.canEditMess" ng-click="editButton($index)">
+                    <button type="button" class="btn btn-primary" ng-show="mine(mess)" ng-click="editButton($index)">
                         <span class="glyphicon glyphicon-edit" aria-hidden="true" ng-hide="mess.editMode"></span>
                         <span class="glyphicon glyphicon-ok" aria-hidden="true" ng-show="mess.editMode"></span>
                     </button>
-                    <button type="button" class="btn btn-danger" ng-show="session.canDelMess" ng-click="delMess($index)">
+                    <button type="button" class="btn btn-danger" ng-show="mine(mess)" ng-click="delMess($index)">
                         <span class="glyphicon glyphicon-remove" aria-hidden="true"></span>
                     </button>
                 </td>
             </tr>
-            <tr ng-show="session.canAddMess">
+            <tr ng-show="session.membre">
                 <form ng-init="mess = {content: '', editMode: true, viewSource: true}">
                     <td>
                         <label for="nvMessTitre">Nouveau message</label>
@@ -31,16 +31,16 @@
                     {{ doss.date }}
                 </td>
                 <td>
-                    <button type="button" class="btn btn-default" ng-show="session.canAbonner">
+                    <button type="button" class="btn btn-default" ng-show="session.membre && implemented">
                         <span class="glyphicon glyphicon-eye-open" aria-hidden="true" ng-show="doss.abonne"></span>
                         <span class="glyphicon glyphicon-eye-close" aria-hidden="true" ng-hide="doss.abonne"></span>
                     </button>
-                    <button type="button" class="btn btn-danger" ng-show="session.canDelDoss" ng-click="delDoss($index)">
+                    <button type="button" class="btn btn-danger" ng-show="mine(doss)" ng-click="delDoss($index)">
                         <span class="glyphicon glyphicon-remove" aria-hidden="true"></span>
                     </button>
                 </td>
             </tr>
-            <tr ng-show="session.canAddDoss">
+            <tr ng-show="session.membre">
                 <form>
                     <td class="form-group">
                         <label for="nvDossTitre">Nouveau dossier</label>
@@ -73,16 +73,16 @@
                     {{ conv.date }}
                 </td>
                 <td>
-                    <button type="button" class="btn btn-default" ng-show="session.canAbonner">
+                    <button type="button" class="btn btn-default" ng-show="session.membre && implemented">
                         <span class="glyphicon glyphicon-eye-open" aria-hidden="true" ng-show="conv.abonne"></span>
                         <span class="glyphicon glyphicon-eye-close" aria-hidden="true" ng-hide="conv.abonne"></span>
                     </button>
-                    <button type="button" class="btn btn-danger" ng-show="session.canDelConv" ng-click="delConv($index)">
+                    <button type="button" class="btn btn-danger" ng-show="mine(conv)" ng-click="delConv($index)">
                         <span class="glyphicon glyphicon-remove" aria-hidden="true"></span>
                     </button>
                 </td>
             </tr>
-            <tr ng-show="session.canAddConv">
+            <tr ng-show="session.membre">
                 <form>
                     <td class="form-group">
                         <label for="nvConvTitre">Nouvelle conversation</label>