From: José Bollo Date: Wed, 10 Feb 2016 08:28:00 +0000 (+0100) Subject: app.js: avoid ambiguity X-Git-Tag: blowfish_2.0.1~23 X-Git-Url: https://gerrit.automotivelinux.org/gerrit/gitweb?p=src%2Fapp-framework-demo.git;a=commitdiff_plain;h=d663533aad6585b33f4fe3138ae6ed9e0b520e78 app.js: avoid ambiguity Writing: "conf ? expr : var = item;" is wrong because = has lower priority than ?:. It is in fact evaluated as: "(conf ? expr : var) = item;" Change-Id: I876b67b80c9285356d82bcc1c87795a97a501c86 Signed-off-by: José Bollo --- diff --git a/afm-client/app/Frontend/app.js b/afm-client/app/Frontend/app.js index 24d5539..d332894 100644 --- a/afm-client/app/Frontend/app.js +++ b/afm-client/app/Frontend/app.js @@ -6,7 +6,7 @@ location.search.substr(1).split("&").forEach(function(item) { var k = item.split("=")[0]; var v = decodeURIComponent(item.split("=")[1]); - (k in qd) ? qd[k].push(v) : qd[k] = [v]; + if (k in qd) qd[k].push(v); else qd[k] = [v]; }); return qd; }