Minor tweaks

Author: bitwiseman

src/main/java/com/cloudbees/jenkins/plugins/bitbucket/client/BitbucketCloudApiClient.java

@@ -306,8 +306,9 @@ public List<BitbucketPullRequestValue> getPullRequests() throws InterruptedExcep
             pullRequests.addAll(page.getValues());
         } while (page.getNext() != null);
 
-        // PRs with missing destination branch are invalid and should be ignored.
+        // PRs with missing source or destination branch are invalid and should be ignored.
         pullRequests.removeIf(pr -> pr.getSource().getRepository() == null
+                || pr.getSource().getBranch() == null
                 || pr.getSource().getCommit() == null
                 || pr.getDestination().getBranch() == null
                 || pr.getDestination().getCommit() == null);

src/main/java/com/cloudbees/jenkins/plugins/bitbucket/client/events/BitbucketCloudPullRequestEvent.java

@@ -90,8 +90,7 @@ private void reconstructMissingData() {
                         }
                     }
                 }
-            }
-            if (source != null) {
+
                 BitbucketCloudBranch sourceBranch = source.getBranch();
                 BitbucketCommit sourceCommit = source.getCommit();
                 if (sourceCommit != null

src/main/java/com/cloudbees/jenkins/plugins/bitbucket/server/client/BitbucketServerAPIClient.java

@@ -329,10 +329,12 @@ private List<BitbucketServerPullRequest> getPullRequests(UriTemplate template)
             throws IOException, InterruptedException {
         List<BitbucketServerPullRequest> pullRequests = getResources(template, BitbucketServerPullRequests.class);
 
-        // PRs with missing destination branch are invalid and should be ignored.
+        // PRs with missing source or destination branch are invalid and should be ignored.
         pullRequests.removeIf(pr -> pr.getSource().getRepository() == null
                 || pr.getSource().getBranch() == null
-                || pr.getDestination().getBranch() == null);
+                || pr.getSource().getCommit() == null
+                || pr.getDestination().getBranch() == null
+                || pr.getDestination().getCommit() == null);
 
         // set commit closure to make commit informations available when need, in a similar way to when request branches
         for (BitbucketServerPullRequest pullRequest : pullRequests) {