Details

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

      Description

      phpcpd tool always running no matter settings in sonar configuration or in the pom.

        Activity

        Hide
        Sebastian Marek added a comment -

        There is a typo in PhpCpdConfiguration, the option should be sonar.phpcpd.analyzeOnly not sonar.phpcpd.analyseOnly. Also the logic in PhpCpdSensor.analyse() needs changing to take into account this option and an option needs to be added to Sonar configuration panel to allow to configure it from there.

        Show
        Sebastian Marek added a comment - There is a typo in PhpCpdConfiguration, the option should be sonar.phpcpd.analyzeOnly not sonar.phpcpd.analyseOnly . Also the logic in PhpCpdSensor.analyse() needs changing to take into account this option and an option needs to be added to Sonar configuration panel to allow to configure it from there.
        Hide
        Sebastian Marek added a comment -

        There was also a bug in the logic detecting whether the tool should be executed or not. Fixed. phpcpd execution can be controlled with the following 3 options:

        • sonar.cpd.skip (true - don't run, false run)
        • sonar.php.cpd.skip (true - don't run, false run)
        • sonar.phpcpd.shouldRun (true - run, false don't run)

        Sonar will check whether one of them has been set in configuration in the order above. If none of them is set the tool will be executed by default.

        Show
        Sebastian Marek added a comment - There was also a bug in the logic detecting whether the tool should be executed or not. Fixed. phpcpd execution can be controlled with the following 3 options: sonar.cpd.skip (true - don't run, false run) sonar.php.cpd.skip (true - don't run, false run) sonar.phpcpd.shouldRun (true - run, false don't run) Sonar will check whether one of them has been set in configuration in the order above. If none of them is set the tool will be executed by default.

          People

          • Assignee:
            Sebastian Marek
            Reporter:
            Anonymous
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: