From fbe6dbd1e4c95b33fc21d8a7aaf17f181dacab48 Mon Sep 17 00:00:00 2001
From: James Moger <james.moger@gitblit.com>
Date: Thu, 25 Sep 2014 10:31:46 -0400
Subject: [PATCH] Merged #188 "Fix failure to clear/delete ticket topic and description"

---
 src/main/java/com/gitblit/wicket/pages/EditTicketPage.java |    7 ++++---
 1 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/src/main/java/com/gitblit/wicket/pages/EditTicketPage.java b/src/main/java/com/gitblit/wicket/pages/EditTicketPage.java
index 1adc713..074ffa6 100644
--- a/src/main/java/com/gitblit/wicket/pages/EditTicketPage.java
+++ b/src/main/java/com/gitblit/wicket/pages/EditTicketPage.java
@@ -52,6 +52,7 @@
 import com.gitblit.wicket.GitBlitWebSession;
 import com.gitblit.wicket.WicketUtils;
 import com.gitblit.wicket.panels.MarkdownTextArea;
+import com.google.common.base.Optional;
 
 /**
  * Page for editing a ticket.
@@ -268,7 +269,7 @@
 					change.setField(Field.title, title);
 				}
 
-				String description = descriptionEditor.getText();
+				String description = Optional.fromNullable(descriptionEditor.getText()).or("");
 				if ((StringUtils.isEmpty(ticket.body) && !StringUtils.isEmpty(description))
 						|| (!StringUtils.isEmpty(ticket.body) && !ticket.body.equals(description))) {
 					// description change
@@ -287,9 +288,9 @@
 					change.setField(Field.type, type);
 				}
 
-				String topic = topicModel.getObject();
+				String topic = Optional.fromNullable(topicModel.getObject()).or("");
 				if ((StringUtils.isEmpty(ticket.topic) && !StringUtils.isEmpty(topic))
-						|| (!StringUtils.isEmpty(topic) && !topic.equals(ticket.topic))) {
+					|| (!StringUtils.isEmpty(ticket.topic) && !ticket.topic.equals(topic))) {
 					// topic change
 					change.setField(Field.topic, topic);
 				}

--
Gitblit v1.9.1