remult 0.25.6-exp.7 → 0.25.7-exp.0

package/esm/remult-knex/index.js

@@ -435,7 +435,7 @@ function supportsJsonLoadingAndSaving(knex) {
 export function buildColumn(x, dbName, b, supportsJson = true) {
     if (x.valueType == Number) {
         if (!x.valueConverter.fieldTypeInDb) {
-            let c = b.decimal(dbName);
+            let c = b.decimal(dbName, 18, 2);
             if (!x.allowNull) {
                 c.defaultTo(0).notNullable();
             }

package/esm/server/expressBridge.js

@@ -286,32 +286,37 @@ export class RemultServerImplementation {
             }
             let myReq = new ExpressRequestBridgeToDataApiRequest(genReq);
             let myRes = new ExpressResponseBridgeToDataApiResponse(origRes, req);
-            await this.runWithRemult(async (remult) => {
-                if (req) {
-                    let user;
-                    if (this.options.getUser)
-                        user = await this.options.getUser(req);
-                    else {
-                        user = req['user'];
-                        if (!user)
-                            user = req['auth'];
-                    }
-                    if (user)
-                        remult.user = user;
-                    if (this.options.initRequest) {
-                        await this.options.initRequest(req, {
-                            remult,
-                            get liveQueryStorage() {
-                                return remult.liveQueryStorage;
-                            },
-                            set liveQueryStorage(value) {
-                                remult.liveQueryStorage = value;
-                            },
-                        });
+            try {
+                await this.runWithRemult(async (remult) => {
+                    if (req) {
+                        let user;
+                        if (this.options.getUser)
+                            user = await this.options.getUser(req);
+                        else {
+                            user = req['user'];
+                            if (!user)
+                                user = req['auth'];
+                        }
+                        if (user)
+                            remult.user = user;
+                        if (this.options.initRequest) {
+                            await this.options.initRequest(req, {
+                                remult,
+                                get liveQueryStorage() {
+                                    return remult.liveQueryStorage;
+                                },
+                                set liveQueryStorage(value) {
+                                    remult.liveQueryStorage = value;
+                                },
+                            });
+                        }
                     }
-                }
-                await what(remult, myReq, myRes, genReq, origRes, req);
-            });
+                    await what(remult, myReq, myRes, genReq, origRes, req);
+                });
+            }
+            catch (err) {
+                myRes.error(serializeError(err));
+            }
         };
     }
     async getRemult(req) {

package/esm/src/remult3/remult3.js

@@ -10,8 +10,18 @@ export {};
 #
 
 //y1 - getFields didn't work for kobi in the home component
-//y1 - talk about export api is not possible in +server - how did we face that in others, and how do we doc it - ended up using _api
-//y1 - number default storage in knex is decimal and it by default has 8,2 - meaning only 999,999 - I think it should be way bigger than that. in firefly we do at least 18 - https://discord.com/channels/975754286384418847/1195162533447876658
+//p1 - JYC - change name of add parameter and .... to token - param
+//p1 - Hint names for sql parameters
+//p1 - add section to Fields doc, explaining field type in db
+/*p1 - JYC - Delete many, update many, insert many
+  - post + __action=delet
+  */
+//p1 - JYC - there should be a required that says not null or undefined, but space, 0 etc... should be ok - NOAM- check if removing zero is good enough
+/*y1 - Talk JYC - JYC - add some integrity checks on delete
+  - soft delete
+  - delete restrict (implicit, or user selected - and if so, how) (delete & update of id)
+
+*/
 //y1 - admin url!
 //y1 - consider sql expression gets a dbnames of it's own (that already has the "tableName" defined correctly) maybe also the filter translator
 //y1 - talk about modules in init express with entities/controllers,initRequest,initApi
@@ -20,9 +30,9 @@ export {};
 //y1 - support get with backend method, with url search params as the first parameter, & url as second parameter
 //y1 - talk about the parameter issue with backend methods
 //y1 - select data provider per entity https://discord.com/channels/975754286384418847/976006081748807690/1201415305885397003
+//y1 - migrations
 //y2 - live query refresh of view on table update
 //y2 - livequery for findfirst (@JY)
-
 /*y2 -
 //y2 - allow api update only for new rows
   @Fields.string<Category>({

package/esm/src/validators.js

@@ -12,7 +12,7 @@ export class Validators {
             return true;
     }, 'already exists');
     /**
-     * @deprecated is `unique` instead - it also runs only on the backend
+     * @deprecated use `unique` instead - it also runs only on the backend
      */
     static uniqueOnBackend = createValidator(async (_, e) => {
         if (e.isBackend() && (e.isNew || e.valueChanged())) {

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "remult",
-    "version": "0.25.6-exp.7",
+    "version": "0.25.7-exp.0",
     "description": "A CRUD framework for full-stack TypeScript",
     "homepage": "https://remult.dev",
     "repository": {

package/remult-knex/index.js

@@ -894,7 +894,7 @@ function buildColumn(x, dbName, b, supportsJson) {
     if (supportsJson === void 0) { supportsJson = true; }
     if (x.valueType == Number) {
         if (!x.valueConverter.fieldTypeInDb) {
-            var c = b.decimal(dbName);
+            var c = b.decimal(dbName, 18, 2);
             if (!x.allowNull) {
                 c.defaultTo(0).notNullable();
             }

package/server/expressBridge.js

@@ -527,7 +527,7 @@ var RemultServerImplementation = /** @class */ (function () {
     RemultServerImplementation.prototype.process = function (what) {
         var _this = this;
         return function (req, origRes) { return tslib_1.__awaiter(_this, void 0, void 0, function () {
-            var genReq, myReq, myRes;
+            var genReq, myReq, myRes, err_1;
             var _this = this;
             return tslib_1.__generator(this, function (_a) {
                 switch (_a.label) {
@@ -544,6 +544,9 @@ var RemultServerImplementation = /** @class */ (function () {
                         }
                         myReq = new ExpressRequestBridgeToDataApiRequest(genReq);
                         myRes = new ExpressResponseBridgeToDataApiResponse(origRes, req);
+                        _a.label = 1;
+                    case 1:
+                        _a.trys.push([1, 3, , 4]);
                         return [4 /*yield*/, this.runWithRemult(function (remult) { return tslib_1.__awaiter(_this, void 0, void 0, function () {
                                 var user;
                                 return tslib_1.__generator(this, function (_a) {
@@ -584,9 +587,14 @@ var RemultServerImplementation = /** @class */ (function () {
                                     }
                                 });
                             }); })];
-                    case 1:
+                    case 2:
                         _a.sent();
-                        return [2 /*return*/];
+                        return [3 /*break*/, 4];
+                    case 3:
+                        err_1 = _a.sent();
+                        myRes.error((0, data_api_js_1.serializeError)(err_1));
+                        return [3 /*break*/, 4];
+                    case 4: return [2 /*return*/];
                 }
             });
         }); };

package/src/context.d.ts

@@ -132,8 +132,9 @@ export interface ApiClient {
     subscriptionClient?: SubscriptionClient;
     /**
      * A function that wraps message handling for subscriptions. This is useful for executing some code before
-     * or after any message arrives from the subscription. For example, in Angular, this can be used to trigger
-     * a render by calling the `NgZone` run method.
+     * or after any message arrives from the subscription.
+     * For example, in Angular, to refresh a specific part of the UI,
+     * you can call the `NgZone` run method at this time.
      *
      * @example
      * // Angular example

package/src/remult3/remult3.d.ts

@@ -194,7 +194,7 @@ export interface EntityMetadata<entityType = any> {
      */
     readonly dbName: string;
     /** The options send to the `Entity`'s decorator */
-    readonly options: EntityOptions<entityType>;
+    readonly options: EntityOptions;
     /** The class type of the entity */
     readonly entityType: ClassType<entityType>;
     /** true if the current user is allowed to update an entity instance
@@ -347,7 +347,7 @@ export interface Repository<entityType> {
      */
     metadata: EntityMetadata<entityType>;
     addEventListener(listener: entityEventListener<entityType>): Unsubscribe;
-    relations: (item: entityType) => RepositoryRelations<entityType>;
+    relations(item: entityType): RepositoryRelations<entityType>;
 }
 /**
  * The `LiveQuery` interface represents a live query that allows subscribing to changes in the query results.

package/src/remult3/remult3.js

@@ -11,8 +11,18 @@ Object.defineProperty(exports, "__esModule", { value: true });
 #
 
 //y1 - getFields didn't work for kobi in the home component
-//y1 - talk about export api is not possible in +server - how did we face that in others, and how do we doc it - ended up using _api
-//y1 - number default storage in knex is decimal and it by default has 8,2 - meaning only 999,999 - I think it should be way bigger than that. in firefly we do at least 18 - https://discord.com/channels/975754286384418847/1195162533447876658
+//p1 - JYC - change name of add parameter and .... to token - param
+//p1 - Hint names for sql parameters
+//p1 - add section to Fields doc, explaining field type in db
+/*p1 - JYC - Delete many, update many, insert many
+  - post + __action=delet
+  */
+//p1 - JYC - there should be a required that says not null or undefined, but space, 0 etc... should be ok - NOAM- check if removing zero is good enough
+/*y1 - Talk JYC - JYC - add some integrity checks on delete
+  - soft delete
+  - delete restrict (implicit, or user selected - and if so, how) (delete & update of id)
+
+*/
 //y1 - admin url!
 //y1 - consider sql expression gets a dbnames of it's own (that already has the "tableName" defined correctly) maybe also the filter translator
 //y1 - talk about modules in init express with entities/controllers,initRequest,initApi
@@ -21,9 +31,9 @@ Object.defineProperty(exports, "__esModule", { value: true });
 //y1 - support get with backend method, with url search params as the first parameter, & url as second parameter
 //y1 - talk about the parameter issue with backend methods
 //y1 - select data provider per entity https://discord.com/channels/975754286384418847/976006081748807690/1201415305885397003
+//y1 - migrations
 //y2 - live query refresh of view on table update
 //y2 - livequery for findfirst (@JY)
-
 /*y2 -
 //y2 - allow api update only for new rows
   @Fields.string<Category>({

package/src/validators.d.ts

@@ -4,7 +4,7 @@ export declare class Validators {
     static required: Validator<any>;
     static unique: Validator<any>;
     /**
-     * @deprecated is `unique` instead - it also runs only on the backend
+     * @deprecated use `unique` instead - it also runs only on the backend
      */
     static uniqueOnBackend: Validator<any>;
     static regex: ValidatorWithArgs<string, RegExp> & {

package/src/validators.js

@@ -27,7 +27,7 @@ var Validators = /** @class */ (function () {
         });
     }); }, 'already exists');
     /**
-     * @deprecated is `unique` instead - it also runs only on the backend
+     * @deprecated use `unique` instead - it also runs only on the backend
      */
     Validators.uniqueOnBackend = createValidator(function (_, e) { return tslib_1.__awaiter(void 0, void 0, void 0, function () {
         var _b;