eslint-plugin-fp/rules/no-get-set.js

ESLint rules for functional programming

'use strict';

const mutatingMethods = [
  '__defineGetter__',
  '__defineSetter__'
];

function getNameIfPropertyIsIdentifier(property) {
  return property.type === 'Identifier' &&
    mutatingMethods.indexOf(property.name) !== -1 &&
    property.name;
}

function getNameIfPropertyIsLiteral(property) {
  return property.type === 'Literal' &&
    mutatingMethods.indexOf(property.value) !== -1 &&
    property.value;
}

const create = function (context) {
  return {
    Property(node) {
      if (node.kind === 'get' || node.kind === 'set') {
        context.report({
          node,
          message: `Unallowed use of \`${node.kind}\``
        });
      }
    },
    CallExpression(node) {
      if (node.callee.type !== 'MemberExpression' || node.callee.object.type !== 'Identifier') {
        return;
      }

      const name = getNameIfPropertyIsIdentifier(node.callee.property) || getNameIfPropertyIsLiteral(node.callee.property);
      if (name) {
        context.report({
          node,
          message: name === '__defineGetter__' ?
            'Unallowed use of a getter using `__defineGetter__`' :
            'Unallowed use of a setter using `__defineSetter__`'
        });
      }
    }
  };
};

module.exports = {
  create,
  meta: {
    docs: {
      description: 'Forbid the use of getters and setters.',
      recommended: 'error'
    }
  }
};