Merge branch 'master' of https://github.com/less/less.js

Author: matthew-dean

packages/less/src/less-node/url-file-manager.js

@@ -27,8 +27,8 @@ UrlFileManager.prototype = Object.assign(new AbstractFileManager(), {
             const hackUrlStr = urlStr.indexOf('?') === -1 ? urlStr + '?' : urlStr
 
             request.get(hackUrlStr, { follow_max: 5 }, (err, resp, body) => {
-                if (err || resp.statusCode >= 400) {
-                    const message = resp.statusCode === 404
+                if (err || resp && resp.statusCode >= 400) {
+                    const message = resp && resp.statusCode === 404
                         ? `resource '${urlStr}' was not found\n`
                         : `resource '${urlStr}' gave this Error:\n  ${err || resp.statusMessage || resp.statusCode}\n`;
                     reject({ type: 'File', message });

packages/less/src/less/tree/dimension.js

@@ -66,7 +66,8 @@ Dimension.prototype = Object.assign(new Node(), {
     // so `1px + 2` will yield `3px`.
     operate(context, op, other) {
         /* jshint noempty:false */
-        let value = this._operate(context, op, this.value, other.value), unit = this.unit.clone();
+        let value = this._operate(context, op, this.value, other.value);
+        let unit = this.unit.clone();
 
         if (op === '+' || op === '-') {
             if (unit.numerator.length === 0 && unit.denominator.length === 0) {

packages/less/src/less/tree/operation.js

@@ -29,8 +29,11 @@ Operation.prototype = Object.assign(new Node(), {
             if (b instanceof Dimension && a instanceof Color) {
                 b = b.toColor();
             }
-            if (!a.operate) {
-                if (a instanceof Operation && a.op === '/' && context.math === MATH.PARENS_DIVISION) {
+            if (!a.operate || !b.operate) {
+                if (
+                    (a instanceof Operation || b instanceof Operation)
+                    && a.op === '/' && context.math === MATH.PARENS_DIVISION
+                ) {
                     return new Operation(this.op, [a, b], this.isSpaced);
                 }
                 throw { type: 'Operation',

packages/test-data/less/math/parens-division/media-math.less

@@ -1,4 +1,4 @@
-@var: 16;
+@var: 10 + 6;
 
 @media (min-width: @var + 1) {
   .foo { bar: 1; }