backend-plus 2.5.2-betha.6 → 2.5.2-betha.8

package/lib/backend-plus.d.ts

@@ -172,6 +172,9 @@ export type SequenceDefinition = {
     name:string
     firstValue:number
     prefix?:string /* Prefix for the generated value */
+} 
+export type SequenceMadMaxDefinition = {
+    madMax: [] // Si se necesita una secuencia por grupo acá estará la lista de campos group by de la secuencia. Y habrá que modificar la funión de pg-triggers para que lo tome.
 }
 export type ExportMetadataDefinition={ /* TODO: define */ }
 export type PostInputOptions='upperSpanish' | 'upperWithoutDiacritics' | 'parseDecimal'
@@ -218,11 +221,21 @@ export type FieldDefinition = EditableDbDefinition & {
     alwaysShow?:boolean /* show when appears in fixed fields */
     suggestingKeys?:string[]
     postInput?:PostInputOptions
-} & ({} | {
-    sequence:SequenceDefinition
-    nullable:true
-    editable:false
-})
+} & (
+    {
+        sequence?: undefined
+        nullable?: boolean
+        editable?: boolean
+    } | {
+        sequence: SequenceDefinition
+        nullable: true
+        editable: false
+    } | {
+        sequence: SequenceMadMaxDefinition
+        nullable: true
+        editable: boolean
+    }
+);
 export type EditableDbDefinition = {
     editable?:boolean
     allow?:{
@@ -415,7 +428,6 @@ export interface AppConfigLogin
     {
         schema: string                     // schema of the user table
         table: string                      // user table
-        from: string                       // complete expression to get table or join where get the user
         userFieldname: string              // fieldname in user table that stores the user name
         passFieldname: string              // fieldname in user table that stores the password hash
         rolFieldname: string               // fieldname in user table that stores the rol

package/lib/backend-plus.js

@@ -20,6 +20,7 @@ MiniTools.globalOpts.serveErr.propertiesWhiteList=['message','detail','code','ta
 var crypto = require('crypto');
 var serveContent = require('serve-content');
 var pg = require('pg-promise-strict');
+var pgTriggers = require('pg-triggers');
 var SessionFileStore = require('session-file-store');
 var memorystore = require('memorystore');
 var jsToHtml=require('js-to-html');
@@ -2653,7 +2654,16 @@ AppBackend.prototype.dumpFkConstraint = function dumpFkConstraint(fk, tableDef,
     return {consName, clause, sourceFieldList};
 }
 
+AppBackend.prototype.isGeneratedSequence = function isGeneratedSequence(sequence, not){
+    return sequence && ((!sequence.name && !sequence.madMax) == !not)
+}
+
+AppBackend.prototype.isSequenceNonGenerated = function isSequenceNonGenerated(sequence){
+    return this.isGeneratedSequence(sequence, true);
+}
+
 AppBackend.prototype.dumpDbTableFields = function dumpDbTableFields(tableDef, opts = {}, complements = null){
+    var be = this;
     var db = this.db;
     var fields=[];
     tableDef.fields.forEach(function(fieldDef){
@@ -2667,7 +2677,7 @@ AppBackend.prototype.dumpDbTableFields = function dumpDbTableFields(tableDef, op
                 ' '+(fieldDef.dataLength?(fieldType=='text'?'varchar':fieldType)+'('+fieldDef.dataLength+')':fieldType)+
                 (fieldDef.defaultValue!=null?' default '+db.quoteLiteral(fieldDef.defaultValue):'')+
                 (fieldDef.defaultDbValue!=null?' default '+fieldDef.defaultDbValue:'')+
-                (fieldDef.sequence && !fieldDef.sequence.name?' generated always as identity':'')+
+                (be.isGeneratedSequence(fieldDef.sequence)?' generated always as identity':'')+
                 (fieldDef.generatedAs!=null?` generated always as (${fieldDef.generatedAs}) stored`:'')
             );
             if(complements){
@@ -2774,7 +2784,7 @@ AppBackend.prototype.dumpDbSchemaPartial = async function dumpDbSchemaPartial(pa
                 lines.push('create table '+cualQuoteTableName+' (');
                 var fields = be.dumpDbTableFields(tableDef, opts,
                     function complements(fieldDef){
-                        if(fieldDef.sequence && !fieldDef.sequence.name){
+                        if(be.isGeneratedSequence(fieldDef.sequence)){
                             tablesWithStrictSequence[tableName]={}
                         }
                         if(fieldDef.typeName==='text' && !fieldDef.allowEmptyText){
@@ -2798,7 +2808,7 @@ AppBackend.prototype.dumpDbSchemaPartial = async function dumpDbSchemaPartial(pa
                                 ' alter column '+db.quoteIdent(fieldDef.name)+' set not null;'
                             );
                         }
-                        if(fieldDef.sequence && fieldDef.sequence.name){
+                        if(be.isSequenceNonGenerated(fieldDef.sequence)){
                             fieldsForSequences.push(fieldDef);
                         }
                     }
@@ -2901,19 +2911,26 @@ AppBackend.prototype.dumpDbSchemaPartial = async function dumpDbSchemaPartial(pa
                 });
                 lines.push(tableDef.sql.postCreateSqls);
                 lines.push('');
-                fieldsForSequences.forEach(function(fieldDef) {
+                await Promise.all(fieldsForSequences.map(async function(fieldDef) {
                     var sequence = fieldDef.sequence;
-                    lines.push("CREATE SEQUENCE "+db.quoteIdent(sequence.name)+" START "+db.quoteInteger(sequence.firstValue||1)+";");
-                    lines.push(
-                        "ALTER TABLE "+cualQuoteTableName+
-                        " ALTER COLUMN "+db.quoteIdent(fieldDef.name)+
-                        (sequence.prefix==null
-                        ?" SET DEFAULT nextval("+db.quoteLiteral(sequence.name)+"::regclass);"
-                        :" SET DEFAULT ("+db.quoteLiteral(sequence.prefix)+" || nextval("+db.quoteLiteral(sequence.name)+"::regclass)::text);"
-                        )
-                    );
-                    lines.push('GRANT USAGE, SELECT ON SEQUENCE '+db.quoteIdent(sequence.name)+' TO '+user+';');
-                });
+                    if (sequence.name) {
+                        if (sequence.madMax) throw new Error('a sequence with madMax cannot have name');
+                        lines.push("CREATE SEQUENCE "+db.quoteIdent(sequence.name)+" START "+db.quoteInteger(sequence.firstValue||1)+";");
+                        lines.push(
+                            "ALTER TABLE "+cualQuoteTableName+
+                            " ALTER COLUMN "+db.quoteIdent(fieldDef.name)+
+                            (sequence.prefix==null
+                            ?" SET DEFAULT nextval("+db.quoteLiteral(sequence.name)+"::regclass);"
+                            :" SET DEFAULT ("+db.quoteLiteral(sequence.prefix)+" || nextval("+db.quoteLiteral(sequence.name)+"::regclass)::text);"
+                            )
+                        );
+                        lines.push('GRANT USAGE, SELECT ON SEQUENCE '+db.quoteIdent(sequence.name)+' TO '+user+';');
+                    } else if (sequence.madMax) {
+                        lines.push(await pgTriggers.dumpMaxIdTrigger(tableDef.sql.tableName, fieldDef.name));
+                    } else {
+                        throw new Error('a sequence without madMax nor name');
+                    }
+                }));
                 lines.push('');
                 if(tableDef.sql.policies.enabled){
                     policyLines.push(`ALTER TABLE ${cualQuoteTableName} ENABLE ROW LEVEL SECURITY;`);

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "backend-plus",
   "description": "Backend for the anti Pareto rule",
-  "version": "2.5.2-betha.6",
+  "version": "2.5.2-betha.8",
   "author": "Codenautas <codenautas@googlegroups.com>",
   "license": "MIT",
   "repository": "codenautas/backend-plus",
@@ -59,6 +59,7 @@
     "nodemailer": "^7.0.3",
     "numeral": "^2.0.6",
     "pg-promise-strict": "^1.4.2",
+    "pg-triggers": "0.4.2",
     "pikaday": "^1.8.2",
     "pug": "^3.0.3",
     "read-yaml-promise": "^1.0.2",
@@ -77,13 +78,13 @@
   "devDependencies": {
     "@types/big.js": "^6.2.2",
     "@types/expect.js": "~0.3.32",
-    "@types/express": "^5.0.2",
+    "@types/express": "^5.0.3",
     "@types/express-useragent": "^1.0.5",
     "@types/fs-extra": "^11.0.4",
     "@types/js-yaml": "^4.0.9",
     "@types/mocha": "^10.0.10",
     "@types/multiparty": "~4.2.1",
-    "@types/node": "^22.15.29",
+    "@types/node": "^24.0.3",
     "@types/nodemailer": "^6.4.17",
     "@types/numeral": "~2.0.5",
     "@types/session-file-store": "^1.2.5",
@@ -98,10 +99,10 @@
     "karma-ie-launcher": "^1.0.0",
     "karma-mocha": "^2.0.1",
     "kill-9": "~0.4.3",
-    "mocha": "^11.5.0",
+    "mocha": "^11.7.0",
     "nyc": "^17.1.0",
-    "puppeteer": "^24.9.0",
-    "sinon": "^20.0.0",
+    "puppeteer": "^24.10.2",
+    "sinon": "^21.0.0",
     "supertest": "^7.1.1",
     "types.d.ts": "~0.6.22",
     "typescript": "^5.8.3",