SonarQube PHP
  1. SonarQube PHP
  2. SONARPHP-80

Multiple exclude directories for phpcpd do not work.

    Details

    • Type: Bug Bug
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.4
    • Fix Version/s: 0.5
    • Component/s: None
    • Labels:
      None
    • Number of attachments :
      0

      Description

      If you specify multiple directories to exclude via the sonar.phpcpd.excludes property, only the first directory is excluded. This is because phpcpd requires a separate --exclude parameter for each directory to skip, but the Sonar plug-in is generating a space-separated list. I believe that this can be fixed fairly easily through changes to the PhpCpdConfiguration::getExcludePackages() method.

        Activity

        Hide
        Fabrice Bellingard added a comment -

        Gennadiy, would you mind taking care of this issue? This should be pretty straightforward.

        Show
        Fabrice Bellingard added a comment - Gennadiy, would you mind taking care of this issue? This should be pretty straightforward.
        Hide
        Gennadiy Litvinyuk added a comment -

        Fixed in trunk.

        Show
        Gennadiy Litvinyuk added a comment - Fixed in trunk.
        Hide
        Fabrice Bellingard added a comment -

        Tested: fix OK but functionality was not tested in real life, see SONARPLUGINS-1416.

        Show
        Fabrice Bellingard added a comment - Tested: fix OK but functionality was not tested in real life, see SONARPLUGINS-1416 .

          People

          • Assignee:
            Gennadiy Litvinyuk
            Reporter:
            Anonymous
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: