opendev/gerrit
Code Issues Proposed changes

Merge "Link to owner query from user name in ChangeTable2"

Browse Source
This commit is contained in:
Edwin Kempin
2012-05-10 15:27:54 -07:00
committed by gerrit code review
parent e1c8240046 c7426fd77c
commit a9bbb45370
2 changed files with 8 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
gerrit-common/src/main/java/com/google/gerrit/common/PageLinks.java
View File

@@ -59,6 +59,10 @@ public class PageLinks {
return "/admin/projects/" + p.get() + ",access";
}
public static String toAccountQuery(final String fullname) {
return "/q/owner:\"" + KeyUtil.encode(fullname) + "\"," + TOP;
}
public static String toAccountDashboard(final AccountInfo acct) {
return toAccountDashboard(acct.getId());
}

5
gerrit-gwtui/src/main/java/com/google/gerrit/client/changes/ChangeTable2.java
View File

@@ -20,6 +20,7 @@ import com.google.gerrit.client.Gerrit;
import com.google.gerrit.client.changes.ChangeInfo.LabelInfo;
import com.google.gerrit.client.ui.BranchLink;
import com.google.gerrit.client.ui.ChangeLink;
import com.google.gerrit.client.ui.InlineHyperlink;
import com.google.gerrit.client.ui.NavigationTable;
import com.google.gerrit.client.ui.NeedsSignInKeyCommand;
import com.google.gerrit.client.ui.ProjectLink;
@@ -209,7 +210,9 @@ public class ChangeTable2 extends NavigationTable<ChangeInfo> {
if (c.owner() != null && c.owner().name() != null) {
owner = c.owner().name();
}
table.setText(row, C_OWNER, owner);
table.setWidget(row, C_OWNER, new InlineHyperlink(owner,
PageLinks.toAccountQuery(owner)));
table.setWidget(
row, C_PROJECT, new ProjectLink(c.project_name_key(), c.status()));