summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorNasser Grainawi <nasser@codeaurora.org>2021-05-27 19:30:57 +0000
committerGerrit Code Review <noreply-gerritcodereview@google.com>2021-05-27 19:30:57 +0000
commit2161cc67a15375204da88675ed9c2831e8045fee (patch)
treebd299d0851c3c2f3d8e65dd35dc1dcbb1491c9c6
parent2151203b26508f972ab751ac6cf589748c6570e1 (diff)
parent4f2c9711e41c3a18d9700edcfd21f62ae6db6691 (diff)
Merge "Merge branch 'stable-3.1' into stable-3.2" into stable-3.2
-rw-r--r--java/com/google/gerrit/server/query/change/ChangeData.java2
-rw-r--r--java/com/google/gerrit/server/query/change/EqualsLabelPredicate.java10
2 files changed, 10 insertions, 2 deletions
diff --git a/java/com/google/gerrit/server/query/change/ChangeData.java b/java/com/google/gerrit/server/query/change/ChangeData.java
index 69f1a4e922..c74150694c 100644
--- a/java/com/google/gerrit/server/query/change/ChangeData.java
+++ b/java/com/google/gerrit/server/query/change/ChangeData.java
@@ -1011,7 +1011,7 @@ public class ChangeData {
}
draftsByUser = new HashMap<>();
- for (Ref ref : commentsUtil.getDraftRefs(notes.getChangeId())) {
+ for (Ref ref : commentsUtil.getDraftRefs(notes().getChangeId())) {
Account.Id account = Account.Id.fromRefSuffix(ref.getName());
if (account != null
// Double-check that any drafts exist for this user after
diff --git a/java/com/google/gerrit/server/query/change/EqualsLabelPredicate.java b/java/com/google/gerrit/server/query/change/EqualsLabelPredicate.java
index 790bcc5dd6..160e9f9645 100644
--- a/java/com/google/gerrit/server/query/change/EqualsLabelPredicate.java
+++ b/java/com/google/gerrit/server/query/change/EqualsLabelPredicate.java
@@ -108,7 +108,15 @@ public class EqualsLabelPredicate extends ChangeIndexPredicate {
return false;
}
- if (account != null && !account.equals(approver)) {
+ if (account != null
+ && !account.equals(approver)
+ && !account.equals(ChangeQueryBuilder.OWNER_ACCOUNT_ID)) {
+ return false;
+ }
+
+ if (account != null
+ && account.equals(ChangeQueryBuilder.OWNER_ACCOUNT_ID)
+ && !cd.change().getOwner().equals(approver)) {
return false;
}