4794-mdb_block #1170
Labels
No Milestone
No Assignees
4 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: verdnatura/salix#1170
Loading…
Reference in New Issue
No description provided.
Delete Branch "4794-mdb_block"
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?
@ -0,0 +1,11 @@
CREATE TABLE `vn`.`mdbApp` (
`app` varchar(100) COLLATE utf8mb3_unicode_ci DEFAULT NULL,
primary key
@ -0,0 +3,4 @@
`baselineBranchFk` varchar(255) COLLATE utf8mb3_unicode_ci DEFAULT NULL,
`userFk` int(10) unsigned DEFAULT NULL,
`locked` datetime DEFAULT NULL,
UNIQUE KEY `app_UN` (`app`),
quitar
@ -2734,0 +2734,4 @@
INSERT INTO `vn`.`mdbApp` (`app`, `baselineBranchFk`, `userFk`, `locked`)
VALUES
('com', 'master', NULL, NULL),
no usar nombre "conocidos" ejemplo foo, bar
@ -246,2 +246,3 @@
"There aren't records for this week": "No existen registros para esta semana",
"Empty data source": "Origen de datos vacio"
"Empty data source": "Origen de datos vacio",
"App locked": "Aplicación bloquada por el usuario {{userId}}"
bloqueada
@ -0,0 +39,4 @@
try {
const mdbApp = await models.MdbApp.findById(appName, null, myOptions);
const updatedMdbApp = await mdbApp.updateAttributes({
hay que comprobar que no este bloqueado previamente
@ -0,0 +6,4 @@
try {
const options = {transaction: tx};
const appName = 'com';
foo
4794-mdb_blockto WIP: 4794-mdb_block@ -0,0 +41,4 @@
}
try {
const mdbApp = await models.MdbApp.findById(appName, null, myOptions);
traer solo los campos necesarios ejemplo:
{fields: ['locked','userFk']}
@ -0,0 +46,4 @@
const message = $t('App locked', {
userId: mdbApp.userFk
});
if (mdbApp.locked) throw new UserError(message);
si se usa dentro del if crearla dentro del if.
En este caso se puede poner todo el codigo dentro de los parentesis, sin crear la variable
@ -58,0 +67,4 @@
const message = $t('App locked', {
userId: mdbApp.userFk
});
if (mdbApp.locked && mdbApp.userFk != userId) throw new UserError(message);
mensaje dentro del if como arriba.
WIP: 4794-mdb_blockto 4794-mdb_block@ -0,0 +1,4 @@
INSERT INTO `salix`.`ACL` (`model`, `property`, `accessType`, `permission`, `principalType`, `principalId`)
Cambiar a carpeta actual
@ -0,0 +31,4 @@
if (typeof options == 'object')
Object.assign(myOptions, options);
if (!myOptions.transaction) {
En este caso no es necesario transaccionar si no se te pasa una transacción
@ -0,0 +38,4 @@
locked: null
}, myOptions);
if (tx) await tx.commit();
el commit estas segur que es necesari?
No, no está gastantse. Aso es per a cuant se crea la transacció desde dins. Tampoc faría falta el try-cach perque no te que fer rollback la propia ruta.
@ -0,0 +26,4 @@
const models = Self.app.models;
const myOptions = {};
let tx;
Llevar tot lo referent a tx i try-catch
Ya estaba aprobado, simplemente he hecho git pull origin dev.