Merge branch 'stable-2.12'
* stable-2.12: Update replication plugin with merge from stable-2.11 Correct RebaseChangeOp running hooks Set version to 2.11.5 Release notes for Gerrit 2.11.5 Update replication plugin revision Display latest result of branch list screen's filter Change-Id: Ie6b06da12c5bb46a65cd56f36808f8bfd35011a3
This commit is contained in:
commit
0a2d8cb9c3
89
ReleaseNotes/ReleaseNotes-2.11.5.txt
Normal file
89
ReleaseNotes/ReleaseNotes-2.11.5.txt
Normal file
@ -0,0 +1,89 @@
|
|||||||
|
Release notes for Gerrit 2.11.5
|
||||||
|
===============================
|
||||||
|
|
||||||
|
Gerrit 2.11.5 is now available:
|
||||||
|
|
||||||
|
link:https://gerrit-releases.storage.googleapis.com/gerrit-2.11.5.war[
|
||||||
|
https://gerrit-releases.storage.googleapis.com/gerrit-2.11.5.war]
|
||||||
|
|
||||||
|
There are no schema changes from link:ReleaseNotes-2.11.4.html[2.11.4].
|
||||||
|
|
||||||
|
|
||||||
|
Bug Fixes
|
||||||
|
---------
|
||||||
|
|
||||||
|
* link:https://code.google.com/p/gerrit/issues/detail?id=3442[Issue 3442]:
|
||||||
|
Handle commit validation errors when creating/editing changes via REST.
|
||||||
|
+
|
||||||
|
When an exception was thrown by a commit validator during creation of
|
||||||
|
a new change, or during publish of an inline edit, this resulted in an
|
||||||
|
internal server error message which did not include the actual reason
|
||||||
|
for the error.
|
||||||
|
|
||||||
|
* link:https://code.google.com/p/gerrit/issues/detail?id=3616[Issue 3616]:
|
||||||
|
Strip trailing blank lines from commit messages when modified in the inline
|
||||||
|
editor.
|
||||||
|
+
|
||||||
|
Blank lines were not trimmed from the end of commit messages, which caused
|
||||||
|
problems when the commit was merged and then cherry-picked with the `-x`
|
||||||
|
option (from the command line).
|
||||||
|
|
||||||
|
* Tweak JS clipboard API integration to work on Firefox.
|
||||||
|
+
|
||||||
|
The JS 'copy' functionality was working on Chrome, but not on Firefox.
|
||||||
|
|
||||||
|
* Use image instead of unicode character for copy button.
|
||||||
|
+
|
||||||
|
Some browsers were unable to render the unicode character.
|
||||||
|
|
||||||
|
* Include server config module in init step.
|
||||||
|
+
|
||||||
|
This allows SecureStore to be used during plugins' init step.
|
||||||
|
|
||||||
|
* link:https://code.google.com/p/gerrit/issues/detail?id=3659[Issue 3659]:
|
||||||
|
Show inline comments in change screen history when inline edit is active.
|
||||||
|
+
|
||||||
|
It was not possible to see the inline comments in the history on the
|
||||||
|
change screen when in edit mode.
|
||||||
|
|
||||||
|
* Improve rendering of `stream-events` tasks in the `show-queue` output.
|
||||||
|
+
|
||||||
|
Entries for `stream-events` are now rendered as 'Stream Events (username)'.
|
||||||
|
|
||||||
|
* link:https://code.google.com/p/gerrit/issues/detail?id=3655[Issue 3655]:
|
||||||
|
Fix incorrect owner group matching behavior.
|
||||||
|
+
|
||||||
|
When the given group did not match any group, the group was matched
|
||||||
|
on a group whose name starts with the argument, instead of throwing an
|
||||||
|
error to notify the user.
|
||||||
|
|
||||||
|
* link:https://code.google.com/p/gerrit/issues/detail?id=3664[Issue 3664]:
|
||||||
|
Fix double slash on URL when switching account.
|
||||||
|
+
|
||||||
|
One too many slashes on the URL caused redirection back to the root
|
||||||
|
page instead of the intended location.
|
||||||
|
|
||||||
|
* link:https://code.google.com/p/gerrit/issues/detail?id=3666[Issue 3666]:
|
||||||
|
Fix server error when commit validator is invoked on initial commit.
|
||||||
|
+
|
||||||
|
If a commit was uploaded for review as the first commit in a repository
|
||||||
|
that was created with no initial empty commit, invoking a commit validator
|
||||||
|
on the new commit would cause an internal error.
|
||||||
|
|
||||||
|
* Replication plugin.
|
||||||
|
|
||||||
|
** Parse replication delay and retry times as time units.
|
||||||
|
+
|
||||||
|
The replication delay and retry values were interpreted as seconds and
|
||||||
|
minutes respectively, but were being parsed as integers.
|
||||||
|
+
|
||||||
|
This is inconsistent with how time units are handled in other Gerrit
|
||||||
|
configuration settings, and can cause confusion when the user configures
|
||||||
|
them using the time unit syntax such as '15s' and it causes the plugin
|
||||||
|
to fail with 'invalid value'.
|
||||||
|
+
|
||||||
|
The delay and retry now are parsed as time units. The value can be given
|
||||||
|
in any recognized time unit, and the defaults remain the same as before;
|
||||||
|
15 seconds and 1 minute respectively.
|
||||||
|
|
||||||
|
** Remove documentation of obsolete `remote.NAME.timeout` setting.
|
@ -9,6 +9,7 @@ Version 2.12.x
|
|||||||
[[2_11]]
|
[[2_11]]
|
||||||
Version 2.11.x
|
Version 2.11.x
|
||||||
--------------
|
--------------
|
||||||
|
* link:ReleaseNotes-2.11.5.html[2.11.5]
|
||||||
* link:ReleaseNotes-2.11.4.html[2.11.4]
|
* link:ReleaseNotes-2.11.4.html[2.11.4]
|
||||||
* link:ReleaseNotes-2.11.3.html[2.11.3]
|
* link:ReleaseNotes-2.11.3.html[2.11.3]
|
||||||
* link:ReleaseNotes-2.11.2.html[2.11.2]
|
* link:ReleaseNotes-2.11.2.html[2.11.2]
|
||||||
|
@ -207,8 +207,10 @@ public class ProjectBranchesScreen extends PaginatedProjectScreen {
|
|||||||
Query q = new Query(filterTxt.getValue());
|
Query q = new Query(filterTxt.getValue());
|
||||||
if (match.equals(q.qMatch)) {
|
if (match.equals(q.qMatch)) {
|
||||||
q.start(start);
|
q.start(start);
|
||||||
} else if (query == null) {
|
} else {
|
||||||
|
if (query == null) {
|
||||||
q.run();
|
q.run();
|
||||||
|
}
|
||||||
query = q;
|
query = q;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
@ -25,6 +25,7 @@ import com.google.gerrit.reviewdb.client.RevId;
|
|||||||
import com.google.gerrit.server.ChangeUtil;
|
import com.google.gerrit.server.ChangeUtil;
|
||||||
import com.google.gerrit.server.git.BatchUpdate;
|
import com.google.gerrit.server.git.BatchUpdate;
|
||||||
import com.google.gerrit.server.git.BatchUpdate.ChangeContext;
|
import com.google.gerrit.server.git.BatchUpdate.ChangeContext;
|
||||||
|
import com.google.gerrit.server.git.BatchUpdate.Context;
|
||||||
import com.google.gerrit.server.git.BatchUpdate.RepoContext;
|
import com.google.gerrit.server.git.BatchUpdate.RepoContext;
|
||||||
import com.google.gerrit.server.git.MergeUtil;
|
import com.google.gerrit.server.git.MergeUtil;
|
||||||
import com.google.gerrit.server.git.validators.CommitValidators;
|
import com.google.gerrit.server.git.validators.CommitValidators;
|
||||||
@ -148,6 +149,11 @@ public class RebaseChangeOp extends BatchUpdate.Op {
|
|||||||
rebasedPatchSet = patchSetInserter.getPatchSet();
|
rebasedPatchSet = patchSetInserter.getPatchSet();
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public void postUpdate(Context ctx) throws OrmException {
|
||||||
|
patchSetInserter.postUpdate(ctx);
|
||||||
|
}
|
||||||
|
|
||||||
public PatchSet getPatchSet() {
|
public PatchSet getPatchSet() {
|
||||||
checkState(rebasedPatchSet != null,
|
checkState(rebasedPatchSet != null,
|
||||||
"getPatchSet() only valid after executing update");
|
"getPatchSet() only valid after executing update");
|
||||||
|
Loading…
Reference in New Issue
Block a user