#7867 - avoid deactivate user when is supplier #2860

jsegarra · 2024-08-13T10:45:06Z

jsegarra commented

2024-08-13 10:45:06 +00:00

No description provided.

jsegarra added 1 commit 2024-08-13 10:45:06 +00:00

gitea/salix/pipeline/pr-master This commit looks good Details

a178c285c0 fix(salix): #7867 avoid deactivate user when is supplier

jsegarra requested review from guillermo 2024-08-13 10:45:28 +00:00

guillermo requested changes 2024-08-13 10:56:36 +00:00

Dismissed

db/routines/vn/procedures/client_userDisable.sql Outdated

						
				@ -22,2 +22,4 @@

							AND u.active

							AND c.created < util.VN_CURDATE() - INTERVAL vMonths MONTH 

							AND u.role NOT IN (

								SELECT id FROM `role` r WHERE r.name = 'supplier'

guillermo commented

2024-08-13 10:55:33 +00:00

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

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