opendev/gerrit
Code Issues Proposed changes

Merge "Rename PluginConfigProvider to PluginConfigFactory."

Browse Source
This commit is contained in:
Edwin Kempin
2013-09-20 12:11:29 +00:00
committed by Gerrit Code Review
parent 7c4423c158 acdf533ca7
commit 759e9a042a
2 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
Documentation/dev-plugins.txt
View File

@@ -351,14 +351,14 @@ only use the `plugin` subsection with their own name. For example the
language = Latin
----
Via the `com.google.gerrit.server.config.PluginConfigProvider` class a
Via the `com.google.gerrit.server.config.PluginConfigFactory` class a
plugin can easily access its configuration and there is no need for a
plugin to parse the `gerrit.config` file on its own:
[source,java]
----
@Inject
private com.google.gerrit.server.config.PluginConfigProvider cfg;
private com.google.gerrit.server.config.PluginConfigFactory cfg;
...

4
gerrit-server/src/main/java/com/google/gerrit/server/config/PluginConfigProvider.java → gerrit-server/src/main/java/com/google/gerrit/server/config/PluginConfigFactory.java
View File

@@ -20,11 +20,11 @@ import com.google.inject.Singleton;
import org.eclipse.jgit.lib.Config;
@Singleton
public class PluginConfigProvider {
public class PluginConfigFactory {
private final Config cfg;
@Inject
PluginConfigProvider(@GerritServerConfig Config cfg) {
PluginConfigFactory(@GerritServerConfig Config cfg) {
this.cfg = cfg;
}