Steve Howell a3d3d08a80 refactor: Avoid positional params for MessageList. ...

We used to have positional parameters for table_name
and filter, but we don't use them for message_list.all
and we're about to replace filter in some cases.

Passing everything in on opts is more consistent and
self-documenting in the calling code, plus lots of
unit tests can get away with passing in `{}` now
for situations where table_name does not matter.

All of our callers pass in muting_enabled, so we
remove the default value for it.  And then the
collapse_messages variable doesn't have to live on
`this` as it's only being passed through down to the
view.

2018-05-15 14:25:21 -07:00

8.6 KiB

Raw Blame History

View Raw