Reject add if permission selector has not been set

diff --git a/src/com/gitblit/wicket/panels/RegistrantPermissionsPanel.java b/src/com/gitblit/wicket/panels/RegistrantPermissionsPanel.java
index 27e48fbe8bfae3330d4f11c5d2fb6df78cda3849..1d451a672b58292a30014e37777b87c27b1361f5 100644 (file)
--- 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;