From cacf8bff097fbb66a7be1bfe267b5da2605145f8 Mon Sep 17 00:00:00 2001 From: James Moger <james.moger@gitblit.com> Date: Fri, 29 Nov 2013 11:05:46 -0500 Subject: [PATCH] Use Dagger to inject managers into all filters and servlets --- src/main/java/com/gitblit/git/GitblitReceivePack.java | 22 +++++++++++++--------- 1 files changed, 13 insertions(+), 9 deletions(-) diff --git a/src/main/java/com/gitblit/git/GitblitReceivePack.java b/src/main/java/com/gitblit/git/GitblitReceivePack.java index 8da603a..44e2bef 100644 --- a/src/main/java/com/gitblit/git/GitblitReceivePack.java +++ b/src/main/java/com/gitblit/git/GitblitReceivePack.java @@ -91,11 +91,20 @@ protected GroovyScriptEngine gse; - public GitblitReceivePack(Repository db, RepositoryModel repository, UserModel user) { + private final IStoredSettings settings; + + private final IRepositoryManager repositoryManager; + + public GitblitReceivePack( + IRuntimeManager runtimeManager, + IRepositoryManager repositoryManager, + Repository db, + RepositoryModel repository, + UserModel user) { + super(db); - - IRepositoryManager repositoryManager = GitBlit.getManager(IRepositoryManager.class); - + this.settings = runtimeManager.getSettings(); + this.repositoryManager = repositoryManager; this.repository = repository; this.user = user == null ? UserModel.ANONYMOUS : user; this.groovyDir = repositoryManager.getHooksFolder(); @@ -239,7 +248,6 @@ } } - IRepositoryManager repositoryManager = GitBlit.getManager(IRepositoryManager.class); Set<String> scripts = new LinkedHashSet<String>(); scripts.addAll(repositoryManager.getPreReceiveScriptsInherited(repository)); if (!ArrayUtils.isEmpty(repository.preReceiveScripts)) { @@ -265,8 +273,6 @@ LOGGER.debug("skipping post-receive hooks, no refs created, updated, or removed"); return; } - - IStoredSettings settings = GitBlit.getManager(IRuntimeManager.class).getSettings(); // log ref changes for (ReceiveCommand cmd : commands) { @@ -335,8 +341,6 @@ } catch (Exception e) { LOGGER.error(MessageFormat.format("Failed to update {0} pushlog", repository.name), e); } - - IRepositoryManager repositoryManager = GitBlit.getManager(IRepositoryManager.class); // run Groovy hook scripts Set<String> scripts = new LinkedHashSet<String>(); -- Gitblit v1.9.1