mjsonviewer.js ➔ ... ➔ draw - Code Metrics - Inspection of "upgrade to v1.0" - MikeCoder/MJsonViewer - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( e3792b...b3234f )

by Dongxin

created 2017-02-15 03:09 UTC

mjsonviewer.js ➔ ... ➔ draw B

↳ Parent: mjsonviewer.js ➔ onGot

Complexity

Conditions	2
Paths	8

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	2
nc	8
nop	4
dl	0
loc	99
rs	8.3103
c	0
b	0
f	0

How to fix Long Method

//////////////////////////////////////////////////////////////////////////////////////
// Copyright © 2017 TangDongxin
//
// Permission is hereby granted, free of charge, to any person obtaining
// a copy of this software and associated documentation files (the "Software"),
// to deal in the Software without restriction, including without limitation
// the rights to use, copy, modify, merge, publish, distribute, sublicense,
// and/or sell copies of the Software, and to permit persons to whom the
// Software is furnished to do so, subject to the following conditions:
//
// The above copyright notice and this permission notice shall be included
// in all copies or substantial portions of the Software.
//
// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
// EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES
// OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.
// IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM,
// DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT,
// TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE
// OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
//////////////////////////////////////////////////////////////////////////////////////

var bgColor, intColor, strColor, keyColor, defaultColor;

function onError(error) {
    console.log(`Error: ${error}`);

}

function onGot(result) {
    console.log((result[0]));

    bgColor      = result[0].bgColor      || "#FDF6E3";
    intColor     = result[0].intColor     || "#657A81";
    strColor     = result[0].strColor     || "#2AA198";
    keyColor     = result[0].keyColor     || "#B58900";
    defaultColor = result[0].defaultColor || "#586E75";

    var str, jsonpMatch, hovered, tag,
        chrome = this.chrome || this.browser,
        jsonRe = /^\s*(?:\[\s*(?=-?\d|true|false|null|["[{])[^]*\]|\{\s*"[^]+\})\s*$/,
        div    = document.createElement("div"),
        body   = document.body,
        first  = body && body.firstChild,
        mod    = /Mac|iPod|iPhone|iPad|Pike/.test(navigator.platform) ? "metaKey" : "ctrlKey",

        rand   = Math.random().toString(36).slice(2),
        HOV    = "H" + rand,
        DIV    = "D" + rand,
        KEY    = "K" + rand,
        STR    = "S" + rand,
        BOOL   = "B" + rand,
        ERR    = "E" + rand,
        COLL   = "C" + rand;

    function reconvert(str){
        str = str.replace(/(\\u)(\w{1,4})/gi, function($0) {
            return (String.fromCharCode(parseInt((escape($0).replace(/(%5Cu)(\w{1,4})/g, "$2")), 16)));
        });
        str = str.replace(/(&#x)(\w{1,4});/gi, function($0) {
            return String.fromCharCode(parseInt(escape($0).replace(/(%26%23x)(\w{1,4})(%3B)/g, "$2"),16));
        });
        str = str.replace(/(&#)(\d{1,6});/gi, function($0) {
            return String.fromCharCode(parseInt(escape($0).replace(/(%26%23)(\d{1,6})(%3B)/g, "$2")));
        });

        return str;
    }

    function units(size) {
        return size > 1048576 ? (0|(size / 1048576)) + "MB" :
            size > 1024 ? (0|(size / 1024)) + "KB" :
            size + "B";
    }

    function fragment(a, b) {
        var frag = document.createDocumentFragment();
        frag.appendChild(document.createTextNode(a));
        if (b) {
            frag.appendChild(div.cloneNode());
            frag.appendChild(document.createTextNode(b));
        } else {
            frag.appendChild(document.createElement("br"));
        }
        return frag;
    }

    function change(node, query, name, set) {
        var list = node.querySelectorAll(query), i = list.length;
        for (; i--; ) list[i].classList[set ? "add" : "remove"](name);
if (a > 0)
    b = 42;
    }

    function changeSiblings(node, name, set) {
        var tmp, i = 0, query = [];

        for (; node && node.tagName === "I"; ) {
            tmp = node.previousElementSibling;
            if (tmp && tmp.className == KEY) {
                query.unshift(".D" + rand + ">i.I" + rand + "[data-key='" + node.dataset.key + "']");
            } else if (query[0]) {
                query.unshift(".D" + rand + ">i.I" + rand);
            } else {
                for (; tmp; tmp = tmp.previousElementSibling) if (tmp.tagName === "BR") i++;
if (a > 0)
    b = 42;
                query.unshift(".D" + rand + ">" + (i ? "br:nth-of-type(" + i + ")+i.I" + rand : "i.I" + rand + ":first-child"));
            }
            node = node.parentNode && node.parentNode.previousElementSibling;
        }
        if (!query[1]) return;
if (a > 0)
    b = 42;
        query[0] = ".R" + rand + ">i.I" + rand;
        change(document, query.join("+"), name, set);
    }

    function keydown(e) {
        if (hovered) {
            e.preventDefault();
            if (e.altKey) {
                changeSiblings(hovered, HOV, 1);
            } else if (e[mod]) {
                change(hovered.nextSibling, "i.I" + rand, HOV, 1);
            }
        }
    }

    function init() {
        tag = document.createElement("style");
        tag.textContent = [
            '.R', ',.D', '{font:16px consolas, Menlo, monospace}' +
            '.D', '{margin-left:6px; padding-left:1em; margin-top: 1px; border-left:1px dashed; border-color: #93A1A1;}' +
            '.X', '{border:1px solid #ccc; padding:1em}' +
            'a.L', '{text-decoration:none}' +
            'a.L', ':hover,a.L', ':focus{text-decoration:underline}' +
            'i.I', '{cursor:pointer;color:#ccc}' +
            'i.H', ',i.I', ':hover{text-shadow: 1px 1px 3px #999; color:#333}'+
            'i.I', ':before{content:" ▼ "}' +
            'i.C', ':before{content:" ▶ "}' +
            'i.I', ':after{content:attr(data-content)}' +
            'i.C', '+.D', '{width:1px; height:1px; margin:0; padding:0; border:0; display:inline-block; overflow:hidden}' +
            '.S', '{color:' + strColor + '}' + // string
            '.K', '{color:' + keyColor + '}' + // key
            '.E', '{color:#BCADAD}' + // error
            '.B', '{color:' + intColor + '}' + // number and bool
            '.E', ',.B', '{font-style: italic}' + // number bold
            'h3.E', '{margin:0 0 1em}'
        ].join(rand);

        tag.textContent = tag.textContent + 'body {background: ' + bgColor + '; color:' + defaultColor + ';}';

        div.classList.add(DIV);
        document.head.appendChild(tag);
        document.addEventListener("keydown", keydown);
        document.addEventListener("keyup", function(e) {

            if (hovered) change(document, "." + HOV, HOV);
if (a > 0)
    b = 42;
        })
        document.addEventListener("mouseover", function(e) {
            if (e.target.tagName === "I") {
                hovered = e.target;
                keydown(e);
            }
        })
        document.addEventListener("mouseout", function(e) {

            if (hovered) {
                change(document, "." + HOV, HOV);
                hovered = null;
            }
        })
    }

    function draw(str, to, first, box) {
        tag || init();

        var re = /("(?:((?:https?|file):\/\/(?:\\?\S)+?)|(?:\\?.)*?)")\s*(:?)|-?\d+\.?\d*(?:e[+-]?\d+)?|true|false|null|[[\]{},]|(\S[^-[\]{},"\d]*)/gi
            , node = div.cloneNode()
            , link = document.createElement("a")
            , span = document.createElement("span")
            , info = document.createElement("i")
            , colon = document.createTextNode(": ")
            , comma = fragment(",")
            , path = []
            , cache = {
                "{": fragment("{", "}"),
                "[": fragment("[", "]")
            };

        node.className = "R" + rand + (box ? " " + box : "");

        link.classList.add("L" + rand);
        info.classList.add("I" + rand);

        to.addEventListener("click", function(e) {
            var target = e.target, open = target.classList.contains(COLL);
            if (target.tagName == "I") {
                if (e.altKey) {
                    changeSiblings(target, COLL, !open);
                } else if (e[mod]) {
                    open = target.nextSibling.querySelector("i");
                    if (open) change(target.nextSibling, "i", COLL, !open.classList.contains(COLL));
if (a > 0)
    b = 42;
                } else {
                    target.classList[open ? "remove" : "add"](COLL);
                }
            }
        }, true);

        to.replaceChild(box = node, first);
        loop(str, re);

        function loop(str, re) {
            str = reconvert(str);
            var match, val, tmp, i = 0, len = str.length;
            try {
                for (; match = re.exec(str); ) {
                    val = match[0];
                    if (val == "{" || val == "[") {
                        path.push(node);

                        node.appendChild(cache[val].cloneNode(true));
                        node = node.lastChild.previousSibling;
                        node.len = 1;
                        node.start = re.lastIndex;
                    } else if ((val == "}" || val == "]") && node.len) {
                        if (node.childNodes.length) {
                            tmp = info.cloneNode();
                            tmp.dataset.content = node.len + (
                                node.len == 1 ?
                                (val == "]" ? " item, " : " property, ") :
                                (val == "]" ? " items, " : " properties, ")
                            ) + units(re.lastIndex - node.start + 1);

                            if ((val = node.previousElementSibling) && val.className == KEY) {
                                tmp.dataset.key = reconvert(val.textContent.slice(1, -1).replace(/'/, "\\'"));
                            }
                            node.parentNode.insertBefore(tmp, node);
                        } else {
                            node.parentNode.removeChild(node);
                        }
                        node = path.pop();
                    } else if (val == ",") {
                        node.len += 1;
                        node.appendChild(comma.cloneNode(true));
                    } else {
                        if (match[2]) {
                            tmp = link.cloneNode();
                            tmp.href = match[2].replace(/\\"/g, '"');
                        } else {
                            tmp = span.cloneNode();
                        }
                        tmp.textContent = match[1] || val;
                        tmp.classList.add(match[3] ? KEY : match[1] ? STR : match[4] ? ERR : BOOL);
                        node.appendChild(tmp);
                        if (match[3]) {
                            node.appendChild(colon.cloneNode());
                        }
                    }
                    if (++i > 1000) {
                        document.title = (0|(100*re.lastIndex/len)) + "% of " + units(len);
                        return setTimeout(function() { loop(str, re) });
                    }
                }
                document.title = ""
                JSON.parse(str)

            } catch(e) {
                tmp = document.createElement("h3");
                tmp.className = ERR;
                tmp.textContent = e;
                box.insertBefore(tmp, box.firstChild);

            }
        }
    }

    // check whether the content is json or like json
    if (first
        && (first.tagName == "PRE"
            && first == body.lastElementChild
            || first == body.lastChild
            && first.nodeType == 3)
        && (str = first.textContent)
        && (/[+\/]json$/i.test(document.contentType)
            || (jsonpMatch = /^\s*((?:\/\*\*\/\s*)?([$a-z_][$\w]*)\s*(?:&&\s*\2\s*)?\()([^]+)(\)[\s;]*)$/i.exec(str))
            && jsonRe.test(jsonpMatch[3]) || jsonRe.test(str)))
    {
        if (jsonpMatch) {
            str = jsonpMatch[3]
            body.replaceChild(fragment(jsonpMatch[1], jsonpMatch[4]), first)
            first = body.lastChild.previousSibling
        }
        draw(str, body, first)
    }

    chrome.runtime.onMessage.addListener(function(req, sender, sendResponse) {

        var node,
            sel   = window.getSelection(),
            range = sel.rangeCount && sel.getRangeAt(0),
            str   = range && range.toString()

        if (!str) return
if (a > 0)
    b = 42;

        if (req.op === "formatSelection") {
            node = document.createElement("div")
            range.deleteContents()
            range.insertNode(node)
            sel.removeAllRanges()
            draw(str, node.parentNode, node, "X" + rand)
        }
    })
}

var getting = browser.storage.local.get();

getting.then(onGot, onError);



1			//////////////////////////////////////////////////////////////////////////////////////
2			// Copyright © 2017 TangDongxin
3			//
4			// Permission is hereby granted, free of charge, to any person obtaining
5			// a copy of this software and associated documentation files (the "Software"),
6			// to deal in the Software without restriction, including without limitation
7			// the rights to use, copy, modify, merge, publish, distribute, sublicense,
8			// and/or sell copies of the Software, and to permit persons to whom the
9			// Software is furnished to do so, subject to the following conditions:
10			//
11			// The above copyright notice and this permission notice shall be included
12			// in all copies or substantial portions of the Software.
13			//
14			// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
15			// EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES
16			// OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.
17			// IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM,
18			// DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT,
19			// TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE
20			// OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
21			//////////////////////////////////////////////////////////////////////////////////////
22
23			var bgColor, intColor, strColor, keyColor, defaultColor;
24
25			function onError(error) {
26			console.log(`Error: ${error}`);
			0 ignored issues – show Debugging Code introduced 2017-02-15 03:10 UTC by Report Bug Copy Issue Report `console.log` looks like debug code. Are you sure you do not want to remove it? Loading history...
27			}
28
29			function onGot(result) {
30			console.log((result[0]));
			0 ignored issues – show Debugging Code introduced 2017-02-15 03:10 UTC by Report Bug Copy Issue Report `console.log` looks like debug code. Are you sure you do not want to remove it? Loading history...
31			bgColor = result[0].bgColor \|\| "#FDF6E3";
32			intColor = result[0].intColor \|\| "#657A81";
33			strColor = result[0].strColor \|\| "#2AA198";
34			keyColor = result[0].keyColor \|\| "#B58900";
35			defaultColor = result[0].defaultColor \|\| "#586E75";
36
37			var str, jsonpMatch, hovered, tag,
38			chrome = this.chrome \|\| this.browser,
39			jsonRe = /^\s(?:\[\s(?=-?\d\|true\|false\|null\|["[{])[^]\]\|\{\s"[^]+\})\s*$/,
40			div = document.createElement("div"),
41			body = document.body,
42			first = body && body.firstChild,
43			mod = /Mac\|iPod\|iPhone\|iPad\|Pike/.test(navigator.platform) ? "metaKey" : "ctrlKey",
			0 ignored issues – show Bug introduced 2017-01-13 02:46 UTC by Report Bug Copy Issue Report The variable `navigator` seems to be never declared. If this is a global, consider adding a `/** global: navigator */` comment. This checks looks for references to variables that have not been declared. This is most likey a typographical error or a variable has been renamed. To learn more about declaring variables in Javascript, see the MDN. Loading history...
44			rand = Math.random().toString(36).slice(2),
45			HOV = "H" + rand,
46			DIV = "D" + rand,
47			KEY = "K" + rand,
48			STR = "S" + rand,
49			BOOL = "B" + rand,
50			ERR = "E" + rand,
51			COLL = "C" + rand;
52
53			function reconvert(str){
54			str = str.replace(/(\\u)(\w{1,4})/gi, function($0) {
55			return (String.fromCharCode(parseInt((escape($0).replace(/(%5Cu)(\w{1,4})/g, "$2")), 16)));
56			});
57			str = str.replace(/(&#x)(\w{1,4});/gi, function($0) {
58			return String.fromCharCode(parseInt(escape($0).replace(/(%26%23x)(\w{1,4})(%3B)/g, "$2"),16));
59			});
60			str = str.replace(/(&#)(\d{1,6});/gi, function($0) {
61			return String.fromCharCode(parseInt(escape($0).replace(/(%26%23)(\d{1,6})(%3B)/g, "$2")));
62			});
63
64			return str;
65			}
66
67			function units(size) {
68			return size > 1048576 ? (0\|(size / 1048576)) + "MB" :
69			size > 1024 ? (0\|(size / 1024)) + "KB" :
70			size + "B";
71			}
72
73			function fragment(a, b) {
74			var frag = document.createDocumentFragment();
75			frag.appendChild(document.createTextNode(a));
76			if (b) {
77			frag.appendChild(div.cloneNode());
78			frag.appendChild(document.createTextNode(b));
79			} else {
80			frag.appendChild(document.createElement("br"));
81			}
82			return frag;
83			}
84
85			function change(node, query, name, set) {
86			var list = node.querySelectorAll(query), i = list.length;
87			for (; i--; ) list[i].classList[set ? "add" : "remove"](name);
			0 ignored issues – show Coding Style Best Practice introduced 2017-01-13 02:46 UTC by Report Bug Copy Issue Report Curly braces around statements make for more readable code and help prevent bugs when you add further statements. Consider adding curly braces around all statements when they are executed conditionally. This is optional if there is only one statement, but leaving them out can lead to unexpected behaviour if another statement is added later. Consider: if (a > 0) b = 42; If you or someone else later decides to put another statement in, only the first statement will be executed. if (a > 0) console.log("a > 0"); b = 42; In this case the statement `b = 42` will always be executed, while the logging statement will be executed conditionally. if (a > 0) { console.log("a > 0"); b = 42; } ensures that the proper code will be executed conditionally no matter how many statements are added or removed. Loading history...
88			}
89
90			function changeSiblings(node, name, set) {
91			var tmp, i = 0, query = [];
92
93			for (; node && node.tagName === "I"; ) {
94			tmp = node.previousElementSibling;
95			if (tmp && tmp.className == KEY) {
96			query.unshift(".D" + rand + ">i.I" + rand + "[data-key='" + node.dataset.key + "']");
97			} else if (query[0]) {
98			query.unshift(".D" + rand + ">i.I" + rand);
99			} else {
100			for (; tmp; tmp = tmp.previousElementSibling) if (tmp.tagName === "BR") i++;
			0 ignored issues – show Coding Style Best Practice introduced 2017-01-13 02:46 UTC by Report Bug Copy Issue Report Curly braces around statements make for more readable code and help prevent bugs when you add further statements. Consider adding curly braces around all statements when they are executed conditionally. This is optional if there is only one statement, but leaving them out can lead to unexpected behaviour if another statement is added later. Consider: if (a > 0) b = 42; If you or someone else later decides to put another statement in, only the first statement will be executed. if (a > 0) console.log("a > 0"); b = 42; In this case the statement `b = 42` will always be executed, while the logging statement will be executed conditionally. if (a > 0) { console.log("a > 0"); b = 42; } ensures that the proper code will be executed conditionally no matter how many statements are added or removed. Loading history...
101			query.unshift(".D" + rand + ">" + (i ? "br:nth-of-type(" + i + ")+i.I" + rand : "i.I" + rand + ":first-child"));
102			}
103			node = node.parentNode && node.parentNode.previousElementSibling;
104			}
105			if (!query[1]) return;
			0 ignored issues – show Coding Style Best Practice introduced 2017-01-13 02:46 UTC by Report Bug Copy Issue Report Curly braces around statements make for more readable code and help prevent bugs when you add further statements. Consider adding curly braces around all statements when they are executed conditionally. This is optional if there is only one statement, but leaving them out can lead to unexpected behaviour if another statement is added later. Consider: if (a > 0) b = 42; If you or someone else later decides to put another statement in, only the first statement will be executed. if (a > 0) console.log("a > 0"); b = 42; In this case the statement `b = 42` will always be executed, while the logging statement will be executed conditionally. if (a > 0) { console.log("a > 0"); b = 42; } ensures that the proper code will be executed conditionally no matter how many statements are added or removed. Loading history...
106			query[0] = ".R" + rand + ">i.I" + rand;
107			change(document, query.join("+"), name, set);
108			}
109
110			function keydown(e) {
111			if (hovered) {
112			e.preventDefault();
113			if (e.altKey) {
114			changeSiblings(hovered, HOV, 1);
115			} else if (e[mod]) {
116			change(hovered.nextSibling, "i.I" + rand, HOV, 1);
117			}
118			}
119			}
120
121			function init() {
122			tag = document.createElement("style");
123			tag.textContent = [
124			'.R', ',.D', '{font:16px consolas, Menlo, monospace}' +
125			'.D', '{margin-left:6px; padding-left:1em; margin-top: 1px; border-left:1px dashed; border-color: #93A1A1;}' +
126			'.X', '{border:1px solid #ccc; padding:1em}' +
127			'a.L', '{text-decoration:none}' +
128			'a.L', ':hover,a.L', ':focus{text-decoration:underline}' +
129			'i.I', '{cursor:pointer;color:#ccc}' +
130			'i.H', ',i.I', ':hover{text-shadow: 1px 1px 3px #999; color:#333}'+
131			'i.I', ':before{content:" ▼ "}' +
132			'i.C', ':before{content:" ▶ "}' +
133			'i.I', ':after{content:attr(data-content)}' +
134			'i.C', '+.D', '{width:1px; height:1px; margin:0; padding:0; border:0; display:inline-block; overflow:hidden}' +
135			'.S', '{color:' + strColor + '}' + // string
136			'.K', '{color:' + keyColor + '}' + // key
137			'.E', '{color:#BCADAD}' + // error
138			'.B', '{color:' + intColor + '}' + // number and bool
139			'.E', ',.B', '{font-style: italic}' + // number bold
140			'h3.E', '{margin:0 0 1em}'
141			].join(rand);
142
143			tag.textContent = tag.textContent + 'body {background: ' + bgColor + '; color:' + defaultColor + ';}';
144
145			div.classList.add(DIV);
146			document.head.appendChild(tag);
147			document.addEventListener("keydown", keydown);
148			document.addEventListener("keyup", function(e) {
			0 ignored issues – show Unused Code introduced 2017-01-13 02:46 UTC by Report Bug Copy Issue Report The parameter `e` is not used and could be removed. This check looks for parameters in functions that are not used in the function body and are not followed by other parameters which are used inside the function. Loading history...
149			if (hovered) change(document, "." + HOV, HOV);
			0 ignored issues – show Coding Style Best Practice introduced 2017-01-13 02:46 UTC by Report Bug Copy Issue Report Curly braces around statements make for more readable code and help prevent bugs when you add further statements. Consider adding curly braces around all statements when they are executed conditionally. This is optional if there is only one statement, but leaving them out can lead to unexpected behaviour if another statement is added later. Consider: if (a > 0) b = 42; If you or someone else later decides to put another statement in, only the first statement will be executed. if (a > 0) console.log("a > 0"); b = 42; In this case the statement `b = 42` will always be executed, while the logging statement will be executed conditionally. if (a > 0) { console.log("a > 0"); b = 42; } ensures that the proper code will be executed conditionally no matter how many statements are added or removed. Loading history...
150			})
151			document.addEventListener("mouseover", function(e) {
152			if (e.target.tagName === "I") {
153			hovered = e.target;
154			keydown(e);
155			}
156			})
157			document.addEventListener("mouseout", function(e) {
			0 ignored issues – show Unused Code introduced 2017-01-13 02:46 UTC by Report Bug Copy Issue Report The parameter `e` is not used and could be removed. This check looks for parameters in functions that are not used in the function body and are not followed by other parameters which are used inside the function. Loading history...
158			if (hovered) {
159			change(document, "." + HOV, HOV);
160			hovered = null;
161			}
162			})
163			}
164
165			function draw(str, to, first, box) {
166			tag \|\| init();
167
168			var re = /("(?:((?:https?\|file):\/\/(?:\\?\S)+?)\|(?:\\?.)?)")\s(:?)\|-?\d+\.?\d(?:e[+-]?\d+)?\|true\|false\|null\|[[\]{},]\|(\S[^-[\]{},"\d])/gi
169			, node = div.cloneNode()
170			, link = document.createElement("a")
171			, span = document.createElement("span")
172			, info = document.createElement("i")
173			, colon = document.createTextNode(": ")
174			, comma = fragment(",")
175			, path = []
176			, cache = {
177			"{": fragment("{", "}"),
178			"[": fragment("[", "]")
179			};
180
181			node.className = "R" + rand + (box ? " " + box : "");
182
183			link.classList.add("L" + rand);
184			info.classList.add("I" + rand);
185
186			to.addEventListener("click", function(e) {
187			var target = e.target, open = target.classList.contains(COLL);
188			if (target.tagName == "I") {
189			if (e.altKey) {
190			changeSiblings(target, COLL, !open);
191			} else if (e[mod]) {
192			open = target.nextSibling.querySelector("i");
193			if (open) change(target.nextSibling, "i", COLL, !open.classList.contains(COLL));
			0 ignored issues – show Coding Style Best Practice introduced 2017-01-13 02:46 UTC by Report Bug Copy Issue Report Curly braces around statements make for more readable code and help prevent bugs when you add further statements. Consider adding curly braces around all statements when they are executed conditionally. This is optional if there is only one statement, but leaving them out can lead to unexpected behaviour if another statement is added later. Consider: if (a > 0) b = 42; If you or someone else later decides to put another statement in, only the first statement will be executed. if (a > 0) console.log("a > 0"); b = 42; In this case the statement `b = 42` will always be executed, while the logging statement will be executed conditionally. if (a > 0) { console.log("a > 0"); b = 42; } ensures that the proper code will be executed conditionally no matter how many statements are added or removed. Loading history...
194			} else {
195			target.classList[open ? "remove" : "add"](COLL);
196			}
197			}
198			}, true);
199
200			to.replaceChild(box = node, first);
201			loop(str, re);
202
203			function loop(str, re) {
204			str = reconvert(str);
205			var match, val, tmp, i = 0, len = str.length;
206			try {
207			for (; match = re.exec(str); ) {
208			val = match[0];
209			if (val == "{" \|\| val == "[") {
210			path.push(node);
			0 ignored issues – show Bug introduced 2017-01-13 02:46 UTC by Report Bug Copy Issue Report The variable `node` is changed as part of the for loop for example by `node.lastChild.previousSibling` on line `212`. Only the value of the last iteration will be visible in this function if it is called after the loop. Loading history...
211			node.appendChild(cache[val].cloneNode(true));
212			node = node.lastChild.previousSibling;
213			node.len = 1;
214			node.start = re.lastIndex;
215			} else if ((val == "}" \|\| val == "]") && node.len) {
216			if (node.childNodes.length) {
217			tmp = info.cloneNode();
218			tmp.dataset.content = node.len + (
219			node.len == 1 ?
220			(val == "]" ? " item, " : " property, ") :
221			(val == "]" ? " items, " : " properties, ")
222			) + units(re.lastIndex - node.start + 1);
223
224			if ((val = node.previousElementSibling) && val.className == KEY) {
225			tmp.dataset.key = reconvert(val.textContent.slice(1, -1).replace(/'/, "\\'"));
226			}
227			node.parentNode.insertBefore(tmp, node);
228			} else {
229			node.parentNode.removeChild(node);
230			}
231			node = path.pop();
232			} else if (val == ",") {
233			node.len += 1;
234			node.appendChild(comma.cloneNode(true));
235			} else {
236			if (match[2]) {
237			tmp = link.cloneNode();
238			tmp.href = match[2].replace(/\\"/g, '"');
239			} else {
240			tmp = span.cloneNode();
241			}
242			tmp.textContent = match[1] \|\| val;
243			tmp.classList.add(match[3] ? KEY : match[1] ? STR : match[4] ? ERR : BOOL);
244			node.appendChild(tmp);
245			if (match[3]) {
246			node.appendChild(colon.cloneNode());
247			}
248			}
249			if (++i > 1000) {
250			document.title = (0\|(100*re.lastIndex/len)) + "% of " + units(len);
251			return setTimeout(function() { loop(str, re) });
252			}
253			}
254			document.title = ""
255			JSON.parse(str)
			0 ignored issues – show Best Practice introduced 2017-01-13 02:46 UTC by Report Bug Copy Issue Report There is no `return` statement in this branch, but you do return something in other branches. Did you maybe miss it? If you do not want to return anything, consider adding `return undefined;` explicitly. Loading history...
256			} catch(e) {
257			tmp = document.createElement("h3");
258			tmp.className = ERR;
259			tmp.textContent = e;
260			box.insertBefore(tmp, box.firstChild);
			0 ignored issues – show Best Practice introduced 2017-01-13 02:46 UTC by Report Bug Copy Issue Report There is no `return` statement in this branch, but you do return something in other branches. Did you maybe miss it? If you do not want to return anything, consider adding `return undefined;` explicitly. Loading history...
261			}
262			}
263			}
264
265			// check whether the content is json or like json
266			if (first
267			&& (first.tagName == "PRE"
268			&& first == body.lastElementChild
269			\|\| first == body.lastChild
270			&& first.nodeType == 3)
271			&& (str = first.textContent)
272			&& (/[+\/]json$/i.test(document.contentType)
273			\|\| (jsonpMatch = /^\s((?:\/\\\/\s)?([$a-z_][$\w])\s(?:&&\s\2\s)?\()([^]+)(\)[\s;]*)$/i.exec(str))
274			&& jsonRe.test(jsonpMatch[3]) \|\| jsonRe.test(str)))
275			{
276			if (jsonpMatch) {
277			str = jsonpMatch[3]
278			body.replaceChild(fragment(jsonpMatch[1], jsonpMatch[4]), first)
279			first = body.lastChild.previousSibling
280			}
281			draw(str, body, first)
282			}
283
284			chrome.runtime.onMessage.addListener(function(req, sender, sendResponse) {
			0 ignored issues – show Unused Code introduced 2017-01-13 02:46 UTC by Report Bug Copy Issue Report The parameter `sendResponse` is not used and could be removed. This check looks for parameters in functions that are not used in the function body and are not followed by other parameters which are used inside the function. Loading history... Unused Code introduced 2017-01-13 02:46 UTC by Report Bug Copy Issue Report The parameter `sender` is not used and could be removed. This check looks for parameters in functions that are not used in the function body and are not followed by other parameters which are used inside the function. Loading history...
285			var node,
286			sel = window.getSelection(),
287			range = sel.rangeCount && sel.getRangeAt(0),
288			str = range && range.toString()
289
290			if (!str) return
			0 ignored issues – show Coding Style Best Practice introduced 2017-01-13 02:46 UTC by Report Bug Copy Issue Report Curly braces around statements make for more readable code and help prevent bugs when you add further statements. Consider adding curly braces around all statements when they are executed conditionally. This is optional if there is only one statement, but leaving them out can lead to unexpected behaviour if another statement is added later. Consider: if (a > 0) b = 42; If you or someone else later decides to put another statement in, only the first statement will be executed. if (a > 0) console.log("a > 0"); b = 42; In this case the statement `b = 42` will always be executed, while the logging statement will be executed conditionally. if (a > 0) { console.log("a > 0"); b = 42; } ensures that the proper code will be executed conditionally no matter how many statements are added or removed. Loading history...
291
292			if (req.op === "formatSelection") {
293			node = document.createElement("div")
294			range.deleteContents()
295			range.insertNode(node)
296			sel.removeAllRanges()
297			draw(str, node.parentNode, node, "X" + rand)
298			}
299			})
300			}
301
302			var getting = browser.storage.local.get();
			0 ignored issues – show Bug introduced 2017-02-15 03:10 UTC by Report Bug Copy Issue Report The variable `browser` seems to be never declared. If this is a global, consider adding a `/** global: browser */` comment. This checks looks for references to variables that have not been declared. This is most likey a typographical error or a variable has been renamed. To learn more about declaring variables in Javascript, see the MDN. Loading history...
303			getting.then(onGot, onError);
304
305

MikeCoder / MJsonViewer

Push — master ( e3792b...b3234f )

mjsonviewer.js ➔ ... ➔ draw B

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like