feat: fallback sort field init to createdAt field (#1507)

* feat: fallback sort field init to createdAt field when primary key not exists

* chore: set belongsToMany through autoCreated table to sortable false

packages/core/database/src/__tests__/fields/sort-field.test.ts

@@ -17,6 +17,29 @@ describe('string field', () => {
     await db.close();
   });
 
+  it('should init sorted value by createdAt when primaryKey not exists', async () => {
+    const Test = db.collection({
+      autoGenId: false,
+      name: 'tests',
+    });
+
+    await db.sync();
+    await Test.repository.create({
+      values: [{}, {}, {}],
+    });
+
+    // add sort field
+    Test.setField('sort', { type: 'sort' });
+
+    let err;
+    try {
+      await db.sync();
+    } catch (e) {
+      err = e;
+    }
+    expect(err).toBeFalsy();
+  });
+
   it('sort', async () => {
     const Test = db.collection({
       name: 'tests',

packages/core/database/src/fields/sort-field.ts

@@ -53,9 +53,21 @@ export class SortField extends Field {
       transaction,
     });
 
+    const orderKey = (() => {
+      const model = this.collection.model;
+      if (model.primaryKeyAttribute) {
+        return model.primaryKeyAttribute;
+      }
+      if (model.rawAttributes['createdAt']) {
+        return 'createdAt';
+      }
+
+      throw new Error(`can not find order key for collection ${this.collection.name}`);
+    })();
+
     if (emptyCount === totalCount && emptyCount > 0) {
       const records = await this.collection.repository.find({
-        order: [this.collection.model.primaryKeyAttribute],
+        order: [orderKey],
         transaction,
       });
 

packages/plugins/collection-manager/src/__tests__/fields/belongsToMany.test.ts

@@ -17,11 +17,14 @@ describe('belongsToMany', () => {
       values: {
         name: 'posts',
       },
+      context: {},
     });
+
     await Collection.repository.create({
       values: {
         name: 'tags',
       },
+      context: {},
     });
   });
 
@@ -29,5 +32,24 @@ describe('belongsToMany', () => {
     await app.destroy();
   });
 
-  it('should create belongsToMany field', async () => {});
+  it('should create belongsToMany field', async () => {
+    await Field.repository.create({
+      values: {
+        name: 'tags',
+        type: 'belongsToMany',
+        collectionName: 'posts',
+        interface: 'm2m',
+        through: 'post_tags',
+      },
+      context: {},
+    });
+
+    const throughCollection = await Collection.repository.findOne({
+      filter: {
+        name: 'post_tags',
+      },
+    });
+
+    expect(throughCollection.get('sortable')).toEqual(false);
+  });
 });

packages/plugins/collection-manager/src/hooks/afterCreateForForeignKeyField.ts

@@ -150,6 +150,7 @@ export function afterCreateForForeignKeyField(db: Database) {
             hidden: true,
             autoCreate: true,
             isThrough: true,
+            sortable: false,
           },
           transaction,
         });

packages/presets/nocobase/src/index.ts

@@ -80,6 +80,7 @@ export class PresetNocoBase extends Plugin {
       //   }
       // }
     });
+
     this.app.on('beforeUpgrade', async (options) => {
       const result = await this.app.version.satisfies('<0.8.0-alpha.1');
       if (result) {