opendev/gerrit
Code Issues Proposed changes

Merge "Make "My Drafts" limited to own drafts"

Browse Source
This commit is contained in:
Edwin Kempin 2015-01-19 13:41:46 +00:00 committed by Gerrit Code Review
commit b49af60c23
5 changed files with 7 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
Documentation/rest-api-accounts.txt
View File

@ -909,7 +909,7 @@ link:#preferences-info[PreferencesInfo] entity.
"name": "Changes" "name": "Changes"
}, },
{ {
"url": "#/q/is:draft", "url": "#/q/owner:self+is:draft",
"name": "Drafts" "name": "Drafts"
}, },
{ {
@ -963,7 +963,7 @@ link:#preferences-input[PreferencesInput] entity.
"name": "Changes" "name": "Changes"
}, },
{ {
"url": "#/q/is:draft", "url": "#/q/owner:self+is:draft",
"name": "Drafts" "name": "Drafts"
}, },
{ {
@ -1011,7 +1011,7 @@ link:#preferences-info[PreferencesInfo] entity.
"name": "Changes" "name": "Changes"
}, },
{ {
"url": "#/q/is:draft", "url": "#/q/owner:self+is:draft",
"name": "Drafts" "name": "Drafts"
}, },
{ {

2
Documentation/user-search.txt
View File

@ -14,7 +14,7 @@ matches the search, the change will be presented instead of a list.
|All > Open | status:open '(or is:open)' |All > Open | status:open '(or is:open)'
|All > Merged | status:merged |All > Merged | status:merged
|All > Abandoned | status:abandoned |All > Abandoned | status:abandoned
|My > Drafts | is:draft |My > Drafts | owner:self is:draft
|My > Watched Changes | status:open is:watched |My > Watched Changes | status:open is:watched
|My > Starred Changes | is:starred |My > Starred Changes | is:starred
|My > Draft Comments | has:draft |My > Draft Comments | has:draft

2
gerrit-gwtui/src/main/java/com/google/gerrit/client/Dispatcher.java
View File

@ -284,7 +284,7 @@ public class Dispatcher {
} }
if (matchExact("mine,drafts", token)) { if (matchExact("mine,drafts", token)) {
return toChangeQuery("is:draft"); return toChangeQuery("owner:self is:draft");
} }
if (matchExact("mine,comments", token)) { if (matchExact("mine,comments", token)) {

2
gerrit-gwtui/src/main/java/com/google/gerrit/client/JumpKeys.java
View File

@ -69,7 +69,7 @@ public class JumpKeys {
jumps.add(new KeyCommand(0, 'd', Gerrit.C.jumpMineDrafts()) { jumps.add(new KeyCommand(0, 'd', Gerrit.C.jumpMineDrafts()) {
@Override @Override
public void onKeyPress(final KeyPressEvent event) { public void onKeyPress(final KeyPressEvent event) {
Gerrit.display(PageLinks.toChangeQuery("is:draft")); Gerrit.display(PageLinks.toChangeQuery("owner:self is:draft"));
} }
}); });
jumps.add(new KeyCommand(0, 'c', Gerrit.C.jumpMineDraftComments()) { jumps.add(new KeyCommand(0, 'c', Gerrit.C.jumpMineDraftComments()) {

2
gerrit-server/src/main/java/com/google/gerrit/server/account/GetPreferences.java
View File

@ -147,7 +147,7 @@ public class GetPreferences implements RestReadView<AccountResource> {
} }
if (my.isEmpty()) { if (my.isEmpty()) {
my.add(new TopMenu.MenuItem("Changes", "#/dashboard/self", null)); my.add(new TopMenu.MenuItem("Changes", "#/dashboard/self", null));
my.add(new TopMenu.MenuItem("Drafts", "#/q/is:draft", null)); my.add(new TopMenu.MenuItem("Drafts", "#/q/owner:self+is:draft", null));
my.add(new TopMenu.MenuItem("Draft Comments", "#/q/has:draft", null)); my.add(new TopMenu.MenuItem("Draft Comments", "#/q/has:draft", null));
my.add(new TopMenu.MenuItem("Watched Changes", "#/q/is:watched+is:open", null)); my.add(new TopMenu.MenuItem("Watched Changes", "#/q/is:watched+is:open", null));
my.add(new TopMenu.MenuItem("Starred Changes", "#/q/is:starred", null)); my.add(new TopMenu.MenuItem("Starred Changes", "#/q/is:starred", null));