[go_router_builder] Include required and positional query parameters in the location (#4163)

Fixes flutter/flutter#128483

Author: ValentinVignal

packages/go_router_builder/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 2.1.1
+
+* Fixes a bug that the required/positional parameters are not added to query parameters correctly.
+
 ## 2.1.0
 
 * Supports required/positional parameters that are not in the path.

packages/go_router_builder/lib/src/route_config.dart

@@ -441,15 +441,15 @@ GoRouteData.\$route(
 
   late final List<ParameterElement> _ctorParams =
       _ctor.parameters.where((ParameterElement element) {
-    if (element.isRequired && !element.isExtraField) {
+    if (_pathParams.contains(element.name)) {
       return true;
     }
     return false;
   }).toList();
 
   late final List<ParameterElement> _ctorQueryParams = _ctor.parameters
       .where((ParameterElement element) =>
-          element.isOptional && !element.isExtraField)
+          !_pathParams.contains(element.name) && !element.isExtraField)
       .toList();
 
   ConstructorElement get _ctor {

packages/go_router_builder/pubspec.yaml

@@ -2,7 +2,7 @@ name: go_router_builder
 description: >-
   A builder that supports generated strongly-typed route helpers for
   package:go_router
-version: 2.1.0
+version: 2.1.1
 repository: https://github.com/flutter/packages/tree/main/packages/go_router_builder
 issue_tracker: https://github.com/flutter/flutter/issues?q=is%3Aissue+is%3Aopen+label%3A%22p%3A+go_router_builder%22
 

packages/go_router_builder/test/test_inputs/_go_router_builder_test_input.dart

@@ -68,6 +68,9 @@ extension $NullableRequiredParamNotInPathExtension
 
   String get location => GoRouteData.$location(
         'bob',
+        queryParams: {
+          if (id != null) 'id': id!.toString(),
+        },
       );
 
   void go(BuildContext context) => context.go(location);
@@ -108,6 +111,9 @@ extension $NonNullableRequiredParamNotInPathExtension
 
   String get location => GoRouteData.$location(
         'bob',
+        queryParams: {
+          'id': id.toString(),
+        },
       );
 
   void go(BuildContext context) => context.go(location);