summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMika Hamalainen <mika.hamalainen@accenture.com>2011-09-09 12:25:37 +0300
committerMika Hamalainen <mika.hamalainen@accenture.com>2011-09-09 12:25:37 +0300
commitc5f60b28850ab3e1c9d55a7a657056eb4a7508b0 (patch)
treeed4c82641f63e74f954fcbbe855eeee5d205fd22
parentd55d96d122e34a855e2e4df98302f37b1207e5b9 (diff)
Fixed cherry pick options in All-Projects
Cherry pick options was previously visible also in the All-Projects view. Fixed visibility for cherry pick options panel so that it is visible only for actual projects. Change-Id: I9b9fb131eba7b051a8a00ca1d248225622b44a0b
-rw-r--r--gerrit-gwtui/src/main/java/com/google/gerrit/client/admin/ProjectInfoScreen.java12
1 files changed, 9 insertions, 3 deletions
diff --git a/gerrit-gwtui/src/main/java/com/google/gerrit/client/admin/ProjectInfoScreen.java b/gerrit-gwtui/src/main/java/com/google/gerrit/client/admin/ProjectInfoScreen.java
index 07357ff3c6..d6bca78767 100644
--- a/gerrit-gwtui/src/main/java/com/google/gerrit/client/admin/ProjectInfoScreen.java
+++ b/gerrit-gwtui/src/main/java/com/google/gerrit/client/admin/ProjectInfoScreen.java
@@ -189,10 +189,14 @@ public class ProjectInfoScreen extends ProjectScreen {
final boolean isCherryPickSubmitType =
SubmitType.CHERRY_PICK.equals(Project.SubmitType.valueOf(
submitType.getValue(submitType.getSelectedIndex())));
- includeReviewedOn.setEnabled(isCherryPickSubmitType);
- includeOnlyMaxApproval.setEnabled(isCherryPickSubmitType);
+ enableCherryPickOptions(isCherryPickSubmitType);
+ }
+
+ private void enableCherryPickOptions(final boolean enable) {
+ includeReviewedOn.setEnabled(enable);
+ includeOnlyMaxApproval.setEnabled(enable);
for (CheckBox checkBox : approvalsInFooter.values()) {
- checkBox.setEnabled(isCherryPickSubmitType);
+ checkBox.setEnabled(enable);
}
}
@@ -263,6 +267,7 @@ public class ProjectInfoScreen extends ProjectScreen {
includeOnlyMaxApproval.setValue(project.isIncludeOnlyMaxApproval());
includeReviewedOn.setValue(!project.isHideReviewedOn());
+ cherryPickPanel.setVisible(!isall);
Map<String, Boolean> hiddenFooters = project.getHiddenFooters();
for (Entry<String, CheckBox> entry : approvalsInFooter.entrySet()) {
@@ -297,6 +302,7 @@ public class ProjectInfoScreen extends ProjectScreen {
}
enableForm(false, false, false);
+ enableCherryPickOptions(false);
Util.PROJECT_SVC.changeProjectSettings(project,
new GerritCallback<ProjectDetail>() {