From be9c5a6a8b0dc18bd203eb6116b70dbf1a68facc Mon Sep 17 00:00:00 2001 From: Pierre-Yves Chibon Date: Jul 18 2015 11:20:36 +0000 Subject: [PATCH 1/5] Rename self.Token into self.token Variables starting with capital letters are conventionally classes --- diff --git a/libpagure/libpagure.py b/libpagure/libpagure.py index f57ec45..7658b3b 100644 --- a/libpagure/libpagure.py +++ b/libpagure/libpagure.py @@ -36,11 +36,11 @@ class Pagure(object): :param instance_url: the URL of pagure instance name :return: """ - self.Token = pagure_token + self.token = pagure_token self.Repository = pagure_repository self.ForkUsername = fork_username self.InstanceURL = instance_url - self.Header = {"Authorization": "token " + self.Token} + self.Header = {"Authorization": "token " + self.token} self.session = requests.session() self.insecure = insecure From 422999bee7a47bab83c418c92aa65601398d3d11 Mon Sep 17 00:00:00 2001 From: Pierre-Yves Chibon Date: Jul 18 2015 11:20:36 +0000 Subject: [PATCH 2/5] Rename self.Header into self.header Variables starting with capital letters are conventionally classes --- diff --git a/libpagure/libpagure.py b/libpagure/libpagure.py index 7658b3b..5dd5748 100644 --- a/libpagure/libpagure.py +++ b/libpagure/libpagure.py @@ -40,7 +40,7 @@ class Pagure(object): self.Repository = pagure_repository self.ForkUsername = fork_username self.InstanceURL = instance_url - self.Header = {"Authorization": "token " + self.token} + self.header = {"Authorization": "token " + self.token} self.session = requests.session() self.insecure = insecure @@ -62,7 +62,7 @@ class Pagure(object): method=method, url=url, params=params, - headers=self.Header, + headers=self.header, data=data, verify=not self.insecure, ) From 19146f1f20c4051a8465971402f442633b020c29 Mon Sep 17 00:00:00 2001 From: Pierre-Yves Chibon Date: Jul 18 2015 11:20:36 +0000 Subject: [PATCH 3/5] Rename self.Repository into self.repo Variables starting with capital letters are conventionally classes --- diff --git a/libpagure/libpagure.py b/libpagure/libpagure.py index 5dd5748..752a1eb 100644 --- a/libpagure/libpagure.py +++ b/libpagure/libpagure.py @@ -37,7 +37,7 @@ class Pagure(object): :return: """ self.token = pagure_token - self.Repository = pagure_repository + self.repo = pagure_repository self.ForkUsername = fork_username self.InstanceURL = instance_url self.header = {"Authorization": "token " + self.token} @@ -115,10 +115,10 @@ class Pagure(object): """ if self.ForkUsername is None: request_url = "{}/api/0/{}/tags".format( - self.InstanceURL, self.Repository) + self.InstanceURL, self.repo) else: request_url = "{}/api/0/fork/{}/{}/tags".format( - self.InstanceURL, self.ForkUsername, self.Repository) + self.InstanceURL, self.ForkUsername, self.repo) params = None if pattern: params = {'pattern': pattern} @@ -159,10 +159,10 @@ class Pagure(object): """ if self.ForkUsername is None: request_url = "{}/api/0/{}/pull-requests".format( - self.InstanceURL, self.Repository) + self.InstanceURL, self.repo) else: request_url = "{}/api/0/fork/{}/{}/pull-requests".format( - self.InstanceURL, self.ForkUsername, self.Repository) + self.InstanceURL, self.ForkUsername, self.repo) payload = {} if status is not None: payload['status'] = status @@ -182,10 +182,10 @@ class Pagure(object): """ if self.ForkUsername is None: request_url = "{}/api/0/{}/pull-request/{}".format( - self.InstanceURL, self.Repository, request_id) + self.InstanceURL, self.repo, request_id) else: request_url = "{}/api/0/fork/{}/{}/pull-request/{}".format( - self.InstanceURL, self.ForkUsername, self.Repository, + self.InstanceURL, self.ForkUsername, self.repo, request_id) return_value = self.__call_api(request_url) @@ -199,10 +199,10 @@ class Pagure(object): """ if self.ForkUsername is None: request_url = "{}/api/0/{}/pull-request/{}/merge".format( - self.InstanceURL, self.Repository, request_id) + self.InstanceURL, self.repo, request_id) else: request_url = "{}/api/0/fork/{}/{}/pull-request/{}/merge".format( - self.InstanceURL, self.ForkUsername, self.Repository, + self.InstanceURL, self.ForkUsername, self.repo, request_id) return_value = self.__call_api(request_url, method='POST') @@ -218,10 +218,10 @@ class Pagure(object): """ if self.ForkUsername is None: request_url = "{}/api/0/{}/pull-request/{}/close".format( - self.InstanceURL, self.Repository, request_id) + self.InstanceURL, self.repo, request_id) else: request_url = "{}/api/0/fork/{}/{}/pull-request/{}/close".format( - self.InstanceURL, self.ForkUsername, self.Repository, + self.InstanceURL, self.ForkUsername, self.repo, request_id) return_value = self.__call_api(request_url, method='POST') @@ -241,10 +241,10 @@ class Pagure(object): """ if self.ForkUsername is None: request_url = "{}/api/0/{}/pull-request/{}/comment".format( - self.InstanceURL, self.Repository, request_id) + self.InstanceURL, self.repo, request_id) else: request_url = "{}/api/0/fork/{}/{}/pull-request/{}/comment".format( - self.InstanceURL, self.ForkUsername, self.Repository, + self.InstanceURL, self.ForkUsername, self.repo, request_id) payload = {'comment': body} @@ -274,10 +274,10 @@ class Pagure(object): """ if self.ForkUsername is None: request_url = "{}/api/0/{}/pull-request/{}/flag".format( - self.InstanceURL, self.Repository, request_id) + self.InstanceURL, self.repo, request_id) else: request_url = "{}/api/0/fork/{}/{}/pull-request/{}/flag".format( - self.InstanceURL, self.ForkUsername, self.Repository, + self.InstanceURL, self.ForkUsername, self.repo, request_id) payload = {'username': username, 'percent': percent, 'comment': comment, 'url': url} @@ -301,10 +301,10 @@ class Pagure(object): """ if self.ForkUsername is None: request_url = "{}/api/0/{}/new_issue".format( - self.InstanceURL, self.Repository) + self.InstanceURL, self.repo) else: request_url = "{}/api/0/fork/{}/{}/new_issue".format( - self.InstanceURL, self.ForkUsername, self.Repository) + self.InstanceURL, self.ForkUsername, self.repo) payload = {'title': title, 'issue_content': content} if private is not None: @@ -326,10 +326,10 @@ class Pagure(object): """ if self.ForkUsername is None: request_url = "{}/api/0/{}/issues".format( - self.InstanceURL, self.Repository) + self.InstanceURL, self.repo) else: request_url = "{}/api/0/fork/{}/{}/issues".format( - self.InstanceURL, self.ForkUsername, self.Repository) + self.InstanceURL, self.ForkUsername, self.repo) payload = {} if status is not None: @@ -353,10 +353,10 @@ class Pagure(object): """ if self.ForkUsername is None: request_url = "{}/api/0/{}/issue/{}".format( - self.InstanceURL, self.Repository, issue_id) + self.InstanceURL, self.repo, issue_id) else: request_url = "{}/api/0/fork/{}/{}/issue/{}".format( - self.InstanceURL, self.ForkUsername, self.Repository, + self.InstanceURL, self.ForkUsername, self.repo, issue_id) return_value = self.__call_api(request_url) @@ -372,10 +372,10 @@ class Pagure(object): """ if self.ForkUsername is None: request_url = "{}/api/0/{}/issue/{}/comment/{}".format( - self.InstanceURL, self.Repository, issue_id, comment_id) + self.InstanceURL, self.repo, issue_id, comment_id) else: request_url = "{}/api/0/fork/{}/{}/issue/{}/comment/{}".format( - self.InstanceURL, self.ForkUsername, self.Repository, + self.InstanceURL, self.ForkUsername, self.repo, issue_id, comment_id) return_value = self.__call_api(request_url) @@ -391,10 +391,10 @@ class Pagure(object): """ if self.ForkUsername is None: request_url = "{}/api/0/{}/issue/{}/status".format( - self.InstanceURL, self.Repository, issue_id) + self.InstanceURL, self.repo, issue_id) else: request_url = "{}/api/0/fork/{}/{}/issue/{}/status".format( - self.InstanceURL, self.ForkUsername, self.Repository, + self.InstanceURL, self.ForkUsername, self.repo, issue_id) payload = {'status': new_status} @@ -413,10 +413,10 @@ class Pagure(object): """ if self.ForkUsername is None: request_url = "{}/api/0/{}/issue/{}/comment".format( - self.InstanceURL, self.Repository, issue_id) + self.InstanceURL, self.repo, issue_id) else: request_url = "{}/api/0/fork/{}/{}/issue/{}/comment".format( - self.InstanceURL, self.ForkUsername, self.Repository, + self.InstanceURL, self.ForkUsername, self.repo, issue_id) payload = {'comment': body} @@ -433,10 +433,10 @@ class Pagure(object): """ if self.ForkUsername is None: request_url = "{}/api/0/{}/git/tags".format( - self.InstanceURL, self.Repository) + self.InstanceURL, self.repo) else: request_url = "{}/api/0/fork/{}/{}/git/tags".format( - self.InstanceURL, self.ForkUsername, self.Repository) + self.InstanceURL, self.ForkUsername, self.repo) return_value = self.__call_api(request_url) From 8332624e1399fb978f95a93b10ca94c6ddd8ae20 Mon Sep 17 00:00:00 2001 From: Pierre-Yves Chibon Date: Jul 18 2015 11:20:36 +0000 Subject: [PATCH 4/5] Rename self.ForkUsername into self.username Variables starting with capital letters are conventionally classes --- diff --git a/libpagure/libpagure.py b/libpagure/libpagure.py index 752a1eb..6e512b4 100644 --- a/libpagure/libpagure.py +++ b/libpagure/libpagure.py @@ -38,7 +38,7 @@ class Pagure(object): """ self.token = pagure_token self.repo = pagure_repository - self.ForkUsername = fork_username + self.username = fork_username self.InstanceURL = instance_url self.header = {"Authorization": "token " + self.token} self.session = requests.session() @@ -113,12 +113,12 @@ class Pagure(object): :param pattern: filters the starting letters of the return value :return: """ - if self.ForkUsername is None: + if self.username is None: request_url = "{}/api/0/{}/tags".format( self.InstanceURL, self.repo) else: request_url = "{}/api/0/fork/{}/{}/tags".format( - self.InstanceURL, self.ForkUsername, self.repo) + self.InstanceURL, self.username, self.repo) params = None if pattern: params = {'pattern': pattern} @@ -157,12 +157,12 @@ class Pagure(object): :param author: filters the author of the requests :return: """ - if self.ForkUsername is None: + if self.username is None: request_url = "{}/api/0/{}/pull-requests".format( self.InstanceURL, self.repo) else: request_url = "{}/api/0/fork/{}/{}/pull-requests".format( - self.InstanceURL, self.ForkUsername, self.repo) + self.InstanceURL, self.username, self.repo) payload = {} if status is not None: payload['status'] = status @@ -180,12 +180,12 @@ class Pagure(object): :param request_id: the id of the request :return: """ - if self.ForkUsername is None: + if self.username is None: request_url = "{}/api/0/{}/pull-request/{}".format( self.InstanceURL, self.repo, request_id) else: request_url = "{}/api/0/fork/{}/{}/pull-request/{}".format( - self.InstanceURL, self.ForkUsername, self.repo, + self.InstanceURL, self.username, self.repo, request_id) return_value = self.__call_api(request_url) @@ -197,12 +197,12 @@ class Pagure(object): :param request_id: the id of the request :return: """ - if self.ForkUsername is None: + if self.username is None: request_url = "{}/api/0/{}/pull-request/{}/merge".format( self.InstanceURL, self.repo, request_id) else: request_url = "{}/api/0/fork/{}/{}/pull-request/{}/merge".format( - self.InstanceURL, self.ForkUsername, self.repo, + self.InstanceURL, self.username, self.repo, request_id) return_value = self.__call_api(request_url, method='POST') @@ -216,12 +216,12 @@ class Pagure(object): :param request_id: the id of the request :return: """ - if self.ForkUsername is None: + if self.username is None: request_url = "{}/api/0/{}/pull-request/{}/close".format( self.InstanceURL, self.repo, request_id) else: request_url = "{}/api/0/fork/{}/{}/pull-request/{}/close".format( - self.InstanceURL, self.ForkUsername, self.repo, + self.InstanceURL, self.username, self.repo, request_id) return_value = self.__call_api(request_url, method='POST') @@ -239,12 +239,12 @@ class Pagure(object): :param row: which line of code to comment on :return: """ - if self.ForkUsername is None: + if self.username is None: request_url = "{}/api/0/{}/pull-request/{}/comment".format( self.InstanceURL, self.repo, request_id) else: request_url = "{}/api/0/fork/{}/{}/pull-request/{}/comment".format( - self.InstanceURL, self.ForkUsername, self.repo, + self.InstanceURL, self.username, self.repo, request_id) payload = {'comment': body} @@ -272,12 +272,12 @@ class Pagure(object): :param commit: which commit to flag on :return: """ - if self.ForkUsername is None: + if self.username is None: request_url = "{}/api/0/{}/pull-request/{}/flag".format( self.InstanceURL, self.repo, request_id) else: request_url = "{}/api/0/fork/{}/{}/pull-request/{}/flag".format( - self.InstanceURL, self.ForkUsername, self.repo, + self.InstanceURL, self.username, self.repo, request_id) payload = {'username': username, 'percent': percent, 'comment': comment, 'url': url} @@ -299,12 +299,12 @@ class Pagure(object): :param private: whether create this issue as private :return: """ - if self.ForkUsername is None: + if self.username is None: request_url = "{}/api/0/{}/new_issue".format( self.InstanceURL, self.repo) else: request_url = "{}/api/0/fork/{}/{}/new_issue".format( - self.InstanceURL, self.ForkUsername, self.repo) + self.InstanceURL, self.username, self.repo) payload = {'title': title, 'issue_content': content} if private is not None: @@ -324,12 +324,12 @@ class Pagure(object): :param author: filters the author of the issues :return: """ - if self.ForkUsername is None: + if self.username is None: request_url = "{}/api/0/{}/issues".format( self.InstanceURL, self.repo) else: request_url = "{}/api/0/fork/{}/{}/issues".format( - self.InstanceURL, self.ForkUsername, self.repo) + self.InstanceURL, self.username, self.repo) payload = {} if status is not None: @@ -351,12 +351,12 @@ class Pagure(object): :param issue_id: the id of the issue :return: """ - if self.ForkUsername is None: + if self.username is None: request_url = "{}/api/0/{}/issue/{}".format( self.InstanceURL, self.repo, issue_id) else: request_url = "{}/api/0/fork/{}/{}/issue/{}".format( - self.InstanceURL, self.ForkUsername, self.repo, + self.InstanceURL, self.username, self.repo, issue_id) return_value = self.__call_api(request_url) @@ -370,12 +370,12 @@ class Pagure(object): :param comment_id: the id of the comment :return: """ - if self.ForkUsername is None: + if self.username is None: request_url = "{}/api/0/{}/issue/{}/comment/{}".format( self.InstanceURL, self.repo, issue_id, comment_id) else: request_url = "{}/api/0/fork/{}/{}/issue/{}/comment/{}".format( - self.InstanceURL, self.ForkUsername, self.repo, + self.InstanceURL, self.username, self.repo, issue_id, comment_id) return_value = self.__call_api(request_url) @@ -389,12 +389,12 @@ class Pagure(object): :param new_status: the new status fo the issue :return: """ - if self.ForkUsername is None: + if self.username is None: request_url = "{}/api/0/{}/issue/{}/status".format( self.InstanceURL, self.repo, issue_id) else: request_url = "{}/api/0/fork/{}/{}/issue/{}/status".format( - self.InstanceURL, self.ForkUsername, self.repo, + self.InstanceURL, self.username, self.repo, issue_id) payload = {'status': new_status} @@ -411,12 +411,12 @@ class Pagure(object): :param body: the comment body :return: """ - if self.ForkUsername is None: + if self.username is None: request_url = "{}/api/0/{}/issue/{}/comment".format( self.InstanceURL, self.repo, issue_id) else: request_url = "{}/api/0/fork/{}/{}/issue/{}/comment".format( - self.InstanceURL, self.ForkUsername, self.repo, + self.InstanceURL, self.username, self.repo, issue_id) payload = {'comment': body} @@ -431,12 +431,12 @@ class Pagure(object): List all git tags made to the project. :return: """ - if self.ForkUsername is None: + if self.username is None: request_url = "{}/api/0/{}/git/tags".format( self.InstanceURL, self.repo) else: request_url = "{}/api/0/fork/{}/{}/git/tags".format( - self.InstanceURL, self.ForkUsername, self.repo) + self.InstanceURL, self.username, self.repo) return_value = self.__call_api(request_url) From 77905e9fabc49f1db7f1b67731b592a9f5455f87 Mon Sep 17 00:00:00 2001 From: Pierre-Yves Chibon Date: Jul 18 2015 11:20:36 +0000 Subject: [PATCH 5/5] Rename self.InstanceURL into self.instance Variables starting with capital letters are conventionally classes --- diff --git a/libpagure/libpagure.py b/libpagure/libpagure.py index 6e512b4..639b3fc 100644 --- a/libpagure/libpagure.py +++ b/libpagure/libpagure.py @@ -39,7 +39,7 @@ class Pagure(object): self.token = pagure_token self.repo = pagure_repository self.username = fork_username - self.InstanceURL = instance_url + self.instance = instance_url self.header = {"Authorization": "token " + self.token} self.session = requests.session() self.insecure = insecure @@ -90,7 +90,7 @@ class Pagure(object): Get Pagure API version. :return: """ - request_url = "{}/api/0/version".format(self.InstanceURL) + request_url = "{}/api/0/version".format(self.instance) return_value = self.__call_api(request_url) return return_value['version'] @@ -100,7 +100,7 @@ class Pagure(object): :param pattern: filters the starting letters of the return value :return: """ - request_url = "{}/api/0/users".format(self.InstanceURL) + request_url = "{}/api/0/users".format(self.instance) params = None if pattern: params = {'pattern': pattern} @@ -115,10 +115,10 @@ class Pagure(object): """ if self.username is None: request_url = "{}/api/0/{}/tags".format( - self.InstanceURL, self.repo) + self.instance, self.repo) else: request_url = "{}/api/0/fork/{}/{}/tags".format( - self.InstanceURL, self.username, self.repo) + self.instance, self.username, self.repo) params = None if pattern: params = {'pattern': pattern} @@ -132,7 +132,7 @@ class Pagure(object): :param pattern: filters the starting letters of the return value :return: """ - request_url = "{}/api/0/groups".format(self.InstanceURL) + request_url = "{}/api/0/groups".format(self.instance) params = None if pattern: params = {'pattern': pattern} @@ -145,7 +145,7 @@ class Pagure(object): Get a dictionary of all error codes. :return: """ - request_url = "{}/api/0/error_codes".format(self.InstanceURL) + request_url = "{}/api/0/error_codes".format(self.instance) return_value = self.__call_api(request_url) return return_value @@ -159,10 +159,10 @@ class Pagure(object): """ if self.username is None: request_url = "{}/api/0/{}/pull-requests".format( - self.InstanceURL, self.repo) + self.instance, self.repo) else: request_url = "{}/api/0/fork/{}/{}/pull-requests".format( - self.InstanceURL, self.username, self.repo) + self.instance, self.username, self.repo) payload = {} if status is not None: payload['status'] = status @@ -182,10 +182,10 @@ class Pagure(object): """ if self.username is None: request_url = "{}/api/0/{}/pull-request/{}".format( - self.InstanceURL, self.repo, request_id) + self.instance, self.repo, request_id) else: request_url = "{}/api/0/fork/{}/{}/pull-request/{}".format( - self.InstanceURL, self.username, self.repo, + self.instance, self.username, self.repo, request_id) return_value = self.__call_api(request_url) @@ -199,10 +199,10 @@ class Pagure(object): """ if self.username is None: request_url = "{}/api/0/{}/pull-request/{}/merge".format( - self.InstanceURL, self.repo, request_id) + self.instance, self.repo, request_id) else: request_url = "{}/api/0/fork/{}/{}/pull-request/{}/merge".format( - self.InstanceURL, self.username, self.repo, + self.instance, self.username, self.repo, request_id) return_value = self.__call_api(request_url, method='POST') @@ -218,10 +218,10 @@ class Pagure(object): """ if self.username is None: request_url = "{}/api/0/{}/pull-request/{}/close".format( - self.InstanceURL, self.repo, request_id) + self.instance, self.repo, request_id) else: request_url = "{}/api/0/fork/{}/{}/pull-request/{}/close".format( - self.InstanceURL, self.username, self.repo, + self.instance, self.username, self.repo, request_id) return_value = self.__call_api(request_url, method='POST') @@ -241,10 +241,10 @@ class Pagure(object): """ if self.username is None: request_url = "{}/api/0/{}/pull-request/{}/comment".format( - self.InstanceURL, self.repo, request_id) + self.instance, self.repo, request_id) else: request_url = "{}/api/0/fork/{}/{}/pull-request/{}/comment".format( - self.InstanceURL, self.username, self.repo, + self.instance, self.username, self.repo, request_id) payload = {'comment': body} @@ -274,10 +274,10 @@ class Pagure(object): """ if self.username is None: request_url = "{}/api/0/{}/pull-request/{}/flag".format( - self.InstanceURL, self.repo, request_id) + self.instance, self.repo, request_id) else: request_url = "{}/api/0/fork/{}/{}/pull-request/{}/flag".format( - self.InstanceURL, self.username, self.repo, + self.instance, self.username, self.repo, request_id) payload = {'username': username, 'percent': percent, 'comment': comment, 'url': url} @@ -301,10 +301,10 @@ class Pagure(object): """ if self.username is None: request_url = "{}/api/0/{}/new_issue".format( - self.InstanceURL, self.repo) + self.instance, self.repo) else: request_url = "{}/api/0/fork/{}/{}/new_issue".format( - self.InstanceURL, self.username, self.repo) + self.instance, self.username, self.repo) payload = {'title': title, 'issue_content': content} if private is not None: @@ -326,10 +326,10 @@ class Pagure(object): """ if self.username is None: request_url = "{}/api/0/{}/issues".format( - self.InstanceURL, self.repo) + self.instance, self.repo) else: request_url = "{}/api/0/fork/{}/{}/issues".format( - self.InstanceURL, self.username, self.repo) + self.instance, self.username, self.repo) payload = {} if status is not None: @@ -353,10 +353,10 @@ class Pagure(object): """ if self.username is None: request_url = "{}/api/0/{}/issue/{}".format( - self.InstanceURL, self.repo, issue_id) + self.instance, self.repo, issue_id) else: request_url = "{}/api/0/fork/{}/{}/issue/{}".format( - self.InstanceURL, self.username, self.repo, + self.instance, self.username, self.repo, issue_id) return_value = self.__call_api(request_url) @@ -372,10 +372,10 @@ class Pagure(object): """ if self.username is None: request_url = "{}/api/0/{}/issue/{}/comment/{}".format( - self.InstanceURL, self.repo, issue_id, comment_id) + self.instance, self.repo, issue_id, comment_id) else: request_url = "{}/api/0/fork/{}/{}/issue/{}/comment/{}".format( - self.InstanceURL, self.username, self.repo, + self.instance, self.username, self.repo, issue_id, comment_id) return_value = self.__call_api(request_url) @@ -391,10 +391,10 @@ class Pagure(object): """ if self.username is None: request_url = "{}/api/0/{}/issue/{}/status".format( - self.InstanceURL, self.repo, issue_id) + self.instance, self.repo, issue_id) else: request_url = "{}/api/0/fork/{}/{}/issue/{}/status".format( - self.InstanceURL, self.username, self.repo, + self.instance, self.username, self.repo, issue_id) payload = {'status': new_status} @@ -413,10 +413,10 @@ class Pagure(object): """ if self.username is None: request_url = "{}/api/0/{}/issue/{}/comment".format( - self.InstanceURL, self.repo, issue_id) + self.instance, self.repo, issue_id) else: request_url = "{}/api/0/fork/{}/{}/issue/{}/comment".format( - self.InstanceURL, self.username, self.repo, + self.instance, self.username, self.repo, issue_id) payload = {'comment': body} @@ -433,10 +433,10 @@ class Pagure(object): """ if self.username is None: request_url = "{}/api/0/{}/git/tags".format( - self.InstanceURL, self.repo) + self.instance, self.repo) else: request_url = "{}/api/0/fork/{}/{}/git/tags".format( - self.InstanceURL, self.username, self.repo) + self.instance, self.username, self.repo) return_value = self.__call_api(request_url) @@ -450,7 +450,7 @@ class Pagure(object): :param fork: filters whether it is a fork (True) or not (False) :return: """ - request_url = "{}/api/0/projects".format(self.InstanceURL) + request_url = "{}/api/0/projects".format(self.instance) payload = {} if tags is not None: @@ -470,7 +470,7 @@ class Pagure(object): :param username: the username of the user to get info about :return: """ - request_url = "{}/api/0/user/{}".format(self.InstanceURL, username) + request_url = "{}/api/0/user/{}".format(self.instance, username) return_value = self.__call_api(request_url)