REOPENED - bug 349633: BugzillaRepositoryConnector.getTaskData() fails

when repository URL has trailing slash and no path 
https://bugs.eclipse.org/bugs/show_bug.cgi?id=349633

1 files changed, 10 insertions, 0 deletions

diff --git a/org.eclipse.mylyn.bugzilla.core/src/org/eclipse/mylyn/internal/bugzilla/core/BugzillaClientFactory.java b/org.eclipse.mylyn.bugzilla.core/src/org/eclipse/mylyn/internal/bugzilla/core/BugzillaClientFactory.java
index 02cf2e841..fb8330d9d 100644
--- a/org.eclipse.mylyn.bugzilla.core/src/org/eclipse/mylyn/internal/bugzilla/core/BugzillaClientFactory.java
+++ b/org.eclipse.mylyn.bugzilla.core/src/org/eclipse/mylyn/internal/bugzilla/core/BugzillaClientFactory.java
@@ -27,6 +27,16 @@ public class BugzillaClientFactory {
 
 	public static BugzillaClient createClient(TaskRepository taskRepository, BugzillaRepositoryConnector connector)
 			throws MalformedURLException {
+		// to fix bug#349633
+		// (BugzillaRepositoryConnector.getTaskData() fails when repository URL has trailing slash and no path)
+		String repositoryURL = taskRepository.getRepositoryUrl();
+		if (repositoryURL.endsWith("/")) { //$NON-NLS-1$
+			StringBuilder sb = new StringBuilder(repositoryURL.trim());
+			while (sb.length() > 0 && sb.charAt(sb.length() - 1) == '/') {
+				sb.setLength(sb.length() - 1);
+			}
+			taskRepository.setRepositoryUrl(sb.toString());
+		}
 		AbstractWebLocation location = taskRepositoryLocationFactory.createWebLocation(taskRepository);
 
 		BugzillaClient client = new BugzillaClient(location, taskRepository, connector);