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

#7867 - avoid deactivate user when is supplier #2860

Merged
jsegarra merged 2 commits from hotfix_nightTask_supplierDisable into master 2024-08-14 05:52:14 +00:00
Conversation 0 Commits 2 Files Changed 1 +1
1 changed files with 1 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
db/routines/vn/procedures/client_userDisable.sql
View File

@ -21,6 +21,7 @@ BEGIN
AND a.id IS NULL AND a.id IS NULL
AND u.active AND u.active
AND c.created < util.VN_CURDATE() - INTERVAL vMonths MONTH AND c.created < util.VN_CURDATE() - INTERVAL vMonths MONTH
AND NOT u.role = (SELECT id FROM `role` WHERE name = 'supplier')
AND u.id NOT IN ( AND u.id NOT IN (
guillermo commented 2024-08-13 10:55:33 +00:00
Outdated
Review
Copy Link

Como es una sola tabla no hay que especificar el alias.
Además. como siempre va a retornar 1 registro, en vez de poner un IN yo pondría un =

AND NOT u.role = (SELECT id FROM `role` WHERE name = 'supplier')
Como es una sola tabla no hay que especificar el alias. Además. como siempre va a retornar 1 registro, en vez de poner un IN yo pondría un = ``` AND NOT u.role = (SELECT id FROM `role` WHERE name = 'supplier') ```
jsegarra commented 2024-08-13 11:17:28 +00:00
Outdated
Review
Copy Link

"Como es una sola tabla no hay que especificar el alias."...fallo mío por no refrescar la "Convención SQL"

"Como es una sola tabla no hay que especificar el alias."...fallo mío por no refrescar la "Convención SQL"
SELECT DISTINCT c.id SELECT DISTINCT c.id
FROM client c FROM client c