remult 0.13.20 → 0.13.23

package/package.json

@@ -1,7 +1,7 @@
 {
     "name": "remult",
-    "version": "0.13.20",
-    "description": "remult core lib",
+    "version": "0.13.23",
+    "description": "A CRUD framework for full-stack TypeScript",
     "homepage": "https://remult.dev/",
     "repository": {
         "type": "git",
@@ -13,7 +13,6 @@
     },
     "dependencies": {
         "uuid": "^8.3.2",
-        "body-parser": "^1.18.3",
         "reflect-metadata": "^0.1.13",
         "tslib": "^2.3.1"
     }

package/readme.md

@@ -4,7 +4,7 @@
   </a>
   <h1>Remult</h1>
 	<a href="https://circleci.com/gh/remult/remult/tree/master">
-		<img alt="CircleCI" src="https://circleci.com/gh/remult/remult/tree/master.svg?style=svg">
+		<img alt="CircleCI" src="https://circleci.com/gh/remult/remult/tree/master.svg?style=shield">
 	</a>
 	<a href="https://codecov.io/gh/remult/remult">
         	<img src="https://codecov.io/gh/remult/remult/branch/master/graph/badge.svg?token=LYWQRUN3D1"/>
@@ -18,33 +18,36 @@
 	<a href="https://www.npmjs.com/package/remult">
 		<img alt="npm downloads" src="https://img.shields.io/npm/dm/remult">
 	</a>
-	<a href="https://snyk.io/test/npm/remult">
-		<img alt="snyk" src="https://snyk.io/test/npm/remult/badge.svg"/>
-		</a>
-	<a href="https://github.com/remult/remult/pulse" alt="Activity">
-        <img src="https://img.shields.io/github/commit-activity/m/remult/remult" /></a>
+	<a href="https://discord.gg/GXHk7ZfuG5">
+		<img alt="chat on discord" src="https://img.shields.io/badge/chat-on%20discord-blueviolet"/>
+	</a>
 </div>
 
 ## What is Remult?
 
-**Remult** is a fullstack CRUD framework that uses your TypeScript model types to provide:
+**Remult** is a full-stack CRUD framework that uses your TypeScript model types to provide:
 
 * Secure REST API (highly configurable)
 * Type-safe frontend API client
 * Type-safe backend query builder
 
-#### Remult :heart: Monorepos
+#### Remult :heart: Code Sharing
 
-Using a `monorepo` approach, with model types shared between frontend and backend code, Remult can enforce data validation and constraints, defined once, on both frontend and REST API levels.
+With model types shared between frontend and backend code, Remult can **enforce data validation and constraints, defined once, both in the front-end and witin back-end API routes**.
 
 ## Getting started
-The best way to learn Remult is by following a tutorial of a simple Todo app with a Node.js Express backend. There's one [using a React frontend](https://remult.dev/tutorials/tutorial-react.html) and one [using Angular](https://remult.dev/tutorials/tutorial-angular.html).
+The best way to learn Remult is by following a tutorial of a simple Todo web app with a Node.js Express backend. 
+
+* [Tutorial with React](https://remult.dev/tutorials/react/) 
+* [Tutorial with Angular](https://remult.dev/tutorials/angular/)
 
 ## Installation
 ```sh
 npm i remult
 ```
 
+## Usage
+
 ### Setup API backend using a Node.js Express middleware
 ```ts
 import express from 'express';
@@ -60,14 +63,14 @@ app.listen(port, () => {
 });
 ```
 
-## Define model classes
+### Define model classes
 ```ts
-import { Entity, EntityBase, Fields } from 'remult';
+import { Entity, Fields } from 'remult';
 
 @Entity('products', {
     allowApiCrud: true
 })
-export class Product extends EntityBase {
+export class Product {
   @Fields.string()
   name = '';
 
@@ -83,13 +86,14 @@ export class Product extends EntityBase {
 [{"name":"Tofu","unitPrice":5}]
 ```
 
-## Find and manipulate data in type-safe frontend code
+### Find and manipulate data in type-safe frontend code
 ```ts
 async function increasePriceOfTofu(priceIncrease: number) {
-  const product = await remult.repo(Product).findFirst({ name: 'Tofu' });
+  const productsRepo = remult.repo(Product);
 
+  const product = await productsRepo.findFirst({ name: 'Tofu' });
   product.unitPrice += priceIncrease;
-  product.save();
+  productsRepo.save(product);
 }
 ```
 
@@ -97,26 +101,27 @@ async function increasePriceOfTofu(priceIncrease: number) {
 ```ts
 @BackendMethod({ allowed: Allow.authenticated })
 static async increasePriceOfTofu(priceIncrease: number, remult?: Remult) {
-  const product = await remult!.repo(Product).findFirst({ name: 'Tofu' });
+  const productsRepo = remult!.repo(Product);
 
+  const product = await productsRepo.findFirst({ name: 'Tofu' });
   product.unitPrice += priceIncrease;
-  product.save();
+  productsRepo.save(product);
 }
 ```
 
-## :ballot_box_with_check: Data validation and constraints - defined once
+### :ballot_box_with_check: Data validation and constraints - defined once
 
 ```ts
-import { Entity, EntityBase, Fields } from 'remult';
+import { Entity, Fields } from 'remult';
 
 @Entity('products', {
     allowApiCrud: true
 })
-export class Product extends EntityBase {
+export class Product {
     @Fields.string<Product>({
         validate: product => {
             if (product.name.trim().length == 0)
-                product.$.name.error = 'required';
+                throw 'required';
         }
     })
     name = '';
@@ -124,7 +129,7 @@ export class Product extends EntityBase {
     @Fields.number({
         validate: (_, field) => {
             if (field.value < 0)
-                field.error = "must not be less than 0";
+                throw "must not be less than 0";
         }
     })
     unitPrice = 0;
@@ -133,31 +138,31 @@ export class Product extends EntityBase {
 
 ### Enforced in frontend:
 ```ts
-const product = remult.repo(Product).create();
+const product = productsRepo.create();
 
 try {
-  await product.save();
+  await productsRepo.save(product);
 }
-catch {
-  console.error(product._.error); // Browser console will display - "Name: required"
+catch (e: any) {
+  console.error(e.message); // Browser console will display - "Name: required"
 }
 ```
 
 ### Enforced in backend:
 ```sh
-> curl -d "{""unitPrice"":-1}" -H "Content-Type: application/json" -X POST http://localhost:3001/api/products
+> curl http://localhost:3001/api/products -H "Content-Type: application/json" -d "{""unitPrice"":-1}"
 
 {"modelState":{"unitPrice":"must not be less than 0","name":"required"},"message":"Name: required"}
 ```
 
-## :lock: Secure the API with fine-grained authorization
+### :lock: Secure the API with fine-grained authorization
 ```ts
 @Entity<Article>('Articles', {
     allowApiRead: true,
     allowApiInsert: remult => remult.authenticated(),
     allowApiUpdate: (remult, article) => article.author.id == remult.user.id
 })
-export class Article extends EntityBase {
+export class Article {
     @Fields.string({ allowApiUpdate: false })
     slug = '';
     
@@ -168,3 +173,13 @@ export class Article extends EntityBase {
     content = '';
 }
 ```
+
+## Example App
+
+[CRM demo](https://github.com/remult/crm-demo) with a React + [MUI](https://mui.com) front-end and Postgres database.
+
+## Contributing
+Contributions are welcome. See [CONTRIBUTING](CONTRIBUTING.md).
+
+## License
+Remult is [MIT Licensed](LICENSE).

package/server/expressBridge.d.ts

@@ -6,10 +6,10 @@ import { DataApi, DataApiRequest, DataApiResponse } from '../src/data-api';
 import { AllowedForInstance } from '../src/context';
 import { ClassType } from '../classType';
 export declare function remultExpress(options?: {
-    /** Determines the Data Provider
-  
-  * @see  [Optional Databases](https://remult.dev/docs/databases.html)
-  */
+    /** Sets a database connection for Remult.
+     *
+     * @see [Connecting to a Database](https://remult.dev/docs/databases.html).
+    */
     dataProvider?: DataProvider | Promise<DataProvider> | (() => Promise<DataProvider | undefined>);
     bodySizeLimit?: string;
     disableAutoApi?: boolean;
@@ -20,6 +20,7 @@ export declare function remultExpress(options?: {
     defaultGetLimit?: number;
     entities?: ClassType<any>[];
     controllers?: ClassType<any>[];
+    bodyParser?: boolean;
 }): RemultExpressBridge;
 export interface RemultExpressBridge extends express.RequestHandler {
     getRemult(req: express.Request): Promise<Remult>;

package/server/expressBridge.js

@@ -4,7 +4,6 @@ exports.JobsInQueueEntity = exports.EntityQueueStorage = exports.ExpressRequestB
 var tslib_1 = require("tslib");
 var __1 = require("../");
 var express = require("express");
-var bodyParser = require("body-parser");
 var register_actions_on_server_1 = require("./register-actions-on-server");
 var register_entities_on_server_1 = require("./register-entities-on-server");
 var JsonEntityFileStorage_1 = require("./JsonEntityFileStorage");
@@ -29,8 +28,10 @@ function remultExpress(options) {
     if (!options.queueStorage) {
         options.queueStorage = new InMemoryQueueStorage();
     }
-    app.use(bodyParser.json({ limit: options.bodySizeLimit }));
-    app.use(bodyParser.urlencoded({ extended: true, limit: options.bodySizeLimit }));
+    if ((options === null || options === void 0 ? void 0 : options.bodyParser) !== false) {
+        app.use(express.json({ limit: options.bodySizeLimit }));
+        app.use(express.urlencoded({ extended: true, limit: options.bodySizeLimit }));
+    }
     var dataProvider;
     if (typeof options.dataProvider === "function") {
         dataProvider = options.dataProvider();
@@ -62,7 +63,12 @@ function remultExpress(options) {
         bridge.logApiEndPoints = options.logApiEndPoints;
     var apiArea = bridge.addArea(__1.Remult.apiBaseUrl);
     if (!options.disableAutoApi) {
-        register_actions_on_server_1.registerActionsOnServer(apiArea);
+        var actions = [];
+        if (options.entities)
+            actions.push.apply(actions, tslib_1.__spread(options.entities));
+        if (options.controllers)
+            actions.push.apply(actions, tslib_1.__spread(options.controllers));
+        register_actions_on_server_1.registerActionsOnServer(apiArea, actions);
         register_entities_on_server_1.registerEntitiesOnServer(apiArea);
     }
     return Object.assign(app, {

package/server/register-actions-on-server.d.ts

@@ -1,2 +1,3 @@
 import { SiteArea } from "./expressBridge";
-export declare function registerActionsOnServer(area: SiteArea): void;
+import { ClassType } from "../classType";
+export declare function registerActionsOnServer(area: SiteArea, controllers?: ClassType<any>[]): void;

package/server/register-actions-on-server.js

@@ -1,8 +1,10 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.registerActionsOnServer = void 0;
+var tslib_1 = require("tslib");
 var server_action_1 = require("../src/server-action");
-function registerActionsOnServer(area) {
+function registerActionsOnServer(area, controllers) {
+    var e_1, _a, e_2, _b;
     var addAction = function (a) {
         var x = a[server_action_1.serverActionField];
         if (!x) {
@@ -11,8 +13,38 @@ function registerActionsOnServer(area) {
         area.addAction(x);
     };
     server_action_1.actionInfo.runningOnServer = true;
-    server_action_1.actionInfo.allActions.forEach(function (a) {
-        addAction(a);
-    });
+    if (controllers && controllers.length > 0) {
+        try {
+            for (var controllers_1 = tslib_1.__values(controllers), controllers_1_1 = controllers_1.next(); !controllers_1_1.done; controllers_1_1 = controllers_1.next()) {
+                var c = controllers_1_1.value;
+                var z = c[server_action_1.classBackendMethodsArray];
+                if (z)
+                    try {
+                        for (var z_1 = (e_2 = void 0, tslib_1.__values(z)), z_1_1 = z_1.next(); !z_1_1.done; z_1_1 = z_1.next()) {
+                            var a = z_1_1.value;
+                            addAction(a);
+                        }
+                    }
+                    catch (e_2_1) { e_2 = { error: e_2_1 }; }
+                    finally {
+                        try {
+                            if (z_1_1 && !z_1_1.done && (_b = z_1.return)) _b.call(z_1);
+                        }
+                        finally { if (e_2) throw e_2.error; }
+                    }
+            }
+        }
+        catch (e_1_1) { e_1 = { error: e_1_1 }; }
+        finally {
+            try {
+                if (controllers_1_1 && !controllers_1_1.done && (_a = controllers_1.return)) _a.call(controllers_1);
+            }
+            finally { if (e_1) throw e_1.error; }
+        }
+    }
+    else
+        server_action_1.actionInfo.allActions.forEach(function (a) {
+            addAction(a);
+        });
 }
 exports.registerActionsOnServer = registerActionsOnServer;

package/server/register-entities-on-server.d.ts

@@ -1,2 +1,3 @@
 import { SiteArea } from "./expressBridge";
-export declare function registerEntitiesOnServer(area: SiteArea): void;
+import { ClassType } from "../classType";
+export declare function registerEntitiesOnServer(area: SiteArea, entities?: ClassType<any>[]): void;

package/server/register-entities-on-server.js

@@ -4,9 +4,11 @@ exports.registerEntitiesOnServer = void 0;
 var remult3_1 = require("../src/remult3");
 var context_1 = require("../src/context");
 var data_api_1 = require("../src/data-api");
-function registerEntitiesOnServer(area) {
+function registerEntitiesOnServer(area, entities) {
+    if (!entities)
+        entities = context_1.allEntities;
     //add Api Entries
-    context_1.allEntities.forEach(function (e) {
+    entities.forEach(function (e) {
         var key = remult3_1.getEntityKey(e);
         if (key != undefined)
             area.add(key, function (c) {

package/src/data-providers/web-sql-data-provider.d.ts

@@ -6,7 +6,7 @@ export declare class WebSqlDataProvider implements SqlImplementation, __RowsOfDa
     rows: {
         [tableName: string]: any;
     };
-    constructor(databaseName: string);
+    constructor(databaseName: string, databaseSize?: number);
     getLimitSqlSyntax(limit: number, offset: number): string;
     entityIsUsedForTheFirstTime(entity: EntityMetadata): Promise<void>;
     dropTable(entity: EntityMetadata): Promise<void>;

package/src/data-providers/web-sql-data-provider.js

@@ -7,10 +7,11 @@ var sql_database_1 = require("./sql-database");
 var filter_consumer_bridge_to_sql_request_1 = require("../filter/filter-consumer-bridge-to-sql-request");
 //SqlDatabase.LogToConsole = true;
 var WebSqlDataProvider = /** @class */ (function () {
-    function WebSqlDataProvider(databaseName) {
+    function WebSqlDataProvider(databaseName, databaseSize) {
+        if (databaseSize === void 0) { databaseSize = 2 * 1024 * 1024; }
         this.databaseName = databaseName;
         //@ts-ignore
-        this.db = window.openDatabase(databaseName, '1.0', databaseName, 2 * 1024 * 1024);
+        this.db = window.openDatabase(databaseName, '1.0', databaseName, databaseSize);
     }
     WebSqlDataProvider.prototype.getLimitSqlSyntax = function (limit, offset) {
         return ' limit ' + limit + ' offset ' + offset;

package/src/remult3/RepositoryImplementation.d.ts

@@ -26,17 +26,17 @@ export declare class RepositoryImplementation<entityType> implements Repository<
     query(options?: QueryOptions<entityType>): QueryResult<entityType>;
     getEntityRef(entity: entityType): EntityRef<entityType>;
     delete(id: (entityType extends {
-        id: number;
+        id?: number;
     } ? number : entityType extends {
-        id: string;
+        id?: string;
     } ? string : (string | number))): Promise<void>;
     delete(item: entityType): Promise<void>;
     insert(item: Partial<OmitEB<entityType>>[]): Promise<entityType[]>;
     insert(item: Partial<OmitEB<entityType>>): Promise<entityType>;
     update(id: (entityType extends {
-        id: number;
+        id?: number;
     } ? number : entityType extends {
-        id: string;
+        id?: string;
     } ? string : (string | number)), entity: Partial<OmitEB<entityType>>): Promise<entityType>;
     private getRefForExistingRow;
     save(item: Partial<OmitEB<entityType>>[]): Promise<entityType[]>;
@@ -157,6 +157,7 @@ export declare class FieldRefImplementation<entityType, valueType> implements Fi
     get originalValue(): any;
     private rawOriginalValue;
     setId(id: (string | number)): void;
+    getId(): any;
     get inputValue(): string;
     set inputValue(val: string);
     valueChanged(): boolean;
@@ -223,7 +224,7 @@ export declare function isAutoIncrement(f: FieldMetadata): boolean;
 export interface StringFieldOptions<entityType = any> extends FieldOptions<entityType, string> {
     maxLength?: number;
 }
-export declare function ValueListFieldType<entityType = any, valueType extends ValueListItem = any>(...options: (ValueListFieldOptions<entityType, valueType> | ((options: FieldOptions<entityType, valueType>, remult: Remult) => void))[]): (type: ClassType<valueType>) => void;
+export declare function ValueListFieldType<valueType extends ValueListItem = any>(...options: (ValueListFieldOptions<any, valueType> | ((options: FieldOptions<any, valueType>, remult: Remult) => void))[]): (type: ClassType<valueType>) => void;
 export interface ValueListFieldOptions<entityType, valueType> extends FieldOptions<entityType, valueType> {
     getValues?: () => valueType[];
 }

package/src/remult3/RepositoryImplementation.js

@@ -200,49 +200,50 @@ var RepositoryImplementation = /** @class */ (function () {
     };
     RepositoryImplementation.prototype.insert = function (entity) {
         return tslib_1.__awaiter(this, void 0, void 0, function () {
-            var entity_1, entity_1_1, item, e_4_1, ref;
-            var e_4, _a;
-            return tslib_1.__generator(this, function (_b) {
-                switch (_b.label) {
+            var r, entity_1, entity_1_1, item, _a, _b, e_4_1, ref;
+            var e_4, _c;
+            return tslib_1.__generator(this, function (_d) {
+                switch (_d.label) {
                     case 0:
                         if (!Array.isArray(entity)) return [3 /*break*/, 9];
-                        _b.label = 1;
+                        r = [];
+                        _d.label = 1;
                     case 1:
-                        _b.trys.push([1, 6, 7, 8]);
+                        _d.trys.push([1, 6, 7, 8]);
                         entity_1 = tslib_1.__values(entity), entity_1_1 = entity_1.next();
-                        _b.label = 2;
+                        _d.label = 2;
                     case 2:
                         if (!!entity_1_1.done) return [3 /*break*/, 5];
                         item = entity_1_1.value;
+                        _b = (_a = r).push;
                         return [4 /*yield*/, this.insert(item)];
                     case 3:
-                        _b.sent();
-                        _b.label = 4;
+                        _b.apply(_a, [_d.sent()]);
+                        _d.label = 4;
                     case 4:
                         entity_1_1 = entity_1.next();
                         return [3 /*break*/, 2];
                     case 5: return [3 /*break*/, 8];
                     case 6:
-                        e_4_1 = _b.sent();
+                        e_4_1 = _d.sent();
                         e_4 = { error: e_4_1 };
                         return [3 /*break*/, 8];
                     case 7:
                         try {
-                            if (entity_1_1 && !entity_1_1.done && (_a = entity_1.return)) _a.call(entity_1);
+                            if (entity_1_1 && !entity_1_1.done && (_c = entity_1.return)) _c.call(entity_1);
                         }
                         finally { if (e_4) throw e_4.error; }
                         return [7 /*endfinally*/];
-                    case 8: return [3 /*break*/, 13];
+                    case 8: return [2 /*return*/, r];
                     case 9:
                         ref = getEntityRef(entity, false);
                         if (!ref) return [3 /*break*/, 11];
                         if (!ref.isNew())
                             throw "Item is not new";
                         return [4 /*yield*/, ref.save()];
-                    case 10: return [2 /*return*/, _b.sent()];
+                    case 10: return [2 /*return*/, _d.sent()];
                     case 11: return [4 /*yield*/, this.getEntityRef(this.create(entity)).save()];
-                    case 12: return [2 /*return*/, _b.sent()];
-                    case 13: return [2 /*return*/];
+                    case 12: return [2 /*return*/, _d.sent()];
                 }
             });
         });
@@ -298,7 +299,7 @@ var RepositoryImplementation = /** @class */ (function () {
                 switch (_a.label) {
                     case 0:
                         if (!Array.isArray(entity)) return [3 /*break*/, 1];
-                        return [2 /*return*/, Promise.all(entity.map(function (x) { return _this.insert(x); }))];
+                        return [2 /*return*/, Promise.all(entity.map(function (x) { return _this.save(x); }))];
                     case 1:
                         ref = getEntityRef(entity, false);
                         if (!ref) return [3 /*break*/, 3];
@@ -649,23 +650,24 @@ var RepositoryImplementation = /** @class */ (function () {
 exports.RepositoryImplementation = RepositoryImplementation;
 function __updateEntityBasedOnWhere(entityDefs, where, r) {
     var w = filter_interfaces_1.Filter.fromEntityFilter(entityDefs, where);
+    var emptyFunction = function () { };
     if (w) {
         w.__applyToConsumer({
-            custom: function () { },
-            databaseCustom: function () { },
-            containsCaseInsensitive: function () { },
-            isDifferentFrom: function () { },
+            custom: emptyFunction,
+            databaseCustom: emptyFunction,
+            containsCaseInsensitive: emptyFunction,
+            isDifferentFrom: emptyFunction,
             isEqualTo: function (col, val) {
                 r[col.key] = val;
             },
-            isGreaterOrEqualTo: function () { },
-            isGreaterThan: function () { },
-            isIn: function () { },
-            isLessOrEqualTo: function () { },
-            isLessThan: function () { },
-            isNotNull: function () { },
-            isNull: function () { },
-            or: function () { }
+            isGreaterOrEqualTo: emptyFunction,
+            isGreaterThan: emptyFunction,
+            isIn: emptyFunction,
+            isLessOrEqualTo: emptyFunction,
+            isLessThan: emptyFunction,
+            isNotNull: emptyFunction,
+            isNull: emptyFunction,
+            or: emptyFunction
         });
     }
 }
@@ -1895,6 +1897,12 @@ var FieldRefImplementation = /** @class */ (function () {
     FieldRefImplementation.prototype.setId = function (id) {
         this.value = id;
     };
+    FieldRefImplementation.prototype.getId = function () {
+        var lu = this.rowBase.lookups.get(this.metadata.key);
+        if (lu)
+            return lu.id != undefined ? lu.id : null;
+        return this.value;
+    };
     Object.defineProperty(FieldRefImplementation.prototype, "inputValue", {
         get: function () {
             this.reportObserved();

package/src/remult3/remult3.d.ts

@@ -42,7 +42,7 @@ export interface Subscribable {
 }
 export declare type FieldsRef<entityType> = {
     [Properties in keyof OmitEB<entityType>]: entityType[Properties] extends {
-        id: (number | string);
+        id?: (number | string);
     } ? IdFieldRef<entityType, entityType[Properties]> : FieldRef<entityType, entityType[Properties]>;
 } & {
     find(fieldMetadataOrKey: FieldMetadata | string): FieldRef<entityType, any>;
@@ -63,10 +63,15 @@ export declare type SortSegments<entityType> = {
 };
 export interface IdFieldRef<entityType, valueType> extends FieldRef<entityType, valueType> {
     setId(id: valueType extends {
-        id: number;
+        id?: number;
     } ? number : valueType extends {
-        id: string;
+        id?: string;
     } ? string : (string | number)): any;
+    getId(): (valueType extends {
+        id?: number;
+    } ? number : valueType extends {
+        id?: string;
+    } ? string : (string | number));
 }
 export interface FieldRef<entityType = any, valueType = any> extends Subscribable {
     error: string;
@@ -77,7 +82,7 @@ export interface FieldRef<entityType = any, valueType = any> extends Subscribabl
     valueChanged(): boolean;
     entityRef: EntityRef<entityType>;
     container: entityType;
-    metadata: FieldMetadata<entityType>;
+    metadata: FieldMetadata<valueType>;
     load(): Promise<valueType>;
     valueIsNull(): boolean;
     originalValueIsNull(): boolean;
@@ -111,9 +116,9 @@ export interface Repository<entityType> {
     findFirst(where?: EntityFilter<entityType>, options?: FindFirstOptions<entityType>): Promise<entityType>;
     /** returns the items that matches the idm the result is cached unless specified differently in the `options` parameter */
     findId(id: entityType extends {
-        id: number;
+        id?: number;
     } ? number : entityType extends {
-        id: string;
+        id?: string;
     } ? string : (string | number), options?: FindFirstOptionsBase<entityType>): Promise<entityType>;
     /**  An alternative form of fetching data from the API server, which is intended for operating on large numbers of entity objects.
      *
@@ -152,15 +157,15 @@ export interface Repository<entityType> {
      * taskRepo.update(task.id,{...task,completed:true})
     */
     update(id: (entityType extends {
-        id: number;
+        id?: number;
     } ? number : entityType extends {
-        id: string;
+        id?: string;
     } ? string : (string | number)), item: Partial<OmitEB<entityType>>): Promise<entityType>;
     /** Deletes an Item*/
     delete(id: (entityType extends {
-        id: number;
+        id?: number;
     } ? number : entityType extends {
-        id: string;
+        id?: string;
     } ? string : (string | number))): Promise<void>;
     delete(item: entityType): Promise<void>;
     /** Creates an instance of an item. It'll not be saved to the data source unless `save` or `insert` will be called for that item */
@@ -205,7 +210,7 @@ export declare type EntityOrderBy<entityType> = {
  */
 export declare type EntityFilter<entityType> = {
     [Properties in keyof Partial<OmitEB<entityType>>]?: (Partial<OmitEB<entityType>>[Properties] extends (number | Date | undefined) ? ComparisonValueFilter<Partial<OmitEB<entityType>>[Properties]> : Partial<OmitEB<entityType>>[Properties] extends (string | undefined) ? ContainsStringValueFilter & ComparisonValueFilter<string> : Partial<OmitEB<entityType>>[Properties] extends (boolean | undefined) ? ValueFilter<boolean> : Partial<OmitEB<entityType>>[Properties] extends ({
-        id: (string | number);
+        id?: (string | number);
     } | undefined) ? IdFilter<Partial<OmitEB<entityType>>[Properties]> : ValueFilter<Partial<OmitEB<entityType>>[Properties]>) & ContainsStringValueFilter;
 } & {
     $or?: EntityFilter<entityType>[];
@@ -230,7 +235,7 @@ export interface ContainsStringValueFilter {
 }
 export declare type IdFilter<valueType> = ValueFilter<valueType> | {
     $id: ValueFilter<valueType extends {
-        id: number;
+        id?: number;
     } ? number : string>;
 };
 export interface LoadOptions<entityType> {
@@ -240,7 +245,7 @@ export interface FindOptionsBase<entityType> extends LoadOptions<entityType> {
     /** filters the data
     * @example
     * await taskRepo.find({where: { completed:false }})
-    * @see For more usage examples see [EntityFilter](https://remult.github.io/guide/entityFilter.html)
+    * @see For more usage examples see [EntityFilter](https://remult.dev/docs/entityFilter.html)
     */
     where?: EntityFilter<entityType>;
     /** Determines the order of items returned .

package/src/server-action.d.ts

@@ -66,4 +66,5 @@ export declare class ProgressListener {
 }
 export declare function prepareArgsToSend(types: any[], args: any[]): any[];
 export declare function prepareReceivedArgs(types: any[], args: any[], remult: Remult, ds: DataProvider, res: DataApiResponse): Promise<any[]>;
+export declare const classBackendMethodsArray: unique symbol;
 export {};

package/src/server-action.js

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.prepareReceivedArgs = exports.prepareArgsToSend = exports.ProgressListener = exports.BackendMethod = exports.Controller = exports.serverActionField = exports.actionInfo = exports.myServerAction = exports.Action = void 0;
+exports.classBackendMethodsArray = exports.prepareReceivedArgs = exports.prepareArgsToSend = exports.ProgressListener = exports.BackendMethod = exports.Controller = exports.serverActionField = exports.actionInfo = exports.myServerAction = exports.Action = void 0;
 var tslib_1 = require("tslib");
 require("reflect-metadata");
 var context_1 = require("./context");
@@ -216,7 +216,7 @@ function BackendMethod(options) {
                     });
                 });
             };
-            exports.actionInfo.allActions.push(descriptor.value);
+            registerAction(target, descriptor);
             descriptor.value[exports.serverActionField] = serverAction_1;
             return descriptor;
         }
@@ -479,7 +479,7 @@ function BackendMethod(options) {
                 });
             });
         };
-        exports.actionInfo.allActions.push(descriptor.value);
+        registerAction(target.constructor, descriptor);
         descriptor.value[exports.serverActionField] = serverAction;
         return descriptor;
     };
@@ -488,6 +488,10 @@ exports.BackendMethod = BackendMethod;
 var customUndefined = {
     _isUndefined: true
 };
+function registerAction(target, descriptor) {
+    (target[exports.classBackendMethodsArray] || (target[exports.classBackendMethodsArray] = [])).push(descriptor.value);
+    exports.actionInfo.allActions.push(descriptor.value);
+}
 function isCustomUndefined(x) {
     return x && x._isUndefined;
 }
@@ -592,3 +596,4 @@ function prepareReceivedArgs(types, args, remult, ds, res) {
     });
 }
 exports.prepareReceivedArgs = prepareReceivedArgs;
+exports.classBackendMethodsArray = Symbol('classBackendMethodsArray');

package/src/shared-tests/entityWithValidations.js

@@ -64,26 +64,12 @@ var entityWithValidations = /** @class */ (function (_super) {
         remult3_1.Entity('', {
             allowApiCrud: true,
             saving: function (t) { return tslib_1.__awaiter(void 0, void 0, void 0, function () {
-                var e;
                 return tslib_1.__generator(this, function (_a) {
-                    switch (_a.label) {
-                        case 0:
-                            if (!t.name || t.name.length < 3)
-                                t._.fields.name.error = 'invalid';
-                            if (!(t._.isNew() && (!t.myId || t.myId == 0))) return [3 /*break*/, 2];
-                            return [4 /*yield*/, t.remult.repo(entityWithValidations_1).find({
-                                    orderBy: { myId: "desc" },
-                                    limit: 1
-                                })];
-                        case 1:
-                            e = _a.sent();
-                            t.myId = e.length ? e[0].myId + 1 : 1;
-                            _a.label = 2;
-                        case 2:
-                            if (!exports.testConfiguration.restDbRunningOnServer)
-                                entityWithValidations_1.savingRowCount++;
-                            return [2 /*return*/];
-                    }
+                    if (!t.name || t.name.length < 3)
+                        t._.fields.name.error = 'invalid';
+                    if (!exports.testConfiguration.restDbRunningOnServer)
+                        entityWithValidations_1.savingRowCount++;
+                    return [2 /*return*/];
                 });
             }); }
         }),

package/src/tests/TestDataApiResponse.js

@@ -7,22 +7,22 @@ var TestDataApiResponse = /** @class */ (function () {
     TestDataApiResponse.prototype.progress = function (progress) {
     };
     TestDataApiResponse.prototype.success = function (data) {
-        fail('didnt expect success: ' + JSON.stringify(data));
+        throw new Error(('didnt expect success: ' + JSON.stringify(data)));
     };
     TestDataApiResponse.prototype.forbidden = function () {
-        fail('didnt expect forbidden:');
+        throw new Error('didnt expect forbidden:');
     };
     TestDataApiResponse.prototype.created = function (data) {
-        fail('didnt expect created: ' + JSON.stringify(data));
+        throw new Error('didnt expect created: ' + JSON.stringify(data));
     };
     TestDataApiResponse.prototype.deleted = function () {
-        fail('didnt expect deleted:');
+        throw new Error('didnt expect deleted:');
     };
     TestDataApiResponse.prototype.notFound = function () {
-        fail('not found');
+        throw new Error('not found');
     };
     TestDataApiResponse.prototype.error = function (data) {
-        fail('error: ' + data + " " + JSON.stringify(data));
+        throw new Error('error: ' + data + " " + JSON.stringify(data));
     };
     return TestDataApiResponse;
 }());

package/src/tests/d.d.ts

@@ -3,5 +3,4 @@ export declare class d extends EntityBase {
     id: number;
     b: number;
     static count: number;
-    doIt(): Promise<boolean>;
 }

package/src/tests/d.js

@@ -2,22 +2,12 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.d = void 0;
 var tslib_1 = require("tslib");
-var server_action_1 = require("../server-action");
 var remult3_1 = require("../remult3");
-var dWithPrefilter_1 = require("./dWithPrefilter");
 var d = /** @class */ (function (_super) {
     tslib_1.__extends(d, _super);
     function d() {
         return _super !== null && _super.apply(this, arguments) || this;
     }
-    d.prototype.doIt = function () {
-        return tslib_1.__awaiter(this, void 0, void 0, function () {
-            return tslib_1.__generator(this, function (_a) {
-                dWithPrefilter_1.dWithPrefilter.count++;
-                return [2 /*return*/, true];
-            });
-        });
-    };
     d.count = 0;
     tslib_1.__decorate([
         remult3_1.Fields.integer(),
@@ -27,12 +17,6 @@ var d = /** @class */ (function (_super) {
         remult3_1.Fields.integer(),
         tslib_1.__metadata("design:type", Number)
     ], d.prototype, "b", void 0);
-    tslib_1.__decorate([
-        server_action_1.BackendMethod({ allowed: true }),
-        tslib_1.__metadata("design:type", Function),
-        tslib_1.__metadata("design:paramtypes", []),
-        tslib_1.__metadata("design:returntype", Promise)
-    ], d.prototype, "doIt", null);
     d = tslib_1.__decorate([
         remult3_1.Entity('d1', {
             allowApiCrud: true

package/src/tests/testModel/models.d.ts

@@ -5,7 +5,6 @@ export declare class Status {
     static closed: Status;
     static hold: Status;
     constructor(id: number, name: string);
-    toString(): string;
 }
 export declare class TestStatus {
     id?: string;

package/src/tests/testModel/models.js

@@ -9,9 +9,6 @@ var Status = /** @class */ (function () {
         this.name = name;
     }
     Status_1 = Status;
-    Status.prototype.toString = function () {
-        return this.name;
-    };
     var Status_1;
     Status.open = new Status_1(0, "open");
     Status.closed = new Status_1(1, "closed");