SONARPY-3772 Implement S8439: Django view functions should include all URL parameters (#887)

joke1196 · claude · sonartech · commit 68297ba75dc3 · 2026-03-06T07:42:46.000Z
Co-authored-by: Claude Opus 4.5 &lt;noreply@anthropic.com&gt;
GitOrigin-RevId: 11f315cd45432aad488a71d85ecb98401f661a2a
diff --git a/python-checks/src/main/java/org/sonar/python/checks/utils/FunctionParameterUtils.java b/python-checks/src/main/java/org/sonar/python/checks/utils/FunctionParameterUtils.java
@@ -30,7 +30,6 @@
 public final class FunctionParameterUtils {
 
   private FunctionParameterUtils() {
-    // Utility class
   }
 
   /**
@@ -60,7 +59,7 @@ public static Optional<FunctionType> getFunctionType(FunctionDef functionDef) {
     return Optional.empty();
   }
 
-  private static FunctionParameterInfo buildParameterInfo(FunctionType functionType) {
+  public static FunctionParameterInfo buildParameterInfo(FunctionType functionType) {
     Set<String> allParams = new HashSet<>();
     Set<String> positionalOnlyParams = new HashSet<>();
     boolean hasVariadicKeyword = functionType.parameters().stream()

Original file line number	Diff line number	Diff line change
`@@ -30,7 +30,6 @@`
`30`	`30`	`public final class FunctionParameterUtils {`
`31`	`31`
`32`	`32`	`private FunctionParameterUtils() {`
`33`		`- // Utility class`
`34`	`33`	`}`
`35`	`34`
`36`	`35`	`/**`
`@@ -60,7 +59,7 @@ public static Optional<FunctionType> getFunctionType(FunctionDef functionDef) {`
`60`	`59`	`return Optional.empty();`
`61`	`60`	`}`
`62`	`61`
`63`		`- private static FunctionParameterInfo buildParameterInfo(FunctionType functionType) {`
	`62`	`+ public static FunctionParameterInfo buildParameterInfo(FunctionType functionType) {`
`64`	`63`	`Set<String> allParams = new HashSet<>();`
`65`	`64`	`Set<String> positionalOnlyParams = new HashSet<>();`
`66`	`65`	`boolean hasVariadicKeyword = functionType.parameters().stream()`