Merge pull request #88 from gisce/domain-context-helpers

Add helpers to domain and context

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gisce/ooui",
-  "version": "0.22.2",
+  "version": "0.22.3",
   "main": "./dist/ooui.umd.js",
   "module": "./dist/ooui.es.js",
   "types": "./dist/index.d.ts",

src/helpers/contextParser.ts

@@ -79,6 +79,61 @@ export const parseContext = ({
   }
 };
 
+export const parseContextFields = (context:string): string[] => {
+  const fields: string[] = [];
+  try {
+    if (!context) return fields;
+
+    if (isObject(context)) {
+      return fields;
+    }
+
+    if (typeof context !== "string") {
+      return fields;
+    }
+
+    const parsedContextInJson = tryParseJSON(context);
+    if (parsedContextInJson !== null) {
+      return [];
+    }
+
+    if (context.trim().length === 0) {
+      return fields;
+    }
+
+    // TODO: maybe this can be accomplished more performant and elegant with regex
+    const singleQuotesReplace = context.replace(/\"/g, "'");
+    const strNoWhitespaces = singleQuotesReplace.replace(/\s/g, "");
+    var replaceTrue = strNoWhitespaces.replace(/True/g, "true");
+    var replaceFalse = replaceTrue.replace(/False/g, "false");
+    const strNoClauLeft = replaceFalse.replace(/\{/g, "");
+    const strNoClauRight = strNoClauLeft.replace(/\}/g, "");
+
+    const entryValues = strNoClauRight.split(",");
+    const valuesSplitted = entryValues.map((entry) => {
+      return entry.split(":");
+    });
+
+    const parsedContext: any = {};
+
+    valuesSplitted.forEach((entry) => {
+      const fieldName = entry[1];
+
+      if (
+        entry[1].indexOf("'") === -1 &&
+        entry[1] !== "true" &&
+        entry[1] !== "false"
+      ) {
+        fields.push(entry[1].replace(/'/g, ""))
+        }
+    });
+
+    return fields;
+  } catch (e) {
+  }
+  return fields;
+}
+
 function tryParseJSON(str: string): any | null {
   try {
     const parsedJSON = JSON.parse(str.replace(/'/g, '"'));

src/helpers/domainParser.ts

@@ -36,4 +36,14 @@ const transformDomainForChildWidget = ({
   return transformedDomain;
 };
 
+export const parseDomainFields = (domain: string | boolean): string[] => {
+  if (typeof domain != 'string') {
+    return [];
+  }
+  return domain.replace(/[()\[\]]/g, "")
+    .split(',')
+    .map(i => i.trim())
+    .filter(i => i.indexOf("'") === -1)
+}
+
 export { transformDomainForChildWidget };

src/index.ts

@@ -31,8 +31,8 @@ import ButtonGroup from "./ButtonGroup";
 import Reference from "./Reference";
 import Binary from "./Binary";
 import Image from "./Image";
-import { parseContext } from "./helpers/contextParser";
-import { transformDomainForChildWidget } from "./helpers/domainParser";
+import { parseContext, parseContextFields } from "./helpers/contextParser";
+import { transformDomainForChildWidget, parseDomainFields } from "./helpers/domainParser";
 import Timeline from "./Timeline";
 import Indicator from "./Indicator";
 import Dashboard from "./Dashboard";
@@ -96,7 +96,9 @@ export {
   Binary,
   Image,
   parseContext,
+  parseContextFields,
   transformDomainForChildWidget,
+  parseDomainFields,
   Timeline,
   Indicator,
   Dashboard,

src/spec/contextParser.spec.ts

@@ -1,4 +1,4 @@
-import { parseContext } from "../helpers/contextParser";
+import { parseContext, parseContextFields } from "../helpers/contextParser";
 
 describe("A Context Parser", () => {
   describe("in parseContext method", () => {
@@ -129,10 +129,26 @@ describe("A Context Parser", () => {
     expect(parsedContext!["person"].name).toBe("John Doe");
     expect(JSON.stringify(parsedContext)).toBe(JSON.stringify(ctx));
   });
-
-  it.only("should parse context with simple single quotes in a string", () => {
+  it("should parse context with simple single quotes in a string", () => {
     const string = "{'contract_id': 1, 'contract_ids': [1, 3]}";
     const parsedContext = parseContext({ context: string });
     expect(parsedContext!["contract_id"]).toBe(1);
   });
+  describe("Getting evaluable fields from context", () => {
+    test("If not context is empty evaluable fields should be empty", () => {
+      const context: string = "{'power': potencia, 'tarifa_id': tarifa, 'o2m': tensio_o2m, 'tensio_id': tensio_normalitzada, 'model': 'giscedata.polissa', 'field': 'potencia'}";
+      const fields = parseContextFields(context);
+      expect(fields).toEqual([
+        'potencia',
+        'tarifa',
+        'tensio_o2m',
+        'tensio_normalitzada'
+      ])
+    });
+    test("If context is all json an empty list of fields must be returned", () => {
+      const context: string = "{'from_model': 'res.partner'}";
+      const fields = parseContextFields(context);
+      expect(fields).toEqual([])
+    });
+  });
 });

src/spec/domainParser.spec.ts

@@ -1,4 +1,4 @@
-import { transformDomainForChildWidget } from "../helpers/domainParser";
+import { transformDomainForChildWidget, parseDomainFields } from "../helpers/domainParser";
 
 describe("A Domain Parser", () => {
   it("should properly transform domain for inner widgets", () => {
@@ -38,3 +38,14 @@ describe("A Domain Parser", () => {
     expect(domainForStageId![0][2]).toBe("backlog");
   });
 });
+
+describe("Getting evaluable fields for context", () => {
+  test("Should return evaluable fields", () => {
+    const domain: string = "['|', ('a', '=', 'foo'), ('niu', '=', nau), ('state', 'in', ('open', 'draft'))]";
+    const fields = parseDomainFields(domain);
+    expect(fields).toEqual(['nau'])
+  })
+  test("If domain is false fields must be an empty list", () => {
+    expect(parseDomainFields(false)).toEqual([])
+  })
+});