remult 0.25.2 → 0.25.3

package/esm/postgres/schema-builder.js

@@ -1,5 +1,5 @@
 import { allEntities } from '../src/context.js';
-import { dbNamesOf, isDbReadonly, } from '../src/filter/filter-consumer-bridge-to-sql-request.js';
+import { dbNamesOf, shouldNotCreateField, } from '../src/filter/filter-consumer-bridge-to-sql-request.js';
 import { remult as defaultRemult } from '../src/remult-proxy.js';
 import { isAutoIncrement } from '../src/remult3/RepositoryImplementation.js';
 import { ValueConverters } from '../src/valueConverters.js';
@@ -132,7 +132,7 @@ export class PostgresSchemaBuilder {
             if (r.rows.length == 0) {
                 let result = '';
                 for (const x of entity.fields) {
-                    if (!isDbReadonly(x, e) || isAutoIncrement(x)) {
+                    if (!shouldNotCreateField(x, e) || isAutoIncrement(x)) {
                         if (result.length != 0)
                             result += ',';
                         result += '\r\n  ';
@@ -155,7 +155,7 @@ CREATE table ${this.schemaAndName(e)} (${result}\r\n)`;
     }
     async addColumnIfNotExist(entity, c) {
         let e = await dbNamesOf(entity, this.pool.wrapIdentifier);
-        if (isDbReadonly(c(entity), e))
+        if (shouldNotCreateField(c(entity), e))
             return;
         try {
             let cmd = this.pool.createCommand();
@@ -181,7 +181,7 @@ CREATE table ${this.schemaAndName(e)} (${result}\r\n)`;
             let cols = (await cmd.execute(`SELECT column_name FROM information_schema.columns WHERE ` +
                 `${this.whereTableAndSchema(cmd, e)}`)).rows.map((x) => x.column_name.toLocaleLowerCase());
             for (const col of entity.fields) {
-                if (!isDbReadonly(col, e)) {
+                if (!shouldNotCreateField(col, e)) {
                     let colName = e.$dbNameOf(col).toLocaleLowerCase();
                     if (colName.startsWith('"') && colName.endsWith('"'))
                         colName = colName.substring(1, colName.length - 1);

package/esm/remult-knex/index.js

@@ -1,5 +1,5 @@
 import { allEntities } from '../src/context.js';
-import { dbNamesOf, isDbReadonly, } from '../src/filter/filter-consumer-bridge-to-sql-request.js';
+import { dbNamesOf, isDbReadonly, shouldNotCreateField, } from '../src/filter/filter-consumer-bridge-to-sql-request.js';
 import { customDatabaseFilterToken, Filter, } from '../src/filter/filter-interfaces.js';
 import { CompoundIdField } from '../src/CompoundIdField.js';
 import { remult as remultContext } from '../src/remult-proxy.js';
@@ -358,7 +358,7 @@ export class KnexSchemaBuilder {
             for (const f of entity.fields) {
                 cols.set(f, {
                     name: e.$dbNameOf(f),
-                    readonly: isDbReadonly(f, e),
+                    readonly: shouldNotCreateField(f, e),
                 });
             }
             await logSql(this.knex.schema.createTable(e.$entityName, (b) => {
@@ -378,7 +378,7 @@ export class KnexSchemaBuilder {
     }
     async addColumnIfNotExist(entity, c) {
         let e = await dbNamesOf(entity);
-        if (isDbReadonly(c(entity), e))
+        if (shouldNotCreateField(c(entity), e))
             return;
         let col = c(entity);
         let colName = e.$dbNameOf(col);
@@ -392,7 +392,7 @@ export class KnexSchemaBuilder {
         let e = await dbNamesOf(entity);
         try {
             for (const col of entity.fields.toArray()) {
-                if (!isDbReadonly(col, e)) {
+                if (!shouldNotCreateField(col, e)) {
                     await this.addColumnIfNotExist(entity, () => col);
                 }
             }

package/esm/src/filter/filter-consumer-bridge-to-sql-request.js

@@ -154,6 +154,10 @@ export function isDbReadonly(field, dbNames) {
         field.isServerExpression ||
         (field.options.sqlExpression && field.dbName != dbNames.$dbNameOf(field)));
 }
+export function shouldNotCreateField(field, dbNames) {
+    return (field.isServerExpression ||
+        (field.options.sqlExpression && field.dbName != dbNames.$dbNameOf(field)));
+}
 export async function dbNamesOf(repo, wrapIdentifier = (x) => x) {
     var meta = getEntityMetadata(repo);
     const result = {

package/esm/src/remult3/remult3.js

@@ -43,6 +43,11 @@ remult.apiClient.url='localhost:3007/api
 
 
 */
+//p1 -check if checouknt doesn't respect allow read (I think)
+//p1 - fix multiple instances of remult: the defaultRemult and async local hook - and columnsOfType - all can be tests with a vite project loading `remult-admin`
+//p1 - crm demo graphql error
+//p1 - allow api update only for new rows
+//p1 - getFields didn't work for kobi in the home component
 //p2 - document validators
 //p2 - and validators to reference
 //y2 - discuss a default date formatter

package/package.json

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

package/postgres/schema-builder.js

@@ -234,7 +234,7 @@ var PostgresSchemaBuilder = /** @class */ (function () {
                                             try {
                                                 for (_a = tslib_1.__values(entity.fields), _b = _a.next(); !_b.done; _b = _a.next()) {
                                                     x = _b.value;
-                                                    if (!(0, filter_consumer_bridge_to_sql_request_js_1.isDbReadonly)(x, e) || (0, RepositoryImplementation_js_1.isAutoIncrement)(x)) {
+                                                    if (!(0, filter_consumer_bridge_to_sql_request_js_1.shouldNotCreateField)(x, e) || (0, RepositoryImplementation_js_1.isAutoIncrement)(x)) {
                                                         if (result.length != 0)
                                                             result += ',';
                                                         result += '\r\n  ';
@@ -281,7 +281,7 @@ var PostgresSchemaBuilder = /** @class */ (function () {
                     case 0: return [4 /*yield*/, (0, filter_consumer_bridge_to_sql_request_js_1.dbNamesOf)(entity, this.pool.wrapIdentifier)];
                     case 1:
                         e = _a.sent();
-                        if ((0, filter_consumer_bridge_to_sql_request_js_1.isDbReadonly)(c(entity), e))
+                        if ((0, filter_consumer_bridge_to_sql_request_js_1.shouldNotCreateField)(c(entity), e))
                             return [2 /*return*/];
                         _a.label = 2;
                     case 2:
@@ -335,7 +335,7 @@ var PostgresSchemaBuilder = /** @class */ (function () {
                     case 4:
                         if (!!_b.done) return [3 /*break*/, 7];
                         col = _b.value;
-                        if (!!(0, filter_consumer_bridge_to_sql_request_js_1.isDbReadonly)(col, e)) return [3 /*break*/, 6];
+                        if (!!(0, filter_consumer_bridge_to_sql_request_js_1.shouldNotCreateField)(col, e)) return [3 /*break*/, 6];
                         colName = e.$dbNameOf(col).toLocaleLowerCase();
                         if (colName.startsWith('"') && colName.endsWith('"'))
                             colName = colName.substring(1, colName.length - 1);

package/remult-knex/index.js

@@ -719,7 +719,7 @@ var KnexSchemaBuilder = /** @class */ (function () {
                                 f = _b.value;
                                 cols_1.set(f, {
                                     name: e.$dbNameOf(f),
-                                    readonly: (0, filter_consumer_bridge_to_sql_request_js_1.isDbReadonly)(f, e),
+                                    readonly: (0, filter_consumer_bridge_to_sql_request_js_1.shouldNotCreateField)(f, e),
                                 });
                             }
                         }
@@ -771,7 +771,7 @@ var KnexSchemaBuilder = /** @class */ (function () {
                     case 0: return [4 /*yield*/, (0, filter_consumer_bridge_to_sql_request_js_1.dbNamesOf)(entity)];
                     case 1:
                         e = _a.sent();
-                        if ((0, filter_consumer_bridge_to_sql_request_js_1.isDbReadonly)(c(entity), e))
+                        if ((0, filter_consumer_bridge_to_sql_request_js_1.shouldNotCreateField)(c(entity), e))
                             return [2 /*return*/];
                         col = c(entity);
                         colName = e.$dbNameOf(col);
@@ -805,7 +805,7 @@ var KnexSchemaBuilder = /** @class */ (function () {
                             return tslib_1.__generator(this, function (_e) {
                                 switch (_e.label) {
                                     case 0:
-                                        if (!!(0, filter_consumer_bridge_to_sql_request_js_1.isDbReadonly)(col, e)) return [3 /*break*/, 2];
+                                        if (!!(0, filter_consumer_bridge_to_sql_request_js_1.shouldNotCreateField)(col, e)) return [3 /*break*/, 2];
                                         return [4 /*yield*/, this_2.addColumnIfNotExist(entity, function () { return col; })];
                                     case 1:
                                         _e.sent();

package/src/filter/filter-consumer-bridge-to-sql-request.d.ts

@@ -40,6 +40,7 @@ export declare class CustomSqlFilterBuilder {
     filterToRaw<entityType>(repo: RepositoryOverloads<entityType>, condition: EntityFilter<entityType>): Promise<string>;
 }
 export declare function isDbReadonly<entityType>(field: FieldMetadata, dbNames: EntityDbNames<entityType>): boolean;
+export declare function shouldNotCreateField<entityType>(field: FieldMetadata, dbNames: EntityDbNames<entityType>): boolean;
 export declare type EntityDbNamesBase = {
     $entityName: string;
     $dbNameOf(field: FieldMetadata<any> | string): string;

package/src/filter/filter-consumer-bridge-to-sql-request.js

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.fieldDbName = exports.entityDbName = exports.dbNamesOf = exports.isDbReadonly = exports.CustomSqlFilterBuilder = exports.FilterConsumerBridgeToSqlRequest = void 0;
+exports.fieldDbName = exports.entityDbName = exports.dbNamesOf = exports.shouldNotCreateField = exports.isDbReadonly = exports.CustomSqlFilterBuilder = exports.FilterConsumerBridgeToSqlRequest = void 0;
 var tslib_1 = require("tslib");
 var sql_database_js_1 = require("../data-providers/sql-database.js");
 var RepositoryImplementation_js_1 = require("../remult3/RepositoryImplementation.js");
@@ -263,6 +263,11 @@ function isDbReadonly(field, dbNames) {
         (field.options.sqlExpression && field.dbName != dbNames.$dbNameOf(field)));
 }
 exports.isDbReadonly = isDbReadonly;
+function shouldNotCreateField(field, dbNames) {
+    return (field.isServerExpression ||
+        (field.options.sqlExpression && field.dbName != dbNames.$dbNameOf(field)));
+}
+exports.shouldNotCreateField = shouldNotCreateField;
 function dbNamesOf(repo, wrapIdentifier) {
     if (wrapIdentifier === void 0) { wrapIdentifier = function (x) { return x; }; }
     return tslib_1.__awaiter(this, void 0, void 0, function () {

package/src/remult3/remult3.js

@@ -44,6 +44,11 @@ remult.apiClient.url='localhost:3007/api
 
 
 */
+//p1 -check if checouknt doesn't respect allow read (I think)
+//p1 - fix multiple instances of remult: the defaultRemult and async local hook - and columnsOfType - all can be tests with a vite project loading `remult-admin`
+//p1 - crm demo graphql error
+//p1 - allow api update only for new rows
+//p1 - getFields didn't work for kobi in the home component
 //p2 - document validators
 //p2 - and validators to reference
 //y2 - discuss a default date formatter