Sonar Plugins

Add a rule repository containing all PHP rules (phpcodesniffer rules and pmd rules)

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Critical Critical
  • Resolution: Fixed
  • Affects Version/s: PHP-0.3
  • Fix Version/s: PHP-0.3
  • Component/s: PHP
  • Labels:
    None
  • Number of attachments :
    1

Description

PMD is not working.
I know that it's temporary (because the code is commented with a //FIXME) but I prefer to create this bug to keep a trace and don't forget to activate PMD in the future.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Mathias kluba added a comment -

This patch includes the bug fixes of SONARPLUGINS-520
I created a "PhpRuleRepository" which includes both rules of "PmdRulesRepository" and "PhpCodesnifferRulesRepository".
But I didn't implement the "exportConfiguration" and "importConfiguration".

Are they used anyway? See SONARPLUGINS-806

Show
Mathias kluba added a comment - This patch includes the bug fixes of SONARPLUGINS-520 I created a "PhpRuleRepository" which includes both rules of "PmdRulesRepository" and "PhpCodesnifferRulesRepository". But I didn't implement the "exportConfiguration" and "importConfiguration". Are they used anyway? See SONARPLUGINS-806
Hide
Permalink
Akram Ben Aissi added a comment -

Good initiative, I will include this patch, but maybe not in next release because of planning.
Yes import and exportconfiguration are used in 0.3.
To create a profile containing violations from the 2 rules repositories, user can make a copy of the bigger and then add missing rules from the smaller.

Show
Akram Ben Aissi added a comment - Good initiative, I will include this patch, but maybe not in next release because of planning. Yes import and exportconfiguration are used in 0.3. To create a profile containing violations from the 2 rules repositories, user can make a copy of the bigger and then add missing rules from the smaller.
Hide
Permalink
Akram Ben Aissi added a comment -

fixed
the two profiles exists and are used

Show
Akram Ben Aissi added a comment - fixed the two profiles exists and are used

People

  • Assignee:
    Akram Ben Aissi
    Reporter:
    Mathias kluba
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: