opendev/gerrit
Code Issues Proposed changes

Merge "Split up canSubmit()"

Browse Source
This commit is contained in:
Saša Živkov
2012-10-10 01:24:48 -07:00
committed by Gerrit Code Review
parent 6bb9d0e90d 7fcb0f2dce
commit 86b13f8068
1 changed files with 12 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
gerrit-server/src/main/java/com/google/gerrit/server/project/ChangeControl.java
View File

@@ -443,12 +443,18 @@ public class ChangeControl {
return ruleError("Project submit rule has no solution");
}
// Convert the results from Prolog Cafe's format to Gerrit's common format.
// can_submit/1 terminates when an ok(P) record is found. Therefore walk
// the results backwards, using only that ok(P) record if it exists. This
// skips partial results that occur early in the output. Later after the loop
// the out collection is reversed to restore it to the original ordering.
//
return resultsToSubmitRecord(submitRule, results);
}
/**
* Convert the results from Prolog Cafe's format to Gerrit's common format.
*
* can_submit/1 terminates when an ok(P) record is found. Therefore walk
* the results backwards, using only that ok(P) record if it exists. This
* skips partial results that occur early in the output. Later after the loop
* the out collection is reversed to restore it to the original ordering.
*/
public List<SubmitRecord> resultsToSubmitRecord(Term submitRule, List<Term> results) {
List<SubmitRecord> out = new ArrayList<SubmitRecord>(results.size());
for (int resultIdx = results.size() - 1; 0 <= resultIdx; resultIdx--) {
Term submitRecord = results.get(resultIdx);