From ba3ab88ffcda141b4707c3fdbe25b977df69f69b Mon Sep 17 00:00:00 2001 From: Pierre-Yves Chibon Date: Nov 30 2015 11:55:09 +0000 Subject: [PATCH 1/2] If you try to fork a forked project, redirect the user to the fork --- diff --git a/pagure/ui/fork.py b/pagure/ui/fork.py index 2f55525..b531c41 100644 --- a/pagure/ui/fork.py +++ b/pagure/ui/fork.py @@ -725,6 +725,13 @@ def fork_project(repo, username=None): if repo is None: flask.abort(404) + if pagure.lib.get_project( + SESSION, repo.name, user=flask.g.fas_user.username): + flask.flash('You have already forked this project', 'error') + return flask.redirect(flask.url_for( + 'view_repo', repo=repo.name, username=flask.g.fas_user.username + )) + try: message = pagure.lib.fork_project( session=SESSION, From 59554a1cd87a3fdd3f8c1d29317adbd964efd361 Mon Sep 17 00:00:00 2001 From: Pierre-Yves Chibon Date: Nov 30 2015 12:01:52 +0000 Subject: [PATCH 2/2] Let's inform the user and change the tense of the message --- diff --git a/pagure/ui/fork.py b/pagure/ui/fork.py index b531c41..b92b63e 100644 --- a/pagure/ui/fork.py +++ b/pagure/ui/fork.py @@ -727,7 +727,7 @@ def fork_project(repo, username=None): if pagure.lib.get_project( SESSION, repo.name, user=flask.g.fas_user.username): - flask.flash('You have already forked this project', 'error') + 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 ))