remult 0.25.5-exp.6 → 0.25.5-exp.8

package/esm/graphql.js

@@ -413,6 +413,9 @@ export function remultGraphql(options) {
                         const refType = ri.toEntity;
                         ref = entities.find((i) => i.entityType === refType);
                     }
+                    if (!ref) {
+                        throw new Error(`Entity "${ri.toEntity.name}" that is used by the relation "${f.key}" in "${meta.entityType.name}" was not found in the 'entities' array.`);
+                    }
                     notARealField = ri.type === 'toOne' || ri.type === 'toMany';
                     const refKey = ref.key;
                     switch (ri.type) {
@@ -448,11 +451,11 @@ export function remultGraphql(options) {
                                 order: 10,
                                 comment: `List all \`${getMetaType(meta)}\` of \`${refKey}\``,
                             });
-                            currentType.query.resultProcessors.push((r) => {
+                            currentType.query.resultProcessors.push((r, y) => {
                                 r[f.key] = async (args, req, gqlInfo) => {
                                     const remult = await options.getRemultFromRequest(req);
                                     const myRepo = remult.repo(meta.entityType);
-                                    const item = myRepo.fromJson(r);
+                                    const item = myRepo.fromJson(y);
                                     let relRepo = myRepo.relations(item)[f.key];
                                     return handleRequestWithDataApiContextBasedOnRepo(relRepo, async (dApi, response, setResult, arg1, meta) => {
                                         return await connectionImplementation(setResult, itemsKey, createResultPromise, dApi, upsertTypes(getMetaType(ref)), arg1, meta, totalCountKey);

package/esm/remult-knex/index.js

@@ -20,7 +20,7 @@ export class KnexDataProvider {
         return r.knex;
     }
     getEntityDataProvider(entity) {
-        if (!supportsJson(this.knex))
+        if (!supportsJsonLoadingAndSaving(this.knex))
             for (const f of entity.fields.toArray()) {
                 if (f.valueConverter.fieldTypeInDb === 'json') {
                     //@ts-ignore
@@ -367,7 +367,7 @@ export class KnexSchemaBuilder {
                         if (isAutoIncrement(x))
                             b.increments(cols.get(x).name);
                         else {
-                            buildColumn(x, cols.get(x).name, b, supportsJson(this.knex));
+                            buildColumn(x, cols.get(x).name, b, supportsJsonDataStorage(this.knex));
                             if (x == entity.idMetadata.field)
                                 b.primary([cols.get(x).name]);
                         }
@@ -384,7 +384,7 @@ export class KnexSchemaBuilder {
         let colName = e.$dbNameOf(col);
         if (!(await this.knex.schema.hasColumn(e.$entityName, colName))) {
             await logSql(this.knex.schema.alterTable(e.$entityName, (b) => {
-                buildColumn(col, colName, b, supportsJson(this.knex));
+                buildColumn(col, colName, b, supportsJsonDataStorage(this.knex));
             }));
         }
     }
@@ -406,10 +406,17 @@ export class KnexSchemaBuilder {
         this.knex = knex;
     }
 }
-function supportsJson(knex) {
+function supportsJsonDataStorage(knex) {
+    const client = knex.client.config.client;
+    if (client?.includes('sqlite3') || client?.includes('mssql'))
+        return false;
+    return true;
+}
+function supportsJsonLoadingAndSaving(knex) {
     const client = knex.client.config.client;
     if (client?.includes('sqlite3') ||
         client?.includes('mssql') ||
+        client?.includes('mysql2') ||
         client?.includes('mysql'))
         return false;
     return true;

package/esm/src/remult3/remult3.js

@@ -2,6 +2,17 @@ export {};
 //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
 //y1 - talk about the parameter issue with backend methods
 //y1 - tried to upgrade vitest, nuxt tests are failing with loading uuid - sounds familiar?
+//p1 - live query with include
+//p1 - relation from order details to order gave a compound id info - and it is not true - same for the relation to product
+//p1 - include admin in remult
+//p1 - new row when there are relations, looks funny (see product)
+//p1 - the + row in the bottom should extend to the full width
+//p1 - adjust angular starter kit for latest angular (as is in tutorial)
+/*y2 -
+  @Fields.string<Category>({
+    allowApiUpdate: (c) => getEntityRef(c).isNew(),
+  })
+  Description = ""*/
 //y2 - from the crm-demo(https://crm-demo.up.railway.app/deals), after editing a deal: - _updateEntityBasedOnApi
 //y2 - Backend methods are transactions, it's not intuitive and maybe should be optional / opt in
 //y2 - how to run a transaction as a user

package/graphql.js

@@ -505,6 +505,9 @@ function remultGraphql(options) {
                         var refType_1 = ri.toEntity;
                         ref = entities.find(function (i) { return i.entityType === refType_1; });
                     }
+                    if (!ref) {
+                        throw new Error("Entity \"".concat(ri.toEntity.name, "\" that is used by the relation \"").concat(f.key, "\" in \"").concat(meta.entityType.name, "\" was not found in the 'entities' array."));
+                    }
                     notARealField = ri.type === 'toOne' || ri.type === 'toMany';
                     var refKey = ref.key;
                     switch (ri.type) {
@@ -557,7 +560,7 @@ function remultGraphql(options) {
                                 order: 10,
                                 comment: "List all `".concat(getMetaType(meta), "` of `").concat(refKey, "`"),
                             });
-                            currentType.query.resultProcessors.push(function (r) {
+                            currentType.query.resultProcessors.push(function (r, y) {
                                 r[f.key] = function (args, req, gqlInfo) { return tslib_1.__awaiter(_this, void 0, void 0, function () {
                                     var remult, myRepo, item, relRepo;
                                     var _this = this;
@@ -567,7 +570,7 @@ function remultGraphql(options) {
                                             case 1:
                                                 remult = _a.sent();
                                                 myRepo = remult.repo(meta.entityType);
-                                                item = myRepo.fromJson(r);
+                                                item = myRepo.fromJson(y);
                                                 relRepo = myRepo.relations(item)[f.key];
                                                 return [2 /*return*/, handleRequestWithDataApiContextBasedOnRepo_1(relRepo, function (dApi, response, setResult, arg1, meta) { return tslib_1.__awaiter(_this, void 0, void 0, function () {
                                                         return tslib_1.__generator(this, function (_a) {

package/package.json

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

package/remult-knex/index.js

@@ -24,7 +24,7 @@ var KnexDataProvider = /** @class */ (function () {
     };
     KnexDataProvider.prototype.getEntityDataProvider = function (entity) {
         var e_1, _a;
-        if (!supportsJson(this.knex))
+        if (!supportsJsonLoadingAndSaving(this.knex))
             try {
                 for (var _b = tslib_1.__values(entity.fields.toArray()), _c = _b.next(); !_c.done; _c = _b.next()) {
                     var f = _c.value;
@@ -739,7 +739,7 @@ var KnexSchemaBuilder = /** @class */ (function () {
                                             if ((0, RepositoryImplementation_js_1.isAutoIncrement)(x))
                                                 b.increments(cols_1.get(x).name);
                                             else {
-                                                buildColumn(x, cols_1.get(x).name, b, supportsJson(_this.knex));
+                                                buildColumn(x, cols_1.get(x).name, b, supportsJsonDataStorage(_this.knex));
                                                 if (x == entity.idMetadata.field)
                                                     b.primary([cols_1.get(x).name]);
                                             }
@@ -779,7 +779,7 @@ var KnexSchemaBuilder = /** @class */ (function () {
                     case 2:
                         if (!!(_a.sent())) return [3 /*break*/, 4];
                         return [4 /*yield*/, logSql(this.knex.schema.alterTable(e.$entityName, function (b) {
-                                buildColumn(col, colName, b, supportsJson(_this.knex));
+                                buildColumn(col, colName, b, supportsJsonDataStorage(_this.knex));
                             }))];
                     case 3:
                         _a.sent();
@@ -856,10 +856,17 @@ var KnexSchemaBuilder = /** @class */ (function () {
     return KnexSchemaBuilder;
 }());
 exports.KnexSchemaBuilder = KnexSchemaBuilder;
-function supportsJson(knex) {
+function supportsJsonDataStorage(knex) {
+    var client = knex.client.config.client;
+    if ((client === null || client === void 0 ? void 0 : client.includes('sqlite3')) || (client === null || client === void 0 ? void 0 : client.includes('mssql')))
+        return false;
+    return true;
+}
+function supportsJsonLoadingAndSaving(knex) {
     var client = knex.client.config.client;
     if ((client === null || client === void 0 ? void 0 : client.includes('sqlite3')) ||
         (client === null || client === void 0 ? void 0 : client.includes('mssql')) ||
+        (client === null || client === void 0 ? void 0 : client.includes('mysql2')) ||
         (client === null || client === void 0 ? void 0 : client.includes('mysql')))
         return false;
     return true;

package/src/remult3/remult3.js

@@ -3,6 +3,17 @@ Object.defineProperty(exports, "__esModule", { value: true });
 //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
 //y1 - talk about the parameter issue with backend methods
 //y1 - tried to upgrade vitest, nuxt tests are failing with loading uuid - sounds familiar?
+//p1 - live query with include
+//p1 - relation from order details to order gave a compound id info - and it is not true - same for the relation to product
+//p1 - include admin in remult
+//p1 - new row when there are relations, looks funny (see product)
+//p1 - the + row in the bottom should extend to the full width
+//p1 - adjust angular starter kit for latest angular (as is in tutorial)
+/*y2 -
+  @Fields.string<Category>({
+    allowApiUpdate: (c) => getEntityRef(c).isNew(),
+  })
+  Description = ""*/
 //y2 - from the crm-demo(https://crm-demo.up.railway.app/deals), after editing a deal: - _updateEntityBasedOnApi
 //y2 - Backend methods are transactions, it's not intuitive and maybe should be optional / opt in
 //y2 - how to run a transaction as a user