remult 0.27.23-next.2 → 0.27.23-next.4

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

@@ -228,7 +228,7 @@ class LogSQLCommand {
             let r = await this.origin.execute(sql);
             if (this.logToConsole !== false) {
                 var d = new Date().valueOf() - start.valueOf();
-                if (d > SqlDatabase.durationThreshold) {
+                if (d >= SqlDatabase.durationThreshold) {
                     const duration = d / 1000;
                     if (this.logToConsole === 'oneLiner') {
                         const rawSql = sql

package/esm/src/remult3/remult3.js

@@ -17,8 +17,32 @@ const wnn = [
 export const flags = {
     error500RetryCount: 4,
 };
-//p1 - make error info an actual error, important for exception handling, and also for auth.js errors that were mistic
-//p1 - add validators min,max,between(range)
+/*p1 - issues in https://stackblitz.com/edit/demo-allow-delete-based-on-other-entity:
+  - don't like the ensure schema
+  - seems like this didn't work well in their version of sqlite:
+    ```
+    @Entity<TaskUser>('TaskUsers', {
+      id: ['taskId', 'userId'],
+    })
+    ```
+  - This could be better
+    ```
+    sqlExpression: async () => {
+        if (!remult.authenticated()) return 'false';
+        return (
+          '1=' +
+          (await sqlRelations(Task).taskUsers.$count({
+            userId: [remult.user?.id],
+            canDelete: true,
+          }))
+        );
+      },
+    ```
+  
+  -
+*/
+//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...)
 //p2 - add parameter all to deleteMany, and updateMany
 //p2  filter.apply ApiPreFilter

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "remult",
-    "version": "0.27.23-next.2",
+    "version": "0.27.23-next.4",
     "description": "A CRUD framework for full-stack TypeScript",
     "homepage": "https://remult.dev",
     "repository": {

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

@@ -310,7 +310,7 @@ var LogSQLCommand = /** @class */ (function () {
                         r = _a.sent();
                         if (this.logToConsole !== false) {
                             d = new Date().valueOf() - start.valueOf();
-                            if (d > SqlDatabase.durationThreshold) {
+                            if (d >= SqlDatabase.durationThreshold) {
                                 duration = d / 1000;
                                 if (this.logToConsole === 'oneLiner') {
                                     rawSql = sql

package/src/remult3/Fields.d.ts

@@ -100,7 +100,7 @@ export declare class Relations {
      * customer?: Customer;
      * ```
      */
-    static toOne<entityType, toEntityType>(toEntityType: () => ClassType<toEntityType>, options?: (FieldOptions<entityType, toEntityType> & Pick<RelationOptions<entityType, toEntityType, any, any>, 'defaultIncluded'>) | RelationOptions<entityType, toEntityType, entityType> | keyof entityType): (target: any, context: string | ClassFieldDecoratorContextStub<any, toEntityType>, c?: any) => void;
+    static toOne<entityType, toEntityType>(toEntityType: () => ClassType<toEntityType>, options?: (FieldOptions<entityType, toEntityType> & Pick<RelationOptions<entityType, toEntityType, any, any>, 'defaultIncluded'>) | RelationOptions<entityType, toEntityType, entityType> | keyof entityType): (target: any, context: ClassFieldDecoratorContextStub<entityType, toEntityType | undefined> | string, c?: any) => void;
     /**
      * Define a toMany relation between entities, indicating a one-to-many relationship.
      * This method allows you to establish a relationship where one entity can have multiple related entities.

package/src/remult3/remult3.js

@@ -20,8 +20,32 @@ var wnn = [
 exports.flags = {
     error500RetryCount: 4,
 };
-//p1 - make error info an actual error, important for exception handling, and also for auth.js errors that were mistic
-//p1 - add validators min,max,between(range)
+/*p1 - issues in https://stackblitz.com/edit/demo-allow-delete-based-on-other-entity:
+  - don't like the ensure schema
+  - seems like this didn't work well in their version of sqlite:
+    ```
+    @Entity<TaskUser>('TaskUsers', {
+      id: ['taskId', 'userId'],
+    })
+    ```
+  - This could be better
+    ```
+    sqlExpression: async () => {
+        if (!remult.authenticated()) return 'false';
+        return (
+          '1=' +
+          (await sqlRelations(Task).taskUsers.$count({
+            userId: [remult.user?.id],
+            canDelete: true,
+          }))
+        );
+      },
+    ```
+  
+  -
+*/
+//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...)
 //p2 - add parameter all to deleteMany, and updateMany
 //p2  filter.apply ApiPreFilter