#5666 - Loggable to mixin #1826
juan
alexm
Labels
No Milestone
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: verdnatura/salix#1826
Loading…
Reference in New Issue
No description provided.
Delete Branch "5666-loggable_to_mixin"
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?
https://redmine.verdnatura.es/issues/5666
WIP: Loggable to mixinto Loggable to mixinLoggable to mixinto #5666 - Loggable to mixin@ -0,0 +10,4 @@"mixins": {"Loggable": true}}A parte de crear el modelo habria que substituir todas las ocurrencias del modelo Role por VnRole
@ -6,0 +5,4 @@"mixins": {"Loggable": true}}Porque se continua manteniendo el json y el js del modelo Loggable? Habría que hacer que todos los modelos que lo tienen como base utilizaran el nuevo mixin
Tenia entendido que si defines un mixin en un modelo, y este lo usas de base en otro modelo, este último lo hereda al igual que las propiedades.
Lo cambio
Este fichero habría que borrarlo.
He eliminado el fichero json y el mixin se ha definido para cada uno de los modelos que dependía
La he aprobado sin querer, revisar notas.
New commits pushed, approval review dismissed automatically according to repository settings
A parte de las modificaciones solicitadas, quitar el modelo
Rolenativo del fichero loopback/server/model-config.json.@ -0,0 +1,8 @@-- Auto-generated SQL script #202311301038INSERT INTO `salix`.`ACL` (model,property,accessType,permission,principalType,principalId)VALUES ('VnRole','*','*','ALLOW','ROLE','$everyone');Este ACL permite a cualquiera modificar los roles!
Aplicar los mismos ACLs que al modelo
Roleen producción.@ -0,0 +2,4 @@INSERT INTO `salix`.`ACL` (model,property,accessType,permission,principalType,principalId)VALUES ('VnRole','*','*','ALLOW','ROLE','$everyone');INSERT INTO `salix`.`ACL` (model,property,accessType,permission,principalType,principalId)VALUES ('VnRole','*','*','ALLOW','ROLE','employee');Este ACL permite a los empleados modificar los roles!
Debe ser solo READ
@ -0,0 +5,4 @@VALUES ('VnRole','*','*','ALLOW','ROLE','employee');-- Auto-generated SQL script #202311301203UPDATE `salix`.`ACL` SET permission='DENY' WHERE model='Role';DELETE FROM salix.ACL WHERE model='Role';@ -24,3 +24,3 @@let modelName = rule.shift();let fieldName = rule.shift();let modelAlias = $attrs.ruleAlias;He añadido este attrs para que desde las rutas de role/:id/basic-data y create se puedan cargar los datos entrantes del registro
No es necesario modificar rule, hay que usar
rule="VnRule.[propiedad]"para que aplique las reglas de validación del modeloVnRole@ -21,2 +21,2 @@label="Description"ng-model="$ctrl.role.description"label="Description"ng-model="$ctrl.VnRole.description"En lugar de modificar la directiva rule, hay que utilizar
rule="VnRole.[propiedad]"para que aplique las reglas del modelo VnRoleNew commits pushed, approval review dismissed automatically according to repository settings
New commits pushed, approval review dismissed automatically according to repository settings