Merge pull request #149 from balena-io-modules/block-alternate-key-pa…

…rtial-unique-index Properly error when using fields of a partial index as an alternate key
balena-io-modules · Apr 23, 2024 · 56e6c98 · 56e6c98
2 parents 18795c8 + 8652f14
commit 56e6c98
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/package.json b/package.json
@@ -15,7 +15,7 @@
   "author": "",
   "license": "BSD",
   "dependencies": {
-    "@balena/abstract-sql-compiler": "^9.0.4",
+    "@balena/abstract-sql-compiler": "^9.1.4",
     "@balena/odata-parser": "^3.0.3",
     "@types/lodash": "^4.14.202",
     "@types/memoizee": "^0.4.11",

diff --git a/src/odata-to-abstract-sql.ts b/src/odata-to-abstract-sql.ts
@@ -790,7 +790,8 @@ export class OData2AbstractSQL {
 				) &&
 				!resource.indexes.some((index) => {
 					return (
-						(index.type === 'UNIQUE' || index.type === 'PRIMARY KEY') &&
+						((index.type === 'UNIQUE' && index.predicate == null) ||
+							index.type === 'PRIMARY KEY') &&
 						sqlFieldNames.length === index.fields.length &&
 						_.isEqual(index.fields.slice().sort(), sqlFieldNames)
 					);