index.js ➔ decryptAES - Code Metrics - Inspection of "v2.0.5 release" - MikeCoder/hexo-blog-encrypt - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 632daf...1bb9df )

by Dongxin

created 2018-04-10 12:06 UTC

index.js ➔ decryptAES B

↳ Parent: index.js

Complexity

Conditions	6
Paths	51

Size

Total Lines

Duplication

Lines	58
Ratio	100 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	6
nc	51
nop	0
dl	58
loc	58
rs	8.7274
c	1
b	0
f	0

How to fix Long Method

'use strict';


function decryptAES () {

  const pass = String(document.getElementById('pass').value);
  try {

    var decryptionError = String(document.getElementById('decryptionError').innerHTML);
    var noContentError = String(document.getElementById('noContentError').innerHTML);

  } catch (e) {

    decryptionError = 'Incorrect Password!';
    noContentError = 'No content to display!';

  }

  try {

    let content = CryptoJS.AES.decrypt(document.getElementById('encrypt-blog').innerHTML.trim(), pass);

    content = content.toString(CryptoJS.enc.Utf8);
    content = decodeBase64(content);
    content = unescape(content);
    if (content === '') {

      throw new Error(noContentError); // ???

    } else {

      document.getElementById('encrypt-blog').style.display = 'inline';
      document.getElementById('encrypt-blog').innerHTML = '';
      // Use jquery to load some js code
      $('#encrypt-blog').html(content);
      document.getElementById('security').style.display = 'none';
      if (document.getElementById('toc-div')) {

        document.getElementById('toc-div').style.display = 'inline';

      }

    }
    // Call MathJax to render
    if(typeof MathJax !== 'undefined') {


      MathJax.Hub.Queue(
        ['resetEquationNumbers', MathJax.InputJax.TeX, ],
        ['PreProcess', MathJax.Hub, ],
        ['Reprocess', MathJax.Hub, ]
      );

    }

  } catch (e) {

    alert(decryptionError);
    console.log(e);


  }

}

function htmlDecode (str) {

  let s = '';
  if (str.length == 0) {

    return '';

  }

  s = str.replace(/&gt;/g, '&');
  s = s.replace(/&lt;/g, '<');
  s = s.replace(/&gt;/g, '>');
  s = s.replace(/&nbsp;/g, '    '); // ??? why not ' '
  s = s.replace(/'/g, '\'');
  s = s.replace(/&quot;/g, '"');
  s = s.replace(/<br>/g, '\n');
  return s;

}

function decodeBase64 (content) {

  content = CryptoJS.enc.Base64.parse(content);

  content = CryptoJS.enc.Utf8.stringify(content);
  return content;

}

// Since you decided to use jQuery.
$(document).ready(
  function () {

    console.log('Registering Enter for decrypt.');

    document.getElementById('pass').onkeypress = function (keyPressEvent) {

      if (keyPressEvent.keyCode === 13) {

        decryptAES();

      }

    };

  }
);


1	View Code Duplication	'use strict';
		0 ignored issues – show Duplication introduced 2018-04-10 12:10 UTC by Report Bug Copy Issue Report This code seems to be duplicated in your project. Loading history...
2
3		function decryptAES () {
4
5		const pass = String(document.getElementById('pass').value);
6		try {
7
8		var decryptionError = String(document.getElementById('decryptionError').innerHTML);
9		var noContentError = String(document.getElementById('noContentError').innerHTML);
10
11		} catch (e) {
12
13		decryptionError = 'Incorrect Password!';
14		noContentError = 'No content to display!';
15
16		}
17
18		try {
19
20		let content = CryptoJS.AES.decrypt(document.getElementById('encrypt-blog').innerHTML.trim(), pass);
		0 ignored issues – show Bug introduced 2017-12-13 07:18 UTC by Report Bug Copy Issue Report The variable `CryptoJS` seems to be never declared. If this is a global, consider adding a `/** global: CryptoJS */` 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...
21		content = content.toString(CryptoJS.enc.Utf8);
22		content = decodeBase64(content);
23		content = unescape(content);
24		if (content === '') {
25
26		throw new Error(noContentError); // ???
27
28		} else {
29
30		document.getElementById('encrypt-blog').style.display = 'inline';
31		document.getElementById('encrypt-blog').innerHTML = '';
32		// Use jquery to load some js code
33		$('#encrypt-blog').html(content);
34		document.getElementById('security').style.display = 'none';
35		if (document.getElementById('toc-div')) {
36
37		document.getElementById('toc-div').style.display = 'inline';
38
39		}
40
41		}
42		// Call MathJax to render
43		if(typeof MathJax !== 'undefined') {
		0 ignored issues – show Bug introduced 2018-02-19 11:29 UTC by Report Bug Copy Issue Report The variable `MathJax` seems to be never declared. If this is a global, consider adding a `/** global: MathJax */` 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
45		MathJax.Hub.Queue(
46		['resetEquationNumbers', MathJax.InputJax.TeX, ],
47		['PreProcess', MathJax.Hub, ],
48		['Reprocess', MathJax.Hub, ]
49		);
50
51		}
52
53		} catch (e) {
54
55		alert(decryptionError);
56		console.log(e);
		0 ignored issues – show Debugging Code introduced 2017-12-13 07:18 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...
57
58		}
59
60		}
61
62		function htmlDecode (str) {
63
64		let s = '';
65		if (str.length == 0) {
66
67		return '';
68
69		}
70
71		s = str.replace(/>/g, '&');
72		s = s.replace(/</g, '<');
73		s = s.replace(/>/g, '>');
74		s = s.replace(/ /g, ' '); // ??? why not ' '
75		s = s.replace(/'/g, '\'');
76		s = s.replace(/"/g, '"');
77		s = s.replace(/<br>/g, '\n');
78		return s;
79
80		}
81
82		function decodeBase64 (content) {
83
84		content = CryptoJS.enc.Base64.parse(content);
		0 ignored issues – show Bug introduced 2017-12-13 07:18 UTC by Report Bug Copy Issue Report The variable `CryptoJS` seems to be never declared. If this is a global, consider adding a `/** global: CryptoJS */` 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...
85		content = CryptoJS.enc.Utf8.stringify(content);
86		return content;
87
88		}
89
90		// Since you decided to use jQuery.
91		$(document).ready(
92		function () {
93
94		console.log('Registering Enter for decrypt.');
		0 ignored issues – show Debugging Code introduced 2018-02-19 11:29 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...
95		document.getElementById('pass').onkeypress = function (keyPressEvent) {
96
97		if (keyPressEvent.keyCode === 13) {
98
99		decryptAES();
100
101		}
102
103		};
104
105		}
106		);
107

MikeCoder / hexo-blog-encrypt

Push — master ( 632daf...1bb9df )

index.js ➔ decryptAES B

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like