Merge branch 'dev' of https://gitea.verdnatura.es/verdnatura/salix into 3888-ticket.expedition_moveExpedition

modules/claim/back/methods/claim/claimPickupEmail.js

@@ -1,8 +1,9 @@
-const {Report, Email, smtp} = require('vn-print');
+const {Email} = require('vn-print');
 
 module.exports = Self => {
     Self.remoteMethodCtx('claimPickupEmail', {
         description: 'Sends the the claim pickup order email with an attached PDF',
+        accessType: 'WRITE',
         accepts: [
             {
                 arg: 'id',
@@ -40,7 +41,7 @@ module.exports = Self => {
         }
     });
 
-    Self.claimPickupEmail = async(ctx, id) => { 
+    Self.claimPickupEmail = async ctx => {
         const args = Object.assign({}, ctx.args);
         const params = {
             recipient: args.recipient,

modules/client/back/methods/client/consumptionSendQueued.js

@@ -18,43 +18,43 @@ module.exports = Self => {
     Self.consumptionSendQueued = async() => {
         const queues = await Self.rawSql(`
             SELECT
-                    ccq.id,
+                id,
+                params
+            FROM clientConsumptionQueue
+            WHERE status = ''`);
+
+        for (const queue of queues) {
+            try {
+                const params = JSON.parse(queue.params);
+
+                const clients = await Self.rawSql(`
+                    SELECT
                             c.id AS clientFk,
                             c.email AS clientEmail,
-                    eu.email salesPersonEmail,
+                            eu.email salesPersonEmail
-                    REPLACE(json_extract(params, '$.from'), '"', '') AS fromDate,
+                    FROM client c 
-                    REPLACE(json_extract(params, '$.to'), '"', '') AS toDate
-            FROM clientConsumptionQueue ccq
-                JOIN client c ON (
-                    JSON_SEARCH(
-                        JSON_ARRAY(
-                            json_extract(params, '$.clients')
-                        )
-                    , 'all', c.id) IS NOT NULL)
                         JOIN account.emailUser eu ON eu.userFk = c.salesPersonFk
                         JOIN ticket t ON t.clientFk = c.id
                         JOIN sale s ON s.ticketFk = t.id
                         JOIN item i ON i.id = s.itemFk
                         JOIN itemType it ON it.id = i.typeFk
-            WHERE status = ''
+                    WHERE c.id IN(?)
                         AND it.isPackaging = FALSE
-                AND DATE(t.shipped) BETWEEN
+                        AND DATE(t.shipped) BETWEEN ? AND ?
-                    REPLACE(json_extract(params, '$.from'), '"', '') AND
+                    GROUP BY c.id`, [params.clients, params.from, params.to]);
-                    REPLACE(json_extract(params, '$.to'), '"', '')
-            GROUP BY c.id`);
 
-        for (const queue of queues) {
+                for (const client of clients) {
-            try {
                     const args = {
-                    id: queue.clientFk,
+                        id: client.clientFk,
-                    recipient: queue.clientEmail,
+                        recipient: client.clientEmail,
-                    replyTo: queue.salesPersonEmail,
+                        replyTo: client.salesPersonEmail,
-                    from: queue.fromDate,
+                        from: params.from,
-                    to: queue.toDate
+                        to: params.to
                     };
 
                     const email = new Email('campaign-metrics', args);
                     await email.send();
+                }
 
                 await Self.rawSql(`
                     UPDATE clientConsumptionQueue
@@ -69,7 +69,7 @@ module.exports = Self => {
                     WHERE id = ?`,
                 [error.message, queue.id]);
 
-                throw e;
+                throw error;
             }
         }
 

modules/client/front/notification/index.js

@@ -77,15 +77,16 @@ export default class Controller extends Section {
 
     onSendClientConsumption() {
         const clientIds = this.checked.map(client => client.id);
-        const params = {
+        const data = {
             clients: clientIds,
             from: this.campaign.from,
             to: this.campaign.to
         };
 
+        const params = JSON.stringify(data);
         this.$http.post('ClientConsumptionQueues', {params})
             .then(() => this.$.filters.hide())
-            .then(() => this.vnApp.showSuccess(this.$t('Notifications sent!')));
+            .then(() => this.vnApp.showSuccess(this.$t('Notification sent!')));
     }
 
     exprBuilder(param, value) {

modules/client/front/notification/index.spec.js

@@ -69,15 +69,16 @@ describe('Client notification', () => {
                 data[0].$checked = true;
                 data[1].$checked = true;
 
-                const params = Object.assign({
+                const args = Object.assign({
                     clients: [1101, 1102]
                 }, controller.campaign);
+                const params = JSON.stringify(args);
 
                 $httpBackend.expect('POST', `ClientConsumptionQueues`, {params}).respond(200, params);
                 controller.onSendClientConsumption();
                 $httpBackend.flush();
 
-                expect(controller.vnApp.showSuccess).toHaveBeenCalledWith('Notifications sent!');
+                expect(controller.vnApp.showSuccess).toHaveBeenCalledWith('Notification sent!');
             });
         });
 

modules/item/front/diary/index.js

@@ -27,7 +27,7 @@ class Controller extends Section {
             if (this.$params.warehouseFk)
                 this.warehouseFk = this.$params.warehouseFk;
             else if (value)
-                this.warehouseFk = value.itemType.warehouseFk;
+                this.warehouseFk = this.vnConfig.warehouseFk;
 
             if (this.$params.lineFk)
                 this.lineFk = this.$params.lineFk;

modules/route/back/methods/route/driverRouteEmail.js

@@ -3,6 +3,7 @@ const {Email} = require('vn-print');
 module.exports = Self => {
     Self.remoteMethodCtx('driverRouteEmail', {
         description: 'Sends the driver route email with an attached PDF',
+        accessType: 'WRITE',
         accepts: [
             {
                 arg: 'id',

modules/ticket/back/methods/ticket-request/getItemTypeWorker.js

@@ -29,7 +29,7 @@ module.exports = Self => {
             Object.assign(myOptions, options);
 
         const query =
-            `SELECT DISTINCT u.nickname
+            `SELECT DISTINCT u.id, u.nickname
                 FROM itemType it
                     JOIN worker w ON w.id = it.workerFk
                     JOIN account.user u ON u.id = w.id`;

modules/ticket/back/methods/ticket/closeAll.js

@@ -18,6 +18,7 @@ module.exports = Self => {
 
     Self.closeAll = async() => {
         const toDate = new Date();
+        toDate.setHours(0, 0, 0, 0);
         toDate.setDate(toDate.getDate() - 1);
 
         const todayMinDate = new Date();

modules/travel/front/extra-community/index.html

@@ -42,10 +42,10 @@
                         <th field="id" shrink>
                             <span translate>Id</span>
                         </th>
-                        <th field="cargoSupplierFk" expand>
+                        <th field="cargoSupplierFk">
                             <span translate>Supplier</span>
                         </th>
-                        <th field="agencyModeFk" expand>
+                        <th field="agencyModeFk">
                             <span translate>Agency</span>
                         </th>
                         <th field="ref">
@@ -100,37 +100,37 @@
                                 {{::travel.id}}
                             </span>
                         </td>
-                        <td expand vn-click-stop>
+                        <td class="multi-line" vn-click-stop>
                             <span
                                 class="link"
                                 ng-click="supplierDescriptor.show($event, travel.cargoSupplierFk)">
                                 {{::travel.cargoSupplierNickname}}
                             </span>
                         </td>
-                        <td expand>{{::travel.agencyModeName}}</td>
+                        <td>{{::travel.agencyModeName}}</td>
-                        <td
+                        <td vn-click-stop>
-                            name="reference"
+                            <vn-td-editable name="reference" expand>
-                            expand
+                                <text>{{travel.ref}}</text>
-                            vn-click-stop>
+                                <field>
-                            <vn-textfield
+                                    <vn-textfield class="dense" vn-focus
-                                class="dense td-editable"
                                         ng-model="travel.ref"
                                         on-change="$ctrl.save(travel.id, {ref: value})">
                                     </vn-textfield>
-                        </vn-icon>
+                                </field>
+                            </vn-td-editable>
                         </td>
                         <td number>{{::travel.stickers}}</td>
-                        <td
+                        <td vn-click-stop>
-                            name="lockedKg"
+                            <vn-td-editable name="lockedKg" expand style="text-align: right">
-                            expand
+                                <text number>{{travel.kg}}</text>
-                            vn-click-stop>
+                                <field>
-                            <vn-input-number
+                                    <vn-input-number class="dense" vn-focus
-                                number
-                                class="td-editable number"
                                         ng-model="travel.kg"
                                         on-change="$ctrl.save(travel.id, {kg: value})"
                                         min="0">
                                     </vn-input-number>
+                                </field>
+                            </vn-td-editable>
                         </td>
                         <td number>{{::travel.loadedKg}}</td>
                         <td number>{{::travel.volumeKg}}</td>
@@ -150,7 +150,7 @@
                                 {{::entry.id}}
                             </span>
                         </td>
-                        <td>
+                        <td class="multi-line">
                             <span
                                 class="link"
                                 ng-click="supplierDescriptor.show($event, entry.supplierFk)">
@@ -158,7 +158,7 @@
                             </span>
                         </td>
                         <td></td>
-                        <td expand>{{::entry.ref}}</td>
+                        <td class="td-editable">{{::entry.ref}}</td>
                         <td number>{{::entry.stickers}}</td>
                         <td number></td>
                         <td number>{{::entry.loadedkg}}</td>

modules/travel/front/extra-community/style.scss

@@ -3,7 +3,6 @@
 vn-travel-extra-community {
     .header {
         margin-bottom: 16px;
-        font-size: 1.25rem;
         line-height: 1;
         padding: 7px;
         padding-bottom: 7px;
@@ -16,6 +15,10 @@ vn-travel-extra-community {
         overflow: hidden;
         text-overflow: ellipsis;
         cursor: pointer;
+        .multi-line{
+            padding-top: 15px;
+            padding-bottom: 15px;
+        }
     }
 
     table[vn-droppable] {
@@ -29,10 +32,10 @@ vn-travel-extra-community {
         outline: 0;
         height: 65px;
         pointer-events: fill;
-        user-select:all;
+        user-select: all;
     }
 
-    tr[draggable] *::selection{
+    tr[draggable] *::selection {
         background-color: transparent;
     }
 
@@ -43,16 +46,22 @@ vn-travel-extra-community {
     tr[draggable].dragging {
         background-color: $color-primary-light;
         color: $color-font-light;
-        font-weight:bold;
+        font-weight: bold;
     }
 
-    .td-editable{
+
-        input{
+    .multi-line{
-            font-size: 1.25rem!important;
+        max-width: 200px;
-        }
+        word-wrap: normal;
+        white-space: normal;
     }
 
-    .number *{
+    vn-td-editable text {
-        text-align: right;
+        background-color: transparent;
+        padding: 0;
+        border: 0;
+        border-bottom: 1px dashed $color-active;
+        border-radius: 0;
+        color: $color-active
     }
 }

modules/zone/back/methods/zone/getLeaves.js

@@ -59,6 +59,9 @@ module.exports = Self => {
         }
 
         const leaves = map.get(parentId);
+
+        const maxNodes = 250;
+        if (res.length <= maxNodes)
             setLeaves(leaves);
 
         return leaves || [];