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

7648_myEntries #2744

Merged
jsegarra merged 7 commits from 7648_myEntries into master 2024-07-19 10:27:40 +00:00
Conversation 0 Commits 7 Files Changed 13 +251 -63
jsegarra commented 2024-07-16 11:52:48 +00:00
Member
Copy Link
No description provided.
jsegarra added 3 commits 2024-07-16 11:52:49 +00:00
jsegarra added 1 commit 2024-07-16 11:52:58 +00:00
gitea/salix/pipeline/pr-master This commit looks good Details
5293becb6d Merge branch 'master' into 7648_myEntries
jsegarra requested review from jgallego 2024-07-16 11:57:28 +00:00
jgallego approved these changes 2024-07-16 13:54:40 +00:00
jgallego requested review from alexm 2024-07-16 13:54:48 +00:00
alexm requested changes 2024-07-17 06:04:54 +00:00
Dismissed
db/versions/11118-limeCymbidium/00-firstScript.sql
@ -0,0 +9,4 @@
VALUES ('Entry','buyLabel','READ','ALLOW','ROLE','supplier');
INSERT IGNORE INTO `account`.`role` (`name`,`description`,`hasLogin`,`created`,`modified`)
VALUES ('supplier','Proveedores',1,'2017-10-10 14:58:58.000','2017-10-10 14:59:20.000');
alexm commented 2024-07-17 05:37:48 +00:00
Member
Copy Link

Created 2017?, mejor no ponerlo y que se ponga automatico

Created 2017?, mejor no ponerlo y que se ponga automatico
jsegarra commented 2024-07-17 06:23:02 +00:00
Author
Member
Copy Link

Cierto, aunque este rol ya está creado asi que lo podriamos evitar

Cierto, aunque este rol ya está creado asi que lo podriamos evitar
db/versions/11118-limeCymbidium/00-firstScript.sql
@ -0,0 +11,4 @@
INSERT IGNORE INTO `account`.`role` (`name`,`description`,`hasLogin`,`created`,`modified`)
VALUES ('supplier','Proveedores',1,'2017-10-10 14:58:58.000','2017-10-10 14:59:20.000');
SET @supplierFk =LAST_INSERT_ID();
INSERT IGNORE INTO account.roleInherit (`role`,`inheritsFrom`)
alexm commented 2024-07-17 05:40:29 +00:00
Member
Copy Link

Despues de esto hace falta hacer un CALL account.role_sync()

Despues de esto hace falta hacer un `CALL account.role_sync()`
jsegarra commented 2024-07-17 06:23:18 +00:00
Author
Member
Copy Link

Ya existe en prod

Ya existe en prod
jsegarra marked this conversation as resolved
db/versions/11118-limeCymbidium/00-firstScript.sql Outdated
@ -0,0 +16,4 @@
UPDATE salix.ACL
SET principalId='$authenticated'
WHERE id=264;
alexm commented 2024-07-17 05:40:57 +00:00
Member
Copy Link

No poner id, poner a que hacer referencia (model, property, etc)

No poner id, poner a que hacer referencia (model, property, etc)
jsegarra marked this conversation as resolved
modules/entry/back/methods/entry/getBuys.js
@ -36,1 +40,4 @@
if (supplier) {
const isEntryOwner = (await Self.findById(id)).supplierFk === supplier.id;
if (!isEntryOwner) throw new UserError('Access Denied');
alexm commented 2024-07-17 05:45:23 +00:00
Member
Copy Link

Diria que mejor usar un throw new ForbiddenError(...);

Diria que mejor usar un `throw new ForbiddenError(`...`);`
👀 1
jsegarra commented 2024-07-17 06:42:02 +00:00
Author
Member
Copy Link

lo vemos porque en el proyecto también se usa UserError para casos similares

lo vemos porque en el proyecto también se usa UserError para casos similares
jsegarra marked this conversation as resolved
print/templates/reports/buy-label/sql/buys.sql
@ -1,4 +1,15 @@
SELECT ROW_NUMBER() OVER(ORDER BY b.id) labelNum,
WITH RECURSIVE numbers AS (
alexm commented 2024-07-17 06:04:51 +00:00
Member
Copy Link

Este SQL lo veo raro, ver con @pablone

Este SQL lo veo raro, ver con @pablone
jsegarra commented 2024-07-17 06:24:55 +00:00
Author
Member
Copy Link

El SQL es de guillermo, solamente he hecho cherry-pick

El SQL es de guillermo, solamente he hecho cherry-pick
guillermo commented 2024-07-18 05:51:17 +00:00
Member
Copy Link

¿A que te refieres con raro?

Es un SQL recursivo, lo utilizo para generar el número de etiquetas que tiene el campo stickers.

Es decir, antes si filtrábamos en la tabla buy por entryFk = 156841, salían 7 registros por ejemplo, entonces aparecían 7 páginas.

Pako lo que quería es que si la linea tiene stickers 20, de esa línea apareciesen 20 páginas, así con las 6 restantes,

Simplemente se utiliza para eso.

¿A que te refieres con raro? Es un SQL recursivo, lo utilizo para generar el número de etiquetas que tiene el campo stickers. Es decir, antes si filtrábamos en la tabla buy por entryFk = 156841, salían 7 registros por ejemplo, entonces aparecían 7 páginas. Pako lo que quería es que si la linea tiene stickers 20, de esa línea apareciesen 20 páginas, así con las 6 restantes, Simplemente se utiliza para eso.
jsegarra marked this conversation as resolved
jsegarra added 2 commits 2024-07-19 09:54:26 +00:00
gitea/salix/pipeline/pr-master This commit looks good Details
287f9edc83 fix(salix): refs #7648 #7648 improve update ACL
jsegarra added 1 commit 2024-07-19 09:59:38 +00:00
gitea/salix/pipeline/pr-master This commit looks good Details
2a044dd90b Merge branch 'master' into 7648_myEntries
jsegarra requested review from alexm 2024-07-19 09:59:40 +00:00
alexm reviewed 2024-07-19 10:16:44 +00:00
db/versions/11118-limeCymbidium/00-firstScript.sql
@ -0,0 +10,4 @@
UPDATE salix.ACL
SET principalId='$authenticated'
WHERE id=(SELECT id FROM salix.ACL WHERE model='StarredModule' and property='*' and `accessType`='*' );
alexm commented 2024-07-19 10:16:44 +00:00
Member
Copy Link

Podias haber puesto directamente ese where jajaja

Podias haber puesto directamente ese where jajaja
alexm approved these changes 2024-07-19 10:16:51 +00:00
jsegarra merged commit 3d20908a42 into master 2024-07-19 10:27:40 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
jgallego
alexm
Labels
Clear labels   
CR / Tests passed

Unit tests and e2e passed

  
e2e

Implement e2e

  
Missing tests

Missing tests

  
Needs refactor

   
Simple

Simple check / No CR needed

No Label
CR / Tests passed
e2e
Missing tests
Needs refactor
Simple
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
4 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: verdnatura/salix#2744
No description provided.
Delete Branch "7648_myEntries"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?