remult 0.15.2 → 0.15.3

package/esm/remult-knex/index.js

@@ -5,6 +5,7 @@ import { allEntities, Remult } from "../src/context";
 import { isAutoIncrement } from "../src/remult3";
 import { ValueConverters } from "../src/valueConverters";
 import { CompoundIdField } from '../src/column';
+import { Sort } from '../src/sort';
 export class KnexDataProvider {
     constructor(knex) {
         this.knex = knex;
@@ -70,6 +71,9 @@ class KnexEntityDataProvider {
             let r = await br.resolveWhere();
             query.where(b => r.forEach(y => y(b)));
         }
+        if (!options.orderBy) {
+            options.orderBy = Sort.createUniqueSort(this.entity, new Sort());
+        }
         if (options.orderBy) {
             query = query.orderBy(options.orderBy.Segments.map(s => ({
                 column: e.nameOf(s.field),

package/esm/src/backend-tests/my-test.backend-spec.js

@@ -4,6 +4,16 @@ import { testKnexPGSqlImpl, testPostgresImplementation } from "./backend-databas
 import { entityWithValidations } from "../shared-tests/entityWithValidations";
 import { SqlDatabase } from "../data-providers/sql-database";
 config();
+testPostgresImplementation("default order by", async ({ createEntity }) => {
+    let s = await entityWithValidations.create4RowsInDp(createEntity);
+    await s.update(1, { name: "updated name" });
+    expect((await s.find()).map(x => x.myId)).toEqual([1, 2, 3, 4]);
+}, false);
+testKnexPGSqlImpl("default order by", async ({ createEntity }) => {
+    let s = await entityWithValidations.create4RowsInDp(createEntity);
+    await s.update(1, { name: "updated name" });
+    expect((await s.find()).map(x => x.myId)).toEqual([1, 2, 3, 4]);
+}, false);
 testPostgresImplementation("sql filter", async ({ createEntity }) => {
     let s = await entityWithValidations.create4RowsInDp(createEntity);
     expect((await s.find({

package/esm/src/context.d.ts

@@ -40,7 +40,7 @@ export declare class Remult {
     /** Returns the current user's info */
     get user(): UserInfo;
     /** Set's the current user info */
-    setUser(info: UserInfo): Promise<void>;
+    setUser(info: UserInfo | undefined): Promise<void>;
     private _user;
     private _userChangeEvent;
     /** Checks if a user was authenticated */
@@ -66,7 +66,7 @@ export declare class Remult {
      *
      * If no provider is specified, `fetch` will be used as an http provider
      */
-    constructor(provider?: HttpProvider | DataProvider);
+    constructor(provider?: HttpProvider | DataProvider | typeof fetch);
     /** The api Base Url to be used in all remult calls. by default it's set to `/api`.
      *
      * Set this property in case you want to determine a non relative api url

package/esm/src/context.js

@@ -103,10 +103,17 @@ export class Remult {
             this._dataSource = provider;
             return;
         }
-        let http = provider;
         let dataProvider;
-        if (http) {
-            dataProvider = new HttpProviderBridgeToRestDataProviderHttpProvider(http);
+        if (!dataProvider) {
+            let http = provider;
+            if (http && http.get && http.put && http.post && http.delete) {
+                dataProvider = new HttpProviderBridgeToRestDataProviderHttpProvider(http);
+            }
+        }
+        if (!dataProvider) {
+            if (typeof provider === "function") {
+                dataProvider = new RestDataProviderHttpProviderUsingFetch(provider);
+            }
         }
         if (!dataProvider) {
             dataProvider = new RestDataProviderHttpProviderUsingFetch();
@@ -147,6 +154,8 @@ export class Remult {
     /** Set's the current user info */
     async setUser(info) {
         this._user = info;
+        if (this._user === null)
+            this._user = undefined;
         let auth = info;
         if (auth) {
             if (!this._user.id && auth.sub)

package/esm/src/data-providers/sql-database.js

@@ -127,6 +127,9 @@ class ActualSQLServerDataProvider {
             if (options.limit) {
                 options.orderBy = Sort.createUniqueSort(this.entity, options.orderBy);
             }
+            if (!options.orderBy) {
+                options.orderBy = Sort.createUniqueSort(this.entity, new Sort());
+            }
             if (options.orderBy) {
                 let first = true;
                 let segs = [];

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "remult",
-    "version": "0.15.2",
+    "version": "0.15.3",
     "description": "A CRUD framework for full-stack TypeScript",
     "main": "index.js",
     "types": "index.d.ts",

package/remult-knex/index.js

@@ -9,6 +9,7 @@ var context_1 = require("../src/context");
 var remult3_1 = require("../src/remult3");
 var valueConverters_1 = require("../src/valueConverters");
 var column_1 = require("../src/column");
+var sort_1 = require("../src/sort");
 var KnexDataProvider = /** @class */ (function () {
     function KnexDataProvider(knex) {
         this.knex = knex;
@@ -129,6 +130,9 @@ var KnexEntityDataProvider = /** @class */ (function () {
                         query.where(function (b) { return r_1.forEach(function (y) { return y(b); }); });
                         _d.label = 3;
                     case 3:
+                        if (!options.orderBy) {
+                            options.orderBy = sort_1.Sort.createUniqueSort(this.entity, new sort_1.Sort());
+                        }
                         if (options.orderBy) {
                             query = query.orderBy(options.orderBy.Segments.map(function (s) { return ({
                                 column: e.nameOf(s.field),

package/src/backend-tests/my-test.backend-spec.js

@@ -7,6 +7,48 @@ var backend_database_test_setup_backend_spec_1 = require("./backend-database-tes
 var entityWithValidations_1 = require("../shared-tests/entityWithValidations");
 var sql_database_1 = require("../data-providers/sql-database");
 dotenv_1.config();
+backend_database_test_setup_backend_spec_1.testPostgresImplementation("default order by", function (_a) {
+    var createEntity = _a.createEntity;
+    return tslib_1.__awaiter(void 0, void 0, void 0, function () {
+        var s, _b;
+        return tslib_1.__generator(this, function (_c) {
+            switch (_c.label) {
+                case 0: return [4 /*yield*/, entityWithValidations_1.entityWithValidations.create4RowsInDp(createEntity)];
+                case 1:
+                    s = _c.sent();
+                    return [4 /*yield*/, s.update(1, { name: "updated name" })];
+                case 2:
+                    _c.sent();
+                    _b = expect;
+                    return [4 /*yield*/, s.find()];
+                case 3:
+                    _b.apply(void 0, [(_c.sent()).map(function (x) { return x.myId; })]).toEqual([1, 2, 3, 4]);
+                    return [2 /*return*/];
+            }
+        });
+    });
+}, false);
+backend_database_test_setup_backend_spec_1.testKnexPGSqlImpl("default order by", function (_a) {
+    var createEntity = _a.createEntity;
+    return tslib_1.__awaiter(void 0, void 0, void 0, function () {
+        var s, _b;
+        return tslib_1.__generator(this, function (_c) {
+            switch (_c.label) {
+                case 0: return [4 /*yield*/, entityWithValidations_1.entityWithValidations.create4RowsInDp(createEntity)];
+                case 1:
+                    s = _c.sent();
+                    return [4 /*yield*/, s.update(1, { name: "updated name" })];
+                case 2:
+                    _c.sent();
+                    _b = expect;
+                    return [4 /*yield*/, s.find()];
+                case 3:
+                    _b.apply(void 0, [(_c.sent()).map(function (x) { return x.myId; })]).toEqual([1, 2, 3, 4]);
+                    return [2 /*return*/];
+            }
+        });
+    });
+}, false);
 backend_database_test_setup_backend_spec_1.testPostgresImplementation("sql filter", function (_a) {
     var createEntity = _a.createEntity;
     return tslib_1.__awaiter(void 0, void 0, void 0, function () {

package/src/context.d.ts

@@ -40,7 +40,7 @@ export declare class Remult {
     /** Returns the current user's info */
     get user(): UserInfo;
     /** Set's the current user info */
-    setUser(info: UserInfo): Promise<void>;
+    setUser(info: UserInfo | undefined): Promise<void>;
     private _user;
     private _userChangeEvent;
     /** Checks if a user was authenticated */
@@ -66,7 +66,7 @@ export declare class Remult {
      *
      * If no provider is specified, `fetch` will be used as an http provider
      */
-    constructor(provider?: HttpProvider | DataProvider);
+    constructor(provider?: HttpProvider | DataProvider | typeof fetch);
     /** The api Base Url to be used in all remult calls. by default it's set to `/api`.
      *
      * Set this property in case you want to determine a non relative api url

package/src/context.js

@@ -154,10 +154,17 @@ var Remult = /** @class */ (function () {
             this._dataSource = provider;
             return;
         }
-        var http = provider;
         var dataProvider;
-        if (http) {
-            dataProvider = new HttpProviderBridgeToRestDataProviderHttpProvider(http);
+        if (!dataProvider) {
+            var http = provider;
+            if (http && http.get && http.put && http.post && http.delete) {
+                dataProvider = new HttpProviderBridgeToRestDataProviderHttpProvider(http);
+            }
+        }
+        if (!dataProvider) {
+            if (typeof provider === "function") {
+                dataProvider = new rest_data_provider_1.RestDataProviderHttpProviderUsingFetch(provider);
+            }
         }
         if (!dataProvider) {
             dataProvider = new rest_data_provider_1.RestDataProviderHttpProviderUsingFetch();
@@ -207,6 +214,8 @@ var Remult = /** @class */ (function () {
                 switch (_a.label) {
                     case 0:
                         this._user = info;
+                        if (this._user === null)
+                            this._user = undefined;
                         auth = info;
                         if (auth) {
                             if (!this._user.id && auth.sub)

package/src/data-providers/sql-database.js

@@ -211,6 +211,9 @@ var ActualSQLServerDataProvider = /** @class */ (function () {
                         if (options.limit) {
                             options.orderBy = sort_1.Sort.createUniqueSort(this.entity, options.orderBy);
                         }
+                        if (!options.orderBy) {
+                            options.orderBy = sort_1.Sort.createUniqueSort(this.entity, new sort_1.Sort());
+                        }
                         if (options.orderBy) {
                             first = true;
                             segs = [];