James Moger
2014-09-04 ddcebbd982d81dabf8cb1c89f60f9b876a23b58c
src/main/java/com/gitblit/wicket/pages/EditTicketPage.java
@@ -37,6 +37,7 @@
import com.gitblit.Constants;
import com.gitblit.Constants.AccessPermission;
import com.gitblit.Constants.AuthorizationControl;
import com.gitblit.models.RegistrantAccessPermission;
import com.gitblit.models.TicketModel;
import com.gitblit.models.TicketModel.Change;
@@ -163,16 +164,23 @@
         // responsible
         Set<String> userlist = new TreeSet<String>(ticket.getParticipants());
         for (RegistrantAccessPermission rp : app().repositories().getUserAccessPermissions(getRepositoryModel())) {
            if (rp.permission.atLeast(AccessPermission.PUSH) && !rp.isTeam()) {
               userlist.add(rp.registrant);
         if (UserModel.ANONYMOUS.canPush(getRepositoryModel())
               || AuthorizationControl.AUTHENTICATED == getRepositoryModel().authorizationControl) {
            //    authorization is ANONYMOUS or AUTHENTICATED (i.e. all users can be set responsible)
            userlist.addAll(app().users().getAllUsernames());
         } else {
            // authorization is by NAMED users (users with PUSH permission can be set responsible)
            for (RegistrantAccessPermission rp : app().repositories().getUserAccessPermissions(getRepositoryModel())) {
               if (rp.permission.atLeast(AccessPermission.PUSH)) {
                  userlist.add(rp.registrant);
               }
            }
         }
         List<TicketResponsible> responsibles = new ArrayList<TicketResponsible>();
         for (String username : userlist) {
            UserModel user = app().users().getUserModel(username);
            if (user != null) {
            if (user != null && !user.disabled) {
               TicketResponsible responsible = new TicketResponsible(user);
               responsibles.add(responsible);
               if (user.username.equals(ticket.responsible)) {