opendev/gerrit
Code Issues Proposed changes

Merge branch 'stable'

Browse Source 
* stable:
  Fix ChangeDetailFactory's invocation of PatchSetDetailFactory
  Release notes for 2.1.7.1
  Fix API breakage on ChangeDetailService
  Do not reset Patch History selection on navigation to next file diff
  Resolve Project Owners when checking access right on any ref

Conflicts:
	gerrit-server/src/main/java/com/google/gerrit/server/project/ProjectControl.java
	gerrit-server/src/test/java/com/google/gerrit/server/project/RefControlTest.java

Change-Id: I6e956625cb4648df35035b9be2d32e6e431fb8f3
This commit is contained in:
Shawn O. Pearce
2011-06-07 13:44:39 -07:00
parent 1118daf779 019d2c4dfb
commit 080a51f4e6
9 changed files with 82 additions and 21 deletions
Download Patch File Download Diff File Expand all files Collapse all files

32
ReleaseNotes/ReleaseNotes-2.1.7.2.txt Normal file
View File

@@ -0,0 +1,32 @@
Release notes for Gerrit 2.1.7.1
================================
Gerrit 2.1.7.1 is now available:
link:http://code.google.com/p/gerrit/downloads/detail?name=gerrit-2.1.7.1.war[http://code.google.com/p/gerrit/downloads/detail?name=gerrit-2.1.7.1.war]
Bug Fixes
---------
* issue 997 Resolve Project Owners when checking access rights
+
Members of the 'Project Owners' magical group did not always have
their project owner privileges when Gerrit Code Review was looking
for "access to any ref" at the project level. This was caused by
not expanding the 'Project Owner's group to the actual ownership
list. Fixed.
* issue 999 Do not reset Patch History selection on navigation
+
Navigating to the next/previous file lost the setting of the
"Old Version" made under the "Patch History" expandable control
on a specific file view. This was accidentally broken when the
"Old Version History" control was added to the change page. Fixed.
* Fix API breakage on ChangeDetailService
+
Version 2.1.7 broke the Gerrit Code Review plugin for Mylyn Reviews
due to an accidential signature change of one of the remote JSON
APIs. The ChangeDetailService.patchSetDetail() method is back to the
old signature and a new patchSetDetail2() method has been added to
handle the newer calling convention used in some contexts of the
web UI.

1
ReleaseNotes/index.txt
View File

@@ -9,6 +9,7 @@ Version 2.2.x
[[2_1]]
Version 2.1.x
-------------
* link:ReleaseNotes-2.1.7.2.html[2.1.7.2],
* link:ReleaseNotes-2.1.7.html[2.1.7],
* link:ReleaseNotes-2.1.6.html[2.1.6],
link:ReleaseNotes-2.1.6.1.html[2.1.6.1]