opendev/gerrit
Code Issues Proposed changes

Fix notes NPE for has:draft search predicate.

Browse Source 
Use notes() instead of accessing the notes field directly since it can
be null.  Calling notes() ensures that it is not null.

Change-Id: Ie7e49cdf38b929e637f650b2cf1f6a2db9b68890
This commit is contained in:
Martin Fick
2016-11-13 08:35:00 -07:00
committed by David Pursehouse
parent 01583ca0ee
commit 62d46660cf
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
gerrit-server/src/main/java/com/google/gerrit/server/query/change/ChangeData.java
View File

@@ -1056,7 +1056,7 @@ public class ChangeData {
return Collections.emptySet();
}
draftsByUser = new HashSet<>();
for (PatchLineComment sc : plcUtil.draftByChange(db, notes)) {
for (PatchLineComment sc : plcUtil.draftByChange(db, notes())) {
draftsByUser.add(sc.getAuthor());
}
}