@gisce/ooui 2.23.0-alpha.4 → 2.23.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gisce/ooui",
-  "version": "2.23.0-alpha.4",
+  "version": "2.23.0",
   "engines": {
     "node": "20.5.0"
   },

package/src/Field.ts

@@ -1,6 +1,5 @@
 import Widget from "./Widget";
 import { replaceEntities, isTrue } from "./helpers/attributeParser";
-import { parseBoolAttribute } from "./helpers/nodeParser";
 
 class Field extends Widget {
   /**
@@ -126,27 +125,6 @@ class Field extends Widget {
     this._selectionValues = value;
   }
 
-  _autoRefresh?: boolean = false;
-  get autoRefresh(): boolean {
-    return this._autoRefresh ?? false;
-  }
-
-  set autoRefresh(value: boolean) {
-    this._autoRefresh = value;
-  }
-
-  get readOnly(): boolean | undefined {
-    if (this.autoRefresh) {
-      return true;
-    } else {
-      return super.readOnly;
-    }
-  }
-
-  set readOnly(value: boolean | undefined) {
-    super.readOnly = value;
-  }
-
   constructor(props: any) {
     super(props);
 
@@ -198,9 +176,6 @@ class Field extends Widget {
       if (props.help_inline) {
         this.tooltipInline = isTrue(props.help_inline);
       }
-      if (props.autorefresh) {
-        this.autoRefresh = parseBoolAttribute(props.autorefresh);
-      }
     }
   }
 

package/src/Form.ts

@@ -124,14 +124,6 @@ class Form {
     this._invisibleFields = value;
   }
 
-  /**
-   * List of autorefreshable fields
-   */
-  _autorefreshableFields: string[] = [];
-  get autorefreshableFields(): string[] {
-    return this._autorefreshableFields;
-  }
-
   /**
    * Context for each field in the form
    */
@@ -178,11 +170,6 @@ class Form {
         this._contextForFields[unknownWidget._id] = widget._context;
       }
     });
-
-    // Also we store all the autorefreshables fields in a list
-    this._autorefreshableFields = allWidgets
-      .filter((widget) => widget instanceof Field && widget.autoRefresh)
-      .map((field) => (field as Field)._id);
   }
 
   parseNode({

package/src/Tree.ts

@@ -1,7 +1,7 @@
 import WidgetFactory from "./WidgetFactory";
 import Widget from "./Widget";
 import { replaceEntities } from "./helpers/attributeParser";
-import { parseBoolAttribute, ParsedNode } from "./helpers/nodeParser";
+import { ParsedNode } from "./helpers/nodeParser";
 import * as txml from "txml";
 import { parseContext } from "./helpers/contextParser";
 
@@ -67,14 +67,6 @@ class Tree {
     this._contextForFields = value;
   }
 
-  /**
-   * List of autorefreshable fields
-   */
-  _autorefreshableFields: string[] = [];
-  get autorefreshableFields(): string[] {
-    return this._autorefreshableFields;
-  }
-
   /**
    * Is infinite
    */
@@ -153,10 +145,6 @@ class Tree {
           const widget = widgetFactory.createWidget(widgetType, mergedAttrs);
           this._columns.push(widget);
         }
-
-        if (parseBoolAttribute(mergedAttrs.autorefresh)) {
-          this._autorefreshableFields.push(name);
-        }
       }
     });
   }

package/src/Widget.ts

@@ -1,5 +1,4 @@
 import { replaceEntities, parseWidgetProps } from "./helpers/attributeParser";
-import { parseBoolAttribute } from "./helpers/nodeParser";
 
 abstract class Widget {
   /**
@@ -133,7 +132,19 @@ abstract class Widget {
         this._colspan = +props.colspan;
       }
       if (props.readonly !== undefined) {
-        this._readOnly = parseBoolAttribute(props.readonly);
+        if (
+          props.readonly === "1" ||
+          props.readonly === 1 ||
+          props.readonly === true
+        ) {
+          this._readOnly = true;
+        } else if (
+          props.readonly === "0" ||
+          props.readonly === 0 ||
+          props.readonly === false
+        ) {
+          this._readOnly = false;
+        }
       }
       if (props.invisible) {
         if (

package/src/WidgetFactory.ts

@@ -41,7 +41,6 @@ import Comments from "./Comments";
 import JSONField from "./JSONField";
 import Email from "./Email";
 import Spinner from "./Spinner";
-import Carousel from "./Carousel";
 
 class WidgetFactory {
   /**
@@ -185,9 +184,6 @@ class WidgetFactory {
       case "spinner":
         this._widgetClass = Spinner;
         break;
-      case "carousel":
-        this._widgetClass = Carousel;
-        break;
       default:
         break;
     }

package/src/helpers/nodeParser.ts

@@ -5,13 +5,7 @@ type ParsedNode = {
 };
 
 const parseBoolAttribute = (attr: any): boolean => {
-  if (
-    attr === 1 ||
-    attr === "1" ||
-    attr === true ||
-    attr === "True" ||
-    attr === "true"
-  ) {
+  if (attr === 1 || attr === "1" || attr === true || attr === "True") {
     return true;
   } else {
     return false;

package/src/index.ts

@@ -53,7 +53,6 @@ import JSONField from "./JSONField";
 import Comments from "./Comments";
 import Email from "./Email";
 import Spinner from "./Spinner";
-import Carousel from "./Carousel";
 
 import {
   Graph,
@@ -143,5 +142,4 @@ export {
   JSONField,
   Email,
   Spinner,
-  Carousel,
 };

package/src/spec/Form.spec.ts

@@ -6015,56 +6015,6 @@ describe("A Form", () => {
     expect(field_char?.type).toBe("arrow_steps");
     expect(field_char?.id).toBe("field_char");
   });
-  it("a field with autorefresh evaluated in attrs should be present in form autorefreshable fields property", () => {
-    const fields = {
-      field_char: {
-        string: "Etapa",
-        type: "char",
-      },
-      state: {
-        readonly: true,
-        required: true,
-        selection: [
-          ["esborrany", "Borrador"],
-          ["validar", "Validar"],
-          ["pendent", "Pendiente"],
-          ["activa", "Activa"],
-          ["cancelada", "Cancelada"],
-          ["contracte", "Activación Contrato"],
-          ["novapolissa", "Creación nuevo contrato"],
-          ["modcontractual", "Modificación Contractual"],
-          ["impagament", "Impago"],
-          ["tall", "Corte"],
-          ["running", "En ejecución"],
-          ["baixa", "Baja"],
-          ["facturacio", "Facturación"],
-        ],
-        string: "Estado",
-        type: "selection",
-        views: {},
-      },
-    };
-
-    const xmlViewForm = `<?xml version="1.0"?>
-    <form string="Form1">
-    	<field name="field_char" widget="arrow_steps" colspan="4" nolabel="1" attrs="{'autorefresh':[('state', '=', 'running')]}" />
-    </form>`;
-
-    const form = new Form(fields);
-    form.parse(xmlViewForm, {
-      values: {
-        field_char: "test",
-        state: "running",
-      },
-    });
-
-    const field_char = form.findById("field_char") as Field;
-    expect(field_char).toBeDefined();
-    expect(field_char?.autoRefresh).toBeTruthy();
-    expect(field_char?.readOnly).toBeTruthy();
-    expect(form.autorefreshableFields.length).toBe(1);
-    expect(form.autorefreshableFields[0]).toBe("field_char");
-  });
   describe("If the field has widget_props", () => {
     it("should merge widget_props from fields definition and xml", () => {
       const fields = {

package/src/spec/Tree.spec.ts

@@ -377,23 +377,4 @@ describe("A Tree", () => {
     const nameWidget = tree.findById("name") as Char;
     expect(nameWidget.isFunction).toBeTruthy();
   });
-  it("Should parse autorefreshable fields", () => {
-    const tree = new Tree({
-      name: {
-        required: true,
-        select: true,
-        size: 128,
-        string: "Potència contractada (kW)",
-        type: "char",
-        views: {},
-      },
-    });
-    tree.parse(
-      `<tree string="Partners" colors="red:type=='updated'"><field name="name" sum="Pot&#232;ncia contractada (kW)" autorefresh="1"/></tree>`,
-    );
-
-    const nameWidget = tree.findById("name") as Char;
-    expect(nameWidget.autoRefresh).toBeTruthy();
-    expect(tree._autorefreshableFields.length).toBe(1);
-  });
 });

package/dist/Carousel.d.ts

@@ -1,11 +0,0 @@
-import ContainerWidget from "./ContainerWidget";
-import Group from "./Group";
-declare class Carousel extends ContainerWidget {
-    _autoPlay: boolean;
-    get autoPlay(): boolean;
-    set autoPlay(value: boolean);
-    get items(): Group[];
-    constructor(props?: any);
-}
-export default Carousel;
-//# sourceMappingURL=Carousel.d.ts.map

package/dist/Carousel.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"Carousel.d.ts","sourceRoot":"","sources":["../src/Carousel.ts"],"names":[],"mappings":"AAAA,OAAO,eAAe,MAAM,mBAAmB,CAAC;AAChD,OAAO,KAAK,MAAM,SAAS,CAAC;AAG5B,cAAM,QAAS,SAAQ,eAAe;IACpC,SAAS,UAAQ;IAEjB,IAAI,QAAQ,IAAI,OAAO,CAEtB;IAED,IAAI,QAAQ,CAAC,KAAK,EAAE,OAAO,EAE1B;IAED,IAAI,KAAK,IAAI,KAAK,EAAE,CAEnB;gBAEW,KAAK,CAAC,EAAE,GAAG;CAQxB;AAED,eAAe,QAAQ,CAAC"}

package/src/Carousel.ts

@@ -1,30 +0,0 @@
-import ContainerWidget from "./ContainerWidget";
-import Group from "./Group";
-import { parseBoolAttribute } from "./helpers/nodeParser";
-
-class Carousel extends ContainerWidget {
-  _autoPlay = true;
-
-  get autoPlay(): boolean {
-    return this._autoPlay;
-  }
-
-  set autoPlay(value: boolean) {
-    this._autoPlay = value;
-  }
-
-  get items(): Group[] {
-    return this._container.rows.flat().filter((g) => !g.invisible) as Group[];
-  }
-
-  constructor(props?: any) {
-    super(props);
-    if (props) {
-      if ("auto_play" in props) {
-        this._autoPlay = parseBoolAttribute(props.auto_play);
-      }
-    }
-  }
-}
-
-export default Carousel;

package/src/spec/Carousel.spec.ts

@@ -1,75 +0,0 @@
-import WidgetFactory from "../WidgetFactory";
-import Form from "../Form";
-import Carousel from "../Carousel";
-import { it, expect, describe } from "vitest";
-
-describe("A Carousel", () => {
-  it("should have an id corresponding to field name", () => {
-    const widgetFactory = new WidgetFactory();
-    const props = {
-      name: "carousel",
-    };
-
-    const widget = widgetFactory.createWidget("carousel", props);
-    expect(widget).toBeInstanceOf(Carousel);
-  });
-  it("should have autoPlay as true by default", () => {
-    const widgetFactory = new WidgetFactory();
-    const props = {
-      name: "carousel",
-    };
-    const widget = widgetFactory.createWidget("carousel", props);
-    expect(widget.autoPlay).toBe(true);
-  });
-  it("should allow autoPlay to be set", () => {
-    const widgetFactory = new WidgetFactory();
-    const props = {
-      name: "carousel",
-      auto_play: false,
-    };
-    const widget = widgetFactory.createWidget("carousel", props);
-    expect(widget.autoPlay).toBe(false);
-  });
-  it("should have items with the first childs group items", () => {
-    const xml = `
-      <form>
-        <carousel name="carousel">
-            <group string="Group 1">
-                <field name="field1" string="Field 1" />
-            </group>
-            <group string="Group 2">
-                <field name="field2" string="Field 2" />
-                <group string="Group 3">
-                    <field name="field3" string="Field 3" />
-                </group>
-            </group>
-        </carousel>
-      </form>
-    `;
-    const fields = {
-      field1: {
-        string: "Field 1",
-        type: "char",
-        size: 10,
-      },
-      field2: {
-        string: "Field 2",
-        type: "char",
-        size: 10,
-      },
-      field3: {
-        string: "Field 3",
-        type: "char",
-        size: 10,
-      },
-    };
-
-    const form = new Form(fields);
-    form.parse(xml);
-    const carousel = form.findById("carousel") as Carousel;
-    expect(carousel).toBeInstanceOf(Carousel);
-    expect(carousel.items.length).toBe(2);
-    expect(carousel.items[0].label).toBe("Group 1");
-    expect(carousel.items[1].label).toBe("Group 2");
-  });
-});

package/src/spec/Field.spec.ts

@@ -1,35 +0,0 @@
-import { describe, it, expect } from "vitest";
-import Field from "../Field";
-
-describe("Field", () => {
-  describe("with the autoRefresh property", () => {
-    it("should be false as default", () => {
-      const props = {};
-      const field = new Field(props);
-      expect(field.autoRefresh).toBe(false);
-    });
-    it("should work with text", () => {
-      const props = {
-        autorefresh: "1",
-      };
-      const field = new Field(props);
-      expect(field.autoRefresh).toBe(true);
-    });
-    describe("if autorefresh is not a valid boold", () => {
-      it("should return false", () => {
-        const props = {
-          autorefresh: "abc",
-        };
-        const field = new Field(props);
-        expect(field.autoRefresh).toBe(false);
-      });
-    });
-    it("should return true for readOnly if autoRefresh is set", () => {
-      const props = {
-        autorefresh: true,
-      };
-      const field = new Field(props);
-      expect(field.readOnly).toBe(true);
-    });
-  });
-});

package/src/spec/nodeParser.spec.ts

@@ -1,53 +0,0 @@
-import { describe, it, expect } from "vitest";
-import { parseBoolAttribute } from "../helpers/nodeParser";
-
-describe("parseBoolAttribute", () => {
-  it("returns true for numeric 1", () => {
-    expect(parseBoolAttribute(1)).toBe(true);
-  });
-
-  it('returns true for string "1"', () => {
-    expect(parseBoolAttribute("1")).toBe(true);
-  });
-
-  it("returns true for boolean true", () => {
-    expect(parseBoolAttribute(true)).toBe(true);
-  });
-
-  it('returns true for string "True"', () => {
-    expect(parseBoolAttribute("True")).toBe(true);
-  });
-
-  it('returns true for string "true"', () => {
-    expect(parseBoolAttribute("true")).toBe(true);
-  });
-
-  it("returns false for numeric 0", () => {
-    expect(parseBoolAttribute(0)).toBe(false);
-  });
-
-  it('returns false for string "0"', () => {
-    expect(parseBoolAttribute("0")).toBe(false);
-  });
-
-  it("returns false for boolean false", () => {
-    expect(parseBoolAttribute(false)).toBe(false);
-  });
-
-  it('returns false for string "False"', () => {
-    expect(parseBoolAttribute("False")).toBe(false);
-  });
-
-  it("returns false for null", () => {
-    expect(parseBoolAttribute(null)).toBe(false);
-  });
-
-  it("returns false for undefined", () => {
-    expect(parseBoolAttribute(undefined)).toBe(false);
-  });
-
-  it("returns false for non-boolean strings", () => {
-    expect(parseBoolAttribute("yes")).toBe(false);
-    expect(parseBoolAttribute("no")).toBe(false);
-  });
-});