events.js ➔ ??? - Code Metrics - Inspection of "Updated Linter & Rules" - prometheas/jira-2-omnifocus - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#17)

unknown

created 2017-08-22 23:51 UTC

events.js ➔ ??? A

↳ Parent: source/js/events.js

Complexity

Conditions	2
Paths	2

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	2
nc	2
nop	3
dl	0
loc	9
rs	9.6666
c	0
b	0
f	0

/* eslint-env webextensions */

function encodeTaskProperties(omnifocusTask) {
  return {
    name: encodeURIComponent(omnifocusTask.name),
    note: encodeURIComponent(omnifocusTask.note),
  };
}

const senders = {
  app(config, encodedTask) {
    return `omnifocus:///add?name=${encodedTask.name}&note=${encodedTask.note}`;
  },
  maildrop(config, encodedTask) {
    return `mailto:${config.address}@sync.omnigroup.com?subject=${encodedTask.name}&body=${encodedTask.note}`;
  },
};

const actions = {
  createTask(taskInfo) {
    const sender = localStorage.sender || 'app';

    const encodedTask = encodeTaskProperties(taskInfo);

    return {
      url: senders[sender](localStorage, encodedTask),
    };
  },
};

chrome.runtime.onMessage.addListener((request, sender, sendResponse) => {

  let responseData = false;

  if (actions[request.method]) {
    responseData = actions[request.method](request.params);
  }

  sendResponse(responseData);
});


1			/* eslint-env webextensions */
2
3			function encodeTaskProperties(omnifocusTask) {
4			return {
5			name: encodeURIComponent(omnifocusTask.name),
6			note: encodeURIComponent(omnifocusTask.note),
7			};
8			}
9
10			const senders = {
11			app(config, encodedTask) {
12			return `omnifocus:///add?name=${encodedTask.name}&note=${encodedTask.note}`;
13			},
14			maildrop(config, encodedTask) {
15			return `mailto:${config.address}@sync.omnigroup.com?subject=${encodedTask.name}&body=${encodedTask.note}`;
16			},
17			};
18
19			const actions = {
20			createTask(taskInfo) {
21			const sender = localStorage.sender \|\| 'app';
			0 ignored issues – show Bug introduced 2017-08-22 22:58 UTC by Report Bug Copy Issue Report The variable `localStorage` seems to be never declared. If this is a global, consider adding a `/** global: localStorage */` 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...
22			const encodedTask = encodeTaskProperties(taskInfo);
23
24			return {
25			url: senders[sender](localStorage, encodedTask),
26			};
27			},
28			};
29
30			chrome.runtime.onMessage.addListener((request, sender, sendResponse) => {
			0 ignored issues – show Bug introduced 2017-04-27 18:36 UTC by Report Bug Copy Issue Report The variable `chrome` seems to be never declared. If this is a global, consider adding a `/** global: chrome */` 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			let responseData = false;
32
33			if (actions[request.method]) {
34			responseData = actions[request.method](request.params);
35			}
36
37			sendResponse(responseData);
38			});
39

prometheas / jira-2-omnifocus

Pull Request — master (#17)

events.js ➔ ??? A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like