lib/devmind-beautifier.js - Code Metrics - LukasMeine/devmind-beautifier - Measure and Improve Code Quality continuously with Scrutinizer

lib/devmind-beautifier.js A
last analyzed 2018-01-02 17:28 UTC

↳ Parent: Project

Complexity

Total Complexity	14
Complexity/F	1.4

Size

Lines of Code	93
Function Count	10

Duplication

Duplicated Lines	0
Ratio	0 %

Importance

Changes	5
Bugs	0	Features	0

Metric	Value
cc	0
c	5
b	0
f	0
nc	1
dl	0
loc	93
rs	10
wmc	14
mnd	1
bc	13
fnc	10
bpm	1.3
cpm	1.4
noi	14

8 Functions

Rating	Name	Size	Complexity
A	devmind-beautifier.js ➔ request	8	1
A	???.deactivate	5	1
A	???.activate	19	1
A	???.serialize	5	1
A	devmind-beautifier.js ➔ changeState	10	1
A	devmind-beautifier.js ➔ success	9	2
A	devmind-beautifier.js ➔ fail	6	1
A	???.toggle	12	1

"use babel";

import DevmindBeautifierView from "./devmind-beautifier-view";
import { CompositeDisposable } from "atom";
var qs = require("qs");

function request(ext, text, url) {
  data = { extension: ext, content: text };

  var http = new XMLHttpRequest();

  http.open("POST", url, true);
  http.setRequestHeader("Content-type", "application/x-www-form-urlencoded");
  http.send(qs.stringify(data));
  changeState(http);
}

function changeState(http) {
  http.onreadystatechange = function() {
    if (http.readyState == 4 && http.status == 200) {
      success(http);
    }
    if (http.status !== 200) {
      fail();
    }
  };
}

function success(http) {
  if (typeof http.responseText.status === "undefined") {
    editor.setText(http.responseText);

    atom.notifications.addSuccess("Now your code looks fancy");

    panel.modalPanel.hide();

  } else {
    atom.notifications.addInfo("File extension not supported. Yet.");
  }
}

function fail() {
  panel.modalPanel.hide();

  atom.notifications.addError(

    "Something went wrong, please check your internet connection"
  );
}

export default {
  devmindBeautifierView: null,
  modalPanel: null,
  subscriptions: null,

  activate(state) {
    this.devmindBeautifierView = new DevmindBeautifierView(
      state.devmindBeautifierViewState
    );
    this.modalPanel = atom.workspace.addModalPanel({

      item: this.devmindBeautifierView.getElement(),
      visible: false
    });

    // Events subscribed to in atom's system can be easily cleaned up with a CompositeDisposable
    this.subscriptions = new CompositeDisposable();

    // Register command that toggles this view
    this.subscriptions.add(
      atom.commands.add("atom-workspace", {
        "devmind-beautifier:toggle": () => this.toggle()
      })
    );
  },

  deactivate() {
    this.modalPanel.destroy();
    this.subscriptions.dispose();
    this.devmindBeautifierView.destroy();
  },

  serialize() {
    return {
      devmindBeautifierViewState: this.devmindBeautifierView.serialize()
    };
  },

  toggle() {
    panel = this;


    editor = atom.workspace.getActiveTextEditor();

    text = editor.getText().trim();

    fileName = editor.getTitle();

    ext = fileName

      .split(".")
      .pop()
      .trim();
    request(ext, text, "https://beautifier.devmind.io/format");
  }
};


1			"use babel";
2
3			import DevmindBeautifierView from "./devmind-beautifier-view";
4			import { CompositeDisposable } from "atom";
5			var qs = require("qs");
6
7			function request(ext, text, url) {
8			data = { extension: ext, content: text };
			0 ignored issues – show Bug introduced 2017-12-29 14:51 UTC by Report Bug Copy Issue Report The variable `data` seems to be never declared. Assigning variables without defining them first makes them global. If this was intended, consider making it explicit like using `window.data`. Loading history...
9			var http = new XMLHttpRequest();
			0 ignored issues – show Bug introduced 2017-12-29 14:48 UTC by Report Bug Copy Issue Report The variable `XMLHttpRequest` seems to be never declared. If this is a global, consider adding a `/** global: XMLHttpRequest */` 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...
10			http.open("POST", url, true);
11			http.setRequestHeader("Content-type", "application/x-www-form-urlencoded");
12			http.send(qs.stringify(data));
13			changeState(http);
14			}
15
16			function changeState(http) {
17			http.onreadystatechange = function() {
18			if (http.readyState == 4 && http.status == 200) {
19			success(http);
20			}
21			if (http.status !== 200) {
22			fail();
23			}
24			};
25			}
26
27			function success(http) {
28			if (typeof http.responseText.status === "undefined") {
29			editor.setText(http.responseText);
			0 ignored issues – show Bug introduced 2017-12-29 14:48 UTC by Report Bug Copy Issue Report The variable `editor` seems to be never declared. If this is a global, consider adding a `/** global: editor */` 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...
30			atom.notifications.addSuccess("Now your code looks fancy");
			0 ignored issues – show Bug introduced 2016-10-11 19:47 UTC by Report Bug Copy Issue Report The variable `atom` seems to be never declared. If this is a global, consider adding a `/** global: atom */` 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...
31			panel.modalPanel.hide();
			0 ignored issues – show Bug introduced 2017-12-29 14:48 UTC by Report Bug Copy Issue Report The variable `panel` seems to be never declared. If this is a global, consider adding a `/** global: panel */` 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...
32			} else {
33			atom.notifications.addInfo("File extension not supported. Yet.");
34			}
35			}
36
37			function fail() {
38			panel.modalPanel.hide();
			0 ignored issues – show Bug introduced 2017-12-29 15:05 UTC by Report Bug Copy Issue Report The variable `panel` seems to be never declared. If this is a global, consider adding a `/** global: panel */` 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...
39			atom.notifications.addError(
			0 ignored issues – show Bug introduced 2017-12-29 15:05 UTC by Report Bug Copy Issue Report The variable `atom` seems to be never declared. If this is a global, consider adding a `/** global: atom */` 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...
40			"Something went wrong, please check your internet connection"
41			);
42			}
43
44			export default {
45			devmindBeautifierView: null,
46			modalPanel: null,
47			subscriptions: null,
48
49			activate(state) {
50			this.devmindBeautifierView = new DevmindBeautifierView(
51			state.devmindBeautifierViewState
52			);
53			this.modalPanel = atom.workspace.addModalPanel({
			0 ignored issues – show Bug introduced 2016-10-11 19:47 UTC by Report Bug Copy Issue Report The variable `atom` seems to be never declared. If this is a global, consider adding a `/** global: atom */` 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...
54			item: this.devmindBeautifierView.getElement(),
55			visible: false
56			});
57
58			// Events subscribed to in atom's system can be easily cleaned up with a CompositeDisposable
59			this.subscriptions = new CompositeDisposable();
60
61			// Register command that toggles this view
62			this.subscriptions.add(
63			atom.commands.add("atom-workspace", {
64			"devmind-beautifier:toggle": () => this.toggle()
65			})
66			);
67			},
68
69			deactivate() {
70			this.modalPanel.destroy();
71			this.subscriptions.dispose();
72			this.devmindBeautifierView.destroy();
73			},
74
75			serialize() {
76			return {
77			devmindBeautifierViewState: this.devmindBeautifierView.serialize()
78			};
79			},
80
81			toggle() {
82			panel = this;
			0 ignored issues – show Bug introduced 2016-10-11 19:47 UTC by Report Bug Copy Issue Report The variable `panel` seems to be never declared. Assigning variables without defining them first makes them global. If this was intended, consider making it explicit like using `window.panel`. Loading history...
83
84			editor = atom.workspace.getActiveTextEditor();
			0 ignored issues – show Bug introduced 2016-10-11 19:47 UTC by Report Bug Copy Issue Report The variable `editor` seems to be never declared. Assigning variables without defining them first makes them global. If this was intended, consider making it explicit like using `window.editor`. Loading history... Bug introduced 2016-10-11 19:47 UTC by Report Bug Copy Issue Report The variable `atom` seems to be never declared. If this is a global, consider adding a `/** global: atom */` 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			text = editor.getText().trim();
			0 ignored issues – show Bug introduced 2016-10-11 19:47 UTC by Report Bug Copy Issue Report The variable `text` seems to be never declared. Assigning variables without defining them first makes them global. If this was intended, consider making it explicit like using `window.text`. Loading history...
86			fileName = editor.getTitle();
			0 ignored issues – show Bug introduced 2017-12-29 14:48 UTC by Report Bug Copy Issue Report The variable `fileName` seems to be never declared. Assigning variables without defining them first makes them global. If this was intended, consider making it explicit like using `window.fileName`. Loading history...
87			ext = fileName
			0 ignored issues – show Bug introduced 2017-12-29 14:48 UTC by Report Bug Copy Issue Report The variable `ext` seems to be never declared. Assigning variables without defining them first makes them global. If this was intended, consider making it explicit like using `window.ext`. Loading history...
88			.split(".")
89			.pop()
90			.trim();
91			request(ext, text, "https://beautifier.devmind.io/format");
92			}
93			};
94

LukasMeine / devmind-beautifier

lib/devmind-beautifier.js A last analyzed 2018-01-02 17:28 UTC

Complexity

Size

Duplication

Importance

8 Functions

Duplication Side-by-Side

Filter issues like

lib/devmind-beautifier.js A
last analyzed 2018-01-02 17:28 UTC