Uploaded image for project: 'SonarJava'
  1. SonarJava
  2. SONARJAVA-2874

FP on S107 (too many parameter in method declaration) with spring annotations

    Details

    • Type: False-Positive
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 5.8
    • Component/s: Rules
    • Labels:
      None

      Description

      Implementation of RSPEC-107 already covers `RequestMapping` annotation from Spring framework and allow to any number of parameters (these parameters are injected when mapped on requests).

      The implementation should be updated to also cover the case of other spring annotations relying on the same principle:

          "org.springframework.web.bind.annotation.GetMapping",
          "org.springframework.web.bind.annotation.PostMapping",
          "org.springframework.web.bind.annotation.PutMapping",
          "org.springframework.web.bind.annotation.DeleteMapping",
          "org.springframework.web.bind.annotation.PatchMapping"
      

      Note that RSPEC should be udpated as well in order to mention this cases in exclusions.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                michael.gumowski Michael Gumowski
                Reporter:
                michael.gumowski Michael Gumowski
              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Due:
                  Created:
                  Updated:
                  Resolved: