e45d24a2a489609fc12b565dcc187451c2f1b676 - gerrit

commit	e45d24a2a489609fc12b565dcc187451c2f1b676	[log] [tgz]
author	Yacob Yonas <yyonas@google.com>	Fri Jul 25 17:05:29 2014 -0700
committer	Yacob Yonas <yyonas@google.com>	Thu Aug 07 13:38:27 2014 -0700
tree	61301109eff7db50149fead9748d6222805771f6
parent	5020ff2fd06e3ada4ca163b4f5b24f13cdd9928b [diff]

Refactor byChange() into two methods: drafts and published

Callers of the PatchLineCommentUtils.byChange() mostly just want
either the published comments or the drafts comments. For that reason,
I refactored this method into two pieces and updated its callers.

I modified ChangeData.comments() to be ChangeData.publishedComments()
in order to more accurately the contents of the returned list.

Change-Id: I4222b894896af1ae635fd8c390fe014db9f6778d

gerrit-server/src/main/java/com/google/gerrit/server/PatchLineCommentsUtil.java[diff]
gerrit-server/src/main/java/com/google/gerrit/server/index/ChangeField.java[diff]
gerrit-server/src/main/java/com/google/gerrit/server/query/change/ChangeData.java[diff]
gerrit-server/src/main/java/com/google/gerrit/server/query/change/QueryProcessor.java[diff]
gerrit-server/src/test/java/com/google/gerrit/server/change/CommentsTest.java[diff]

5 files changed