Merge "Add parent(s) revision information to output of query."
This commit is contained in:
@@ -39,6 +39,8 @@ import com.google.inject.Provider;
|
|||||||
import com.google.inject.Singleton;
|
import com.google.inject.Singleton;
|
||||||
|
|
||||||
import org.eclipse.jgit.lib.ObjectId;
|
import org.eclipse.jgit.lib.ObjectId;
|
||||||
|
import org.slf4j.Logger;
|
||||||
|
import org.slf4j.LoggerFactory;
|
||||||
|
|
||||||
import java.util.ArrayList;
|
import java.util.ArrayList;
|
||||||
import java.util.Collection;
|
import java.util.Collection;
|
||||||
@@ -48,6 +50,7 @@ import javax.annotation.Nullable;
|
|||||||
|
|
||||||
@Singleton
|
@Singleton
|
||||||
public class EventFactory {
|
public class EventFactory {
|
||||||
|
private static final Logger log = LoggerFactory.getLogger(EventFactory.class);
|
||||||
private final AccountCache accountCache;
|
private final AccountCache accountCache;
|
||||||
private final Provider<String> urlProvider;
|
private final Provider<String> urlProvider;
|
||||||
private final ApprovalTypes approvalTypes;
|
private final ApprovalTypes approvalTypes;
|
||||||
@@ -273,6 +276,20 @@ public class EventFactory {
|
|||||||
p.ref = patchSet.getRefName();
|
p.ref = patchSet.getRefName();
|
||||||
p.uploader = asAccountAttribute(patchSet.getUploader());
|
p.uploader = asAccountAttribute(patchSet.getUploader());
|
||||||
p.createdOn = patchSet.getCreatedOn().getTime() / 1000L;
|
p.createdOn = patchSet.getCreatedOn().getTime() / 1000L;
|
||||||
|
try {
|
||||||
|
final ReviewDb db = schema.open();
|
||||||
|
try {
|
||||||
|
p.parents = new ArrayList<String>();
|
||||||
|
for (PatchSetAncestor a : db.patchSetAncestors().ancestorsOf(
|
||||||
|
patchSet.getId())) {
|
||||||
|
p.parents.add(a.getAncestorRevision().get());
|
||||||
|
}
|
||||||
|
} finally {
|
||||||
|
db.close();
|
||||||
|
}
|
||||||
|
} catch (OrmException e) {
|
||||||
|
log.error("Cannot load patch set data for " + patchSet.getId(), e);
|
||||||
|
}
|
||||||
return p;
|
return p;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -19,6 +19,7 @@ import java.util.List;
|
|||||||
public class PatchSetAttribute {
|
public class PatchSetAttribute {
|
||||||
public String number;
|
public String number;
|
||||||
public String revision;
|
public String revision;
|
||||||
|
public List<String> parents;
|
||||||
public String ref;
|
public String ref;
|
||||||
public AccountAttribute uploader;
|
public AccountAttribute uploader;
|
||||||
public Long createdOn;
|
public Long createdOn;
|
||||||
|
|||||||
Reference in New Issue
Block a user