opendev/gerrit
Code Issues Proposed changes

Merge "Add extension points for adding download schemes and commands"

Browse Source
This commit is contained in:
David Pursehouse
2013-10-17 02:24:44 +00:00
committed by Gerrit Code Review
parent 5bba0b245e 29324b1fa6
commit ce423b139e
3 changed files with 71 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

33
gerrit-extension-api/src/main/java/com/google/gerrit/extensions/config/DownloadCommand.java Normal file
View File

@@ -0,0 +1,33 @@
// Copyright (C) 2013 The Android Open Source Project
//
// Licensed under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
// You may obtain a copy of the License at
//
// http://www.apache.org/licenses/LICENSE-2.0
//
// Unless required by applicable law or agreed to in writing, software
// distributed under the License is distributed on an "AS IS" BASIS,
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
// See the License for the specific language governing permissions and
// limitations under the License.
package com.google.gerrit.extensions.config;
import com.google.gerrit.extensions.annotations.ExtensionPoint;
@ExtensionPoint
public abstract class DownloadCommand {
/**
* Returns the download command for the given download scheme, project and
* ref.
*
* @param scheme the download scheme for which the command should be returned
* @param project the name of the project for which the download command
* should be returned
* @param ref the change ref
* @return the download command
*/
public abstract String getCommand(DownloadScheme scheme, String project,
String ref);
}

34
gerrit-extension-api/src/main/java/com/google/gerrit/extensions/config/DownloadScheme.java Normal file
View File

@@ -0,0 +1,34 @@
// Copyright (C) 2013 The Android Open Source Project
//
// Licensed under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
// You may obtain a copy of the License at
//
// http://www.apache.org/licenses/LICENSE-2.0
//
// Unless required by applicable law or agreed to in writing, software
// distributed under the License is distributed on an "AS IS" BASIS,
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
// See the License for the specific language governing permissions and
// limitations under the License.
package com.google.gerrit.extensions.config;
import com.google.gerrit.extensions.annotations.ExtensionPoint;
@ExtensionPoint
public abstract class DownloadScheme {
/**
* Returns the URL of this download scheme.
*
* @param project the name of the project for which the URL should be returned
* @return URL of the download scheme
*/
public abstract String getUrl(String project);
/** @return whether this scheme requires authentication */
public abstract boolean isAuthRequired();
/** @return whether the download scheme is enabled */
public abstract boolean isEnabled();
}

4
gerrit-server/src/main/java/com/google/gerrit/server/config/GerritGlobalModule.java
View File

@@ -20,6 +20,8 @@ import com.google.common.cache.Cache;
import com.google.gerrit.audit.AuditModule;
import com.google.gerrit.common.ChangeListener;
import com.google.gerrit.extensions.config.CapabilityDefinition;
import com.google.gerrit.extensions.config.DownloadCommand;
import com.google.gerrit.extensions.config.DownloadScheme;
import com.google.gerrit.extensions.events.GitReferenceUpdatedListener;
import com.google.gerrit.extensions.events.LifecycleListener;
import com.google.gerrit.extensions.events.NewProjectCreatedListener;
@@ -241,6 +243,8 @@ public class GerritGlobalModule extends FactoryModule {
DynamicItem.itemOf(binder(), AvatarProvider.class);
DynamicSet.setOf(binder(), LifecycleListener.class);
DynamicSet.setOf(binder(), TopMenu.class);
DynamicMap.mapOf(binder(), DownloadScheme.class);
DynamicMap.mapOf(binder(), DownloadCommand.class);
bind(AnonymousUser.class);