]> cat aescling's git repositories - mastodon.git/commit
DRY up reblog vs original status check
authorJoël Quenneville <joelq@thoughtbot.com>
Fri, 7 Apr 2017 18:18:30 +0000 (14:18 -0400)
committerJoël Quenneville <joelq@thoughtbot.com>
Fri, 7 Apr 2017 18:18:30 +0000 (14:18 -0400)
commitd4c94fa004117fdb7226b1b846a12d12dc0542d9
tree93d2a3a70ad6bb6f23a63c9f6fc086c9f197384f
parent4e41cd9ab8f51120d558b70528b163c98993be53
DRY up reblog vs original status check

Checking reblog vs original status was happening in multiple places
across the app. For views, this logic was encapsulated in a helper
method named `proper_status` but in the other layers of the app, the
logic was duplicated.

Because the logic is used at all layers of the app, we extracted it into
a `Status#proper` method on the model and changed all uses of the logic
to use this method. There is now a single source of truth for this
condition.

We added test coverage to untested methods that got refactored.
app/helpers/stream_entries_helper.rb
app/lib/atom_serializer.rb
app/models/account.rb
app/models/status.rb
app/views/stream_entries/_status.html.haml
spec/models/account_spec.rb
spec/models/status_spec.rb