From 5a3a59bc6c6f0a7fd123b6bb8db8cd91bd4d0b46 Mon Sep 17 00:00:00 2001 From: farhaanbukhsh Date: Aug 11 2016 08:44:16 +0000 Subject: Fix test for repo and fork ui `get_project` function was left in the file which got caught because of tests and the other function an extra parameter `username` was passed which was not there. --- diff --git a/pagure/ui/fork.py b/pagure/ui/fork.py index a7fa38d..0621ad4 100644 --- a/pagure/ui/fork.py +++ b/pagure/ui/fork.py @@ -847,8 +847,8 @@ def fork_project(repo, username=None): if repo is None: flask.abort(404) - if pagure.lib.get_project( - SESSION, repo.name, is_repo_admin, user=flask.g.fas_user.username): + if get_authorized_project( + SESSION, repo.name, user=flask.g.fas_user.username): flask.flash('You had already forked this project') return flask.redirect(flask.url_for( 'view_repo', repo=repo.name, username=flask.g.fas_user.username diff --git a/pagure/ui/repo.py b/pagure/ui/repo.py index 23e0d49..f42e6d8 100644 --- a/pagure/ui/repo.py +++ b/pagure/ui/repo.py @@ -1957,7 +1957,7 @@ def view_project_activity(repo): if not APP.config.get('DATAGREPPER_URL'): flask.abort(404) - repo_obj = get_authorized_project(SESSION, repo, user=username) + repo_obj = get_authorized_project(SESSION, repo) if not repo_obj: flask.abort(404, 'Project not found')