fixes #5036 Regularizar históricos #1244
No reviewers
Labels
No Milestone
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: verdnatura/salix#1244
Loading…
Reference in New Issue
No description provided.
Delete Branch "5036-regularizar-historicos"
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?
WIP: #5036 Regularizar historicosto fixes #5036 Regularizar históricos@ -222,0 +304,4 @@
'updateOrCreate',
'replaceOrCreate',
'replaceById',
'updateAttributes',
Llevar
updateAttributes
@ -222,0 +306,4 @@
'replaceById',
'updateAttributes',
'update'
]);
Declarar-ho fora de la funció per a que no ho faça cada vegada.
@ -222,0 +314,4 @@
update: [
'update',
'replaceById',
'updateAttributes',
Llevar
updateAttributes
@ -222,0 +333,4 @@
for (const op in opOpts) {
for (const met of opOpts[op])
opMap.set(met, op);
}
Declarar
opOpts
iopMap
fora de la funció per a que no ho faça cada vegada.@ -222,0 +357,4 @@
const res = await new Promise(resolve => {
const fnArgs = args.slice(0, -2);
fnArgs.push(opts);
fnArgs.push((...args) => resolve(args));
Pots afegir els dos arguments en una instrucció
fnArgs.push(opts, (...args) => resolve(args));
@ -222,0 +514,4 @@
// Delete unchanged properties
if (oldI) {
Object.keys(oldI).forEach(prop => {
if (newI[prop] == oldI[prop]) {
Ficar igual estricte
===
, no està tenint en compte les datesCom has resolt açò? Continue vejent
===
, les dates no es poden comparar aixina@ -222,0 +324,4 @@
try {
// Fetch old values (update|delete) or login
await this.grabUserLog(model, opts, 'login');
Si grabUser està habilitat, els disparadors de la taula ja creen els logs, despres de açò no has de fer res mes a part de logout i finalitzar transacció
@ -222,0 +414,4 @@
buildSelectStmt(op, data, idName, model, where, limit) {
const Model = this.getModelDefinition(model).model;
const settings = Model.definition.settings;
Nomes es gasta en la línia 419, no es necesari definir la variable
@ -222,0 +416,4 @@
const Model = this.getModelDefinition(model).model;
const settings = Model.definition.settings;
const properties = Object.keys(Model.definition.properties);
const log = settings.log;
Nomes es gasta en el
else
, declarar dins del else:const log = Model.definition.settings.log;
Si grabUser està habilitat salix no ha de crear logs, nomes fer login i logout
@ -222,0 +411,4 @@
else if(action == 'logout')
await this.executeP(`CALL account.myUser_logout()`, null, opts);
}
El funcionamiento de esta función es confuso y además su código no se reutiliza.
Quitar la función y mover cada bloque de código a su sitio correspondiente.
9b11987d7
feat: desaparecen los botones para las semanas que no tienen registros