@qooxdoo/framework 8.0.0-beta.1 → 8.0.0-beta.2

package/source/class/qx/core/property/Property.js

@@ -13,22 +13,13 @@
 
    Authors:
      * John Spackman (github.com/johnspackman)
+     * Patryk Malinowski (https://github.com/patryk-m-malinowski, pmalinowski116@gmail.com)
 
 ************************************************************************ */
 
 /**
  * Property implementation for actual properties
  *
- * TODO:
- *
- * `validate` implementation
- * `delegate` implementation
- * `inheritable` implementation (pass onto `obj._getChildren`; check for special `inherit` value)
- * Array check
- * FunctionCheck
- *
- * how does init of property values work?  `init` per class and `initFunction` per instance?
- *
  */
 qx.Bootstrap.define("qx.core.property.Property", {
   implement: qx.core.property.IProperty,
@@ -54,7 +45,7 @@ qx.Bootstrap.define("qx.core.property.Property", {
      * If set to true, then properties with init values will have their apply method called during construction.
      * Default is false to maintain backward compatibility with v7 behavior.
      */
-    "qx.core.property.Property.applyDuringConstruct": false,
+    "qx.core.property.Property.applyDuringConstruct": false,//TODO should this be true?
 
     /**
      * Only relevant when applyDuringConstruct is true.
@@ -83,14 +74,14 @@ qx.Bootstrap.define("qx.core.property.Property", {
     /** @type {String} the name of the property */
     __propertyName: null,
 
-    /** @type {qx.Class} the class that defined the property */
+    /** @type {new () => qx.core.Object} the class that defined the property */
     __clazz: null,
 
     /** @type {Boolean} whether this is a pseudo property or not */
     __pseudoProperty: false,
 
     /**
-     * @type {qx.Class} the class that original defined this property, before it was cloned and
+     * @type {new () => qx.core.Object} the class that original defined this property, before it was cloned and
      * refined for the current `__clazz`
      */
     __superClass: null,
@@ -134,11 +125,9 @@ qx.Bootstrap.define("qx.core.property.Property", {
     /** @type {Boolean} whether the property needs to be dereferenced */
     __needsDereference: false,
 
-    isRefineAllowed(def) {},
-
     /**
      * The class where this property is defined
-     * @returns {Function}
+     * @returns {new () => qx.core.Object}
      */
     getClass() {
       return this.__clazz;
@@ -274,11 +263,21 @@ qx.Bootstrap.define("qx.core.property.Property", {
         }
       }
 
-      if (def.init !== undefined) {
+      let hasInitVal = def.init !== undefined;
+
+      if (qx.core.Environment.get("qx.debug")) {
+        if (hasInitVal && def.initFunction) {
+          throw new Error(`${this}: init and initFunction are mutually exclusive. Please specify only one of them.`);
+        }
+      }
+
+      if (hasInitVal) {
         this.__initValue = def.init;
+        this.__initFunction = undefined;
       }
       if (def.initFunction) {
         this.__initFunction = def.initFunction;
+        this.__initValue = undefined;
       }
 
       if (qx.core.Environment.get("qx.debug")) {
@@ -433,6 +432,8 @@ qx.Bootstrap.define("qx.core.property.Property", {
       let initValue = this.__initValue;
       if (initValue !== undefined) {
         clazz.prototype["$$init_" + propertyName] = initValue;
+      } else if (this.__initFunction) {
+        clazz.prototype["$$init_" + propertyName] = undefined;
       }
 
       addMethod("init" + upname, function (...args) {
@@ -567,9 +568,9 @@ qx.Bootstrap.define("qx.core.property.Property", {
       }
       state.initMethodCalled = true;
 
-      if (value !== undefined && this.__definition?.init !== undefined) {
+      if (value !== undefined && (this.__initValue !== undefined || this.__initFunction !== undefined)) {
         this.warn(
-          `${this}: init() called with a value, ignoring - use deferredInit and do not specify an init value in the property definition`
+          `${this}: init() called with a value, ignoring - use deferredInit and do not specify an init/function value in the property definition`
         );
         value = undefined;
       }

package/source/class/qx/data/SingleValueBinding.js

@@ -13,7 +13,7 @@
 
    Authors:
      * John Spackman (https://github.com/johnspackman, john.spackman@zenesis.com)
-     * Patryk Malinowski (https://github.com/patryk-m-malinowski, pmalinowski@vmn.digital)
+     * Patryk Malinowski (https://github.com/patryk-m-malinowski, pmalinowski116@gmail.com)
 
 ************************************************************************ */
 
@@ -172,7 +172,7 @@ qx.Class.define("qx.data.SingleValueBinding", {
     /**
      * @type {*}
      */
-    __value: null,
+    __value: undefined,
     /**
      * @type {qx.core.Object}
      */
@@ -284,7 +284,7 @@ qx.Class.define("qx.data.SingleValueBinding", {
         SingleValueBinding.__bindingsBySource[value.toHashCode()][this.toHashCode()] = this;
 
         //Set the input on the first segment of the source path
-        let promise = this.__sourceSegments[0].setInput(value);
+        let out = this.__sourceSegments[0].setInput(value);
         const cb = () => {
           if (qx.core.Environment.get("qx.debug")) {
             this.assertNotNull(
@@ -294,10 +294,8 @@ qx.Class.define("qx.data.SingleValueBinding", {
           }
         };
 
-        let out;
-
-        if (qx.Promise.isPromise(promise)) {
-          out = promise.then(cb);
+        if (qx.Promise.isPromise(out)) {
+          out = out.then(cb);
         } else {
           out = cb();
         }

package/source/class/qx/data/binding/AbstractSegment.js

@@ -12,7 +12,7 @@
      See the LICENSE file in the project's top-level directory for details.
 
    Authors:
-     * John Spackman (https://github.com/johnspackman, john.spackman@zenesis.com)
+   * Patryk Malinowski (https://github.com/patryk-m-malinowski, pmalinowski116@gmail.com)
 
 ************************************************************************ */
 

package/source/class/qx/data/binding/ArrayIndexSegment.js

@@ -12,7 +12,7 @@
      See the LICENSE file in the project's top-level directory for details.
 
    Authors:
-     * John Spackman (https://github.com/johnspackman, john.spackman@zenesis.com)
+    * Patryk Malinowski (https://github.com/patryk-m-malinowski, pmalinowski116@gmail.com)
 
 ************************************************************************ */
 
@@ -51,7 +51,7 @@ qx.Class.define("qx.data.binding.ArrayIndexSegment", {
   destruct() {
     let input = this.getInput();
     if (input) {
-      if (this.getInput() instanceof qx.data.Array) {
+      if (this.__isListData(this.getInput())) {
         this.getInput().removeListener("change", this.__onChangeContents, this);
       }
     }
@@ -71,19 +71,23 @@ qx.Class.define("qx.data.binding.ArrayIndexSegment", {
       return this.__string;
     },
 
+    __isListData(input) {
+      return qx.Interface.objectImplements(input, qx.data.IListData);
+    },
+
     /**
      * @override
      */
     _applyInput(value, oldValue) {
       if (oldValue) {
         this.setEventName(null);
-        if (oldValue instanceof qx.data.Array) {
+        if (this.__isListData(oldValue)) {
           oldValue.removeListener("change", this.__onChangeContents, this);
         }
       }
 
       if (value) {
-        if (value instanceof qx.data.Array) {
+        if (this.__isListData(value)) {
           value.addListener("change", this.__onChangeContents, this);
           this.setEventName("change");
         }
@@ -108,8 +112,10 @@ qx.Class.define("qx.data.binding.ArrayIndexSegment", {
         return undefined;
       }
 
-      if (input instanceof qx.data.Array) {
-        input = input.toArray();
+      if (this.__isListData(input)) {
+        const len = input.getLength();
+        const idx = this.__index === -1 ? len - 1 : this.__index;
+        return idx >= len ? undefined : input.getItem(idx);
       }
 
       if (this.__index >= input.length) {
@@ -126,11 +132,12 @@ qx.Class.define("qx.data.binding.ArrayIndexSegment", {
       if (!input) {
         return;
       }
-      let index = this.__index == -1 ? input.length - 1 : this.__index;
-      if (input instanceof qx.data.Array) {
-        input.setItem(index, targetValue);
+      if (this.__isListData(input)) {
+        const idx = this.__index === -1 ? input.getLength() - 1 : this.__index;
+        input.setItem(idx, targetValue);
       } else {
-        input[index] = targetValue;
+        const idx = this.__index === -1 ? input.length - 1 : this.__index;
+        input[idx] = targetValue;
       }
     },
 

package/source/class/qx/data/binding/IInputReceiver.js

@@ -7,7 +7,7 @@ qx.Interface.define("qx.data.binding.IInputReceiver", {
      *
      * @param {*} value
      * @returns {Promise?} If the operation is asynchronous, it should return a Promise which resolves when it has completed.
-     * If it's synchrous, it should return null.
+     * If it's synchronous, it should return null.
      */
     setInput(input) {}
   }

package/source/class/qx/data/binding/PropNameSegment.js

@@ -12,7 +12,7 @@
      See the LICENSE file in the project's top-level directory for details.
 
    Authors:
-     * John Spackman (https://github.com/johnspackman, john.spackman@zenesis.com)
+     * Patryk Malinowski (https://github.com/patryk-m-malinowski, pmalinowski116@gmail.com)
 
 ************************************************************************ */
 

package/source/class/qx/dev/unit/TestCase.js

@@ -201,7 +201,10 @@ qx.Class.define("qx.dev.unit.TestCase", {
       if (this.__autoDispose) {
         this.__autoDispose.forEach(function (obj) {
           if (!obj.isDisposed()) {
-            if (obj instanceof qx.ui.core.Widget) {
+            // Do not make an actual reference to qx.ui.core.Widget otherwise the compiler will load
+            // dependencies and that prevents this class from being used in node-only builds
+            const Widget = qx.Class.getByName("qx.ui.core.Widget");
+            if (obj instanceof Widget) {
               obj.destroy();
             } else if (obj instanceof qx.core.Object) {
               obj.dispose();

package/source/class/qx/event/handler/Focus.js

@@ -1344,8 +1344,9 @@ qx.Class.define("qx.event.handler.Focus", {
           return focusedElement;
         }
         if (qx.Class.isClass("qx.ui.core.Widget")) {
+          const Widget = qx.Class.getByName("qx.ui.core.Widget");
           // Check compound widgets
-          var widget = qx.ui.core.Widget.getWidgetByElement(focusedElement),
+          var widget = Widget.getWidgetByElement(focusedElement),
             textField =
               widget &&
               widget.getChildControl &&

package/source/class/qx/html/Jsx.js

@@ -48,11 +48,10 @@ qx.Class.define("qx.html.Jsx", {
      *
      * @param tagname {String|Function} the name of the tag
      * @param attributes {Record<string, any>} map of attribute values
-     * @param children {qx.html.Node[]} array of children
+     * @param children {...qx.html.Node} children
      * @return {qx.html.Element|qx.data.Array}
      */
-    createElement(tagname, attributes) {
-      const children = qx.lang.Array.fromArguments(arguments, 2);
+    createElement(tagname, attributes, ...children) {
       attributes ??= {};
 
       // CSS CUSTOM PROPERTIES

package/source/class/qx/html/Node.js

@@ -889,10 +889,10 @@ qx.Class.define("qx.html.Node", {
     /**
      * Append all given children at the end of this element.
      *
-     * @param varargs {qx.html.Element} elements to insert
+     * @param {...qx.html.Element} children elements to insert
      * @return {qx.html.Element} this object (for chaining support)
      */
-    add(varargs) {
+    add(...children) {
       var self = this;
       function addImpl(arr) {
         arr.forEach(function (child) {
@@ -917,7 +917,7 @@ qx.Class.define("qx.html.Node", {
           self._children.push(child);
         });
       }
-      addImpl(qx.lang.Array.fromArguments(arguments));
+      addImpl(children);
 
       // Chaining support
       return this;

package/source/class/qx/io/jsonrpc/Client.js

@@ -406,6 +406,6 @@ qx.Class.define("qx.io.jsonrpc.Client", {
      * @deprecated
      * Behavior in v8 will be as if the environment variable value is `true`, but the environment variable will no longer be available.
      */
-    "qx.io.jsonrpc.forwardTransportPromiseRejectionToRequest": false
+    "qx.io.jsonrpc.forwardTransportPromiseRejectionToRequest": true
   }
 });

package/source/class/qx/promise/NativeWrapper.js

@@ -13,7 +13,7 @@
 
    Authors:
  * John Spackman (john.spackman@zenesis.com)
- * Patryk Malinowski (pmalinowski@vmn.digital)
+ * Patryk Malinowski (https://github.com/patryk-m-malinowski, pmalinowski116@gmail.com)
 
  ************************************************************************ */
 

package/source/class/qx/test/core/Property.js

@@ -36,7 +36,7 @@ qx.Class.define("qx.test.core.Property", {
         extend: qx.core.Object
       });
 
-      qx.Class.define("qx.test.cpnfv8.Superclass", {
+      qx.Class.define("qx.test.cpnfv8.Superclass", {//cbh
         extend: qx.test.cpnfv8.Object,
 
         construct(bRunning = true) {
@@ -107,7 +107,18 @@ qx.Class.define("qx.test.core.Property", {
 
           readOnly: {
             initFunction: () => 42,
-            immutable: "readonly"
+            immutable: "readonly",
+            apply() {
+              this.appliedReadOnly = true;
+            }
+          },
+
+          noAutoApply: {
+            initFunction:() => 24,
+            autoApply: false,
+            apply() {
+              throw new Error("Should not call here!")
+            }
           }
         },
 
@@ -540,6 +551,7 @@ qx.Class.define("qx.test.core.Property", {
 
     testApply() {
       let obj = new qx.test.cpnfv8.Superclass();
+      this.assertTrue(obj.appliedReadOnly, "Expected apply method for property 'readOnly' to be called, but wasn't");
       this.assertEquals(3, obj.applyCount, "initial apply count should be 3");
       this.assertArrayEquals([null, undefined], obj.lastApply, "last apply call during construction");
       obj.setRunning(false);
@@ -574,6 +586,7 @@ qx.Class.define("qx.test.core.Property", {
       //old, deprecated behaviour
       qx.core.Environment.set("qx.core.property.Property.applyDuringConstruct", false);
       obj = new qx.test.cpnfv8.Superclass();
+      this.assertTrue(obj.appliedReadOnly, "Expected apply method for property 'readOnly' to be called, but wasn't");
       this.assertEquals(1, obj.applyCount, "old behaviour: initial apply count should be 1");
       obj.resetAnyProp();
       this.assertEquals(1, obj.applyCount, "old behaviour: apply count after reset should still be 1");
@@ -821,6 +834,21 @@ qx.Class.define("qx.test.core.Property", {
       let second = inst.getJsdocProp();
       this.assertArrayEquals([10, 20, 30], initial, "properties created with initFunction returned by getter should be the same");
       this.assertIdentical(initial, second, "properties created with initFunction returned by getter should be the same");      
+
+      try {
+        qx.Class.define(null, {
+          extend: qx.core.Object,
+          properties: {
+            badProp: {
+              init: 5,
+              initFunction: () => 10
+            }
+          }
+        });
+        this.fail("Defining a property with both init and initFunction should throw an error");
+      } catch (e) {
+        // expected
+      }
     },
 
     testDefinesThanSubClassWithInterface() {

package/source/class/qx/test/data/singlevalue/Simple.js

@@ -791,6 +791,12 @@ qx.Class.define("qx.test.data.singlevalue.Simple", {
       person2.setFriend(null);
       this.assertEquals(3, timesCalled, "Converter should not be called when chain is broken and ignoreConverter is set and matches.");
       this.assertEquals("Patryk", person3.getName(), "Target should be reset when chain is broken");
+
+      //ensure that during the initial set, if the chain is incomplete (i.e. has a null value),
+      //the target property is reset (not set to null)
+      let person4 = new Person("Harry");
+      person3.bind("friend.friend.name", person4, "name");
+      this.assertEquals("Patryk", person4.getName(), "Target should be reset when chain has null element during initial set");
     }
   }
 });

package/source/class/qx/test/locale/Date.js

@@ -32,7 +32,7 @@ qx.Class.define("qx.test.locale.Date", {
     testGetAmMarker() {
       var Date = qx.locale.Date;
       var items = [
-        { locale: "ko", expected: "오전" },
+        { locale: "zh", expected: "上午" },
         { locale: "fr", expected: "AM" },
         { locale: "af", expected: "vm." }
       ];
@@ -46,7 +46,7 @@ qx.Class.define("qx.test.locale.Date", {
     testGetPMMarker() {
       var Date = qx.locale.Date;
       var items = [
-        { locale: "ko", expected: "오후" },
+        { locale: "zh", expected: "下午" },
         { locale: "fr", expected: "PM" },
         { locale: "af", expected: "nm." }
       ];

package/source/class/qx/theme/classic/Appearance.js

@@ -1465,6 +1465,27 @@ qx.Theme.define("qx.theme.classic.Appearance", {
       }
     },
 
+    "selectbox-arrow-button": "widget",
+
+    /*
+    ---------------------------------------------------------------------------
+      CHECKED SELECT BOX
+    ---------------------------------------------------------------------------
+    */
+
+    "checked-selectbox": "selectbox",
+
+    "checked-selectbox/allNone": {
+      include: "button"
+    },
+
+    "checked-selectbox/tag": "tag",
+
+    tag: {
+      alias: "button",
+      include: "button"
+    },
+
     /*
     ---------------------------------------------------------------------------
       DATE CHOOSER

package/source/class/qx/theme/modern/Appearance.js

@@ -1728,6 +1728,27 @@ qx.Theme.define("qx.theme.modern.Appearance", {
       }
     },
 
+    "selectbox-arrow-button": "widget",
+
+    /*
+    ---------------------------------------------------------------------------
+      CHECKED SELECT BOX
+    ---------------------------------------------------------------------------
+    */
+
+    "checked-selectbox": "selectbox",
+
+    "checked-selectbox/allNone": {
+      include: "button"
+    },
+
+    "checked-selectbox/tag": "tag",
+
+    tag: {
+      alias: "button",
+      include: "button"
+    },
+
     /*
     ---------------------------------------------------------------------------
       DATE CHOOSER

package/source/class/qx/theme/simple/Appearance.js

@@ -1204,6 +1204,27 @@ qx.Theme.define("qx.theme.simple.Appearance", {
       }
     },
 
+    "selectbox-arrow-button": "widget",
+
+    /*
+    ---------------------------------------------------------------------------
+      CHECKED SELECT BOX
+    ---------------------------------------------------------------------------
+    */
+
+    "checked-selectbox": "selectbox",
+
+    "checked-selectbox/allNone": {
+      include: "button"
+    },
+
+    "checked-selectbox/tag": "tag",
+
+    tag: {
+      alias: "button",
+      include: "button"
+    },
+
     /*
     ---------------------------------------------------------------------------
       COMBO BOX

package/source/class/qx/theme/tangible/Appearance.js

@@ -1313,6 +1313,8 @@ qx.Theme.define("qx.theme.tangible.Appearance", {
       }
     },
 
+    "selectbox-arrow-button": "widget",
+
     /*
     ---------------------------------------------------------------------------
       CHECKED SELECT BOX

package/source/class/qx/tool/cli/AbstractCliApp.js

@@ -41,12 +41,28 @@ qx.Class.define("qx.tool.cli.AbstractCliApp", {
         console.log((cmd || rootCmd).usage());
         process.exit((!cmd || errors) ? 1 : 0);
       }
+      let exitCode = 0;
       try {
-        process.exit(await run.call(cmd, cmd) ?? 0);
+        exitCode = await run.call(cmd, cmd) ?? 0;
       } catch (ex) {
         console.error("ERROR:\n" + (ex.stack ?? ex.message) + "\n");
-        process.exit(1);
+        exitCode = 1;
       }
+      // Close undici's global connection pool before exiting.
+      // Native fetch (undici) keeps HTTP keep-alive connections open which would prevent
+      // natural process exit. node:undici is available as a built-in from Node.js 22+.
+      try {
+        const { getGlobalDispatcher } = require("node:undici");
+        await getGlobalDispatcher().close();
+      } catch (e) {
+        // node:undici not available on this Node.js version — proceed
+      }
+      // Use process.exitCode instead of process.exit() to avoid a libuv race on Windows + Node 24:
+      // close() resolves, but undici's internal uv_async_t handles are still tearing down; calling
+      // process.exit() immediately marks them UV_HANDLE_CLOSING, then a background task fires
+      // uv_async_send() on one → assertion fails (libuv 1.50.x made this a hard crash, not a no-op).
+      // Setting exitCode and returning lets the event loop drain naturally instead.
+      process.exitCode = exitCode;
     },
 
     /**

package/source/class/qx/tool/compiler/ClassFile.js

@@ -3164,10 +3164,6 @@ qx.Class.define("qx.tool.compiler.ClassFile", {
      * These options are passed to Babel for JSX compilation; they can be changed by the CLI etc
      * as needed.
      *
-     * Note that at the moment they use a class that does not exist!  `qx.html.Jsx` is coming soon
-     * to a PR near you, but in the mean time you could use the compile.json `jsx` setting to
-     * change these to something else, eg `{ pragma: "jsx.dom", pragmaFrag: "jsx.Fragment }` and
-     * use https://github.com/alecsgone/jsx-render in your application's code.
      */
     JSX_OPTIONS: {
       pragma: "qx.html.Jsx.createElement",

package/source/class/qx/tool/compiler/MetaDatabase.js

@@ -211,6 +211,53 @@ qx.Class.define("qx.tool.compiler.MetaDatabase", {
       }
     },
 
+    /**
+     * Scans directories recursively for .js files and populates the database.
+     * Calls addFile() for each discovered file, then reparseAll().
+     *
+     * @param {String[]} dirs directories to scan
+     * @param {Object} [options]
+     * @param {Boolean} [options.force] reparse even if up to date (default: false)
+     * @param {Object}  [options.ignore] an `ignore` instance for exclude patterns
+     * @param {Boolean} [options.verbose] log each processed file
+     */
+    async loadFromDirectories(dirs, options = {}) {
+      const classFiles = [];
+
+      const scanImpl = async filename => {
+        let basename = path.basename(filename);
+
+        if (options.ignore) {
+          let relativePath = path.relative(process.cwd(), filename);
+          if (options.ignore.ignores(relativePath)) {
+            return;
+          }
+        }
+
+        let stat = await fs.promises.stat(filename);
+        if (stat.isFile() && basename.match(/\.js$/)) {
+          classFiles.push(filename);
+        } else if (stat.isDirectory() && (basename == "." || basename[0] != ".")) {
+          let files = await fs.promises.readdir(filename);
+          for (let file of files) {
+            await scanImpl(path.join(filename, file));
+          }
+        }
+      };
+
+      for (let dir of dirs) {
+        await scanImpl(dir);
+      }
+
+      for (let filename of classFiles) {
+        if (options.verbose) {
+          qx.tool.compiler.Console.info(`Processing ${filename} ...`);
+        }
+        await this.addFile(filename, options.force || false);
+      }
+      await this.reparseAll();
+    },
+
     __createDerivedClassLookup() {
       const lookup = {};