opendev/gerrit
Code Issues Proposed changes

SetAccess: fix 500 error when no parameter force in access api

Browse Source 
The parameter `force` is optional, if it is not present, it should be set
to false by default.

Change-Id: Ia1f787edfe60680b49798eb72220c6de13aee586
This commit is contained in:
飞 李
2016-10-01 18:06:37 +00:00
committed by Björn Pedersen
parent ca40aad0b9
commit 09ea06b8bf
1 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
gerrit-server/src/main/java/com/google/gerrit/server/project/SetAccess.java
View File

@@ -267,8 +267,10 @@ public class SetAccess implements
r.setMin(pri.min); r.setMin(pri.min);
} }
r.setAction(GetAccess.ACTION_TYPE.inverse().get(pri.action)); r.setAction(GetAccess.ACTION_TYPE.inverse().get(pri.action));
if (pri.force != null) {
r.setForce(pri.force); r.setForce(pri.force);
} }
}
p.add(r); p.add(r);
} }
accessSection.getPermissions().add(p); accessSection.getPermissions().add(p);