verdnatura/salix
verdnatura
/
salix
3
5
Fork 2
Code Issues Pull Requests 70 Releases Wiki Activity

#6321 - Negative tickets #1945

Merged
jsegarra merged 146 commits from 6321_negative_tickets into dev 2025-02-11 08:45:33 +00:00
Conversation 0 Commits 146 Files Changed 27 +12 -17
1 changed files with 12 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit ed6b25455b - Show all commits

29
modules/ticket/back/methods/ticket/itemLackDetail.js
View File

@ -53,29 +53,24 @@ module.exports = Self => {
t.hour minTimed t.hour minTimed
FROM FROM
vn.sale s vn.sale s
jsegarra marked this conversation as resolved Outdated
jgallego commented 2025-01-31 14:30:24 +00:00
Outdated
Review
Copy Link

es una chorrada pero comovas a cambiar cosas, no solemos poner los AS en los select

es una chorrada pero comovas a cambiar cosas, no solemos poner los AS en los select
JOIN vn.ticket t ON t.id=s.ticketFk -- vn.ticket JOIN vn.ticket t ON t.id=s.ticketFk
jsegarra marked this conversation as resolved Outdated
jgallego commented 2025-02-06 11:59:13 +00:00
Outdated
Review
Copy Link

tls.alertLevel

tls.alertLevel
LEFT JOIN vn.zone z ON z.id = t.zoneFk -- vn.zone LEFT JOIN vn.zone z ON z.id = t.zoneFk
LEFT JOIN vn.zoneClosure zc ON zc.zoneFk = t.zoneFk -- AND zc.dated = DateValue(t.Fecha) LEFT JOIN vn.zoneClosure zc ON zc.zoneFk = t.zoneFk
JOIN vn.client c ON c.id=t.clientFk -- vn.client JOIN vn.client c ON c.id=t.clientFk
LEFT JOIN bs.clientNewBorn cn ON cn.clientFk=c.id LEFT JOIN bs.clientNewBorn cn ON cn.clientFk=c.id
JOIN vn.agencyMode agm ON agm.id=t.agencyModeFk -- vn.agencyMode JOIN vn.agencyMode agm ON agm.id=t.agencyModeFk
jgallego marked this conversation as resolved Outdated
jgallego commented 2025-02-05 06:37:33 +00:00
Outdated
Review
Copy Link

aqui puedes devolver el codigo y en front gestionas como necesites, sino estas haciendo 2 funciones por cada linea..lo cual tiene un coste en velocidad al select

aqui puedes devolver el codigo y en front gestionas como necesites, sino estas haciendo 2 funciones por cada linea..lo cual tiene un coste en velocidad al select
jsegarra commented 2025-02-05 14:43:33 +00:00
Outdated
Review
Copy Link

cual select? al de zona? El siguiente es de hora y a continuación son booleanos

cual select? al de zona? El siguiente es de hora y a continuación son booleanos
jsegarra commented 2025-02-06 11:42:53 +00:00
Outdated
Review
Copy Link

cambiado

cambiado
JOIN vn.agency ag ON ag.id=agm.id -- vn.agencyMode JOIN vn.agency ag ON ag.id=agm.id
JOIN vn.ticketState ts ON ts.ticketFk=t.id -- vn.sale JOIN vn.ticketState ts ON ts.ticketFk=t.id
LEFT JOIN vn.state st ON st.id=ts.state LEFT JOIN vn.state st ON st.id=ts.state
LEFT JOIN vn.alertLevel al ON al.id = st.alertLevel LEFT JOIN vn.alertLevel al ON al.id = st.alertLevel
LEFT JOIN vn.saleCloned sc ON sc.saleClonedFk = s.id LEFT JOIN vn.saleCloned sc ON sc.saleClonedFk = s.id
jgallego marked this conversation as resolved Outdated
jgallego commented 2025-02-05 06:39:39 +00:00
Outdated
Review
Copy Link

esto lo confirmamos con ellos, yo intentaria llevarlo a front sin este campo hasToIgnore y que se lo filtren, de lo contrario..mañana cuando cambien codigos esto habra que cambiarlo

esto lo confirmamos con ellos, yo intentaria llevarlo a front sin este campo hasToIgnore y que se lo filtren, de lo contrario..mañana cuando cambien codigos esto habra que cambiarlo
jsegarra commented 2025-02-05 14:41:33 +00:00
Outdated
Review
Copy Link

La consulta que estas viendo está copiada y traducida desde access
Solo la he migrado.
Este campo sirve para ordenar los resultados

La consulta que estas viendo está copiada y traducida desde access Solo la he migrado. Este campo sirve para ordenar los resultados
jsegarra commented 2025-02-06 11:42:47 +00:00
Outdated
Review
Copy Link

cambiado

cambiado
LEFT JOIN vn.ticketRequest tr ON tr.saleFk = s.id LEFT JOIN vn.ticketRequest tr ON tr.saleFk = s.id
LIMIT 1 WHERE
s.itemFk = ?
AND t.landed >= util.VN_CURDATE()
AND t.landed < util.VN_CURDATE() + INTERVAL ? + 1 DAY
jsegarra marked this conversation as resolved Outdated
jgallego commented 2025-02-06 12:00:21 +00:00
Outdated
Review
Copy Link

(d.code = 'spainTeamVip') hasToIgnore

(d.code = 'spainTeamVip') hasToIgnore
`, `,
jgallego marked this conversation as resolved
jgallego commented 2025-02-05 06:40:54 +00:00
Review
Copy Link

esto hace que las consultas sean muy lentas, hay que buscar alternativa, busca si encuentras otra consulta similar por la BBDD

esto hace que las consultas sean muy lentas, hay que buscar alternativa, busca si encuentras otra consulta similar por la BBDD
😕 1
jsegarra commented 2025-02-06 11:42:40 +00:00
Review
Copy Link

cambiado

cambiado
null [id, 2]);
);
// WHERE
// s.itemFk = ?
// AND t.landed >= util.VN_CURDATE()
// AND t.landed < INTERVAL util.VN_CURDATE() + INTERVAL ? + 1 DAY
// ORDER BY s.id DESC+
// [id,2]
const sql = ParameterizedSQL.join([stmt], ';'); const sql = ParameterizedSQL.join([stmt], ';');
const result = await conn.executeStmt(sql, myOptions); const result = await conn.executeStmt(sql, myOptions);