fix for inverting operator on negative numbers

andyjansson · ben-eb · commit 7908443dcf55 · 2017-05-08T20:33:05.000+01:00
diff --git a/src/lib/reducer.js b/src/lib/reducer.js
@@ -121,11 +121,11 @@ function reduceAddSubExpression(node, precision) {
       if (left.operator === '-') {
         node.right = reduce({
           type: 'MathExpression',
-          operator: op === left.operator ? '+' : '-',
+          operator: op === '-' ? '+' : '-',
           left: right,
           right: left.right
         }, precision)
-        node.operator = op === left.operator ? '-' : '+';
+        node.operator = op === '-' ? '-' : '+';
       }
       else {
         node.right = reduce({
@@ -137,7 +137,7 @@ function reduceAddSubExpression(node, precision) {
       }
       if (node.right.value < 0) {
         node.right.value *= -1;
-        node.operator = '-';
+        node.operator = node.operator === '-' ? '+' : '-';
       }
       return reduce(node, precision)
     }

Original file line number	Diff line number	Diff line change
`@@ -121,11 +121,11 @@ function reduceAddSubExpression(node, precision) {`
`121`	`121`	`if (left.operator === '-') {`
`122`	`122`	`node.right = reduce({`
`123`	`123`	`type: 'MathExpression',`
`124`		`- operator: op === left.operator ? '+' : '-',`
	`124`	`+ operator: op === '-' ? '+' : '-',`
`125`	`125`	`left: right,`
`126`	`126`	`right: left.right`
`127`	`127`	`}, precision)`
`128`		`- node.operator = op === left.operator ? '-' : '+';`
	`128`	`+ node.operator = op === '-' ? '-' : '+';`
`129`	`129`	`}`
`130`	`130`	`else {`
`131`	`131`	`node.right = reduce({`
`@@ -137,7 +137,7 @@ function reduceAddSubExpression(node, precision) {`
`137`	`137`	`}`
`138`	`138`	`if (node.right.value < 0) {`
`139`	`139`	`node.right.value *= -1;`
`140`		`- node.operator = '-';`
	`140`	`+ node.operator = node.operator === '-' ? '+' : '-';`
`141`	`141`	`}`
`142`	`142`	`return reduce(node, precision)`
`143`	`143`	`}`