opendev/gerrit
Code Issues Proposed changes

Trim ProjectApiImpl

Browse Source 
Do not need inject CreateBranch.Factory, delete it.
Rename ProjectResource parameter from 'change' to 'project'.

Change-Id: I039d07fa4499bdc6acce96437cf884be6431a049
This commit is contained in:
Bruce Zu
2013-11-26 15:37:53 +08:00
parent abce5ad014
commit 9959e6380e
1 changed files with 1 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
gerrit-server/src/main/java/com/google/gerrit/server/api/projects/ProjectApiImpl.java
View File

@@ -16,14 +16,13 @@ package com.google.gerrit.server.api.projects;
import com.google.gerrit.extensions.api.projects.BranchApi;
import com.google.gerrit.extensions.api.projects.ProjectApi;
import com.google.gerrit.server.project.CreateBranch;
import com.google.gerrit.server.project.ProjectResource;
import com.google.inject.Inject;
import com.google.inject.assistedinject.Assisted;
public class ProjectApiImpl implements ProjectApi {
interface Factory {
ProjectApiImpl create(ProjectResource change);
ProjectApiImpl create(ProjectResource project);
}
private final ProjectResource project;
@@ -31,7 +30,6 @@ public class ProjectApiImpl implements ProjectApi {
@Inject
ProjectApiImpl(
CreateBranch.Factory createBranchFactory,
BranchApiImpl.Factory branchApiFactory,
@Assisted ProjectResource project) {
this.project = project;