Sonar Plugins

Replace Java CPD with phpcpd

Details

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

Description

For coherence and performance reasons, this solution is fairly better than the actual one.
After last modifications (see SONARPLUGINS-218) CPD is very ressource consuming and pratically must be desactivated for large projects.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Akram Ben Aissi added a comment -

Due to perfomance issue with our current implementation, we are setting this to high priority and planning to use phpcpd on version 0.3

Show
Akram Ben Aissi added a comment - Due to perfomance issue with our current implementation, we are setting this to high priority and planning to use phpcpd on version 0.3
Hide
Permalink
Juergen Kellerer added a comment -

The current 0.3-SNAPSHOT version (revision 2237 of PhpCpdResultParser.java) causes a NullPointerException for the case that phpcpd did not find any duplicates.

The attached patch fixes this issue.

Show
Juergen Kellerer added a comment - The current 0.3-SNAPSHOT version (revision 2237 of PhpCpdResultParser.java) causes a NullPointerException for the case that phpcpd did not find any duplicates . The attached patch fixes this issue .
Hide
Permalink
Akram Ben Aissi added a comment -

Sonar PHP Plugin 0.3 rely on phpcpd !

Show
Akram Ben Aissi added a comment - Sonar PHP Plugin 0.3 rely on phpcpd !
Hide
Permalink
Akram Ben Aissi added a comment - - edited

fix was implemeted before patch was submitted, anyway, thank you for the patch !

Show
Akram Ben Aissi added a comment - - edited fix was implemeted before patch was submitted, anyway, thank you for the patch !

People

  • Assignee:
    Akram Ben Aissi
    Reporter:
    Gabriele Santini
Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: