From 1aec17636d8c548257cb307f8cc1e41354e00421 Mon Sep 17 00:00:00 2001 From: marco <mw.sunshine@gmx.de> Date: Mon, 23 Nov 2015 13:58:14 -0500 Subject: [PATCH] Fix 2 possible NullPointer occurences --- src/main/java/com/gitblit/wicket/pages/RepositoryPage.java | 5 ++++- 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/src/main/java/com/gitblit/wicket/pages/RepositoryPage.java b/src/main/java/com/gitblit/wicket/pages/RepositoryPage.java index 9639a0c..7e164a8 100644 --- a/src/main/java/com/gitblit/wicket/pages/RepositoryPage.java +++ b/src/main/java/com/gitblit/wicket/pages/RepositoryPage.java @@ -64,6 +64,7 @@ import com.gitblit.utils.BugtraqProcessor; import com.gitblit.utils.DeepCopier; import com.gitblit.utils.JGitUtils; +import com.gitblit.utils.ModelUtils; import com.gitblit.utils.RefLogUtils; import com.gitblit.utils.StringUtils; import com.gitblit.wicket.CacheControl; @@ -370,8 +371,10 @@ add(new ExternalLink("myForkLink", "").setVisible(false)); } else { String fork = app().repositories().getFork(user.username, model.name); + String userRepo = ModelUtils.getPersonalPath(user.username) + "/" + StringUtils.stripDotGit(StringUtils.getLastPathElement(model.name)); + boolean hasUserRepo = app().repositories().hasRepository(userRepo); boolean hasFork = fork != null; - boolean canFork = user.canFork(model) && model.hasCommits; + boolean canFork = user.canFork(model) && model.hasCommits && !hasUserRepo; if (hasFork || !canFork) { // user not allowed to fork or fork already exists or repo forbids forking -- Gitblit v1.9.1