fixed a bug with wrapped expressions in ?:

sokra · sokra · commit cd5252b96245 · 2015-03-06T17:02:24.000+01:00
diff --git a/lib/BasicEvaluatedExpression.js b/lib/BasicEvaluatedExpression.js
@@ -45,7 +45,7 @@ BasicEvaluatedExpression.prototype.asBool = function() {
 	else if(this.isRegExp()) return true;
 	else if(this.isArray()) return true;
 	else if(this.isConstArray()) return true;
-	else if(this.isWrapped()) return this.prefix || this.postfix ? true : undefined;
+	else if(this.isWrapped()) return this.prefix && this.prefix.asBool() || this.postfix && this.postfix.asBool() ? true : undefined;
 	return undefined;
 };
 BasicEvaluatedExpression.prototype.set = function(value) {
diff --git a/test/cases/parsing/evaluate/index.js b/test/cases/parsing/evaluate/index.js
@@ -11,6 +11,12 @@ it("should evaluate null", function() {
 		require("fail");
 });
 
+if("shouldn't evaluate expression", function() {
+	var value = "";
+	var x = (value + "") ? "fail" : "ok";
+	x.should.be.eql("ok");
+});
+
 it("should short-circut evaluating", function() {
 	var expr;
 	var a = DEBUG && expr ? require("fail") : require("./a");
