Sonar Plugins

Rule: Avoid empty block

Details

  • Type: New Feature New Feature
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: JAVASCRIPT-1.2
  • Component/s: JavaScript
  • Labels:
    None
  • Number of attachments :
    0

Description

  • Title
    Avoid empty block
  • Description
    In most cases, en empty block represents a missing implementation.

examples :

if(something){   /*Non-Compliant*/
   //What ?
}

for (var i=0;i<cars.length;i++) { /*Non-Compliant*/
   //Empty on purpose or missing piece of code ?
}
  • Violation message
    Provide the missing piece of code
  • Default severity
    Major

Part of Sonar way quality profile

Issue Links

is related to

Task - A task that needs to be done. SONARPLUGINS-2515 Update the SQALE mapping for new rules

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Fabrice Bellingard added a comment -

Empty function blocks should be considered as well.

Show
Fabrice Bellingard added a comment - Empty function blocks should be considered as well.
Hide
Permalink
Evgeny Mandrikov added a comment -

Fixed.

Show
Evgeny Mandrikov added a comment - Fixed.
Hide
Permalink
Fabrice Bellingard added a comment -

Manually tested.

Show
Fabrice Bellingard added a comment - Manually tested.
Hide
Permalink
Evgeny Mandrikov added a comment -

Integration tests were updated.

Show
Evgeny Mandrikov added a comment - Integration tests were updated.
Hide
Permalink
Evgeny Mandrikov added a comment -

In fact we decided to revert change on functions as it generates false-positives.

Show
Evgeny Mandrikov added a comment - In fact we decided to revert change on functions as it generates false-positives.

People

  • Assignee:
    Evgeny Mandrikov
    Reporter:
    Freddy Mallet
Vote (0)
Watch (3)

Dates

  • Created:
    Updated:
    Resolved: