From 560f88201f1bce58dea681e4f6cd4639ee3cede1 Mon Sep 17 00:00:00 2001 From: farhaanbukhsh Date: Aug 11 2016 08:44:16 +0000 Subject: Fix pep8 issues --- diff --git a/pagure/ui/fork.py b/pagure/ui/fork.py index 22ca62b..61e8aa9 100644 --- a/pagure/ui/fork.py +++ b/pagure/ui/fork.py @@ -45,7 +45,8 @@ def _get_parent_request_repo_path(repo): provided Repository object from the DB. """ if repo.parent: - parentpath = os.path.join(APP.config['REQUESTS_FOLDER'], repo.parent.path) + parentpath = os.path.join( + APP.config['REQUESTS_FOLDER'], repo.parent.path) else: parentpath = os.path.join(APP.config['REQUESTS_FOLDER'], repo.path) diff --git a/pagure/ui/issues.py b/pagure/ui/issues.py index f8325ed..d1a6ba2 100644 --- a/pagure/ui/issues.py +++ b/pagure/ui/issues.py @@ -279,7 +279,7 @@ def edit_tag(repo, tag, username=None): tags = pagure.lib.get_tags_of_project(SESSION, repo) if not tags or tag not in [t.tag for t in tags]: - flask.abort(404, 'Tag %s not found in this project' % tag ) + flask.abort(404, 'Tag %s not found in this project' % tag) form = pagure.forms.AddIssueTagForm() if form.validate_on_submit(): @@ -554,7 +554,6 @@ def view_roadmap(repo, username=None): ) - @APP.route('//new_issue/', methods=('GET', 'POST')) @APP.route('//new_issue', methods=('GET', 'POST')) @APP.route('/fork///new_issue/', methods=('GET', 'POST')) @@ -650,7 +649,7 @@ def new_issue(repo, username=None): bail_on_tree=True) if default_file: default, _ = pagure.doc_utils.convert_readme( - default_file.data, 'md') + default_file.data, 'md') return flask.render_template( 'new_issue.html', diff --git a/pagure/ui/repo.py b/pagure/ui/repo.py index 60a01ba..67386de 100644 --- a/pagure/ui/repo.py +++ b/pagure/ui/repo.py @@ -200,7 +200,7 @@ def view_repo_branch(repo, branchname, username=None): orig_repo = pygit2.Repository(parentname) tree = None - safe=False + safe = False readme = None if not repo_obj.is_empty and not orig_repo.is_empty: @@ -227,7 +227,7 @@ def view_repo_branch(repo, branchname, username=None): break diff_commits.append(commit.oid.hex) - tree=sorted(last_commits[0].tree, key=lambda x: x.filemode) + tree = sorted(last_commits[0].tree, key=lambda x: x.filemode) for i in tree: name, ext = os.path.splitext(i.name) if name == 'README': @@ -555,7 +555,7 @@ def view_file(repo, identifier, filename, username=None): headers['Content-Encoding'] = encoding return ( - flask.render_template( + flask.render_template( 'file.html', select='tree', repo=repo, @@ -1027,7 +1027,7 @@ def view_settings(repo, username=None): tags=tags, plugins=plugins, repo_admin=repo_admin, - branchname = branchname, + branchname=branchname, ) @@ -1266,7 +1266,6 @@ def change_ref_head(repo, username=None): return flask.redirect( flask.url_for('auth_login', next=url)) - repo = pagure.lib.get_project(SESSION, repo, user=username, is_repo_admin) if not repo: @@ -1284,14 +1283,15 @@ def change_ref_head(repo, username=None): if form.validate_on_submit(): branchname = form.branches.data try: - reference = repo_obj.lookup_reference('refs/heads/%s'%branchname).resolve() + reference = repo_obj.lookup_reference( + 'refs/heads/%s' % branchname).resolve() repo_obj.set_head(reference.name) - flask.flash('Default branch updated to %s'%branchname) + flask.flash('Default branch updated to %s' % branchname) except Exception as err: # pragma: no cover APP.logger.exception(err) return flask.redirect(flask.url_for( - 'view_settings', username=username, repo=repo.name)) + 'view_settings', username=username, repo=repo.name)) @APP.route('//delete', methods=['POST']) @@ -1463,7 +1463,8 @@ def add_user(repo, username=None): """ if not pagure.APP.config.get('ENABLE_USER_MNGT', True): - flask.abort(404, 'User management is not allowed in this pagure instance') + flask.abort( + 404, 'User management is not allowed in this pagure instance') if admin_session_timedout(): if flask.request.method == 'POST': @@ -1521,7 +1522,8 @@ def remove_group_project(repo, groupid, username=None): """ if not pagure.APP.config.get('ENABLE_USER_MNGT', True): - flask.abort(404, 'User management is not allowed in this pagure instance') + flask.abort( + 404, 'User management is not allowed in this pagure instance') if admin_session_timedout(): flask.flash('Action canceled, try it again', 'error') @@ -1580,7 +1582,8 @@ def add_group_project(repo, username=None): """ if not pagure.APP.config.get('ENABLE_USER_MNGT', True): - flask.abort(404, 'User management is not allowed in this pagure instance') + flask.abort( + 404, 'User management is not allowed in this pagure instance') if admin_session_timedout(): if flask.request.method == 'POST':