@qooxdoo/framework 7.9.2 → 8.0.0-beta.2

package/source/class/qx/Interface.js

@@ -93,10 +93,7 @@ qx.Bootstrap.define("qx.Interface", {
     define(name, config) {
       if (config) {
         // Normalize include
-        if (
-          config.extend &&
-          !(qx.Bootstrap.getClass(config.extend) === "Array")
-        ) {
+        if (config.extend && !(qx.Bootstrap.getClass(config.extend) === "Array")) {
           config.extend = [config.extend];
         }
 
@@ -218,8 +215,7 @@ qx.Bootstrap.define("qx.Interface", {
         for (var key in members) {
           if (qx.Bootstrap.isFunction(members[key])) {
             var isPropertyMethod = this.__isPropertyMethod(clazz, key);
-            var hasMemberFunction =
-              isPropertyMethod || qx.Bootstrap.isFunction(object[key]);
+            var hasMemberFunction = isPropertyMethod || qx.Bootstrap.isFunction(object[key]);
 
             if (!hasMemberFunction) {
               if (shouldThrow) {
@@ -239,18 +235,10 @@ qx.Bootstrap.define("qx.Interface", {
 
             // Only wrap members if the interface was not been applied yet. This
             // can easily be checked by the recursive hasInterface method.
-            var shouldWrapFunction =
-              wrap === true &&
-              !isPropertyMethod &&
-              !qx.util.OOUtil.hasInterface(clazz, iface);
+            var shouldWrapFunction = wrap === true && !isPropertyMethod && !qx.util.OOUtil.hasInterface(clazz, iface);
 
             if (shouldWrapFunction) {
-              object[key] = this.__wrapInterfaceMember(
-                iface,
-                object[key],
-                key,
-                members[key]
-              );
+              object[key] = this.__wrapInterfaceMember(iface, object[key], key, members[key]);
             }
           } else {
             // Other members are not checked more detailed because of
@@ -296,10 +284,7 @@ qx.Bootstrap.define("qx.Interface", {
       }
 
       var propertyName = qx.Bootstrap.firstLow(match[2]);
-      var isPropertyMethod = qx.util.OOUtil.getPropertyDefinition(
-        clazz,
-        propertyName
-      );
+      var isPropertyMethod = qx.Class.getByProperty(clazz, propertyName);
 
       if (!isPropertyMethod) {
         return false;
@@ -307,10 +292,7 @@ qx.Bootstrap.define("qx.Interface", {
 
       var isBoolean = match[0] === "is" || match[0] === "toggle";
       if (isBoolean) {
-        return (
-          qx.util.OOUtil.getPropertyDefinition(clazz, propertyName).check ===
-          "Boolean"
-        );
+        return qx.Class.getByProperty(clazz, propertyName).getCheck() instanceof qx.core.check.standard.BooleanCheck;
       }
 
       return true;
@@ -328,15 +310,9 @@ qx.Bootstrap.define("qx.Interface", {
     __checkProperties(clazz, iface, shouldThrow) {
       if (iface.$$properties) {
         for (var key in iface.$$properties) {
-          if (!qx.util.OOUtil.getPropertyDefinition(clazz, key)) {
+          if (!qx.Class.getByProperty(clazz, key)) {
             if (shouldThrow) {
-              throw new Error(
-                'The property "' +
-                  key +
-                  '" is not supported by Class "' +
-                  clazz.classname +
-                  '"!'
-              );
+              throw new Error('The property "' + key + '" is not supported by Class "' + clazz.classname + '"!');
             } else {
               return false;
             }
@@ -362,13 +338,7 @@ qx.Bootstrap.define("qx.Interface", {
         for (var key in iface.$$events) {
           if (!qx.util.OOUtil.supportsEvent(clazz, key)) {
             if (shouldThrow) {
-              throw new Error(
-                'The event "' +
-                  key +
-                  '" is not supported by Class "' +
-                  clazz.classname +
-                  '"!'
-              );
+              throw new Error('The event "' + key + '" is not supported by Class "' + clazz.classname + '"!');
             } else {
               return false;
             }
@@ -389,16 +359,18 @@ qx.Bootstrap.define("qx.Interface", {
      *  @param iface {Interface} The interface to verify
      */
     assertObject(object, iface) {
-      var clazz = object.constructor;
-      this.__checkMembers(object, clazz, iface, false, true);
-      this.__checkProperties(clazz, iface, true);
-      this.__checkEvents(clazz, iface, true);
-
-      // Validate extends, recursive
-      var extend = iface.$$extends;
-      if (extend) {
-        for (var i = 0, l = extend.length; i < l; i++) {
-          this.assertObject(object, extend[i]);
+      if (qx.core.Environment.get("qx.debug")) {
+        var clazz = object.constructor;
+        this.__checkMembers(object, clazz, iface, false, true);
+        this.__checkProperties(clazz, iface, true);
+        this.__checkEvents(clazz, iface, true);
+
+        // Validate extends, recursive
+        var extend = iface.$$extends;
+        if (extend) {
+          for (var i = 0, l = extend.length; i < l; i++) {
+            this.assertObject(object, extend[i]);
+          }
         }
       }
     },
@@ -412,15 +384,17 @@ qx.Bootstrap.define("qx.Interface", {
      *     check parameters etc.
      */
     assert(clazz, iface, wrap) {
-      this.__checkMembers(clazz.prototype, clazz, iface, wrap, true);
-      this.__checkProperties(clazz, iface, true);
-      this.__checkEvents(clazz, iface, true);
-
-      // Validate extends, recursive
-      var extend = iface.$$extends;
-      if (extend) {
-        for (var i = 0, l = extend.length; i < l; i++) {
-          this.assert(clazz, extend[i], wrap);
+      if (qx.core.Environment.get("qx.debug")) {
+        this.__checkMembers(clazz.prototype, clazz, iface, wrap, true);
+        this.__checkProperties(clazz, iface, true);
+        this.__checkEvents(clazz, iface, true);
+
+        // Validate extends, recursive
+        var extend = iface.$$extends;
+        if (extend) {
+          for (var i = 0, l = extend.length; i < l; i++) {
+            this.assert(clazz, extend[i], wrap);
+          }
         }
       }
     },
@@ -435,11 +409,7 @@ qx.Bootstrap.define("qx.Interface", {
      */
     objectImplements(object, iface) {
       var clazz = object.constructor;
-      if (
-        !this.__checkMembers(object, clazz, iface) ||
-        !this.__checkProperties(clazz, iface) ||
-        !this.__checkEvents(clazz, iface)
-      ) {
+      if (!this.__checkMembers(object, clazz, iface) || !this.__checkProperties(clazz, iface) || !this.__checkEvents(clazz, iface)) {
         return false;
       }
 
@@ -570,34 +540,16 @@ qx.Bootstrap.define("qx.Interface", {
 
           for (var key in config) {
             if (allowed[key] === undefined) {
-              throw new Error(
-                'The configuration key "' +
-                  key +
-                  '" in class "' +
-                  name +
-                  '" is not allowed!'
-              );
+              throw new Error('The configuration key "' + key + '" in class "' + name + '" is not allowed!');
             }
 
             if (config[key] == null) {
-              throw new Error(
-                "Invalid key '" +
-                  key +
-                  "' in interface '" +
-                  name +
-                  "'! The value is undefined/null!"
-              );
+              throw new Error("Invalid key '" + key + "' in interface '" + name + "'! The value is undefined/null!");
             }
 
             if (allowed[key] !== null && typeof config[key] !== allowed[key]) {
               throw new Error(
-                'Invalid type of key "' +
-                  key +
-                  '" in interface "' +
-                  name +
-                  '"! The type of the key must be "' +
-                  allowed[key] +
-                  '"!'
+                'Invalid type of key "' + key + '" in interface "' + name + '"! The type of the key must be "' + allowed[key] + '"!'
               );
             }
           }
@@ -609,18 +561,9 @@ qx.Bootstrap.define("qx.Interface", {
 
             if (
               config[key] !== undefined &&
-              (["Array", "RegExp", "Date"].indexOf(
-                qx.Bootstrap.getClass(config[key])
-              ) != -1 ||
-                config[key].classname !== undefined)
+              (["Array", "RegExp", "Date"].indexOf(qx.Bootstrap.getClass(config[key])) != -1 || config[key].classname !== undefined)
             ) {
-              throw new Error(
-                'Invalid key "' +
-                  key +
-                  '" in interface "' +
-                  name +
-                  '"! The value needs to be a map!'
-              );
+              throw new Error('Invalid key "' + key + '" in interface "' + name + '"! The value needs to be a map!');
             }
           }
 
@@ -655,13 +598,7 @@ qx.Bootstrap.define("qx.Interface", {
           if (config.statics) {
             for (var key in config.statics) {
               if (key.toUpperCase() !== key) {
-                throw new Error(
-                  'Invalid key "' +
-                    key +
-                    '" in interface "' +
-                    name +
-                    '"! Static constants must be all uppercase.'
-                );
+                throw new Error('Invalid key "' + key + '" in interface "' + name + '"! Static constants must be all uppercase.');
               }
 
               switch (typeof config.statics[key]) {
@@ -672,11 +609,7 @@ qx.Bootstrap.define("qx.Interface", {
 
                 default:
                   throw new Error(
-                    'Invalid key "' +
-                      key +
-                      '" in interface "' +
-                      name +
-                      '"! Static constants must be all of a primitive type.'
+                    'Invalid key "' + key + '" in interface "' + name + '"! Static constants must be all of a primitive type.'
                   );
               }
             }

package/source/class/qx/Mixin.js

@@ -83,10 +83,7 @@ qx.Bootstrap.define("qx.Mixin", {
     define(name, config) {
       if (config) {
         // Normalize include
-        if (
-          config.include &&
-          !(qx.Bootstrap.getClass(config.include) === "Array")
-        ) {
+        if (config.include && !(qx.Bootstrap.getClass(config.include) === "Array")) {
           config.include = [config.include];
         }
 
@@ -148,7 +145,7 @@ qx.Bootstrap.define("qx.Mixin", {
 
       // Add basics
       mixin.$$type = "Mixin";
-      mixin.name = name;
+      mixin.mixinName = name;
 
       // Attach toString
       mixin.toString = this.genericToString;
@@ -188,50 +185,26 @@ qx.Bootstrap.define("qx.Mixin", {
 
         for (var key in mixin.events) {
           if (events[key]) {
-            throw new Error(
-              'Conflict between mixin "' +
-                mixin.name +
-                '" and "' +
-                events[key] +
-                '" in member "' +
-                key +
-                '"!'
-            );
+            throw new Error('Conflict between mixin "' + mixin.mixinName + '" and "' + events[key] + '" in member "' + key + '"!');
           }
 
-          events[key] = mixin.name;
+          events[key] = mixin.mixinName;
         }
 
         for (var key in mixin.properties) {
           if (properties[key]) {
-            throw new Error(
-              'Conflict between mixin "' +
-                mixin.name +
-                '" and "' +
-                properties[key] +
-                '" in property "' +
-                key +
-                '"!'
-            );
+            throw new Error('Conflict between mixin "' + mixin.mixinName + '" and "' + properties[key] + '" in property "' + key + '"!');
           }
 
-          properties[key] = mixin.name;
+          properties[key] = mixin.mixinName;
         }
 
         for (var key in mixin.members) {
           if (members[key]) {
-            throw new Error(
-              'Conflict between mixin "' +
-                mixin.name +
-                '" and "' +
-                members[key] +
-                '" in member "' +
-                key +
-                '"!'
-            );
+            throw new Error('Conflict between mixin "' + mixin.mixinName + '" and "' + members[key] + '" in member "' + key + '"!');
           }
 
-          members[key] = mixin.name;
+          members[key] = mixin.mixinName;
         }
       }
 
@@ -306,50 +279,58 @@ qx.Bootstrap.define("qx.Mixin", {
     },
 
     /**
-     * This method is used to determine the base method to call at runtime, and is used
-     * by Mixins where the mixin method calls `this.base()`.  It is only required by the
-     * compiler, and not the generator.
+     * This method is used to determine the base method to call at runtime,
+     * and is used by Mixins where the mixin method calls `this.base()`. It is
+     * only required by the compiler, and not the generator.
+     *
+     * The problem is that while Mixin's cannot override the same methods in a
+     * single class, they can override methods that were implemented in a base
+     * base - but the compiler cannot emit compile-time code which knows the
+     * base class method because that depends on the class that the mixin is
+     * mixed-into.
      *
-     * The problem is that while Mixin's cannot override the same methods in a single class,
-     * they can override methods that were implemented in a base base - but the compiler
-     * cannot emit compile-time code which knows the base class method because that depends
-     * on the class that the mixin is mixed-into.
+     * This method will search the hierarchy of the class at runtime, and
+     * figure out the nearest superclass method to call; the result is cached,
+     * and it is acceptable for a mixin's method to override a method mixed
+     * into a superclass.
      *
-     * This method will search the hierarchy of the class at runtime, and figure out the
-     * nearest superclass method to call; the result is cached, and it is acceptable for
-     * a mixin's method to override a method mixed into a superclass.
+     * Technically, this method should be private - it is internal and no
+     * notification will be given if the API changes. However, because it
+     * needs to be called by generated code in any class, it has to appear as
+     * public. Do not use it directly.
      *
-     * Technically, this method should be private - it is internal and no notification will
-     * be given if the API changes.  However, because it needs to be called by generated code
-     * in any class, it has to appear as public.  Do not use it directly.
+     * @param clazz {Class}
+     *   The class that is to be examined
      *
-     * @param clazz {Class} the class that is to be examined
-     * @param mixin {Mixin} the mixin that is calling `this.base`
-     * @param methodName {String} the name of the method in `mixin` that is calling `this.base`
-     * @return {Function} the base class function to call
+     * @param mixin {Mixin}
+     *   The mixin that is calling `this.base`
+     *
+     * @param methodName {String}
+     *   The name of the method in `mixin` that is calling `this.base` @return
+     *   {Function} the base class function to call
      */
     baseClassMethod(clazz, mixin, methodName) {
       if (!qx.core.Environment.get("qx.compiler")) {
         qx.log.Logger.error(
-          "qx.Mixin.baseClassMethod should not be used except with code compiled by the compiler (ie NOT the generator / python toolchain)"
+          "qx.Mixin.baseClassMethod should not be used " +
+            "except with code compiled by the compiler " +
+            "(ie NOT the generator / python toolchain)"
         );
+        return undefined;
       } else {
         if (
           clazz.$$mixinBaseClassMethods &&
-          clazz.$$mixinBaseClassMethods[mixin.name] !== undefined &&
-          clazz.$$mixinBaseClassMethods[mixin.name][methodName] !== undefined
+          clazz.$$mixinBaseClassMethods[mixin.mixinName] !== undefined &&
+          clazz.$$mixinBaseClassMethods[mixin.mixinName][methodName] !== undefined
         ) {
-          return clazz.$$mixinBaseClassMethods[mixin.name][methodName];
+          return clazz.$$mixinBaseClassMethods[mixin.mixinName][methodName];
         }
 
-        // Find the class which added the mixin; if it is mixed in twice, we pick the super-most class
+        // Find the class which added the mixin; if it is mixed in
+        // twice, we pick the super-most class
         var mixedInAt = null;
         var mixedInIndex = -1;
-        for (
-          var searchClass = clazz;
-          searchClass;
-          searchClass = searchClass.superclass
-        ) {
+        for (var searchClass = clazz; searchClass; searchClass = searchClass.superclass) {
           if (searchClass.$$flatIncludes) {
             var pos = searchClass.$$flatIncludes.indexOf(mixin);
             if (pos > -1) {
@@ -362,8 +343,9 @@ qx.Bootstrap.define("qx.Mixin", {
         var fn = null;
 
         if (mixedInAt) {
-          // Multiple mixins can provide an implementation, in which case the mixin which was
-          //  added second's "base" implementation is the first mixin's method
+          // Multiple mixins can provide an implementation, in which
+          //  case the mixin which was added second's "base"
+          //  implementation is the first mixin's method
           for (var i = mixedInIndex - 1; i > -1; i--) {
             var peerMixin = mixedInAt.$$flatIncludes[i];
             if (peerMixin.$$members[methodName]) {
@@ -394,18 +376,13 @@ qx.Bootstrap.define("qx.Mixin", {
           if (!clazz.$$mixinBaseClassMethods) {
             clazz.$$mixinBaseClassMethods = {};
           }
-          if (!clazz.$$mixinBaseClassMethods[mixin.name]) {
-            clazz.$$mixinBaseClassMethods[mixin.name] = {};
+          if (!clazz.$$mixinBaseClassMethods[mixin.mixinName]) {
+            clazz.$$mixinBaseClassMethods[mixin.mixinName] = {};
           }
-          clazz.$$mixinBaseClassMethods[mixin.name][methodName] = fn;
+          clazz.$$mixinBaseClassMethods[mixin.mixinName][methodName] = fn;
         } else if (qx.core.Environment.get("qx.debug")) {
           throw new Error(
-            "Cannot find base class method called " +
-              methodName +
-              " for mixin " +
-              mixin.name +
-              ", when viewed from " +
-              clazz.classname
+            "Cannot find base class method called " + methodName + " for mixin " + mixin.mixinName + ", when viewed from " + clazz.classname
           );
         }
 
@@ -427,7 +404,7 @@ qx.Bootstrap.define("qx.Mixin", {
      * @return {String} The mixin identifier
      */
     genericToString() {
-      return "[Mixin " + this.name + "]";
+      return "[Mixin " + this.mixinName + "]";
     },
 
     /** Registers all defined mixins */
@@ -462,34 +439,16 @@ qx.Bootstrap.define("qx.Mixin", {
         var allowed = this.__allowedKeys;
         for (var key in config) {
           if (!allowed[key]) {
-            throw new Error(
-              'The configuration key "' +
-                key +
-                '" in mixin "' +
-                name +
-                '" is not allowed!'
-            );
+            throw new Error('The configuration key "' + key + '" in mixin "' + name + '" is not allowed!');
           }
 
           if (config[key] == null) {
-            throw new Error(
-              'Invalid key "' +
-                key +
-                '" in mixin "' +
-                name +
-                '"! The value is undefined/null!'
-            );
+            throw new Error('Invalid key "' + key + '" in mixin "' + name + '"! The value is undefined/null!');
           }
 
           if (allowed[key] !== null && typeof config[key] !== allowed[key]) {
             throw new Error(
-              'Invalid type of key "' +
-                key +
-                '" in mixin "' +
-                name +
-                '"! The type of the key must be "' +
-                allowed[key] +
-                '"!'
+              'Invalid type of key "' + key + '" in mixin "' + name + '"! The type of the key must be "' + allowed[key] + '"!'
             );
           }
         }
@@ -501,18 +460,9 @@ qx.Bootstrap.define("qx.Mixin", {
 
           if (
             config[key] !== undefined &&
-            (["Array", "RegExp", "Date"].indexOf(
-              qx.Bootstrap.getClass(config[key])
-            ) != -1 ||
-              config[key].classname !== undefined)
+            (["Array", "RegExp", "Date"].indexOf(qx.Bootstrap.getClass(config[key])) != -1 || config[key].classname !== undefined)
           ) {
-            throw new Error(
-              'Invalid key "' +
-                key +
-                '" in mixin "' +
-                name +
-                '"! The value needs to be a map!'
-            );
+            throw new Error('Invalid key "' + key + '" in mixin "' + name + '"! The value needs to be a map!');
           }
         }
 
@@ -521,21 +471,13 @@ qx.Bootstrap.define("qx.Mixin", {
           for (var i = 0, a = config.include, l = a.length; i < l; i++) {
             if (a[i] == null) {
               throw new Error(
-                "Includes of mixins must be mixins. The include number '" +
-                  (i + 1) +
-                  "' in mixin '" +
-                  name +
-                  "'is undefined/null!"
+                "Includes of mixins must be mixins. The include number '" + (i + 1) + "' in mixin '" + name + "'is undefined/null!"
               );
             }
 
             if (a[i].$$type !== "Mixin") {
               throw new Error(
-                "Includes of mixins must be mixins. The include number '" +
-                  (i + 1) +
-                  "' in mixin '" +
-                  name +
-                  "'is not a mixin!"
+                "Includes of mixins must be mixins. The include number '" + (i + 1) + "' in mixin '" + name + "'is not a mixin!"
               );
             }
           }

package/source/class/qx/Promise.js

@@ -233,6 +233,7 @@ qx.Class.define("qx.Promise", {
           );
         } else if (!qx.Promise.__cancelWarningShown) {
           qx.Promise.__cancelWarningShown = true;
+          // eslint-disable-next-line
           console.warn(
             "qx.Promise.cancel is deprecated and will be removed in the future"
           );

package/source/class/qx/Theme.js

@@ -69,7 +69,7 @@ qx.Bootstrap.define("qx.Theme", {
      */
     define(name, config) {
       if (!config) {
-        var config = {};
+        config = {};
       }
 
       config.include = this.__normalizeArray(config.include);

package/source/class/qx/application/Routing.js

@@ -83,6 +83,8 @@
  * Note this class must be disposed after use
  */
 qx.Bootstrap.define("qx.application.Routing", {
+  extend: qx.core.Object,
+
   implement: [qx.core.IDisposable],
 
   construct() {

package/source/class/qx/bom/Font.js

@@ -305,6 +305,9 @@ qx.Class.define("qx.bom.Font", {
 
     // property apply
     _applyFamily(value, old) {
+      if (!value) {
+        return;
+      }
       var family = "";
 
       for (var i = 0, l = value.length; i < l; i++) {

package/source/class/qx/bom/client/Locale.js

@@ -78,6 +78,11 @@ qx.Bootstrap.define("qx.bom.client.Locale", {
      */
     __getNavigatorLocale() {
       var locale = navigator.userLanguage || navigator.language || "";
+      // Since 146 version of Firefox the value of navigator.language
+      // equals a string value "undefined".
+      if (locale === "undefined") {
+        locale = "";
+      }
 
       // Android Bug: Android does not return the system language from the
       // navigator language before version 4.4.x. Try to parse the language

package/source/class/qx/bom/request/Jsonp.js

@@ -53,6 +53,7 @@ qx.Bootstrap.define("qx.bom.request.Jsonp", {
   extend: qx.bom.request.Script,
 
   construct() {
+    super();
     // Borrow super-class constructor
     qx.bom.request.Script.apply(this);
 
@@ -128,9 +129,7 @@ qx.Bootstrap.define("qx.bom.request.Jsonp", {
       this.__callbackCalled = false;
 
       callbackParam = this.__callbackParam || "callback";
-      callbackName =
-        this.__callbackName ||
-        this.__prefix + "qx.bom.request.Jsonp." + this.__id + ".callback";
+      callbackName = this.__callbackName || this.__prefix + "qx.bom.request.Jsonp." + this.__id + ".callback";
 
       // Default callback
       if (!this.__callbackName) {
@@ -149,19 +148,13 @@ qx.Bootstrap.define("qx.bom.request.Jsonp", {
           };
         } else {
           if (qx.core.Environment.get("qx.debug.io")) {
-            qx.Bootstrap.debug(
-              qx.bom.request.Jsonp,
-              "Callback " + this.__callbackName + " already exists"
-            );
+            qx.Bootstrap.debug(qx.bom.request.Jsonp, "Callback " + this.__callbackName + " already exists");
           }
         }
       }
 
       if (qx.core.Environment.get("qx.debug.io")) {
-        qx.Bootstrap.debug(
-          qx.bom.request.Jsonp,
-          "Expecting JavaScript response to call: " + callbackName
-        );
+        qx.Bootstrap.debug(qx.bom.request.Jsonp, "Expecting JavaScript response to call: " + callbackName);
       }
 
       query[callbackParam] = callbackName;
@@ -313,8 +306,7 @@ qx.Bootstrap.define("qx.bom.request.Jsonp", {
     __generateId() {
       // Add random digits to date to allow immediately following requests
       // that may be send at the same time
-      this.__id =
-        "qx" + new Date().valueOf() + ("" + Math.random()).substring(2, 5);
+      this.__id = "qx" + new Date().valueOf() + ("" + Math.random()).substring(2, 5);
     }
   }
 });