Override hasCredentials method for those provider subclasses that rely on tokens for authentication instead of modifying the parent class method

Author: tcrespog

modules/nextflow/src/main/groovy/nextflow/scm/BitbucketRepositoryProvider.groovy

@@ -48,7 +48,7 @@ final class BitbucketRepositoryProvider extends RepositoryProvider {
 
     @Override
     boolean hasCredentials() {
-        return config.token
+        return getToken()
             ? true
             : super.hasCredentials()
     }

modules/nextflow/src/main/groovy/nextflow/scm/GiteaRepositoryProvider.groovy

@@ -53,6 +53,13 @@ final class GiteaRepositoryProvider extends RepositoryProvider {
             return null
     }
 
+    @Override
+    boolean hasCredentials() {
+        return getToken()
+            ? true
+            : super.hasCredentials()
+    }
+
     @Override
     @CompileDynamic
     List<BranchInfo> getBranches() {

modules/nextflow/src/main/groovy/nextflow/scm/GitlabRepositoryProvider.groovy

@@ -50,6 +50,13 @@ class GitlabRepositoryProvider extends RepositoryProvider {
         return null
     }
 
+    @Override
+    boolean hasCredentials() {
+        return getToken()
+            ? true
+            : super.hasCredentials()
+    }
+
     @Override
     String getName() { "GitLab" }
 

modules/nextflow/src/main/groovy/nextflow/scm/RepositoryProvider.groovy

@@ -126,7 +126,7 @@ abstract class RepositoryProvider {
     }
 
     boolean hasCredentials() {
-        return (getUser() && getPassword()) || getToken()
+        getUser() && getPassword()
     }
 
     String getUser() { config?.user }