CodeQL documentation
CodeQL resources

Useless return in setter

ID: js/setter-return Kind: problem Security severity: Severity: warning Precision: very-high Tags: - quality - maintainability - useless-code - language-features Query suites: - javascript-code-quality.qls - javascript-security-and-quality.qls

Click to see the query in the CodeQL repository

Returning a value from a property setter function is useless, since it will always be ignored.

Recommendation

Remove the return statement altogether, or replace it with a simple return; statement that does not return a value.

Example

The following example shows a constructor function Point that uses property accessors on x and y to ensure that they are only set to integer values. It returns the new values for x and y from their setters, but these return values will simply be ignored.

function Point(x, y) { return { get x() { return x; }, set x(_x) { x = _x|0; return x; }, get y() { return y; }, set y(_y) { y = _y|0; return y; } }; }

It would be clearer to omit the return statements:

function Point(x, y) { return { get x() { return x; }, set x(_x) { x = _x|0; }, get y() { return y; }, set y(_y) { y = _y|0; } }; }

References