| commit | author | age | ||
| 5e3521 | 1 | /* |
| JM | 2 | * Copyright 2014 gitblit.com. |
| 3 | * | |
| 4 | * Licensed under the Apache License, Version 2.0 (the "License"); | |
| 5 | * you may not use this file except in compliance with the License. | |
| 6 | * You may obtain a copy of the License at | |
| 7 | * | |
| 8 | * http://www.apache.org/licenses/LICENSE-2.0 | |
| 9 | * | |
| 10 | * Unless required by applicable law or agreed to in writing, software | |
| 11 | * distributed under the License is distributed on an "AS IS" BASIS, | |
| 12 | * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | |
| 13 | * See the License for the specific language governing permissions and | |
| 14 | * limitations under the License. | |
| 15 | */ | |
| 16 | package com.gitblit.wicket.pages; | |
| 17 | ||
| 18 | import java.util.ArrayList; | |
| 19 | import java.util.Arrays; | |
| 20 | import java.util.Collections; | |
| 21 | import java.util.List; | |
| 22 | import java.util.Set; | |
| 23 | import java.util.TreeSet; | |
| 24 | ||
| 25 | import org.apache.wicket.PageParameters; | |
| 9d60ff | 26 | import org.apache.wicket.ajax.AjaxRequestTarget; |
| JM | 27 | import org.apache.wicket.ajax.markup.html.form.AjaxButton; |
| 5e3521 | 28 | import org.apache.wicket.markup.html.basic.Label; |
| JM | 29 | import org.apache.wicket.markup.html.form.Button; |
| 30 | import org.apache.wicket.markup.html.form.DropDownChoice; | |
| 31 | import org.apache.wicket.markup.html.form.Form; | |
| 32 | import org.apache.wicket.markup.html.form.TextField; | |
| 33 | import org.apache.wicket.markup.html.panel.Fragment; | |
| 34 | import org.apache.wicket.model.IModel; | |
| 35 | import org.apache.wicket.model.Model; | |
| 06eb51 | 36 | import org.eclipse.jgit.lib.Repository; |
| 5e3521 | 37 | |
| 06eb51 | 38 | import com.gitblit.Constants; |
| 5e3521 | 39 | import com.gitblit.Constants.AccessPermission; |
| c211e9 | 40 | import com.gitblit.Constants.AuthorizationControl; |
| 5e3521 | 41 | import com.gitblit.models.RegistrantAccessPermission; |
| JM | 42 | import com.gitblit.models.TicketModel; |
| 43 | import com.gitblit.models.TicketModel.Change; | |
| 44 | import com.gitblit.models.TicketModel.Field; | |
| 45 | import com.gitblit.models.TicketModel.Status; | |
| 46 | import com.gitblit.models.TicketModel.Type; | |
| 47 | import com.gitblit.models.UserModel; | |
| 48 | import com.gitblit.tickets.TicketMilestone; | |
| 49 | import com.gitblit.tickets.TicketNotifier; | |
| 50 | import com.gitblit.tickets.TicketResponsible; | |
| 51 | import com.gitblit.utils.StringUtils; | |
| 52 | import com.gitblit.wicket.GitBlitWebSession; | |
| 53 | import com.gitblit.wicket.WicketUtils; | |
| 54 | import com.gitblit.wicket.panels.MarkdownTextArea; | |
| 388f49 | 55 | import com.google.common.base.Optional; |
| 5e3521 | 56 | |
| JM | 57 | /** |
| 58 | * Page for editing a ticket. | |
| 59 | * | |
| 60 | * @author James Moger | |
| 61 | * | |
| 62 | */ | |
| 63 | public class EditTicketPage extends RepositoryPage { | |
| 64 | ||
| 65 | static final String NIL = "<nil>"; | |
| 66 | ||
| 67 | static final String ESC_NIL = StringUtils.escapeForHtml(NIL, false); | |
| 68 | ||
| 69 | private IModel<TicketModel.Type> typeModel; | |
| 70 | ||
| 71 | private IModel<String> titleModel; | |
| 72 | ||
| 73 | private MarkdownTextArea descriptionEditor; | |
| 74 | ||
| 75 | private IModel<String> topicModel; | |
| 06eb51 | 76 | |
| JM | 77 | private IModel<String> mergeToModel; |
| 5e3521 | 78 | |
| 7ae1fa | 79 | private IModel<Status> statusModel; |
| JM | 80 | |
| 5e3521 | 81 | private IModel<TicketResponsible> responsibleModel; |
| JM | 82 | |
| 83 | private IModel<TicketMilestone> milestoneModel; | |
| 84 | ||
| 85 | private Label descriptionPreview; | |
| f9c78c | 86 | |
| PM | 87 | private IModel<TicketModel.Priority> priorityModel; |
| 88 | ||
| 89 | private IModel<TicketModel.Severity> severityModel; | |
| 5e3521 | 90 | |
| JM | 91 | public EditTicketPage(PageParameters params) { |
| 92 | super(params); | |
| 93 | ||
| 94 | UserModel currentUser = GitBlitWebSession.get().getUser(); | |
| 95 | if (currentUser == null) { | |
| 96 | currentUser = UserModel.ANONYMOUS; | |
| 97 | } | |
| 98 | ||
| 99 | long ticketId = 0L; | |
| 100 | try { | |
| 101 | String h = WicketUtils.getObject(params); | |
| 102 | ticketId = Long.parseLong(h); | |
| 103 | } catch (Exception e) { | |
| 104 | setResponsePage(TicketsPage.class, WicketUtils.newRepositoryParameter(repositoryName)); | |
| 105 | } | |
| 106 | ||
| 107 | TicketModel ticket = app().tickets().getTicket(getRepositoryModel(), ticketId); | |
| 7ca053 | 108 | if (ticket == null |
| JM | 109 | || !currentUser.canEdit(ticket, getRepositoryModel()) |
| 110 | || !app().tickets().isAcceptingTicketUpdates(getRepositoryModel())) { | |
| 111 | setResponsePage(TicketsPage.class, WicketUtils.newObjectParameter(repositoryName, "" + ticketId)); | |
| 856a13 | 112 | |
| JM | 113 | // create a placeholder object so we don't trigger NPEs |
| 114 | ticket = new TicketModel(); | |
| 5e3521 | 115 | } |
| JM | 116 | |
| 117 | typeModel = Model.of(ticket.type); | |
| a59627 | 118 | titleModel = Model.of(ticket.title); |
| JM | 119 | topicModel = Model.of(ticket.topic == null ? "" : ticket.topic); |
| 5e3521 | 120 | responsibleModel = Model.of(); |
| JM | 121 | milestoneModel = Model.of(); |
| f1b882 | 122 | mergeToModel = Model.of(ticket.mergeTo == null ? getRepositoryModel().mergeTo : ticket.mergeTo); |
| 7ae1fa | 123 | statusModel = Model.of(ticket.status); |
| f9c78c | 124 | priorityModel = Model.of(ticket.priority); |
| PM | 125 | severityModel = Model.of(ticket.severity); |
| 5e3521 | 126 | |
| JM | 127 | setStatelessHint(false); |
| 128 | setOutputMarkupId(true); | |
| 129 | ||
| 9d60ff | 130 | Form<Void> form = new Form<Void>("editForm"); |
| 5e3521 | 131 | add(form); |
| JM | 132 | |
| 133 | List<Type> typeChoices; | |
| 134 | if (ticket.isProposal()) { | |
| 135 | typeChoices = Arrays.asList(Type.Proposal); | |
| 136 | } else { | |
| 137 | typeChoices = Arrays.asList(TicketModel.Type.choices()); | |
| 138 | } | |
| 139 | form.add(new DropDownChoice<TicketModel.Type>("type", typeModel, typeChoices)); | |
| 7ae1fa | 140 | |
| 5e3521 | 141 | form.add(new TextField<String>("title", titleModel)); |
| JM | 142 | form.add(new TextField<String>("topic", topicModel)); |
| 143 | ||
| a59627 | 144 | final IModel<String> markdownPreviewModel = Model.of(ticket.body == null ? "" : ticket.body); |
| 5e3521 | 145 | descriptionPreview = new Label("descriptionPreview", markdownPreviewModel); |
| JM | 146 | descriptionPreview.setEscapeModelStrings(false); |
| 147 | descriptionPreview.setOutputMarkupId(true); | |
| 148 | form.add(descriptionPreview); | |
| 149 | ||
| 150 | descriptionEditor = new MarkdownTextArea("description", markdownPreviewModel, descriptionPreview); | |
| 151 | descriptionEditor.setRepository(repositoryName); | |
| 152 | descriptionEditor.setText(ticket.body); | |
| 153 | form.add(descriptionEditor); | |
| 154 | ||
| 7ca053 | 155 | // status |
| JM | 156 | List<Status> statusChoices; |
| 157 | if (ticket.isClosed()) { | |
| 158 | statusChoices = Arrays.asList(ticket.status, Status.Open); | |
| 159 | } else if (ticket.isProposal()) { | |
| 160 | statusChoices = Arrays.asList(TicketModel.Status.proposalWorkflow); | |
| 161 | } else if (ticket.isBug()) { | |
| 162 | statusChoices = Arrays.asList(TicketModel.Status.bugWorkflow); | |
| 5e3521 | 163 | } else { |
| 7ca053 | 164 | statusChoices = Arrays.asList(TicketModel.Status.requestWorkflow); |
| 5e3521 | 165 | } |
| 7ca053 | 166 | Fragment status = new Fragment("status", "statusFragment", this); |
| JM | 167 | status.add(new DropDownChoice<TicketModel.Status>("status", statusModel, statusChoices)); |
| 168 | form.add(status); | |
| 169 | ||
| cc1c3f | 170 | if (currentUser.canAdmin(ticket, getRepositoryModel())) { |
| JM | 171 | // responsible |
| 172 | Set<String> userlist = new TreeSet<String>(ticket.getParticipants()); | |
| 7ca053 | 173 | |
| c211e9 | 174 | if (UserModel.ANONYMOUS.canPush(getRepositoryModel()) |
| JM | 175 | || AuthorizationControl.AUTHENTICATED == getRepositoryModel().authorizationControl) { |
| 176 | // authorization is ANONYMOUS or AUTHENTICATED (i.e. all users can be set responsible) | |
| eb6129 | 177 | userlist.addAll(app().users().getAllUsernames()); |
| JM | 178 | } else { |
| c211e9 | 179 | // authorization is by NAMED users (users with PUSH permission can be set responsible) |
| eb6129 | 180 | for (RegistrantAccessPermission rp : app().repositories().getUserAccessPermissions(getRepositoryModel())) { |
| 805468 | 181 | if (rp.permission.atLeast(AccessPermission.PUSH)) { |
| eb6129 | 182 | userlist.add(rp.registrant); |
| JM | 183 | } |
| 7ca053 | 184 | } |
| JM | 185 | } |
| 186 | ||
| cc1c3f | 187 | List<TicketResponsible> responsibles = new ArrayList<TicketResponsible>(); |
| JM | 188 | for (String username : userlist) { |
| 189 | UserModel user = app().users().getUserModel(username); | |
| eb6129 | 190 | if (user != null && !user.disabled) { |
| cc1c3f | 191 | TicketResponsible responsible = new TicketResponsible(user); |
| JM | 192 | responsibles.add(responsible); |
| 193 | if (user.username.equals(ticket.responsible)) { | |
| 194 | responsibleModel.setObject(responsible); | |
| 195 | } | |
| 196 | } | |
| 7ca053 | 197 | } |
| cc1c3f | 198 | Collections.sort(responsibles); |
| JM | 199 | responsibles.add(new TicketResponsible(NIL, "", "")); |
| 200 | Fragment responsible = new Fragment("responsible", "responsibleFragment", this); | |
| 201 | responsible.add(new DropDownChoice<TicketResponsible>("responsible", responsibleModel, responsibles)); | |
| 202 | form.add(responsible.setVisible(!responsibles.isEmpty())); | |
| 7ca053 | 203 | |
| cc1c3f | 204 | // milestone |
| JM | 205 | List<TicketMilestone> milestones = app().tickets().getMilestones(getRepositoryModel(), Status.Open); |
| 206 | for (TicketMilestone milestone : milestones) { | |
| 207 | if (milestone.name.equals(ticket.milestone)) { | |
| 208 | milestoneModel.setObject(milestone); | |
| 209 | break; | |
| 210 | } | |
| 7ca053 | 211 | } |
| cc1c3f | 212 | if (milestoneModel.getObject() == null && !StringUtils.isEmpty(ticket.milestone)) { |
| JM | 213 | // ensure that this unrecognized milestone is listed |
| 214 | // so that we get the <nil> selection. | |
| 215 | TicketMilestone tms = new TicketMilestone(ticket.milestone); | |
| 216 | milestones.add(tms); | |
| 217 | milestoneModel.setObject(tms); | |
| 218 | } | |
| 219 | if (!milestones.isEmpty()) { | |
| 220 | milestones.add(new TicketMilestone(NIL)); | |
| 221 | } | |
| 222 | ||
| 223 | Fragment milestone = new Fragment("milestone", "milestoneFragment", this); | |
| 224 | ||
| 225 | milestone.add(new DropDownChoice<TicketMilestone>("milestone", milestoneModel, milestones)); | |
| 226 | form.add(milestone.setVisible(!milestones.isEmpty())); | |
| 227 | ||
| f9c78c | 228 | // priority |
| PM | 229 | List<TicketModel.Priority> priorityChoices = Arrays.asList(TicketModel.Priority.choices()); |
| 230 | form.add(new DropDownChoice<TicketModel.Priority>("priority", priorityModel, priorityChoices)); | |
| 231 | ||
| 232 | // severity | |
| 233 | List<TicketModel.Severity> severityChoices = Arrays.asList(TicketModel.Severity.choices()); | |
| 234 | form.add(new DropDownChoice<TicketModel.Severity>("severity", severityModel, severityChoices)); | |
| 235 | ||
| cc1c3f | 236 | // mergeTo (integration branch) |
| JM | 237 | List<String> branches = new ArrayList<String>(); |
| 238 | for (String branch : getRepositoryModel().getLocalBranches()) { | |
| 239 | // exclude ticket branches | |
| 240 | if (!branch.startsWith(Constants.R_TICKET)) { | |
| 241 | branches.add(Repository.shortenRefName(branch)); | |
| 242 | } | |
| 243 | } | |
| f1b882 | 244 | branches.remove(Repository.shortenRefName(getRepositoryModel().mergeTo)); |
| JM | 245 | branches.add(0, Repository.shortenRefName(getRepositoryModel().mergeTo)); |
| cc1c3f | 246 | |
| JM | 247 | Fragment mergeto = new Fragment("mergeto", "mergeToFragment", this); |
| 248 | mergeto.add(new DropDownChoice<String>("mergeto", mergeToModel, branches)); | |
| 249 | form.add(mergeto.setVisible(!branches.isEmpty())); | |
| 250 | } else { | |
| 251 | // user can not admin this ticket | |
| 252 | form.add(new Label("responsible").setVisible(false)); | |
| 253 | form.add(new Label("milestone").setVisible(false)); | |
| 254 | form.add(new Label("mergeto").setVisible(false)); | |
| 7ca053 | 255 | } |
| 9d60ff | 256 | |
| JM | 257 | form.add(new AjaxButton("update") { |
| 258 | ||
| 259 | private static final long serialVersionUID = 1L; | |
| 260 | ||
| 261 | @Override | |
| 262 | protected void onSubmit(AjaxRequestTarget target, Form<?> form) { | |
| 263 | long ticketId = 0L; | |
| 264 | try { | |
| 265 | String h = WicketUtils.getObject(getPageParameters()); | |
| 266 | ticketId = Long.parseLong(h); | |
| 267 | } catch (Exception e) { | |
| 268 | setResponsePage(TicketsPage.class, WicketUtils.newRepositoryParameter(repositoryName)); | |
| 269 | } | |
| 270 | ||
| 271 | TicketModel ticket = app().tickets().getTicket(getRepositoryModel(), ticketId); | |
| 272 | ||
| 273 | String createdBy = GitBlitWebSession.get().getUsername(); | |
| 274 | Change change = new Change(createdBy); | |
| 275 | ||
| 276 | String title = titleModel.getObject(); | |
| 277 | if (StringUtils.isEmpty(title)) { | |
| 278 | return; | |
| 279 | } | |
| 280 | ||
| 281 | if (!ticket.title.equals(title)) { | |
| 282 | // title change | |
| 283 | change.setField(Field.title, title); | |
| 284 | } | |
| 285 | ||
| 388f49 | 286 | String description = Optional.fromNullable(descriptionEditor.getText()).or(""); |
| 9d60ff | 287 | if ((StringUtils.isEmpty(ticket.body) && !StringUtils.isEmpty(description)) |
| JM | 288 | || (!StringUtils.isEmpty(ticket.body) && !ticket.body.equals(description))) { |
| 289 | // description change | |
| 290 | change.setField(Field.body, description); | |
| 291 | } | |
| 292 | ||
| 293 | Status status = statusModel.getObject(); | |
| 294 | if (!ticket.status.equals(status)) { | |
| 295 | // status change | |
| 296 | change.setField(Field.status, status); | |
| 297 | } | |
| 298 | ||
| 299 | Type type = typeModel.getObject(); | |
| 300 | if (!ticket.type.equals(type)) { | |
| 301 | // type change | |
| 302 | change.setField(Field.type, type); | |
| 303 | } | |
| 304 | ||
| 388f49 | 305 | String topic = Optional.fromNullable(topicModel.getObject()).or(""); |
| 9d60ff | 306 | if ((StringUtils.isEmpty(ticket.topic) && !StringUtils.isEmpty(topic)) |
| 388f49 | 307 | || (!StringUtils.isEmpty(ticket.topic) && !ticket.topic.equals(topic))) { |
| 9d60ff | 308 | // topic change |
| JM | 309 | change.setField(Field.topic, topic); |
| 310 | } | |
| 311 | ||
| 312 | TicketResponsible responsible = responsibleModel == null ? null : responsibleModel.getObject(); | |
| 313 | if (responsible != null && !responsible.username.equals(ticket.responsible)) { | |
| 314 | // responsible change | |
| 315 | change.setField(Field.responsible, responsible.username); | |
| 316 | if (!StringUtils.isEmpty(responsible.username)) { | |
| 317 | if (!ticket.isWatching(responsible.username)) { | |
| 318 | change.watch(responsible.username); | |
| 319 | } | |
| 320 | } | |
| 321 | } | |
| 322 | ||
| 323 | TicketMilestone milestone = milestoneModel == null ? null : milestoneModel.getObject(); | |
| 324 | if (milestone != null && !milestone.name.equals(ticket.milestone)) { | |
| 325 | // milestone change | |
| 326 | if (NIL.equals(milestone.name)) { | |
| 327 | change.setField(Field.milestone, ""); | |
| 328 | } else { | |
| 329 | change.setField(Field.milestone, milestone.name); | |
| 330 | } | |
| 331 | } | |
| f9c78c | 332 | |
| PM | 333 | TicketModel.Priority priority = priorityModel.getObject(); |
| 334 | if (!ticket.priority.equals(priority)) | |
| 335 | { | |
| 336 | change.setField(Field.priority, priority); | |
| 337 | } | |
| 9d60ff | 338 | |
| f9c78c | 339 | TicketModel.Severity severity = severityModel.getObject(); |
| PM | 340 | if (!ticket.severity.equals(severity)) |
| 341 | { | |
| 342 | change.setField(Field.severity, severity); | |
| 343 | } | |
| 344 | ||
| 9d60ff | 345 | String mergeTo = mergeToModel.getObject(); |
| JM | 346 | if ((StringUtils.isEmpty(ticket.mergeTo) && !StringUtils.isEmpty(mergeTo)) |
| 347 | || (!StringUtils.isEmpty(mergeTo) && !mergeTo.equals(ticket.mergeTo))) { | |
| 348 | // integration branch change | |
| 349 | change.setField(Field.mergeTo, mergeTo); | |
| 350 | } | |
| 351 | ||
| 352 | if (change.hasFieldChanges()) { | |
| 353 | if (!ticket.isWatching(createdBy)) { | |
| 354 | change.watch(createdBy); | |
| 355 | } | |
| 356 | ticket = app().tickets().updateTicket(getRepositoryModel(), ticket.number, change); | |
| 357 | if (ticket != null) { | |
| 358 | TicketNotifier notifier = app().tickets().createNotifier(); | |
| 359 | notifier.sendMailing(ticket); | |
| 60bbdf | 360 | redirectTo(TicketsPage.class, WicketUtils.newObjectParameter(getRepositoryModel().name, "" + ticket.number)); |
| 9d60ff | 361 | } else { |
| JM | 362 | // TODO error |
| 363 | } | |
| 364 | } else { | |
| 365 | // nothing to change?! | |
| 60bbdf | 366 | redirectTo(TicketsPage.class, WicketUtils.newObjectParameter(getRepositoryModel().name, "" + ticket.number)); |
| 9d60ff | 367 | } |
| JM | 368 | } |
| 369 | }); | |
| 370 | ||
| 5e3521 | 371 | Button cancel = new Button("cancel") { |
| JM | 372 | private static final long serialVersionUID = 1L; |
| 373 | ||
| 374 | @Override | |
| 375 | public void onSubmit() { | |
| 376 | setResponsePage(TicketsPage.class, getPageParameters()); | |
| 377 | } | |
| 378 | }; | |
| 379 | cancel.setDefaultFormProcessing(false); | |
| 380 | form.add(cancel); | |
| 381 | } | |
| 382 | ||
| 383 | @Override | |
| 384 | protected String getPageName() { | |
| 385 | return getString("gb.editTicket"); | |
| 386 | } | |
| 387 | ||
| 388 | @Override | |
| 389 | protected Class<? extends BasePage> getRepoNavPageClass() { | |
| 390 | return TicketsPage.class; | |
| 391 | } | |
| 392 | } | |
