Fixed blockUser() and unblockUser() for V3 (#251).

gmessner · gmessner · commit 7b6f8b322d25 · 2018-09-12T20:25:22.000-07:00
diff --git a/src/main/java/org/gitlab4j/api/UserApi.java b/src/main/java/org/gitlab4j/api/UserApi.java
@@ -127,7 +127,11 @@ public void blockUser(Integer userId) throws GitLabApiException {
             throw new RuntimeException("userId cannot be null");
         }
 
-        post(Response.Status.CREATED, (Form) null, "users", userId, "block");
+        if (isApiVersion(ApiVersion.V3)) {
+            put(Response.Status.CREATED, null, "users", userId, "block");
+        } else {
+            post(Response.Status.CREATED, (Form) null, "users", userId, "block");
+        }
     }
 
     /**
@@ -144,7 +148,11 @@ public void unblockUser(Integer userId) throws GitLabApiException {
             throw new RuntimeException("userId cannot be null");
         }
 
-        post(Response.Status.CREATED, (Form) null, "users", userId, "unblock");
+        if (isApiVersion(ApiVersion.V3)) {
+            put(Response.Status.CREATED, null, "users", userId, "unblock");
+        } else {
+            post(Response.Status.CREATED, (Form) null, "users", userId, "unblock");
+        }
     }
 
     /**

Original file line number	Diff line number	Diff line change
`@@ -127,7 +127,11 @@ public void blockUser(Integer userId) throws GitLabApiException {`
`127`	`127`	`throw new RuntimeException("userId cannot be null");`
`128`	`128`	`}`
`129`	`129`
`130`		`- post(Response.Status.CREATED, (Form) null, "users", userId, "block");`
	`130`	`+ if (isApiVersion(ApiVersion.V3)) {`
	`131`	`+ put(Response.Status.CREATED, null, "users", userId, "block");`
	`132`	`+ } else {`
	`133`	`+ post(Response.Status.CREATED, (Form) null, "users", userId, "block");`
	`134`	`+ }`
`131`	`135`	`}`
`132`	`136`
`133`	`137`	`/**`
`@@ -144,7 +148,11 @@ public void unblockUser(Integer userId) throws GitLabApiException {`
`144`	`148`	`throw new RuntimeException("userId cannot be null");`
`145`	`149`	`}`
`146`	`150`
`147`		`- post(Response.Status.CREATED, (Form) null, "users", userId, "unblock");`
	`151`	`+ if (isApiVersion(ApiVersion.V3)) {`
	`152`	`+ put(Response.Status.CREATED, null, "users", userId, "unblock");`
	`153`	`+ } else {`
	`154`	`+ post(Response.Status.CREATED, (Form) null, "users", userId, "unblock");`
	`155`	`+ }`
`148`	`156`	`}`
`149`	`157`
`150`	`158`	`/**`