@mikro-orm/sql 7.0.17-dev.7 → 7.0.17-dev.9

package/dialects/mysql/MySqlSchemaHelper.js

@@ -150,36 +150,34 @@ export class MySqlSchemaHelper extends SchemaHelper {
      * MySQL requires collation to be specified as an expression: (column_name COLLATE collation_name)
      */
     getIndexColumns(index) {
-        if (index.columns?.length) {
-            return index.columns
-                .map(col => {
-                const quotedName = this.quote(col.name);
-                // MySQL supports collation via expression: (column_name COLLATE collation_name)
-                // When collation is specified, wrap in parentheses as an expression
-                if (col.collation) {
-                    let expr = col.length ? `${quotedName}(${col.length})` : quotedName;
-                    expr = `(${expr} collate ${col.collation})`;
-                    // Sort order comes after the expression
-                    if (col.sort) {
-                        expr += ` ${col.sort}`;
-                    }
-                    return expr;
-                }
-                // Standard column definition without collation
-                let colDef = quotedName;
-                // MySQL supports prefix length
-                if (col.length) {
-                    colDef += `(${col.length})`;
-                }
-                // MySQL supports sort order
+        return index.columnNames
+            .map(name => {
+            const col = index.columns?.find(c => c.name === name);
+            const quotedName = this.quote(name);
+            // MySQL supports collation via expression: (column_name COLLATE collation_name)
+            // When collation is specified, wrap in parentheses as an expression
+            if (col?.collation) {
+                let expr = col.length ? `${quotedName}(${col.length})` : quotedName;
+                expr = `(${expr} collate ${col.collation})`;
+                // Sort order comes after the expression
                 if (col.sort) {
-                    colDef += ` ${col.sort}`;
+                    expr += ` ${col.sort}`;
                 }
-                return colDef;
-            })
-                .join(', ');
-        }
-        return index.columnNames.map(c => this.quote(c)).join(', ');
+                return expr;
+            }
+            // Standard column definition without collation
+            let colDef = quotedName;
+            // MySQL supports prefix length
+            if (col?.length) {
+                colDef += `(${col.length})`;
+            }
+            // MySQL supports sort order
+            if (col?.sort) {
+                colDef += ` ${col.sort}`;
+            }
+            return colDef;
+        })
+            .join(', ');
     }
     /**
      * Append MySQL-specific index suffixes like INVISIBLE.

package/dialects/postgresql/PostgreSqlSchemaHelper.js

@@ -687,27 +687,25 @@ export class PostgreSqlSchemaHelper extends SchemaHelper {
      * Build the column list for a PostgreSQL index.
      */
     getIndexColumns(index) {
-        if (index.columns?.length) {
-            return index.columns
-                .map(col => {
-                let colDef = this.quote(col.name);
-                // PostgreSQL supports collation with double quotes
-                if (col.collation) {
-                    colDef += ` collate ${this.quote(col.collation)}`;
-                }
-                // PostgreSQL supports sort order
-                if (col.sort) {
-                    colDef += ` ${col.sort}`;
-                }
-                // PostgreSQL supports NULLS FIRST/LAST
-                if (col.nulls) {
-                    colDef += ` nulls ${col.nulls}`;
-                }
-                return colDef;
-            })
-                .join(', ');
-        }
-        return index.columnNames.map(c => this.quote(c)).join(', ');
+        return index.columnNames
+            .map(name => {
+            const col = index.columns?.find(c => c.name === name);
+            let colDef = this.quote(name);
+            // PostgreSQL supports collation with double quotes
+            if (col?.collation) {
+                colDef += ` collate ${this.quote(col.collation)}`;
+            }
+            // PostgreSQL supports sort order
+            if (col?.sort) {
+                colDef += ` ${col.sort}`;
+            }
+            // PostgreSQL supports NULLS FIRST/LAST
+            if (col?.nulls) {
+                colDef += ` nulls ${col.nulls}`;
+            }
+            return colDef;
+        })
+            .join(', ');
     }
     /**
      * PostgreSQL-specific index options like fill factor.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mikro-orm/sql",
-  "version": "7.0.17-dev.7",
+  "version": "7.0.17-dev.9",
   "description": "TypeScript ORM for Node.js based on Data Mapper, Unit of Work and Identity Map patterns. Supports MongoDB, MySQL, PostgreSQL and SQLite databases as well as usage with vanilla JavaScript.",
   "keywords": [
     "data-mapper",
@@ -53,7 +53,7 @@
     "@mikro-orm/core": "^7.0.16"
   },
   "peerDependencies": {
-    "@mikro-orm/core": "7.0.17-dev.7"
+    "@mikro-orm/core": "7.0.17-dev.9"
   },
   "engines": {
     "node": ">= 22.17.0"

package/schema/SchemaHelper.js

@@ -132,27 +132,25 @@ export class SchemaHelper {
      * Note: Prefix length is only supported by MySQL/MariaDB which override this method.
      */
     getIndexColumns(index) {
-        if (index.columns?.length) {
-            return index.columns
-                .map(col => {
-                let colDef = this.quote(col.name);
-                // Collation comes after column name (SQLite syntax: column COLLATE name)
-                if (col.collation) {
-                    colDef += ` collate ${col.collation}`;
-                }
-                // Sort order
-                if (col.sort) {
-                    colDef += ` ${col.sort}`;
-                }
-                // NULLS ordering (PostgreSQL)
-                if (col.nulls) {
-                    colDef += ` nulls ${col.nulls}`;
-                }
-                return colDef;
-            })
-                .join(', ');
-        }
-        return index.columnNames.map(c => this.quote(c)).join(', ');
+        return index.columnNames
+            .map(name => {
+            const col = index.columns?.find(c => c.name === name);
+            let colDef = this.quote(name);
+            // Collation comes after column name (SQLite syntax: column COLLATE name)
+            if (col?.collation) {
+                colDef += ` collate ${col.collation}`;
+            }
+            // Sort order
+            if (col?.sort) {
+                colDef += ` ${col.sort}`;
+            }
+            // NULLS ordering (PostgreSQL)
+            if (col?.nulls) {
+                colDef += ` nulls ${col.nulls}`;
+            }
+            return colDef;
+        })
+            .join(', ');
     }
     /** Returns SQL to drop an index. */
     getDropIndexSQL(tableName, index) {