kumabook/stickynotes

File background.js has 580 lines of code (exceeds 250 allowed). Consider refactoring.

Open

/* global setTimeout: false, clearTimeout: false */
/* eslint no-use-before-define: ["error", { "functions": false }] */
import browser from 'webextension-polyfill';
import idb     from './utils/indexedDB';
import logger  from './utils/logger';

Function handleContentScriptMessage has 88 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function handleContentScriptMessage(msg) {
  logger.info(`handleContentScriptMessage ${JSON.stringify(msg)}`);
  const port = getPort(msg.portName);
  switch (msg.type) {
    case 'load-options': {

Function handlePopupMessage has 84 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function handlePopupMessage(msg) {
  logger.info(`handlePopupMessage ${msg.type} from ${msg.portName}`);
  const port = getPort(msg.portName);
  switch (msg.type) {
    case 'list-menu': {

Function importStickies has 53 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function importStickies(stickies, db) {
  const createdStickies = [];
  const updatedStickies = [];
  return stickies.reduce((p, s) => {
    s = Sticky.normalize(s);

Function setupContextMenus has 42 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function setupContextMenus() {
  browser.contextMenus.create({
    id:       'create-sticky',
    title:    'create sticky',
    contexts: ['all'],

Function handleSidebarMessage has 42 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function handleSidebarMessage(msg) {
  const port = getPort(msg.portName);
  switch (msg.type) {
    case 'fetch-stickies': {
      idb.open(dbName).then(db => Promise.all([

Function handleOptionsUIMessage has 37 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function handleOptionsUIMessage(msg) {
  const port = getPort(msg.portName);
  switch (msg.type) {
    case 'import': {
      const stickies = msg.payload;

Avoid too many return statements within this function.

Open

    return optionsUIPorts[name];

Avoid too many return statements within this function.

Open

  return null;

Similar blocks of code found in 2 locations. Consider refactoring.

Open

function createStickyOnActiveTab() {
  browser.tabs.query({ currentWindow: true, active: true }).then((tabs) => {
    if (tabs.length > 0) {
      getContentScriptPorts().forEach(p => p.postMessage({
        type:      'create-sticky',

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

function toggleStickies() {
  browser.tabs.query({ currentWindow: true, active: true }).then((tabs) => {
    if (tabs.length > 0) {
      getContentScriptPorts().forEach(p => p.postMessage({
        type:      'toggle-visibility',

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

  } else if (name.startsWith('sidebar')) {
    sidebarPorts[port.name] = port;
    port.onDisconnect.addListener(() => {
      delete sidebarPorts[port.name];
      port.onMessage.removeListener(handleSidebarMessage);

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

  } else if (name.startsWith('options-ui')) {
    optionsUIPorts[port.name] = port;
    port.onDisconnect.addListener(() => {
      delete optionsUIPorts[port.name];
      port.onMessage.removeListener(handleOptionsUIMessage);

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

  if (name.startsWith('content-script')) {
    contentScriptPorts[port.name] = port;
    port.onDisconnect.addListener(() => {
      delete contentScriptPorts[port.name];
      port.onMessage.removeListener(handleContentScriptMessage);

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

          return Sticky.update(item, db)
            .then((v) => {
              logger.trace(`Updated ${v.id}`);
              updatedStickies.push(v);
            })

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

        return Sticky.new(s, db).then((st) => {
          logger.trace(`Created ${s.id}`);
          createdStickies.push(st);
        }).catch((e) => {
          logger.fatal(`Failed to created ${s.id}`);

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

      idb.open(dbName)
        .then(db => Sticky.update(sticky, db))
        .then(() => {
          getSidebarPorts().concat(port).forEach(p => p.postMessage({
            type:    'saved-sticky',

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

      api.isLoggedIn().then((isLoggedIn) => {
        if (isLoggedIn) {
          sticky.state = Sticky.State.Deleted;
          sticky.updated_at = new Date();
          return idb.open(dbName).then(db => Sticky.update(sticky, db));

• Read up
  Read up