opendev/gerrit
Code Issues Proposed changes

Fix Schema_86 to use correct UUID for Change Owner group

Browse Source 
The constant UUID must be used, not a generated UUID.

Change-Id: I83e330b999f32794fa051f6325528962a9a8e9fd
This commit is contained in:
Shawn Pearce 2013-12-02 10:30:01 -08:00
parent 5081c306ea
commit 7f1bb28e9a
1 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
gerrit-server/src/main/java/com/google/gerrit/server/schema/Schema_86.java
View File

@ -22,7 +22,6 @@ import com.google.gerrit.reviewdb.client.AccountGroup;
import com.google.gerrit.reviewdb.client.AccountGroupName;
import com.google.gerrit.reviewdb.server.ReviewDb;
import com.google.gerrit.server.GerritPersonIdent;
import com.google.gerrit.server.account.GroupUUID;
import com.google.gerrit.server.config.AllProjectsName;
import com.google.gerrit.server.extensions.events.GitReferenceUpdated;
import com.google.gerrit.server.git.GitRepositoryManager;
@ -87,8 +86,8 @@ public class Schema_86 extends SchemaVersion {
AccountGroup.UUID adminGroupUUID, String description) throws OrmException {
AccountGroup.Id groupId = new AccountGroup.Id(db.nextAccountGroupId());
AccountGroup.NameKey nameKey = new AccountGroup.NameKey(groupName);
AccountGroup.UUID uuid = GroupUUID.make(groupName, serverUser);
AccountGroup group = new AccountGroup(nameKey, groupId, uuid);
AccountGroup group =
new AccountGroup(nameKey, groupId, AccountGroup.CHANGE_OWNER);
group.setOwnerGroupUUID(adminGroupUUID);
group.setDescription(description);
group.setType(AccountGroup.Type.SYSTEM);