Merge branch 'stable-2.14' * stable-2.14: Fix java.lang.ArrayIndexOutOfBoundsException when checking for parent ProjectTagsScreen: Fix title of revision field for new tag Fix cyclic dependency when using site_path from system_config table Change-Id: I1f4ed09818885284b9686d38705a20cf21cfce23
diff --git a/gerrit-gwtui/src/main/java/com/google/gerrit/client/admin/AdminConstants.java b/gerrit-gwtui/src/main/java/com/google/gerrit/client/admin/AdminConstants.java index 1aecd08..36119d0 100644 --- a/gerrit-gwtui/src/main/java/com/google/gerrit/client/admin/AdminConstants.java +++ b/gerrit-gwtui/src/main/java/com/google/gerrit/client/admin/AdminConstants.java
@@ -181,6 +181,8 @@ String initialRevision(); + String revision(); + String buttonAddBranch(); String buttonDeleteBranch();
diff --git a/gerrit-gwtui/src/main/java/com/google/gerrit/client/admin/AdminConstants.properties b/gerrit-gwtui/src/main/java/com/google/gerrit/client/admin/AdminConstants.properties index 4c7153e..ba3175b 100644 --- a/gerrit-gwtui/src/main/java/com/google/gerrit/client/admin/AdminConstants.properties +++ b/gerrit-gwtui/src/main/java/com/google/gerrit/client/admin/AdminConstants.properties
@@ -86,6 +86,7 @@ columnTagName = Tag Name columnTagRevision = Revision initialRevision = Initial Revision +revision = Revision buttonAddBranch = Create Branch buttonAddTag = Create Tag buttonDeleteBranch = Delete
diff --git a/gerrit-gwtui/src/main/java/com/google/gerrit/client/admin/ProjectTagsScreen.java b/gerrit-gwtui/src/main/java/com/google/gerrit/client/admin/ProjectTagsScreen.java index 63433b9..57a6c3c 100644 --- a/gerrit-gwtui/src/main/java/com/google/gerrit/client/admin/ProjectTagsScreen.java +++ b/gerrit-gwtui/src/main/java/com/google/gerrit/client/admin/ProjectTagsScreen.java
@@ -150,7 +150,7 @@ } } }); - addGrid.setText(1, 0, AdminConstants.I.initialRevision() + ":"); + addGrid.setText(1, 0, AdminConstants.I.revision() + ":"); addGrid.setWidget(1, 1, irevTxtBox); addTag = new Button(AdminConstants.I.buttonAddTag());
diff --git a/gerrit-server/src/main/java/com/google/gerrit/server/change/ChangeKindCacheImpl.java b/gerrit-server/src/main/java/com/google/gerrit/server/change/ChangeKindCacheImpl.java index 7a6c209..14b7207 100644 --- a/gerrit-server/src/main/java/com/google/gerrit/server/change/ChangeKindCacheImpl.java +++ b/gerrit-server/src/main/java/com/google/gerrit/server/change/ChangeKindCacheImpl.java
@@ -242,7 +242,7 @@ } if ((prior.getParentCount() != 1 || next.getParentCount() != 1) - && (!onlyFirstParentChanged(prior, next) || prior.getParentCount() == 0)) { + && (prior.getParentCount() == 0 || !onlyFirstParentChanged(prior, next))) { // Trivial rebases done by machine only work well on 1 parent. return ChangeKind.REWORK; }
diff --git a/gerrit-war/src/main/java/com/google/gerrit/httpd/SitePathFromSystemConfigProvider.java b/gerrit-war/src/main/java/com/google/gerrit/httpd/SitePathFromSystemConfigProvider.java index 0535f26..e1eb9de 100644 --- a/gerrit-war/src/main/java/com/google/gerrit/httpd/SitePathFromSystemConfigProvider.java +++ b/gerrit-war/src/main/java/com/google/gerrit/httpd/SitePathFromSystemConfigProvider.java
@@ -17,6 +17,7 @@ import com.google.gerrit.reviewdb.client.SystemConfig; import com.google.gerrit.reviewdb.server.ReviewDb; import com.google.gerrit.server.config.SitePath; +import com.google.gerrit.server.schema.ReviewDbFactory; import com.google.gwtorm.server.OrmException; import com.google.gwtorm.server.SchemaFactory; import com.google.inject.Inject; @@ -30,7 +31,8 @@ private final Path path; @Inject - SitePathFromSystemConfigProvider(SchemaFactory<ReviewDb> schemaFactory) throws OrmException { + SitePathFromSystemConfigProvider(@ReviewDbFactory SchemaFactory<ReviewDb> schemaFactory) + throws OrmException { path = read(schemaFactory); }