Fixing PULL_REQUEST_URL-bug correctly with getSlug #82

Author: tomasbjerre

CHANGELOG.md

@@ -2,6 +2,9 @@
 
 Changelog of Pull Request Notifier for Bitbucket.
 
+## 2.12
+* Fixing PULL_REQUEST_URL-bug correctly with getSlug.
+
 ## 2.11
 * Bugfix: Replacing spaces with dashes in ${PULL_REQUEST_URL}. Was evaluating to wrong URL if repo name included spaces.
 

src/main/java/se/bjurr/prnfb/listener/PrnfbRenderer.java

@@ -420,9 +420,8 @@ private static String cloneUrlFromRepository(REPO_PROTOCOL protocol, Repository
   }
 
   private static String getPullRequestUrl(ApplicationPropertiesService propertiesService, PullRequest pullRequest) {
-   return (propertiesService.getBaseUrl() + "/projects/" + pullRequest.getToRef().getRepository().getProject().getKey()
-     + "/repos/" + pullRequest.getToRef().getRepository().getName() + "/pull-requests/" + pullRequest.getId())
-     .replaceAll(" ", "-");
+   return propertiesService.getBaseUrl() + "/projects/" + pullRequest.getToRef().getRepository().getProject().getKey()
+     + "/repos/" + pullRequest.getToRef().getRepository().getSlug() + "/pull-requests/" + pullRequest.getId();
   }
 
   PrnfbVariable(Resolver resolver) {

src/test/java/se/bjurr/prnfb/admin/PrnfbPullRequestEventListenerTest.java

@@ -411,7 +411,7 @@ public void testThatAUrlWithVariablesExceptFromAndToCanBeInvoked() throws Except
           pullRequestRefBuilder() //
             .withProjectKey("theProject") //
             .withRepositoryName("the Repo Name") //
-        ) //
+            .withRepositorySlug("the-repo-name")) //
         .withPullRequestAction(OPENED) //
         .withAuthor( //
           prnfbParticipantBuilder() //
@@ -426,7 +426,7 @@ public void testThatAUrlWithVariablesExceptFromAndToCanBeInvoked() throws Except
     ) //
     .invokedUrl(
       0,
-      "http://bjurr.se/id=10&action=OPENED&displayName=authorDisplayName&authorEmail=authorEmail&authorId=100&authorName=authorName&authorSlug=authorSlug&pullRequestUrl=http%3A%2F%2Fbitbucket.server%2Fprojects%2FtheProject%2Frepos%2Fthe-Repo-Name%2Fpull-requests%2F10" //
+      "http://bjurr.se/id=10&action=OPENED&displayName=authorDisplayName&authorEmail=authorEmail&authorId=100&authorName=authorName&authorSlug=authorSlug&pullRequestUrl=http%3A%2F%2Fbitbucket.server%2Fprojects%2FtheProject%2Frepos%2Fthe-repo-name%2Fpull-requests%2F10" //
     );
  }