chore: rename _groupId() -> encodeGroupId()

bevzzz · bevzzz · commit 9f44e94d53e2 · 2025-09-10T18:38:00.000+02:00
diff --git a/src/main/java/io/weaviate/client/v1/async/groups/api/oidc/AssignedRolesGetter.java b/src/main/java/io/weaviate/client/v1/async/groups/api/oidc/AssignedRolesGetter.java
@@ -33,7 +33,7 @@ public AssignedRolesGetter includePermissions(boolean include) {
     return this;
   }
 
-  private String _groupId() {
+  private String encodeGroupId() {
     return UrlEncoder.encode(this.groupId);
   }
 
@@ -43,6 +43,6 @@ public Future<Result<List<Role>>> run(FutureCallback<Result<List<Role>>> callbac
   }
 
   private String path() {
-    return String.format("/authz/groups/%s/roles/oidc?includeFullRoles=%s", _groupId(), includePermissions);
+    return String.format("/authz/groups/%s/roles/oidc?includeFullRoles=%s", encodeGroupId(), includePermissions);
   }
 }
diff --git a/src/main/java/io/weaviate/client/v1/async/groups/api/oidc/RoleAssigner.java b/src/main/java/io/weaviate/client/v1/async/groups/api/oidc/RoleAssigner.java
@@ -36,7 +36,7 @@ public RoleAssigner witRoles(String... roles) {
     return this;
   }
 
-  private String _groupId() {
+  private String encodeGroupId() {
     return UrlEncoder.encode(this.groupId);
   }
 
@@ -56,6 +56,6 @@ public Future<Result<Boolean>> run(FutureCallback<Result<Boolean>> callback) {
   }
 
   private String path() {
-    return String.format("/authz/groups/%s/assign", _groupId());
+    return String.format("/authz/groups/%s/assign", encodeGroupId());
   }
 }
diff --git a/src/main/java/io/weaviate/client/v1/async/groups/api/oidc/RoleRevoker.java b/src/main/java/io/weaviate/client/v1/async/groups/api/oidc/RoleRevoker.java
@@ -36,7 +36,7 @@ public RoleRevoker witRoles(String... roles) {
     return this;
   }
 
-  private String _groupId() {
+  private String encodeGroupId() {
     return UrlEncoder.encode(this.groupId);
   }
 
@@ -56,6 +56,6 @@ public Future<Result<Boolean>> run(FutureCallback<Result<Boolean>> callback) {
   }
 
   private String path() {
-    return String.format("/authz/groups/%s/revoke", _groupId());
+    return String.format("/authz/groups/%s/revoke", encodeGroupId());
   }
 }
diff --git a/src/main/java/io/weaviate/client/v1/groups/api/oidc/AssignedRolesGetter.java b/src/main/java/io/weaviate/client/v1/groups/api/oidc/AssignedRolesGetter.java
@@ -29,7 +29,7 @@ public AssignedRolesGetter includePermissions(boolean include) {
     return this;
   }
 
-  private String _groupId() {
+  private String encodeGroupId() {
     return UrlEncoder.encode(this.groupId);
   }
 
@@ -39,6 +39,6 @@ public Result<List<Role>> run() {
   }
 
   private String path() {
-    return String.format("/authz/groups/%s/roles/oidc?includeFullRoles=%s", _groupId(), includePermissions);
+    return String.format("/authz/groups/%s/roles/oidc?includeFullRoles=%s", encodeGroupId(), includePermissions);
   }
 }
diff --git a/src/main/java/io/weaviate/client/v1/groups/api/oidc/RoleAssigner.java b/src/main/java/io/weaviate/client/v1/groups/api/oidc/RoleAssigner.java
@@ -32,7 +32,7 @@ public RoleAssigner witRoles(String... roles) {
     return this;
   }
 
-  private String _groupId() {
+  private String encodeGroupId() {
     return UrlEncoder.encode(this.groupId);
   }
 
@@ -51,6 +51,6 @@ public Result<Boolean> run() {
   }
 
   private String path() {
-    return String.format("/authz/groups/%s/assign", _groupId());
+    return String.format("/authz/groups/%s/assign", encodeGroupId());
   }
 }
diff --git a/src/main/java/io/weaviate/client/v1/groups/api/oidc/RoleRevoker.java b/src/main/java/io/weaviate/client/v1/groups/api/oidc/RoleRevoker.java
@@ -32,7 +32,7 @@ public RoleRevoker witRoles(String... roles) {
     return this;
   }
 
-  private String _groupId() {
+  private String encodeGroupId() {
     return UrlEncoder.encode(this.groupId);
   }
 
@@ -51,6 +51,6 @@ public Result<Boolean> run() {
   }
 
   private String path() {
-    return String.format("/authz/groups/%s/revoke", _groupId());
+    return String.format("/authz/groups/%s/revoke", encodeGroupId());
   }
 }

Original file line number	Diff line number	Diff line change
`@@ -33,7 +33,7 @@ public AssignedRolesGetter includePermissions(boolean include) {`
`33`	`33`	`return this;`
`34`	`34`	`}`
`35`	`35`
`36`		`- private String _groupId() {`
	`36`	`+ private String encodeGroupId() {`
`37`	`37`	`return UrlEncoder.encode(this.groupId);`
`38`	`38`	`}`
`39`	`39`
`@@ -43,6 +43,6 @@ public Future<Result<List<Role>>> run(FutureCallback<Result<List<Role>>> callbac`
`43`	`43`	`}`
`44`	`44`
`45`	`45`	`private String path() {`
`46`		`- return String.format("/authz/groups/%s/roles/oidc?includeFullRoles=%s", _groupId(), includePermissions);`
	`46`	`+ return String.format("/authz/groups/%s/roles/oidc?includeFullRoles=%s", encodeGroupId(), includePermissions);`
`47`	`47`	`}`
`48`	`48`	`}`
Original file line number	Diff line number	Diff line change
`@@ -36,7 +36,7 @@ public RoleAssigner witRoles(String... roles) {`
`36`	`36`	`return this;`
`37`	`37`	`}`
`38`	`38`
`39`		`- private String _groupId() {`
	`39`	`+ private String encodeGroupId() {`
`40`	`40`	`return UrlEncoder.encode(this.groupId);`
`41`	`41`	`}`
`42`	`42`
`@@ -56,6 +56,6 @@ public Future<Result<Boolean>> run(FutureCallback<Result<Boolean>> callback) {`
`56`	`56`	`}`
`57`	`57`
`58`	`58`	`private String path() {`
`59`		`- return String.format("/authz/groups/%s/assign", _groupId());`
	`59`	`+ return String.format("/authz/groups/%s/assign", encodeGroupId());`
`60`	`60`	`}`
`61`	`61`	`}`
Original file line number	Diff line number	Diff line change
`@@ -36,7 +36,7 @@ public RoleRevoker witRoles(String... roles) {`
`36`	`36`	`return this;`
`37`	`37`	`}`
`38`	`38`
`39`		`- private String _groupId() {`
	`39`	`+ private String encodeGroupId() {`
`40`	`40`	`return UrlEncoder.encode(this.groupId);`
`41`	`41`	`}`
`42`	`42`
`@@ -56,6 +56,6 @@ public Future<Result<Boolean>> run(FutureCallback<Result<Boolean>> callback) {`
`56`	`56`	`}`
`57`	`57`
`58`	`58`	`private String path() {`
`59`		`- return String.format("/authz/groups/%s/revoke", _groupId());`
	`59`	`+ return String.format("/authz/groups/%s/revoke", encodeGroupId());`
`60`	`60`	`}`
`61`	`61`	`}`
Original file line number	Diff line number	Diff line change
`@@ -29,7 +29,7 @@ public AssignedRolesGetter includePermissions(boolean include) {`
`29`	`29`	`return this;`
`30`	`30`	`}`
`31`	`31`
`32`		`- private String _groupId() {`
	`32`	`+ private String encodeGroupId() {`
`33`	`33`	`return UrlEncoder.encode(this.groupId);`
`34`	`34`	`}`
`35`	`35`
`@@ -39,6 +39,6 @@ public Result<List<Role>> run() {`
`39`	`39`	`}`
`40`	`40`
`41`	`41`	`private String path() {`
`42`		`- return String.format("/authz/groups/%s/roles/oidc?includeFullRoles=%s", _groupId(), includePermissions);`
	`42`	`+ return String.format("/authz/groups/%s/roles/oidc?includeFullRoles=%s", encodeGroupId(), includePermissions);`
`43`	`43`	`}`
`44`	`44`	`}`
Original file line number	Diff line number	Diff line change
`@@ -32,7 +32,7 @@ public RoleAssigner witRoles(String... roles) {`
`32`	`32`	`return this;`
`33`	`33`	`}`
`34`	`34`
`35`		`- private String _groupId() {`
	`35`	`+ private String encodeGroupId() {`
`36`	`36`	`return UrlEncoder.encode(this.groupId);`
`37`	`37`	`}`
`38`	`38`
`@@ -51,6 +51,6 @@ public Result<Boolean> run() {`
`51`	`51`	`}`
`52`	`52`
`53`	`53`	`private String path() {`
`54`		`- return String.format("/authz/groups/%s/assign", _groupId());`
	`54`	`+ return String.format("/authz/groups/%s/assign", encodeGroupId());`
`55`	`55`	`}`
`56`	`56`	`}`
Original file line number	Diff line number	Diff line change
`@@ -32,7 +32,7 @@ public RoleRevoker witRoles(String... roles) {`
`32`	`32`	`return this;`
`33`	`33`	`}`
`34`	`34`
`35`		`- private String _groupId() {`
	`35`	`+ private String encodeGroupId() {`
`36`	`36`	`return UrlEncoder.encode(this.groupId);`
`37`	`37`	`}`
`38`	`38`
`@@ -51,6 +51,6 @@ public Result<Boolean> run() {`
`51`	`51`	`}`
`52`	`52`
`53`	`53`	`private String path() {`
`54`		`- return String.format("/authz/groups/%s/revoke", _groupId());`
	`54`	`+ return String.format("/authz/groups/%s/revoke", encodeGroupId());`
`55`	`55`	`}`
`56`	`56`	`}`