Not create triggers with null in sql param

__tests__/DbTable.js

@@ -28,7 +28,23 @@ test ('good', () => {
 			expect (this.name).toBe('t')
 
 			return [ {id: 1} ]
-		}
+		},
+		triggers: [
+			{
+				phase  : 'BEFORE INSERT',
+				action : 'FOR EACH ROW',
+				sql    : /*sql*/`
+					BEGIN
+						RETURN NEW;
+					END;
+				`,
+			},
+			{
+				phase  : 'AFTER INSERT',
+				action : 'FOR EACH ROW',
+				sql    : null,
+			},
+		],
 	})
 
 	expect (r.columns.id.type).toBe ('int')

__tests__/DbTrigger.js

@@ -11,7 +11,7 @@ test ('bad', () => {
 	expect (() => new DbTrigger ({name: 't', phase: 'BEFORE', sql: 1})).toThrow ()
 	expect (() => new DbTrigger ({name: 't', phase: true, sql: 'NULL;'})).toThrow ()
 	expect (() => new DbTrigger ({name: 't', phase: true, sql: function () {return ''}})).toThrow ()
-
+	expect (() => new DbTrigger ({name: 't', phase: true, sql: null})).toThrow ()
 })
 
 test ('not bad', () => {
@@ -24,6 +24,7 @@ test ('not bad', () => {
 		columns: {id: 'int'},
 		pk: ['id'],
 		triggers: [
+			{name: 't', phase: 'BEFORE UPDATE', sql: 'NULL;'},
 			{name: 't', phase: 'BEFORE UPDATE', sql: 'NULL;'},
 			{name: 't1', phase: 'AFTER UPDATE', sql: function () {
 
@@ -34,6 +35,7 @@ test ('not bad', () => {
 
 				return 'NULL;'
 			}},
+			{name: 't', phase: 'BEFORE UPDATE', sql: null},
 		]
 	})
 

lib/model/DbTable.js

@@ -30,17 +30,18 @@ class DbTable extends DbRelation {
 
 		for (let i = 0; i < length; i ++) {
 
-			const o = triggers [i]
+			const o = triggers [i]; if (o.sql !== null) {
 
-			o.table = this
-
-			if (!o.name) o.name = lang.getTriggerName (this, i)
-
-			const trigger = new DbTrigger (o)
+				o.table = this
 
-			trigger.setLang (lang)
+				if (!o.name) o.name = lang.getTriggerName (this, i)
+
+				const trigger = new DbTrigger (o)
+
+				trigger.setLang (lang)
 
-			triggers [i] = trigger
+				triggers [i] = trigger
+			}
 
 		}
 

package.json

@@ -1,6 +1,6 @@
 {
   "name": "doix-db",
-  "version": "1.0.68",
+  "version": "1.0.69",
   "description": "Shared database related code for doix",
   "main": "index.js",
   "files": [