diff options
| author | James Moger <james.moger@gitblit.com> | 2012-10-31 08:50:20 -0400 |
|---|---|---|
| committer | James Moger <james.moger@gitblit.com> | 2012-10-31 08:50:20 -0400 |
| commit | 717267cdf6fff130865c194dc33620ac1cd10a51 (patch) | |
| tree | 5409c662b9d8901aa8498299d90bdf754a9dd270 /src | |
| parent | 70e3d195c738cd190c46754d2d1b0c2eed9a4d4b (diff) | |
| download | gitblit-717267cdf6fff130865c194dc33620ac1cd10a51.tar.gz gitblit-717267cdf6fff130865c194dc33620ac1cd10a51.zip | |
Reject add if permission selector has not been set
Diffstat (limited to 'src')
| -rw-r--r-- | src/com/gitblit/wicket/panels/RegistrantPermissionsPanel.java | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/src/com/gitblit/wicket/panels/RegistrantPermissionsPanel.java b/src/com/gitblit/wicket/panels/RegistrantPermissionsPanel.java index 27e48fbe..1d451a67 100644 --- a/src/com/gitblit/wicket/panels/RegistrantPermissionsPanel.java +++ b/src/com/gitblit/wicket/panels/RegistrantPermissionsPanel.java @@ -169,6 +169,9 @@ public class RegistrantPermissionsPanel extends BasePanel { protected void onSubmit(AjaxRequestTarget target, Form<?> form) { // add permission to our list RegistrantAccessPermission rp = (RegistrantAccessPermission) form.getModel().getObject(); + if (rp.permission == null) { + return; + } RegistrantAccessPermission copy = DeepCopier.copy(rp); if (StringUtils.findInvalidCharacter(copy.registrant) != null) { copy.permissionType = PermissionType.REGEX; |