remult 3.2.3 → 3.3.0

package/src/remult3/remult3.d.ts

@@ -13,7 +13,7 @@ export interface UpsertOptions<entityType> {
 export interface EntityRefBase<entityType> extends Subscribable {
     hasErrors(): boolean;
     undoChanges(): void;
-    save(): Promise<entityType>;
+    save(options?: InsertOrUpdateOptions): Promise<entityType>;
     reload(): Promise<entityType>;
     delete(): Promise<void>;
     isNew(): boolean;
@@ -555,19 +555,19 @@ export interface Repository<entityType> {
      * @example
      * await taskRepo.insert([{title:"task a"}, {title:"task b", completed:true }])
      */
-    insert(item: Partial<MembersOnly<entityType>>[]): Promise<entityType[]>;
-    insert(item: Partial<MembersOnly<entityType>>): Promise<entityType>;
+    insert(item: Partial<MembersOnly<entityType>>[], options?: InsertOrUpdateOptions): Promise<entityType[]>;
+    insert(item: Partial<MembersOnly<entityType>>, options?: InsertOrUpdateOptions): Promise<entityType>;
     /** Updates an item, based on its `id`
      * @example
      * taskRepo.update(task.id,{...task,completed:true})
      */
-    update(id: idType<entityType>, item: Partial<MembersOnly<entityType>>): Promise<entityType>;
-    update(id: Partial<MembersOnly<entityType>>, item: Partial<MembersOnly<entityType>>): Promise<entityType>;
+    update(id: idType<entityType>, item: Partial<MembersOnly<entityType>>, options?: InsertOrUpdateOptions): Promise<entityType>;
+    update(id: Partial<MembersOnly<entityType>>, item: Partial<MembersOnly<entityType>>, options?: InsertOrUpdateOptions): Promise<entityType>;
     /**
      * Updates all items that match the `where` condition.
      */
     updateMany(options: {
-        where: EntityFilter<entityType>;
+        where: EntityFilter<entityType> | 'all';
         set: Partial<MembersOnly<entityType>>;
     }): Promise<number>;
     /**
@@ -607,7 +607,7 @@ export interface Repository<entityType> {
      * Deletes all items that match the `where` condition.
      */
     deleteMany(options: {
-        where: EntityFilter<entityType>;
+        where: EntityFilter<entityType> | 'all';
     }): Promise<number>;
     /** Creates an instance of an item. It'll not be saved to the data source unless `save` or `insert` will be called.
      *
@@ -1069,6 +1069,7 @@ export interface FindOptionsBase<entityType> extends LoadOptions<entityType> {
      * await repo(Products).find({ orderBy: { price: "desc", name: "asc" }})
      */
     orderBy?: EntityOrderBy<entityType>;
+    args?: any;
 }
 export interface FindFirstOptions<entityType> extends FindOptionsBase<entityType>, FindFirstOptionsBase<entityType> {
 }
@@ -1197,6 +1198,9 @@ export declare type EntityIdFields<entityType> = {
 export declare type EntitySelectFields<entityType> = {
     [Properties in keyof Partial<MembersOnly<entityType>>]?: boolean;
 };
+export declare type InsertOrUpdateOptions = {
+    select: 'none';
+};
 export interface ClassFieldDecoratorContextStub<entityType, valueType> {
     readonly access: {
         set(object: entityType, value: valueType | null): void;

package/src/remult3/remult3.js

@@ -44,6 +44,14 @@ exports.flags = {
   
   -
 */
+/*p1 - https://github.com/remult/remult/discussions/438
+  - should we use the arg for update and insert? for the returning query?
+  - Does dbNamesOf still makes sense? I think that abstraction, regarding sql expression has lost it's merit
+  - remember caching of sql expression  calculations that took a recursive amount of time for JYC
+  - maybe introduce a ready dbNamesOf of argument that will be aware of prefixes
+  - I've changed the order by to support order by 1
+
+*/
 //p1 - deleteAll({title:undefined}) should throw an error - not return 0 (with direct call to db)
 //p1 - remult-create, move db question ahead of auth - everyone needs a database, not everyone need auth
 //p1 - allow experimental route registration on remult server, with at least get route, and support redirect, read header and set header - (and the existing get html etc...)

package/src/valueConverters.d.ts

@@ -2,7 +2,9 @@ import type { ValueConverter } from './column-interfaces.js';
 export declare class ValueConverters {
     static readonly Date: ValueConverter<Date>;
     static readonly DateOnly: ValueConverter<Date>;
-    static readonly DateOnlyString: ValueConverter<Date>;
+    static readonly DateOnlyString: ValueConverter<Date> & {
+        zeroDate: Date;
+    };
     static readonly Boolean: ValueConverter<Boolean>;
     static readonly Number: ValueConverter<number>;
     static readonly String: ValueConverter<String>;

package/src/valueConverters.js

@@ -3,6 +3,7 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.ValueConverters = void 0;
 var tslib_1 = require("tslib");
 var inputTypes_js_1 = require("../inputTypes.js");
+var zeroDate = new Date('0000-01-01');
 var ValueConverters = /** @class */ (function () {
     function ValueConverters() {
     }
@@ -71,6 +72,8 @@ var ValueConverters = /** @class */ (function () {
         fromJson: function (value) {
             if (!value || value == '' || value == '0000-00-00')
                 return null;
+            if (value == '0000-01-01')
+                return zeroDate;
             var d = new Date(Date.parse(value));
             d.setMinutes(d.getMinutes() + d.getTimezoneOffset());
             return d;
@@ -92,6 +95,8 @@ var ValueConverters = /** @class */ (function () {
         },
     };
     ValueConverters.DateOnlyString = tslib_1.__assign(tslib_1.__assign({}, ValueConverters.DateOnly), { toDb: function (d) {
+            if (d === zeroDate)
+                return '00000000';
             var val = ValueConverters.DateOnly.toJson(d);
             if (!val)
                 return undefined;
@@ -101,12 +106,10 @@ var ValueConverters = /** @class */ (function () {
                 return null;
             if (!val)
                 return undefined;
-            return new Date(val.substring(0, 4) +
-                '-' +
-                val.substring(4, 6) +
-                '-' +
-                val.substring(6, 8));
-        } });
+            if (val === '00000000')
+                return zeroDate;
+            return new Date(Number(val.substring(0, 4)), Number(val.substring(4, 6)) - 1, Number(val.substring(6, 8)));
+        }, zeroDate: zeroDate });
     ValueConverters.Boolean = {
         toDb: function (val) { return val; },
         inputType: inputTypes_js_1.InputTypes.checkbox,
@@ -173,6 +176,7 @@ var ValueConverters = /** @class */ (function () {
         toDb: function (x) { return ValueConverters.JsonString.toDb(x); },
         fromInput: function (x) { return ValueConverters.Default.fromJson(x); },
         toInput: function (x) { return ValueConverters.Default.toJson(x); },
+        toDbSql: function (x) { return x; },
         displayValue: function (x) { return x + ''; },
         fieldTypeInDb: '',
         inputType: 'text',