verdnatura/salix
verdnatura
/
salix
3
5
Fork 0
Code Issues Pull Requests 64 Releases Wiki Activity

#7648 - Customer View Entries #2659

Merged
jsegarra merged 17 commits from 7648_dev_customerEntries into dev 2024-07-03 07:29:02 +00:00
Conversation 0 Commits 17 Files Changed 8 +6 -1
1 changed files with 6 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit f0326860ba - Show all commits

7
modules/entry/back/methods/entry/filter.js
View File

@ -177,10 +177,15 @@ module.exports = Self => {
s.name AS supplierName,
s.nickname AS supplierAlias,
co.code AS companyCode,
cu.code AS currencyCode
cu.code AS currencyCode,
jsegarra marked this conversation as resolved Outdated
jgallego commented 2024-07-03 05:39:22 +00:00
Outdated
Review
Copy Link

por convencion va sin AS
cu.code currencyCode,
esta linea y las demás tb

por convencion va sin AS `cu.code currencyCode,` esta linea y las demás tb
jsegarra commented 2024-07-03 06:17:28 +00:00
Outdated
Review
Copy Link

946d6f553a

https://gitea.verdnatura.es/verdnatura/salix/commit/946d6f553af0be220e5b9d882397c6fda4252daf
t.shipped AS shipped,
t.landed AS landed,
t.warehouseInFk AS warehouseId,
jsegarra marked this conversation as resolved Outdated
jgallego commented 2024-07-02 16:05:06 +00:00
Outdated
Review
Copy Link

se podria devolver el warehouseInFk tal cual sin el alias..lo apruebo porque no es un fallo, pero si lo ves mas conveniente lo cambias para la proxima

se podria devolver el warehouseInFk tal cual sin el alias..lo apruebo porque no es un fallo, pero si lo ves mas conveniente lo cambias para la proxima
jsegarra commented 2024-07-02 18:41:04 +00:00
Outdated
Review
Copy Link

Lo cambio porque tampoco es correcto que t.shipped y t.landed lleven alias.
Además, queremos mostrar el almacén de entrada así que el alias de warehouseName no es correcto, debería ser warehouseInName,
Puede llevar a confusión, así que lo cambio

Lo cambio porque tampoco es correcto que t.shipped y t.landed lleven alias. Además, queremos mostrar el almacén de entrada así que el alias de warehouseName no es correcto, debería ser warehouseInName, Puede llevar a confusión, así que lo cambio
w.name AS warehouseName
FROM vn.entry e
JOIN vn.supplier s ON s.id = e.supplierFk
JOIN vn.travel t ON t.id = e.travelFk
JOIN vn.warehouse w ON w.id = t.warehouseInFk
JOIN vn.company co ON co.id = e.companyFk
JOIN vn.currency cu ON cu.id = e.currencyFk`
);