From 09ea06b8bf74aec641b50c56c969fc47aee34db9 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=E9=A3=9E=20=E6=9D=8E?= Date: Sat, 1 Oct 2016 18:06:37 +0000 Subject: [PATCH] SetAccess: fix 500 error when no parameter `force` in access api The parameter `force` is optional, if it is not present, it should be set to false by default. Change-Id: Ia1f787edfe60680b49798eb72220c6de13aee586 --- .../main/java/com/google/gerrit/server/project/SetAccess.java | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/gerrit-server/src/main/java/com/google/gerrit/server/project/SetAccess.java b/gerrit-server/src/main/java/com/google/gerrit/server/project/SetAccess.java index 4574a7a55a..1e5a7c9c11 100644 --- a/gerrit-server/src/main/java/com/google/gerrit/server/project/SetAccess.java +++ b/gerrit-server/src/main/java/com/google/gerrit/server/project/SetAccess.java @@ -267,7 +267,9 @@ public class SetAccess implements r.setMin(pri.min); } r.setAction(GetAccess.ACTION_TYPE.inverse().get(pri.action)); - r.setForce(pri.force); + if (pri.force != null) { + r.setForce(pri.force); + } } p.add(r); }