Merge pull request #245 from themost-framework/fix-association-callback

anthioikonomou · web-flow · commit 5ecef99717d4 · 2026-02-02T11:56:35.000+02:00
Throw association error if expand association is missing or is not correctly set
diff --git a/data-queryable.js b/data-queryable.js
@@ -2744,6 +2744,8 @@ function afterExecute_(result, callback) {
                                 return cb(err);
                             });
                     }
+                    // throw association error if mapping is not set
+                    return cb(new DataError('EASSOC', sprintf('Data association mapping (%s) for %s cannot be determined.', mapping.refersTo, self.model.name)));
                 }
                 else {
                     return cb(new Error('Not yet implemented'));
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@themost/data",
-  "version": "2.6.98",
+  "version": "2.6.99",
   "description": "MOST Web Framework Codename Blueshift - Data module",
   "main": "index.js",
   "types": "index.d.ts",

Original file line number	Diff line number	Diff line change
`@@ -2744,6 +2744,8 @@ function afterExecute_(result, callback) {`
`2744`	`2744`	`return cb(err);`
`2745`	`2745`	`});`
`2746`	`2746`	`}`
	`2747`	`+ // throw association error if mapping is not set`
	`2748`	`+ return cb(new DataError('EASSOC', sprintf('Data association mapping (%s) for %s cannot be determined.', mapping.refersTo, self.model.name)));`
`2747`	`2749`	`}`
`2748`	`2750`	`else {`
`2749`	`2751`	`return cb(new Error('Not yet implemented'));`
Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "@themost/data",`
`3`		`- "version": "2.6.98",`
	`3`	`+ "version": "2.6.99",`
`4`	`4`	`"description": "MOST Web Framework Codename Blueshift - Data module",`
`5`	`5`	`"main": "index.js",`
`6`	`6`	`"types": "index.d.ts",`