Merge branch 'test' of https://gitea.verdnatura.es/verdnatura/salix into dev

back/methods/chat/notifyIssues.js

@@ -8,7 +8,7 @@ module.exports = Self => {
         },
         http: {
             path: `/notifyIssues`,
-            verb: 'GET'
+            verb: 'POST'
         }
     });
 

back/methods/chat/sendQueued.js

@@ -10,7 +10,7 @@ module.exports = Self => {
         },
         http: {
             path: `/sendQueued`,
-            verb: 'GET'
+            verb: 'POST'
         }
     });
 

back/methods/dms/deleteTrashFiles.js

@@ -12,7 +12,7 @@ module.exports = Self => {
         },
         http: {
             path: `/deleteTrashFiles`,
-            verb: 'GET'
+            verb: 'POST'
         }
     });
 

back/methods/edi/updateData.js

@@ -12,7 +12,7 @@ module.exports = Self => {
         },
         http: {
             path: `/updateData`,
-            verb: 'GET'
+            verb: 'POST'
         }
     });
 

modules/item/back/methods/item-image-queue/downloadImages.js

@@ -12,7 +12,7 @@ module.exports = Self => {
         },
         http: {
             path: `/downloadImages`,
-            verb: 'GET'
+            verb: 'POST'
         }
     });
 

modules/worker/back/methods/worker-time-control-mail/checkInbox.js

@@ -11,7 +11,7 @@ module.exports = Self => {
             },
         http: {
             path: `/checkInbox`,
-            verb: 'GET'
+            verb: 'POST'
         }
     });
 
@@ -58,8 +58,8 @@ module.exports = Self => {
                                 emailBody = bufferCopy.toUpperCase().trim();
 
                                 const bodyPositionOK = emailBody.match(/\bOK\b/i);
-
-                                if (bodyPositionOK != null && (bodyPositionOK.index == 0 || bodyPositionOK.index == 122))
+                                const bodyPositionIndex = (bodyPositionOK.index == 0 || bodyPositionOK.index == 122);
+                                if (bodyPositionOK != null && bodyPositionIndex)
                                     isEmailOk = true;
                                 else
                                     isEmailOk = false;

modules/zone/front/events/index.html

@@ -91,7 +91,7 @@
     </vn-data-viewer>
 </vn-side-menu>
 <vn-float-button
-    ng-click="$ctrl.create('weekday')"
+    ng-click="$ctrl.createInclusion('weekday')"
     icon="add"
     vn-tooltip="Add event"
     vn-bind="+"

print/methods/closure/closeAll.js

@@ -3,13 +3,22 @@ const closure = require('./closure');
 
 module.exports = async function(request, response, next) {
     try {
-        const reqArgs = request.query;
-        if (!reqArgs.to)
-            throw new Error('The argument to is required');
+        const reqArgs = request.body;
 
-        response.status(200).json({
-            message: 'Success'
-        });
+        let toDate = new Date();
+        toDate.setDate(toDate.getDate() - 1);
+
+        if (reqArgs.to) toDate = reqArgs.to;
+
+        const todayMinDate = new Date();
+        minDate.setHours(0, 0, 0, 0);
+
+        const todayMaxDate = new Date();
+        maxDate.setHours(23, 59, 59, 59);
+
+        // Prevent closure for current day
+        if (toDate >= todayMinDate && toDate <= todayMaxDate)
+            throw new Error('You cannot close tickets for today');
 
         const tickets = await db.rawSql(`
             SELECT 
@@ -36,7 +45,7 @@ module.exports = async function(request, response, next) {
                 AND DATE(t.shipped) BETWEEN DATE_ADD(?, INTERVAL -2 DAY) 
                     AND util.dayEnd(?)
                 AND t.refFk IS NULL
-            GROUP BY t.id`, [reqArgs.to, reqArgs.to]);
+            GROUP BY t.id`, [toDate, toDate]);
 
         await closure.start(tickets, response.locals);
 
@@ -52,7 +61,11 @@ module.exports = async function(request, response, next) {
                     AND util.dayEnd(?)
                     AND al.code NOT IN('DELIVERED','PACKED')
                     AND t.routeFk
-                    AND z.name LIKE '%MADRID%'`, [reqArgs.to, reqArgs.to]);
+                    AND z.name LIKE '%MADRID%'`, [toDate, toDate]);
+
+        response.status(200).json({
+            message: 'Success'
+        });
     } catch (error) {
         next(error);
     }

print/methods/closure/closeByAgency.js

@@ -3,7 +3,7 @@ const closure = require('./closure');
 
 module.exports = async function(request, response, next) {
     try {
-        const reqArgs = request.query;
+        const reqArgs = request.body;
 
         if (!reqArgs.agencyModeId)
             throw new Error('The argument agencyModeId is required');
@@ -14,10 +14,6 @@ module.exports = async function(request, response, next) {
         if (!reqArgs.to)
             throw new Error('The argument to is required');
 
-        response.status(200).json({
-            message: 'Success'
-        });
-
         const agencyIds = reqArgs.agencyModeId.split(',');
         const tickets = await db.rawSql(`
             SELECT 
@@ -53,6 +49,10 @@ module.exports = async function(request, response, next) {
         ]);
 
         await closure.start(tickets, response.locals);
+
+        response.status(200).json({
+            message: 'Success'
+        });
     } catch (error) {
         next(error);
     }

print/methods/closure/closeByRoute.js

@@ -4,15 +4,11 @@ const closure = require('./closure');
 
 module.exports = async function(request, response, next) {
     try {
-        const reqArgs = request.query;
+        const reqArgs = request.body;
 
         if (!reqArgs.routeId)
             throw new Error('The argument routeId is required');
 
-        response.status(200).json({
-            message: 'Success'
-        });
-
         const tickets = await db.rawSql(`
             SELECT 
                 t.id,
@@ -56,6 +52,10 @@ module.exports = async function(request, response, next) {
             const email = new Email('driver-route', args);
             await email.send();
         }
+
+        response.status(200).json({
+            message: 'Success'
+        });
     } catch (error) {
         next(error);
     }

print/methods/closure/closeByTicket.js

@@ -3,15 +3,11 @@ const closure = require('./closure');
 
 module.exports = async function(request, response, next) {
     try {
-        const reqArgs = request.query;
+        const reqArgs = request.body;
 
         if (!reqArgs.ticketId)
             throw new Error('The argument ticketId is required');
 
-        response.status(200).json({
-            message: 'Success'
-        });
-
         const tickets = await db.rawSql(`
             SELECT 
                 t.id,
@@ -38,6 +34,10 @@ module.exports = async function(request, response, next) {
             GROUP BY e.ticketFk`, [reqArgs.ticketId]);
 
         await closure.start(tickets, response.locals);
+
+        response.status(200).json({
+            message: 'Success'
+        });
     } catch (error) {
         next(error);
     }

print/methods/closure/index.js

@@ -1,9 +1,9 @@
 const express = require('express');
 const router = new express.Router();
 
-router.get('/all', require('./closeAll'));
-router.get('/by-ticket', require('./closeByTicket'));
-router.get('/by-agency', require('./closeByAgency'));
-router.get('/by-route', require('./closeByRoute'));
+router.post('/all', require('./closeAll'));
+router.post('/by-ticket', require('./closeByTicket'));
+router.post('/by-agency', require('./closeByAgency'));
+router.post('/by-route', require('./closeByRoute'));
 
 module.exports = router;