Details

    • Type: Improvement Improvement
    • Status: Closed Closed
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: 0.2
    • Fix Version/s: 0.3
    • Component/s: None
    • 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

        Hide
        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
        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
        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
        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
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: