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

#6964 _ itemOlder #2197

Merged
sergiodt merged 26 commits from 6964_itemOlder into dev 2024-04-04 14:18:20 +00:00
Conversation 0 Commits 26 Files Changed 5 +28 -15
1 changed files with 28 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 027aa86662 - Show all commits

43
modules/item/back/methods/item-shelving/hasItemOlder.js
View File

@ -1,9 +1,10 @@
module.exports = Self => {
Self.remoteMethod('hasItemOlder', {
description: 'Get boolean if any item of the shelving has older created in another shelving ',
description:
'Get boolean if any or specific item of the shelving has older created in another shelving or parking',
accessType: 'WRITE',
accepts: [{
arg: 'shelvingFk',
arg: 'shelvingFkIn',
type: 'string',
required: true,
description: 'Shelving code'
@ -11,8 +12,17 @@ module.exports = Self => {
{
arg: 'parking',
type: 'string',
required: true,
description: 'Parking code'
},
{
arg: 'shelvingFkOut',
type: 'string',
description: 'Shelving code'
},
{
arg: 'itemFk',
type: 'int',
description: 'Item id'
sergiodt marked this conversation as resolved Outdated
alexm commented 2024-03-27 06:32:29 +00:00
Outdated
Review
Copy Link

He buscat y type int no ho hem gastat mai,
Seria type: 'integer',

He buscat y type int no ho hem gastat mai, Seria `type: 'integer',`
sergiodt commented 2024-03-27 06:50:36 +00:00
Outdated
Review
Copy Link

vaig mesclant llenguatges :-)

vaig mesclant llenguatges :-)
}],
returns: {
type: 'boolean',
@ -24,21 +34,24 @@ module.exports = Self => {
}
});
Self.hasItemOlder = async(shelvingFk, parking, options) => {
Self.hasItemOlder = async(shelvingFkIn, parking, shelvingFkOut, itemFk, options) => {
sergiodt marked this conversation as resolved Outdated
alexm commented 2024-03-25 06:10:53 +00:00
Outdated
Review
Copy Link

Els 3(parking, shelvingFkOut, itemFk) poden ser null a la vega? O faria falta al menys uno?
Per si fera falta ficar un if(!parking && !shelvingFkOut && !itemFk) return throw new UserError(...)

Els 3(parking, shelvingFkOut, itemFk) poden ser null a la vega? O faria falta al menys uno? Per si fera falta ficar un if(!parking && !shelvingFkOut && !itemFk) return throw new UserError(...)
sergiodt commented 2024-03-25 06:55:40 +00:00
Outdated
Review
Copy Link

Posat

Posat
const result = await Self.rawSql(`
sergiodt marked this conversation as resolved
alexm commented 2024-03-26 07:02:51 +00:00
Review
Copy Link
    const myOptions = {};

    if (typeof options == 'object')
        Object.assign(myOptions, options);
const myOptions = {}; if (typeof options == 'object') Object.assign(myOptions, options);
SELECT COUNT(ish.id)
sergiodt marked this conversation as resolved Outdated
carlosap commented 2024-03-22 15:34:37 +00:00
Outdated
Review
Copy Link

Millor fiquem un alias al cam

Millor fiquem un alias al cam
alexm commented 2024-03-25 12:17:49 +00:00
Review
Copy Link

Faltaria la traduccion

Faltaria la traduccion
sergiodt commented 2024-03-25 15:10:48 +00:00
Review
Copy Link

Afegida

Afegida
FROM vn.itemShelving ish
JOIN (
SELECT ish.itemFk, created,shelvingFk, p.code
FROM vn.itemShelving ish
JOIN vn.shelving s ON ish.shelvingFk = s.code
JOIN vn.parking p ON p.id = s.parkingFk
FROM vn.itemShelving ish
sergiodt marked this conversation as resolved Outdated
carlosap commented 2024-03-22 15:35:10 +00:00
Outdated
Review
Copy Link

Tabulació

Tabulació
JOIN (
SELECT ish.itemFk, created,shelvingFk, p.code
FROM vn.itemShelving ish
JOIN vn.shelving s ON ish.shelvingFk = s.code
LEFT JOIN vn.parking p ON p.id = s.parkingFk
WHERE ish.shelvingFk = ?
)sub ON sub.itemFK = ish.itemFk
JOIN vn.shelving s ON s.code = ish.shelvingFk
JOIN vn.parking p ON p.id = s.parkingFk
WHERE ish.shelvingFk <> ? AND sub.created > ish.created AND p.code <> ?`,
[shelvingFk, shelvingFk, parking]);
)sub ON sub.itemFK = ish.itemFk
JOIN vn.shelving s ON s.code = ish.shelvingFk
sergiodt marked this conversation as resolved Outdated
carlosap commented 2024-03-22 15:35:38 +00:00
Outdated
Review
Copy Link

Tabulació

Tabulació
JOIN vn.parking p ON p.id = s.parkingFk
sergiodt marked this conversation as resolved Outdated
carlosap commented 2024-03-27 18:16:31 +00:00
Outdated
Review
Copy Link

en la subconsulta el p.code y el LEFT JOIN vn.parking p ON p.id = s.parkingFk sirven para algo??

en la subconsulta el p.code y el LEFT JOIN vn.parking p ON p.id = s.parkingFk sirven para algo??
sergiodt commented 2024-03-28 06:16:30 +00:00
Outdated
Review
Copy Link

Llevat

Llevat
WHERE sub.created > ish.created
AND (p.code <> ? OR ? IS NULL)
AND (ish.shelvingFk <> ? OR ? IS NULL)
AND (ish.itemFk <> ? OR ? IS NULL)`,
[shelvingFkIn, parking, parking, shelvingFkOut, shelvingFkOut, itemFk, itemFk]);
sergiodt marked this conversation as resolved Outdated
alexm commented 2024-03-25 06:11:25 +00:00
Outdated
Review
Copy Link

falta gastar la transaccio (...shelvingFkOut, itemFk, itemFk], options)

falta gastar la transaccio (...shelvingFkOut, itemFk, itemFk], options)
sergiodt commented 2024-03-25 06:55:48 +00:00
Outdated
Review
Copy Link

Llevat options

Llevat options
alexm commented 2024-03-25 07:07:00 +00:00
Outdated
Review
Copy Link

Es ficarlo sino no gastarà la transaccio

Es ficarlo sino no gastarà la transaccio
sergiodt commented 2024-03-25 07:23:38 +00:00
Outdated
Review
Copy Link

L'he posat però no veia que fera falta...

L'he posat però no veia que fera falta...
return result[0]['COUNT(ish.id)'] > 0;
sergiodt marked this conversation as resolved Outdated
alexm commented 2024-03-25 06:06:43 +00:00
Outdated
Review
Copy Link

Si li poses el alias aci hi ha que canviarlo

Si li poses el alias aci hi ha que canviarlo
};
sergiodt marked this conversation as resolved Outdated
alexm commented 2024-03-25 07:07:09 +00:00
Outdated
Review
Copy Link

Aci

Aci
alexm commented 2024-03-26 07:03:01 +00:00
Outdated
Review
Copy Link

myOptions

myOptions
};