Fix, hopefully for the last time, quote escaping

CHANGELOG.md

@@ -1,3 +1,7 @@
+JSONView 1.2.4
+---
+* Fixed a case where JSON would fail to parse if a string containing a number was preceded by a quote that was preceded by a lot of escaped slashes, or occurred near another number.
+
 JSONView 1.2.3
 ---
 * Fixed a case where JSON would fail to parse if a string containing a number was preceded by a quote that was preceded by an escaped slash.

lib/jsonview.js

@@ -105,9 +105,14 @@ var JSONView = Class({
  var inQuotes = false;
  for (var i = 0; i < str.length; i++) {
  if (str[i] === '"') {
- var escaped =
- (i > 0 && str[i - 1] === '\\') &&
- (i > 1 && str[i - 2] !== '\\');
+ var escaped = false;
+ for (var lookback = i - 1; lookback >= 0; lookback--) {
+ if (str[lookback] === '\\') {
+ escaped = !escaped;
+ } else {
+ break;
+ }
+ }
  if (!escaped) {
  inQuotes = !inQuotes;
  }

package.json

@@ -1,6 +1,6 @@
 {
  "id": "[email protected]",
- "version": "1.2.3",
+ "version": "1.2.4",
  "name": "jsonview",
  "title": "JSONView",
  "description": "View JSON documents in the browser.",

tests/issue141c.json

@@ -0,0 +1,3 @@
+{
+"dots":"\"v4.0\" \"v4.0\""
+}

tests/issue141d.json

@@ -0,0 +1,3 @@
+{
+"dots":"\\\"v4.0\" \\\\\\\\\\\"v4.0\""
+}

tests/issue141e.json

@@ -0,0 +1,6 @@
+{
+ "abc": {
+ "value": "{\"user\":\"NN12345\"}"
+ },
+ "MM98765": {}
+}