verdnatura/salix
verdnatura
/
salix
3
4
Fork 0
Code Issues Pull Requests 59 Releases Wiki Activity

WIP: fix: refs #7906 remake method #3079

Draft
carlossa wants to merge 1 commits from 7906-remakeMethod into dev
pull from: 7906-remakeMethod
merge into: verdnatura:dev
Conversation 0 Commits 1 Files Changed 1 +18 -15
1 changed files with 18 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 520346d4a5 - Show all commits

33
modules/zone/back/methods/zone/deleteZone.js
View File

@ -1,3 +1,4 @@
const UserError = require('vn-loopback/util/user-error');
module.exports = Self => { module.exports = Self => {
Self.remoteMethodCtx('deleteZone', { Self.remoteMethodCtx('deleteZone', {
description: 'Delete a zone', description: 'Delete a zone',
@ -52,23 +53,25 @@ module.exports = Self => {
const promises = []; const promises = [];
const ticketList = await models.Ticket.find(filter, myOptions); const ticketList = await models.Ticket.find(filter, myOptions);
const fixingState = await models.State.findOne({where: {code: 'FIXING'}}, myOptions); // const fixingState = await models.State.findOne({where: {code: 'FIXING'}}, myOptions);
const worker = await models.Worker.findOne({ // const worker = await models.Worker.findOne({
where: {id: userId} // where: {id: userId}
}, myOptions); // }, myOptions);
if (ticketList.length > 0)
throw new UserError('You cannot delete a zone that is being used');
await models.Ticket.rawSql('UPDATE ticket SET zoneFk = NULL WHERE zoneFk = ?', [id], myOptions); // await models.Ticket.rawSql('UPDATE ticket SET zoneFk = NULL WHERE zoneFk = ?', [id], myOptions);
for (ticket of ticketList) { // for (ticket of ticketList) {
if (ticket.ticketState().alertLevel == 0) { // if (ticket.ticketState().alertLevel == 0) {
promises.push(models.Ticket.state(ctx, { // promises.push(models.Ticket.state(ctx, {
ticketFk: ticket.id, // ticketFk: ticket.id,
stateFk: fixingState.id, // stateFk: fixingState.id,
userFk: worker.id // userFk: worker.id
}, myOptions)); // }, myOptions));
} // }
} // }
await Promise.all(promises); // await Promise.all(promises);
await models.Zone.destroyById(id, myOptions); await models.Zone.destroyById(id, myOptions);
if (tx) await tx.commit(); if (tx) await tx.commit();