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. |
||
|---|---|---|
| .. | ||
| settings | ||
| application_extension.rb | ||
| atom_serializer.rb | ||
| email_validator.rb | ||
| exceptions.rb | ||
| feed_manager.rb | ||
| formatter.rb | ||
| hash_object.rb | ||
| inline_rabl_scope.rb | ||
| inline_renderer.rb | ||
| status_length_validator.rb | ||
| tag_manager.rb | ||
| url_validator.rb | ||