Sign upLogin

huridocs/uwazi

View on GitHub
Star

Showing 3,439 of 3,635 total issues

Definition for rule 'node/no-restricted-import' was not found.
Open

import { actions as formActions } from 'react-redux-form';
Severity: Minor
Found in app/react/Templates/actions/templateActions.js by eslint

For more information visit Source: http://eslint.org/docs/rules/

'thesauris' PropType is defined but prop is never used
Open

  thesauris: PropTypes.object,
Severity: Minor
Found in app/react/Templates/components/FormConfigNested.js by eslint

For more information visit Source: http://eslint.org/docs/rules/

Prefer named exports.
Open

export default connect(mapStateToProps, mapDispatchToProps)(RemovePropertyConfirm);
Severity: Minor
Found in app/react/Templates/components/RemovePropertyConfirm.js by eslint

For more information visit Source: http://eslint.org/docs/rules/

Prop spreading is forbidden
Open

    component = shallow(<FormConfigCommon {...props} />);
Severity: Minor
Found in app/react/Templates/components/specs/FormConfigCommon.spec.js by eslint

For more information visit Source: http://eslint.org/docs/rules/

Arrow function should not return assignment.
Open

          <ComponentToRender ref={ref => (result = ref)} {...props} index={0} />
Severity: Minor
Found in app/react/Templates/components/specs/MetadataProperty.spec.js by eslint

title: no-return-assign

rule_type: suggestion

One of the interesting, and sometimes confusing, aspects of JavaScript is that assignment can happen at almost any point. Because of this, an errant equals sign can end up causing assignment when the true intent was to do a comparison. This is especially true when using a return statement. For example:

function doSomething() {
    return foo = bar + 2;
}

It is difficult to tell the intent of the return statement here. It's possible that the function is meant to return the result of bar + 2, but then why is it assigning to foo? It's also possible that the intent was to use a comparison operator such as == and that this code is an error.

Because of this ambiguity, it's considered a best practice to not use assignment in return statements.

Rule Details

This rule aims to eliminate assignments from return statements. As such, it will warn whenever an assignment is found as part of return.

Options

The rule takes one option, a string, which must contain one of the following values:

  • except-parens (default): Disallow assignments unless they are enclosed in parentheses.
  • always: Disallow all assignments.

except-parens

This is the default option. It disallows assignments unless they are enclosed in parentheses.

Examples of incorrect code for the default "except-parens" option:

::: incorrect

/*eslint no-return-assign: "error"*/

function doSomething() {
    return foo = bar + 2;
}

function doSomething() {
    return foo += 2;
}

const foo = (a, b) => a = b

const bar = (a, b, c) => (a = b, c == b)

function doSomething() {
    return foo = bar && foo > 0;
}

:::

Examples of correct code for the default "except-parens" option:

::: correct

/*eslint no-return-assign: "error"*/

function doSomething() {
    return foo == bar + 2;
}

function doSomething() {
    return foo === bar + 2;
}

function doSomething() {
    return (foo = bar + 2);
}

const foo = (a, b) => (a = b)

const bar = (a, b, c) => ((a = b), c == b)

function doSomething() {
    return (foo = bar) && foo > 0;
}

:::

always

This option disallows all assignments in return statements. All assignments are treated as problems.

Examples of incorrect code for the "always" option:

::: incorrect

/*eslint no-return-assign: ["error", "always"]*/

function doSomething() {
    return foo = bar + 2;
}

function doSomething() {
    return foo += 2;
}

function doSomething() {
    return (foo = bar + 2);
}

:::

Examples of correct code for the "always" option:

::: correct

/*eslint no-return-assign: ["error", "always"]*/

function doSomething() {
    return foo == bar + 2;
}

function doSomething() {
    return foo === bar + 2;
}

:::

When Not To Use It

If you want to allow the use of assignment operators in a return statement, then you can safely disable this rule. Source: http://eslint.org/docs/rules/

Prop spreading is forbidden
Open

      const component = shallow(<MetadataTemplate {...props} />);
Severity: Minor
Found in app/react/Templates/components/specs/MetadataTemplate.spec.js by eslint

For more information visit Source: http://eslint.org/docs/rules/

Definition for rule 'node/no-restricted-import' was not found.
Open

const properties = {
Severity: Minor
Found in database/elastic_mapping/toc_properties.js by eslint

For more information visit Source: http://eslint.org/docs/rules/

Definition for rule 'node/no-restricted-import' was not found.
Open

/* eslint-disable max-lines */
Severity: Minor
Found in .eslintrc.js by eslint

For more information visit Source: http://eslint.org/docs/rules/

Prop spreading is forbidden
Open

    component = shallow(<TemplateLabel store={store} {...props} />);
Severity: Minor
Found in app/react/Layout/specs/TemplateLabel.spec.js by eslint

For more information visit Source: http://eslint.org/docs/rules/

Definition for rule 'node/no-restricted-import' was not found.
Open

import React from 'react';
Severity: Minor
Found in app/react/Layout/specs/Tip.spec.js by eslint

For more information visit Source: http://eslint.org/docs/rules/

Prefer named exports.
Open

export default SearchInput;
Severity: Minor
Found in app/react/Layout/SearchInput.js by eslint

For more information visit Source: http://eslint.org/docs/rules/

Export statements should appear at the end of the file
Open

export default class Modal extends Component {
Severity: Minor
Found in app/react/Layout/Modal.js by eslint

For more information visit Source: http://eslint.org/docs/rules/

propType "children" is not required, but has no corresponding defaultProps declaration.
Open

  children: childrenType,
Severity: Minor
Found in app/react/Layout/Modal.js by eslint

For more information visit Source: http://eslint.org/docs/rules/

Definition for rule 'node/no-restricted-import' was not found.
Open

import PropTypes from 'prop-types';
Severity: Minor
Found in app/react/Layout/BackButton.js by eslint

For more information visit Source: http://eslint.org/docs/rules/

Export statements should appear at the end of the file
Open

export class TemplateLabel extends Component {
Severity: Minor
Found in app/react/Layout/TemplateLabel.js by eslint

For more information visit Source: http://eslint.org/docs/rules/

Definition for rule 'node/no-restricted-import' was not found.
Open

import PropTypes from 'prop-types';
Severity: Minor
Found in app/react/Layout/Tip.js by eslint

For more information visit Source: http://eslint.org/docs/rules/

propType "templates" is not required, but has no corresponding defaultProps declaration.
Open

  templates: PropTypes.object,
Severity: Minor
Found in app/react/Layout/Item.js by eslint

For more information visit Source: http://eslint.org/docs/rules/

propType "onSnippetClick" is not required, but has no corresponding defaultProps declaration.
Open

  onSnippetClick: PropTypes.func,
Severity: Minor
Found in app/react/Layout/Item.js by eslint

For more information visit Source: http://eslint.org/docs/rules/

propType "additionalIcon" is not required, but has no corresponding defaultProps declaration.
Open

  additionalIcon: PropTypes.object,
Severity: Minor
Found in app/react/Layout/Item.js by eslint

For more information visit Source: http://eslint.org/docs/rules/

Prop type "object" is forbidden
Open

  buttons: PropTypes.object,
Severity: Minor
Found in app/react/Layout/Item.js by eslint

For more information visit Source: http://eslint.org/docs/rules/

Severity
Category
Status
Source
Language