.vscode/settings.json

@@ -6,5 +6,9 @@
         "source.fixAll.eslint": true
     },
     "search.useIgnoreFiles": false,
-    "editor.defaultFormatter": "dbaeumer.vscode-eslint"
+    "editor.defaultFormatter": "dbaeumer.vscode-eslint",
+    "eslint.format.enable": true,
+    "[javascript]": {
+        "editor.defaultFormatter": "dbaeumer.vscode-eslint"
+    }
 }

CHANGELOG.md

@@ -5,6 +5,19 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [2314.01] - 2023-04-20
+
+### Added
+- (Clientes -> Morosos) Ahora se puede filtrar por las columnas "Desde" y "Fecha Ú. O.". También se envia un email al comercial cuando se añade una nota.
+- (Monitor tickets) Muestra un icono al lado de la zona, si el ticket es frágil y se envía por agencia
+- (Facturas recibidas -> Bases negativas) Nueva sección
+
+### Changed
+-
+
+### Fixed
+- (Clientes -> Morosos) Ahora se mantienen los elementos seleccionados al hacer sroll.
+
 ## [2312.01] - 2023-04-06
 
 ### Added
@@ -15,9 +28,6 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - (Envíos -> Extra comunitarios) Se agrupan las entradas del mismo travel. Añadidos campos Referencia y Importe.
 - (Envíos -> Índice) Cambiado el buscador superior por uno lateral
 
-### Fixed
--
-
 ## [2310.01] - 2023-03-23
 
 ### Added

back/methods/account/change-password.js

@@ -2,6 +2,7 @@
 module.exports = Self => {
     Self.remoteMethod('changePassword', {
         description: 'Changes the user password',
+        accessType: 'WRITE',
         accepts: [
             {
                 arg: 'id',

back/methods/account/set-password.js

@@ -1,6 +1,7 @@
 module.exports = Self => {
     Self.remoteMethod('setPassword', {
         description: 'Sets the user password',
+        accessType: 'WRITE',
         accepts: [
             {
                 arg: 'id',

back/models/user.json

@@ -4,7 +4,8 @@
   "options": {
     "mysql": {
       "table": "salix.User"
-    }
+    },
+    "resetPasswordTokenTTL": "604800"
   },
   "properties": {
     "id": {

db/changes/231201/00-mailACL.sql

@@ -0,0 +1,2 @@
+INSERT INTO `salix`.`ACL` ( model, property, accessType, permission, principalType, principalId)
+VALUES('Mail', '*', '*', 'ALLOW', 'ROLE', 'employee');

db/changes/231401/00-invoiceOutAfterInsert.sql

@@ -0,0 +1,13 @@
+DROP TRIGGER IF EXISTS `vn`.`invoiceOut_afterInsert`;
+USE vn;
+
+DELIMITER $$
+$$
+CREATE DEFINER=`root`@`localhost` TRIGGER `vn`.`invoiceOut_afterInsert`
+	AFTER INSERT ON `invoiceOut`
+	FOR EACH ROW
+BEGIN
+	CALL clientRisk_update(NEW.clientFk, NEW.companyFk, NEW.amount);
+END$$
+DELIMITER ;
+

db/changes/231401/00-negativeBases.sql

@@ -0,0 +1,4 @@
+INSERT INTO `salix`.`ACL` (`model`, `property`, `accessType`, `permission`, `principalType`, `principalId`)
+    VALUES
+        ('InvoiceIn', 'negativeBases', 'READ', 'ALLOW', 'ROLE', 'administrative'),
+        ('InvoiceIn', 'negativeBasesCsv', 'READ', 'ALLOW', 'ROLE', 'administrative');

e2e/paths/09-invoice-in/05_negative_bases.spec.js

@@ -0,0 +1,29 @@
+import getBrowser from '../../helpers/puppeteer';
+
+describe('InvoiceIn negative bases path', () => {
+    let browser;
+    let page;
+    const httpRequests = [];
+
+    beforeAll(async() => {
+        browser = await getBrowser();
+        page = browser.page;
+        page.on('request', req => {
+            if (req.url().includes(`InvoiceIns/negativeBases`))
+                httpRequests.push(req.url());
+        });
+        await page.loginAndModule('administrative', 'invoiceIn');
+        await page.accessToSection('invoiceIn.negative-bases');
+    });
+
+    afterAll(async() => {
+        await browser.close();
+    });
+
+    it('should show negative bases in a date range', async() => {
+        const request = httpRequests.find(req =>
+            req.includes(`from`) && req.includes(`to`));
+
+        expect(request).toBeDefined();
+    });
+});

loopback/locale/es.json

@@ -268,8 +268,11 @@
 	"Exists an invoice with a previous date": "Existe una factura con fecha anterior",
 	"Invoice date can't be less than max date": "La fecha de factura no puede ser inferior a la fecha límite",
 	"Warehouse inventory not set": "El almacén inventario no está establecido",
-    "This locker has already been assigned": "Esta taquilla ya ha sido asignada",
+	"This locker has already been assigned": "Esta taquilla ya ha sido asignada",
 	"Tickets with associated refunds": "No se pueden borrar tickets con abonos asociados. Este ticket está asociado al abono Nº {{id}}",
 	"Not exist this branch": "La rama no existe",
-    "This ticket cannot be signed because it has not been boxed": "Este ticket no puede firmarse porque no ha sido encajado"
+	"This ticket cannot be signed because it has not been boxed": "Este ticket no puede firmarse porque no ha sido encajado",
+	"Insert a date range": "Inserte un rango de fechas",
+    "Added observation": "{{user}} añadió esta observacion: {{text}}",
+    "Comment added to client": "Observación añadida al cliente {{clientFk}}"
 }

modules/client/back/methods/client/checkDuplicated.js

@@ -1,63 +0,0 @@
-module.exports = Self => {
-    Self.remoteMethod('checkDuplicatedData', {
-        description: 'Checks if a client has same email, mobile or phone than other client and send an email',
-        accepts: [{
-            arg: 'id',
-            type: 'number',
-            required: true,
-            description: 'The client id'
-        }],
-        returns: {
-            type: 'object',
-            root: true
-        },
-        http: {
-            verb: 'GET',
-            path: '/:id/checkDuplicatedData'
-        }
-    });
-
-    Self.checkDuplicatedData = async function(id, options) {
-        const myOptions = {};
-
-        if (typeof options == 'object')
-            Object.assign(myOptions, options);
-
-        const client = await Self.app.models.Client.findById(id, myOptions);
-
-        const findParams = [];
-        if (client.email) {
-            const emails = client.email.split(',');
-            for (let email of emails)
-                findParams.push({email: email});
-        }
-
-        if (client.phone)
-            findParams.push({phone: client.phone});
-
-        if (client.mobile)
-            findParams.push({mobile: client.mobile});
-
-        const filterObj = {
-            where: {
-                and: [
-                    {or: findParams},
-                    {id: {neq: client.id}}
-                ]
-            }
-        };
-
-        const clientSameData = await Self.findOne(filterObj, myOptions);
-
-        if (clientSameData) {
-            await Self.app.models.Mail.create({
-                receiver: 'direccioncomercial@verdnatura.es',
-                subject: `Cliente con email/teléfono/móvil duplicados`,
-                body: 'El cliente ' + client.id + ' comparte alguno de estos datos con el cliente ' + clientSameData.id +
-                        '\n- Email: ' + client.email +
-                        '\n- Teléfono: ' + client.phone +
-                        '\n- Móvil: ' + client.mobile
-            }, myOptions);
-        }
-    };
-};

modules/client/back/methods/client/specs/checkDuplicated.spec.js

@@ -1,24 +0,0 @@
-const models = require('vn-loopback/server/server').models;
-
-describe('client checkDuplicated()', () => {
-    it('should send an mail if mobile/phone/email is duplicated', async() => {
-        const tx = await models.Client.beginTransaction({});
-
-        try {
-            const options = {transaction: tx};
-
-            const id = 1110;
-            const mailModel = models.Mail;
-            spyOn(mailModel, 'create');
-
-            await models.Client.checkDuplicatedData(id, options);
-
-            expect(mailModel.create).toHaveBeenCalled();
-
-            await tx.rollback();
-        } catch (e) {
-            await tx.rollback();
-            throw e;
-        }
-    });
-});

modules/client/back/methods/defaulter/observationEmail.js

@@ -0,0 +1,52 @@
+module.exports = Self => {
+    Self.remoteMethodCtx('observationEmail', {
+        description: 'Send an email with the observation',
+        accessType: 'WRITE',
+        accepts: [
+            {
+                arg: 'defaulters',
+                type: ['object'],
+                required: true,
+                description: 'The defaulters to send the email'
+            },
+            {
+                arg: 'observation',
+                type: 'string',
+                required: true,
+                description: 'The observation'
+            }],
+        returns: {
+            arg: 'observationEmail'
+        },
+        http: {
+            path: `/observationEmail`,
+            verb: 'POST'
+        }
+    });
+
+    Self.observationEmail = async(ctx, defaulters, observation, options) => {
+        const models = Self.app.models;
+        const $t = ctx.req.__; // $translate
+        const myOptions = {};
+        const userId = ctx.req.accessToken.userId;
+
+        if (typeof options == 'object')
+            Object.assign(myOptions, options);
+
+        for (const defaulter of defaulters) {
+            const user = await models.Account.findById(userId, {fields: ['name']}, myOptions);
+
+            const body = $t('Added observation', {
+                user: user.name,
+                text: observation
+            });
+
+            await models.Mail.create({
+                subject: $t('Comment added to client', {clientFk: defaulter.clientFk}),
+                body: body,
+                receiver: `${defaulter.salesPersonName}@verdnatura.es`,
+                replyTo: `${user.name}@verdnatura.es`
+            }, myOptions);
+        }
+    };
+};

modules/client/back/models/client-methods.js

@@ -2,7 +2,6 @@ module.exports = Self => {
     require('../methods/client/addressesPropagateRe')(Self);
     require('../methods/client/canBeInvoiced')(Self);
     require('../methods/client/canCreateTicket')(Self);
-    require('../methods/client/checkDuplicated')(Self);
     require('../methods/client/confirmTransaction')(Self);
     require('../methods/client/consumption')(Self);
     require('../methods/client/createAddress')(Self);

modules/client/back/models/defaulter.js

@@ -1,3 +1,4 @@
 module.exports = Self => {
     require('../methods/defaulter/filter')(Self);
+    require('../methods/defaulter/observationEmail')(Self);
 };

modules/client/front/basic-data/index.js

@@ -9,9 +9,7 @@ export default class Controller extends Section {
     }
 
     onSubmit() {
-        return this.$.watcher.submit().then(() => {
+        return this.$.watcher.submit();
-            this.$http.get(`Clients/${this.$params.id}/checkDuplicatedData`);
-        });
     }
 }
 

modules/client/front/create/index.js

@@ -12,7 +12,6 @@ export default class Controller extends Section {
     onSubmit() {
         return this.$.watcher.submit().then(json => {
             this.$state.go('client.card.basicData', {id: json.data.id});
-            this.$http.get(`Clients/${this.client.id}/checkDuplicatedData`);
         });
     }
 

modules/client/front/defaulter/index.html

@@ -5,6 +5,7 @@
     limit="20"
     order="amount DESC"
     data="defaulters"
+    on-data-change="$ctrl.reCheck()"
     auto-load="true">
 </vn-crud-model>
 <vn-portal slot="topbar">
@@ -17,22 +18,22 @@
     </vn-searchbar>
 </vn-portal>
 <vn-card>
-    <smart-table 
+    <smart-table
         model="model"
-        options="$ctrl.smartTableOptions" 
+        options="$ctrl.smartTableOptions"
         expr-builder="$ctrl.exprBuilder(param, value)">
         <slot-actions>
             <div>
                 <div class="totalBox" style="text-align: center;">
                     <h6 translate>Total</h6>
-                    <vn-label-value 
+                    <vn-label-value
-                        label="Balance due" 
+                        label="Balance due"
                         value="{{$ctrl.balanceDueTotal | currency: 'EUR': 2}}">
                     </vn-label-value>
                 </div>
             </div>
             <div class="vn-pa-md">
-                <vn-button 
+                <vn-button
                     ng-show="$ctrl.checked.length > 0"
                     ng-click="notesDialog.show()"
                     name="notesDialog"
@@ -46,7 +47,7 @@
                 <thead>
                     <tr>
                         <th shrink>
-                            <vn-multi-check 
+                            <vn-multi-check
                                 model="model">
                             </vn-multi-check>
                         </th>
@@ -56,25 +57,25 @@
                         <th field="salesPersonFk">
                             <span translate>Comercial</span>
                         </th>
-                        <th 
+                        <th
-                            field="amount" 
+                            field="amount"
                             vn-tooltip="Balance due">
                             <span translate>Balance D.</span>
                         </th>
-                        <th 
+                        <th
-                            field="workerFk" 
+                            field="workerFk"
                             vn-tooltip="Worker who made the last observation">
                             <span translate>Author</span>
                         </th>
                         <th field="observation" expand>
                             <span translate>Last observation</span>
                         </th>
-                        <th 
+                        <th
                             vn-tooltip="Last observation date"
                             field="created">
                             <span translate>L. O. Date</span>
                         </th>
-                        <th 
+                        <th
                             vn-tooltip="Credit insurance"
                             field="creditInsurance"
                             shrink>
@@ -88,8 +89,9 @@
                 <tbody>
                     <tr ng-repeat="defaulter in defaulters">
                         <td shrink>
-                            <vn-check 
+                            <vn-check
                                 ng-model="defaulter.checked"
+                                on-change="$ctrl.saveChecked(defaulter.clientFk)"
                                 vn-click-stop>
                             </vn-check>
                         </td>
@@ -150,7 +152,7 @@
     </vn-client-summary>
 </vn-popup>
 
-<!-- Dialog of add notes button --> 
+<!-- Dialog of add notes button -->
 <vn-dialog
     vn-id="notesDialog"
     on-accept="$ctrl.onResponse()">
@@ -160,7 +162,7 @@
             <vn-horizontal>
                 <vn-textarea vn-one
                     vn-id="message"
-                    label="Message" 
+                    label="Message"
                     ng-model="$ctrl.defaulter.observation"
                     rows="3"
                     required="true"

modules/client/front/defaulter/index.js

@@ -6,6 +6,7 @@ export default class Controller extends Section {
     constructor($element, $) {
         super($element, $);
         this.defaulter = {};
+        this.checkedDefaulers = [];
 
         this.smartTableOptions = {
             activeButtons: {
@@ -45,11 +46,11 @@ export default class Controller extends Section {
                 },
                 {
                     field: 'created',
-                    searchable: false
+                    datepicker: true
                 },
                 {
                     field: 'defaulterSinced',
-                    searchable: false
+                    datepicker: true
                 }
             ]
         };
@@ -68,6 +69,19 @@ export default class Controller extends Section {
         return checkedLines;
     }
 
+    saveChecked(clientId) {
+        this.checkedDefaulers = this.checkedDefaulers.includes(clientId) ?
+            this.checkedDefaulers.filter(id => id !== clientId) : [...this.checkedDefaulers, clientId];
+    }
+
+    reCheck() {
+        if (!this.$.model.data || !this.checkedDefaulers.length) return;
+
+        this.$.model.data.forEach(defaulter => {
+            defaulter.checked = this.checkedDefaulers.includes(defaulter.clientFk);
+        });
+    }
+
     getBalanceDueTotal() {
         this.$http.get('Defaulters/filter')
             .then(res => {
@@ -109,11 +123,20 @@ export default class Controller extends Section {
         }
 
         this.$http.post(`ClientObservations`, params) .then(() => {
-            this.vnApp.showMessage(this.$t('Observation saved!'));
+            this.vnApp.showSuccess(this.$t('Observation saved!'));
+            this.sendMail();
             this.$state.reload();
         });
     }
 
+    sendMail() {
+        const params = {
+            defaulters: this.checked,
+            observation: this.defaulter.observation
+        };
+        this.$http.post(`Defaulters/observationEmail`, params);
+    }
+
     exprBuilder(param, value) {
         switch (param) {
         case 'creditInsurance':
@@ -122,8 +145,25 @@ export default class Controller extends Section {
         case 'workerFk':
         case 'salesPersonFk':
             return {[`d.${param}`]: value};
+        case 'created':
+            return {'d.created': {
+                between: this.dateRange(value)}
+            };
+        case 'defaulterSinced':
+            return {'d.defaulterSinced': {
+                between: this.dateRange(value)}
+            };
         }
     }
+
+    dateRange(value) {
+        const minHour = new Date(value);
+        minHour.setHours(0, 0, 0, 0);
+        const maxHour = new Date(value);
+        maxHour.setHours(23, 59, 59, 59);
+
+        return [minHour, maxHour];
+    }
 }
 
 ngModule.vnComponent('vnClientDefaulter', {

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

@@ -81,14 +81,15 @@ describe('client defaulter', () => {
 
                 const params = [{text: controller.defaulter.observation, clientFk: data[1].clientFk}];
 
-                jest.spyOn(controller.vnApp, 'showMessage');
+                jest.spyOn(controller.vnApp, 'showSuccess');
                 $httpBackend.expect('GET', `Defaulters/filter`).respond(200);
                 $httpBackend.expect('POST', `ClientObservations`, params).respond(200, params);
+                $httpBackend.expect('POST', `Defaulters/observationEmail`).respond(200);
 
                 controller.onResponse();
                 $httpBackend.flush();
 
-                expect(controller.vnApp.showMessage).toHaveBeenCalledWith('Observation saved!');
+                expect(controller.vnApp.showSuccess).toHaveBeenCalledWith('Observation saved!');
             });
         });
 
@@ -117,5 +118,62 @@ describe('client defaulter', () => {
                 expect(controller.balanceDueTotal).toEqual(875);
             });
         });
+
+        describe('dateRange()', () => {
+            it('should return two dates with the hours at the start and end of the given date', () => {
+                const now = Date.vnNew();
+
+                const today = now.getDate();
+
+                const dateRange = controller.dateRange(now);
+                const start = dateRange[0].toString();
+                const end = dateRange[1].toString();
+
+                expect(start).toContain(today);
+                expect(start).toContain('00:00:00');
+
+                expect(end).toContain(today);
+                expect(end).toContain('23:59:59');
+            });
+        });
+
+        describe('reCheck()', () => {
+            it(`should recheck buys`, () => {
+                controller.$.model.data = [
+                    {checked: false, clientFk: 1},
+                    {checked: false, clientFk: 2},
+                    {checked: false, clientFk: 3},
+                    {checked: false, clientFk: 4},
+                ];
+                controller.checkedDefaulers = [1, 2];
+
+                controller.reCheck();
+
+                expect(controller.$.model.data[0].checked).toEqual(true);
+                expect(controller.$.model.data[1].checked).toEqual(true);
+                expect(controller.$.model.data[2].checked).toEqual(false);
+                expect(controller.$.model.data[3].checked).toEqual(false);
+            });
+        });
+
+        describe('saveChecked()', () => {
+            it(`should check buy`, () => {
+                const buyCheck = 3;
+                controller.checkedDefaulers = [1, 2];
+
+                controller.saveChecked(buyCheck);
+
+                expect(controller.checkedDefaulers[2]).toEqual(buyCheck);
+            });
+
+            it(`should uncheck buy`, () => {
+                const buyUncheck = 3;
+                controller.checkedDefaulers = [1, 2, 3];
+
+                controller.saveChecked(buyUncheck);
+
+                expect(controller.checkedDefaulers[2]).toEqual(undefined);
+            });
+        });
     });
 });

modules/client/front/defaulter/locale/es.yml

@@ -6,4 +6,6 @@ Last observation: Última observación
 L. O. Date: Fecha Ú. O.
 Last observation date: Fecha última observación
 Search client: Buscar clientes
-Worker who made the last observation: Trabajador que ha realizado la última observación
+Worker who made the last observation: Trabajador que ha realizado la última observación
+Email sended!: Email enviado!
+Observation saved!: Observación añadida!

modules/client/front/locale/es.yml

@@ -63,4 +63,4 @@ Consumption: Consumo
 Compensation Account: Cuenta para compensar
 Amount to return: Cantidad a devolver
 Delivered amount: Cantidad entregada
-Unpaid: Impagado
+Unpaid: Impagado

modules/invoiceIn/back/methods/invoice-in/negativeBases.js

@@ -0,0 +1,112 @@
+const UserError = require('vn-loopback/util/user-error');
+const ParameterizedSQL = require('loopback-connector').ParameterizedSQL;
+
+module.exports = Self => {
+    Self.remoteMethodCtx('negativeBases', {
+        description: 'Find all negative bases',
+        accessType: 'READ',
+        accepts: [
+            {
+                arg: 'from',
+                type: 'date',
+                description: 'From date'
+            },
+            {
+                arg: 'to',
+                type: 'date',
+                description: 'To date'
+            },
+            {
+                arg: 'filter',
+                type: 'object',
+                description: 'Filter defining where, order, offset, and limit - must be a JSON-encoded string'
+            },
+        ],
+        returns: {
+            type: ['object'],
+            root: true
+        },
+        http: {
+            path: `/negativeBases`,
+            verb: 'GET'
+        }
+    });
+
+    Self.negativeBases = async(ctx, options) => {
+        const conn = Self.dataSource.connector;
+        const args = ctx.args;
+
+        if (!args.from || !args.to)
+            throw new UserError(`Insert a date range`);
+
+        const myOptions = {};
+
+        if (typeof options == 'object')
+            Object.assign(myOptions, options);
+
+        const stmts = [];
+        let stmt;
+        stmts.push(`DROP TEMPORARY TABLE IF EXISTS tmp.ticket`);
+
+        stmts.push(new ParameterizedSQL(
+            `CREATE TEMPORARY TABLE tmp.ticket
+                (KEY (ticketFk))
+                ENGINE = MEMORY
+                SELECT id ticketFk
+                    FROM ticket t
+                    WHERE shipped BETWEEN ? AND ?
+                        AND refFk IS NULL`, [args.from, args.to]));
+        stmts.push(`CALL vn.ticket_getTax(NULL)`);
+        stmts.push(`DROP TEMPORARY TABLE IF EXISTS tmp.filter`);
+        stmts.push(new ParameterizedSQL(
+            `CREATE TEMPORARY TABLE tmp.filter
+                ENGINE = MEMORY
+                SELECT
+                        co.code company,
+                        cou.country,
+                        c.id clientId,
+                        c.socialName clientSocialName,
+                        SUM(s.quantity * s.price * ( 100 - s.discount ) / 100) amount,
+                        negativeBase.taxableBase,
+                        negativeBase.ticketFk,
+                        c.isActive,
+                        c.hasToInvoice,
+                        c.isTaxDataChecked,
+                        w.id comercialId,
+                        CONCAT(w.firstName, ' ', w.lastName) comercialName
+                        FROM vn.ticket t
+                            JOIN vn.company co ON co.id = t.companyFk
+                            JOIN vn.sale s ON s.ticketFk = t.id
+                            JOIN vn.client c ON c.id = t.clientFk
+                            JOIN vn.country cou ON cou.id = c.countryFk
+                            LEFT JOIN vn.worker w ON w.id = c.salesPersonFk
+                            LEFT JOIN (
+                                SELECT ticketFk, taxableBase
+                                    FROM tmp.ticketAmount
+                                    GROUP BY ticketFk
+                                    HAVING taxableBase < 0
+                            ) negativeBase ON negativeBase.ticketFk = t.id
+                        WHERE t.shipped BETWEEN ? AND ?
+                            AND t.refFk IS NULL
+                            AND c.typeFk IN ('normal','trust')
+                        GROUP BY t.clientFk, negativeBase.taxableBase
+                        HAVING amount <> 0`, [args.from, args.to]));
+
+        stmt = new ParameterizedSQL(`
+            SELECT f.*
+            FROM tmp.filter f`);
+
+        stmt.merge(conn.makeWhere(args.filter.where));
+        stmt.merge(conn.makeOrderBy(args.filter.order));
+
+        const negativeBasesIndex = stmts.push(stmt) - 1;
+
+        stmts.push(`DROP TEMPORARY TABLE tmp.filter, tmp.ticket, tmp.ticketTax, tmp.ticketAmount`);
+
+        const sql = ParameterizedSQL.join(stmts, ';');
+        const result = await conn.executeStmt(sql, myOptions);
+
+        return negativeBasesIndex === 0 ? result : result[negativeBasesIndex];
+    };
+};
+

modules/invoiceIn/back/methods/invoice-in/negativeBasesCsv.js

@@ -0,0 +1,53 @@
+const {toCSV} = require('vn-loopback/util/csv');
+
+module.exports = Self => {
+    Self.remoteMethodCtx('negativeBasesCsv', {
+        description: 'Returns the negative bases as .csv',
+        accessType: 'READ',
+        accepts: [{
+            arg: 'negativeBases',
+            type: ['object'],
+            required: true
+        },
+        {
+            arg: 'from',
+            type: 'date',
+            description: 'From date'
+        },
+        {
+            arg: 'to',
+            type: 'date',
+            description: 'To date'
+        }],
+        returns: [
+            {
+                arg: 'body',
+                type: 'file',
+                root: true
+            }, {
+                arg: 'Content-Type',
+                type: 'String',
+                http: {target: 'header'}
+            }, {
+                arg: 'Content-Disposition',
+                type: 'String',
+                http: {target: 'header'}
+            }
+        ],
+        http: {
+            path: '/negativeBasesCsv',
+            verb: 'GET'
+        }
+    });
+
+    Self.negativeBasesCsv = async ctx => {
+        const args = ctx.args;
+        const content = toCSV(args.negativeBases);
+
+        return [
+            content,
+            'text/csv',
+            `attachment; filename="negative-bases-${new Date(args.from).toLocaleDateString()}-${new Date(args.to).toLocaleDateString()}.csv"`
+        ];
+    };
+};

modules/invoiceIn/back/methods/invoice-in/specs/negativeBases.spec.js

@@ -0,0 +1,47 @@
+const models = require('vn-loopback/server/server').models;
+
+describe('invoiceIn negativeBases()', () => {
+    it('should return all negative bases in a date range', async() => {
+        const tx = await models.InvoiceIn.beginTransaction({});
+        const options = {transaction: tx};
+        const ctx = {
+            args: {
+                from: new Date().setMonth(new Date().getMonth() - 12),
+                to: new Date(),
+                filter: {}
+            }
+        };
+
+        try {
+            const result = await models.InvoiceIn.negativeBases(ctx, options);
+
+            expect(result.length).toBeGreaterThan(0);
+
+            await tx.rollback();
+        } catch (e) {
+            await tx.rollback();
+            throw e;
+        }
+    });
+
+    it('should throw an error if a date range is not in args', async() => {
+        let error;
+        const tx = await models.InvoiceIn.beginTransaction({});
+        const options = {transaction: tx};
+        const ctx = {
+            args: {
+                filter: {}
+            }
+        };
+
+        try {
+            await models.InvoiceIn.negativeBases(ctx, options);
+            await tx.rollback();
+        } catch (e) {
+            error = e;
+            await tx.rollback();
+        }
+
+        expect(error.message).toEqual(`Insert a date range`);
+    });
+});

modules/invoiceIn/back/models/invoice-in.js

@@ -7,4 +7,6 @@ module.exports = Self => {
     require('../methods/invoice-in/invoiceInPdf')(Self);
     require('../methods/invoice-in/invoiceInEmail')(Self);
     require('../methods/invoice-in/getSerial')(Self);
+    require('../methods/invoice-in/negativeBases')(Self);
+    require('../methods/invoice-in/negativeBasesCsv')(Self);
 };

modules/invoiceIn/front/index.js

@@ -15,3 +15,4 @@ import './create';
 import './log';
 import './serial';
 import './serial-search-panel';
+import './negative-bases';

modules/invoiceIn/front/locale/es.yml

@@ -24,3 +24,4 @@ Show agricultural receipt as PDF: Ver recibo agrícola como PDF
 Send agricultural receipt as PDF: Enviar recibo agrícola como PDF
 New InvoiceIn: Nueva Factura
 Days ago: Últimos días
+Negative bases: Bases negativas

modules/invoiceIn/front/negative-bases/index.html

@@ -0,0 +1,133 @@
+<vn-crud-model
+    vn-id="model"
+    url="InvoiceIns/negativeBases"
+    auto-load="true"
+    params="$ctrl.params">
+</vn-crud-model>
+<vn-portal slot="topbar">
+</vn-portal>
+<vn-card>
+    <smart-table
+        model="model"
+        options="$ctrl.smartTableOptions"
+        expr-builder="$ctrl.exprBuilder(param, value)">
+        <slot-actions>
+            <vn-date-picker
+            vn-one
+            label="From"
+            ng-model="$ctrl.params.from"
+            on-change="model.refresh()">
+        </vn-date-picker>
+        <vn-date-picker
+            vn-one
+            label="To"
+            ng-model="$ctrl.params.to"
+            on-change="model.refresh()">
+        </vn-date-picker>
+        <vn-button
+            disabled="model._orgData.length == 0"
+            icon="download"
+            ng-click="$ctrl.downloadCSV()"
+            vn-tooltip="Download as CSV">
+        </vn-button>
+        </slot-actions>
+        <slot-table>
+            <table>
+                <thead>
+                    <tr>
+                        <th field="company">
+                            <span translate>Company</span>
+                        </th>
+                        <th field="country">
+                            <span translate>Country</span>
+                        </th>
+                        <th field="clientId">
+                            <span translate>Id Client</span>
+                        </th>
+                        <th field="clientSocialName">
+                            <span translate>Client</span>
+                        </th>
+                        <th field="amount">
+                            <span translate>Amount</span>
+                        </th>
+                        <th field="taxableBase">
+                            <span translate>Base</span>
+                        </th>
+                        <th field="ticketFk">
+                            <span translate>Id Ticket</span>
+                        </th>
+                        <th field="isActive">
+                            <span translate>Active</span>
+                        </th>
+                        <th field="hasToInvoice">
+                            <span translate>Has To Invoice</span>
+                        </th>
+                        <th field="isTaxDataChecked">
+                            <span translate>Verified data</span>
+                        </th>
+                        <th field="comercialName">
+                            <span translate>Comercial</span>
+                        </th>
+                    </tr>
+                </thead>
+                <tbody>
+                    <tr ng-repeat="client in model.data">
+                        <td>{{client.company | dashIfEmpty}}</td>
+                        <td>{{client.country | dashIfEmpty}}</td>
+                        <td>
+                            <vn-span
+                                class="link"
+                                ng-click="clientDescriptor.show($event, client.clientId)">
+                                {{::client.clientId | dashIfEmpty}}
+                            </vn-span>
+                        </td>
+                        <td>{{client.clientSocialName | dashIfEmpty}}</td>
+                        <td>{{client.amount | currency: 'EUR':2 | dashIfEmpty}}</td>
+                        <td>{{client.taxableBase | dashIfEmpty}}</td>
+                        <td>
+                            <vn-span
+                                class="link"
+                                ng-click="ticketDescriptor.show($event, client.ticketFk)">
+                                {{::client.ticketFk | dashIfEmpty}}
+                            </vn-span>
+                        </td>
+                        <td>
+                            <vn-check
+                                disabled="true"
+                                ng-model="client.isActive">
+                            </vn-check>
+                        </td>
+                        <td>
+                            <vn-check
+                                disabled="true"
+                                ng-model="client.hasToInvoice">
+                            </vn-check>
+                        </td>
+                        <td>
+                            <vn-check
+                                disabled="true"
+                                ng-model="client.isTaxDataChecked">
+                            </vn-check>
+                        </td>
+                        <td>
+                            <vn-span
+                                class="link"
+                                ng-click="workerDescriptor.show($event, client.comercialId)">
+                                {{::client.comercialName | dashIfEmpty}}
+                            </vn-span>
+                        </td>
+                    </tr>
+                </tbody>
+            </table>
+        </slot-table>
+    </smart-table>
+</vn-card>
+<vn-ticket-descriptor-popover
+    vn-id="ticket-descriptor">
+</vn-ticket-descriptor-popover>
+<vn-client-descriptor-popover
+    vn-id="client-descriptor">
+</vn-client-descriptor-popover>
+<vn-worker-descriptor-popover
+    vn-id="worker-descriptor">
+</vn-worker-descriptor-popover>

modules/invoiceIn/front/negative-bases/index.js

@@ -0,0 +1,84 @@
+import ngModule from '../module';
+import Section from 'salix/components/section';
+import './style.scss';
+
+export default class Controller extends Section {
+    constructor($element, $, vnReport) {
+        super($element, $);
+
+        this.vnReport = vnReport;
+        const now = new Date();
+        const firstDayOfMonth = new Date(now.getFullYear(), now.getMonth(), 1);
+        const lastDayOfMonth = new Date(now.getFullYear(), now.getMonth() + 1, 0);
+        this.params = {
+            from: firstDayOfMonth,
+            to: lastDayOfMonth
+        };
+        this.$checkAll = false;
+
+        this.smartTableOptions = {
+            activeButtons: {
+                search: true,
+            }, columns: [
+                {
+                    field: 'isActive',
+                    searchable: false
+                },
+                {
+                    field: 'hasToInvoice',
+                    searchable: false
+                },
+                {
+                    field: 'isTaxDataChecked',
+                    searchable: false
+                },
+            ]
+        };
+    }
+
+    exprBuilder(param, value) {
+        switch (param) {
+        case 'company':
+            return {'company': value};
+        case 'country':
+            return {'country': value};
+        case 'clientId':
+            return {'clientId': value};
+        case 'clientSocialName':
+            return {'clientSocialName': value};
+        case 'amount':
+            return {'amount': value};
+        case 'taxableBase':
+            return {'taxableBase': value};
+        case 'ticketFk':
+            return {'ticketFk': value};
+        case 'comercialName':
+            return {'comercialName': value};
+        }
+    }
+
+    downloadCSV() {
+        const data = [];
+        this.$.model._orgData.forEach(element => {
+            data.push(Object.keys(element).map(key => {
+                return {newName: this.$t(key), value: element[key]};
+            }).filter(item => item !== null)
+                .reduce((result, item) => {
+                    result[item.newName] = item.value;
+                    return result;
+                }, {}));
+        });
+        this.vnReport.show('InvoiceIns/negativeBasesCsv', {
+            negativeBases: data,
+            from: this.params.from,
+            to: this.params.to
+        });
+    }
+}
+
+Controller.$inject = ['$element', '$scope', 'vnReport'];
+
+ngModule.vnComponent('vnNegativeBases', {
+    template: require('./index.html'),
+    controller: Controller
+});

modules/invoiceIn/front/negative-bases/locale/es.yml

@@ -0,0 +1,14 @@
+Has To Invoice: Facturar
+Download as CSV: Descargar como CSV
+company: Compañía
+country: País
+clientId: Id Cliente
+clientSocialName: Cliente
+amount: Importe
+taxableBase: Base
+ticketFk: Id Ticket
+isActive: Activo
+hasToInvoice: Facturar
+isTaxDataChecked: Datos comprobados
+comercialId: Id Comercial
+comercialName: Comercial

modules/invoiceIn/front/negative-bases/style.scss

@@ -0,0 +1,10 @@
+@import "./variables";
+
+vn-negative-bases {
+     vn-date-picker{
+        padding-right: 5%;
+    }
+    slot-actions{
+        align-items: center;
+    }
+}

modules/invoiceIn/front/routes.json

@@ -9,14 +9,9 @@
     ],
     "menus": {
         "main": [
-            {
+            { "state": "invoiceIn.index", "icon": "icon-invoice-in"},
-                "state": "invoiceIn.index",
+            { "state": "invoiceIn.serial", "icon": "icon-invoice-in"},
-                "icon": "icon-invoice-in"
+            { "state": "invoiceIn.negative-bases", "icon": "icon-ticket"}
-            },
-            {
-                "state": "invoiceIn.serial",
-                "icon": "icon-invoice-in"
-            }
         ],
         "card": [
             {
@@ -58,6 +53,15 @@
                 "administrative"
             ]
         },
+        {
+            "url": "/negative-bases",
+            "state": "invoiceIn.negative-bases",
+            "component": "vn-negative-bases",
+            "description": "Negative bases",
+            "acl": [
+                "administrative"
+            ]
+        },
         {
             "url": "/serial",
             "state": "invoiceIn.serial",

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

@@ -1,7 +1,7 @@
 const axios = require('axios');
 const uuid = require('uuid');
 const fs = require('fs/promises');
-const { createWriteStream } = require('fs');
+const {createWriteStream} = require('fs');
 const path = require('path');
 const gm = require('gm');
 
@@ -15,7 +15,7 @@ module.exports = Self => {
         },
     });
 
-    Self.download = async () => {
+    Self.download = async() => {
         const models = Self.app.models;
         const tempContainer = await models.TempContainer.container(
             'salix-image'
@@ -32,13 +32,13 @@ module.exports = Self => {
         let tempFilePath;
         let queueRow;
         try {
-            const myOptions = { transaction: tx };
+            const myOptions = {transaction: tx};
 
             queueRow = await Self.findOne(
                 {
                     fields: ['id', 'itemFk', 'url', 'attempts'],
                     where: {
-                        url: { neq: null },
+                        url: {neq: null},
                         attempts: {
                             lt: maxAttempts,
                         },
@@ -59,7 +59,7 @@ module.exports = Self => {
                         'model',
                         'property',
                     ],
-                    where: { name: collectionName },
+                    where: {name: collectionName},
                     include: {
                         relation: 'sizes',
                         scope: {
@@ -116,16 +116,16 @@ module.exports = Self => {
             const collectionDir = path.join(rootPath, collectionName);
 
             // To max size
-            const { maxWidth, maxHeight } = collection;
+            const {maxWidth, maxHeight} = collection;
             const fullSizePath = path.join(collectionDir, 'full');
             const toFullSizePath = `${fullSizePath}/${fileName}`;
 
-            await fs.mkdir(fullSizePath, { recursive: true });
+            await fs.mkdir(fullSizePath, {recursive: true});
             await new Promise((resolve, reject) => {
                 gm(tempFilePath)
                     .resize(maxWidth, maxHeight, '>')
                     .setFormat('png')
-                    .write(toFullSizePath, function (err) {
+                    .write(toFullSizePath, function(err) {
                         if (err) reject(err);
                         if (!err) resolve();
                     });
@@ -133,12 +133,12 @@ module.exports = Self => {
 
             // To collection sizes
             for (const size of collection.sizes()) {
-                const { width, height } = size;
+                const {width, height} = size;
 
                 const sizePath = path.join(collectionDir, `${width}x${height}`);
                 const toSizePath = `${sizePath}/${fileName}`;
 
-                await fs.mkdir(sizePath, { recursive: true });
+                await fs.mkdir(sizePath, {recursive: true});
                 await new Promise((resolve, reject) => {
                     const gmInstance = gm(tempFilePath);
 
@@ -153,7 +153,7 @@ module.exports = Self => {
 
                     gmInstance
                         .setFormat('png')
-                        .write(toSizePath, function (err) {
+                        .write(toSizePath, function(err) {
                             if (err) reject(err);
                             if (!err) resolve();
                         });

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

@@ -1,105 +0,0 @@
-const https = require('https');
-const fs = require('fs-extra');
-const path = require('path');
-const uuid = require('uuid');
-
-module.exports = Self => {
-    Self.remoteMethod('downloadImages', {
-        description: 'Returns last entries',
-        accessType: 'WRITE',
-        returns: {
-            type: ['Object'],
-            root: true
-        },
-        http: {
-            path: `/downloadImages`,
-            verb: 'POST'
-        }
-    });
-
-    Self.downloadImages = async() => {
-        const models = Self.app.models;
-        const container = await models.TempContainer.container('salix-image');
-        const tempPath = path.join(container.client.root, container.name);
-        const maxAttempts = 3;
-
-        const images = await Self.find({
-            where: {attempts: {eq: maxAttempts}}
-        });
-
-        for (let image of images) {
-            const currentStamp = Date.vnNew().getTime();
-            const updatedStamp = image.updated.getTime();
-            const graceTime = Math.abs(currentStamp - updatedStamp);
-            const maxTTL = 3600 * 48 * 1000; // 48 hours in ms;
-
-            if (graceTime >= maxTTL)
-                await Self.destroyById(image.itemFk);
-        }
-
-        download();
-
-        async function download() {
-            const image = await Self.findOne({
-                where: {url: {neq: null}, attempts: {lt: maxAttempts}},
-                order: 'priority, attempts, updated'
-            });
-
-            if (!image) return;
-
-            const fileName = `${uuid.v4()}.png`;
-            const filePath = path.join(tempPath, fileName);
-            const imageUrl = image.url.replace('http://', 'https://');
-
-            https.get(imageUrl, async response => {
-                if (response.statusCode != 200) {
-                    const error = new Error(`Could not download the image. Status code ${response.statusCode}`);
-
-                    return await errorHandler(image.itemFk, error, filePath);
-                }
-
-                const writeStream = fs.createWriteStream(filePath);
-                writeStream.on('open', () => response.pipe(writeStream));
-                writeStream.on('error', async error =>
-                    await errorHandler(image.itemFk, error, filePath));
-                writeStream.on('finish', () => writeStream.end());
-
-                writeStream.on('close', async function() {
-                    try {
-                        await models.Image.registerImage('catalog', filePath, fileName, image.itemFk);
-                        await image.destroy();
-
-                        download();
-                    } catch (error) {
-                        await errorHandler(image.itemFk, error, filePath);
-                    }
-                });
-            }).on('error', async error => {
-                await errorHandler(image.itemFk, error, filePath);
-            });
-        }
-
-        async function errorHandler(rowId, error, filePath) {
-            try {
-                const row = await Self.findById(rowId);
-
-                if (!row) return;
-
-                if (row.attempts < maxAttempts) {
-                    await row.updateAttributes({
-                        error: error,
-                        attempts: row.attempts + 1,
-                        updated: Date.vnNew()
-                    });
-                }
-
-                if (filePath && fs.existsSync(filePath))
-                    await fs.unlink(filePath);
-
-                download();
-            } catch (err) {
-                throw new Error(`Image download failed: ${err}`);
-            }
-        }
-    };
-};

modules/item/front/request-search-panel/index.html

@@ -38,6 +38,19 @@
                 url="Warehouses">
             </vn-autocomplete>
         </vn-horizontal>
+        <vn-horizontal class="vn-px-lg">
+            <vn-autocomplete
+                vn-one
+                ng-model="filter.requesterFk"
+                url="Workers/activeWithRole"
+                search-function="{firstName: $search}"
+                value-field="id"
+                where="{role: 'salesPerson'}"
+                label="Comercial">
+                <tpl-item>{{firstName}} {{lastName}}</tpl-item>
+            </vn-autocomplete>
+        </vn-horizontal>
+       
     <section class="vn-px-md">
         <vn-horizontal class="manifold-panel vn-pa-md">
             <vn-date-picker

modules/supplier/front/agency-term/locale/es.yml

@@ -5,4 +5,5 @@ M3 Price: Precio M3
 Route Price: Precio ruta
 Minimum Km: Km minimos
 Remove row: Eliminar fila
-Add row: Añadir fila
+Add row: Añadir fila
+New autonomous: Nuevo autónomo

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

@@ -37,6 +37,10 @@ module.exports = Self => {
                 type: 'number',
                 description: `Search requests attended by a given worker id`
             },
+            {
+                arg: 'requesterFk',
+                type: 'number'
+            },
             {
                 arg: 'mine',
                 type: 'boolean',
@@ -89,6 +93,8 @@ module.exports = Self => {
                 return {'t.id': value};
             case 'attenderFk':
                 return {'tr.attenderFk': value};
+            case 'requesterFk':
+                return {'tr.requesterFk': value};
             case 'state':
                 switch (value) {
                 case 'pending':
@@ -125,6 +131,7 @@ module.exports = Self => {
                 tr.description,
                 tr.response,
                 tr.saleFk,
+                tr.requesterFk,
                 tr.isOk,
                 s.quantity AS saleQuantity,
                 s.itemFk,

package.json

@@ -1,6 +1,6 @@
 {
     "name": "salix-back",
-    "version": "23.12.01",
+    "version": "23.14.01",
     "author": "Verdnatura Levante SL",
     "description": "Salix backend",
     "license": "GPL-3.0",