Actions
Refactor #8426
closedMove scoped search into concerns
Description
Scoped search definitions can be cumbersome at times and don't really define the business logic of a model. I think hiding them under the carpet by moving most of the definitions to concerns (as we do with hostext) we can later on include would keep the code cleaner. There are 3 or 4 models with just 1 scoped_search attribute I didn't bother to move, later commits that move them into concerns when they become bigger can just ref this issue.
Updated by The Foreman Bot over 9 years ago
- Status changed from New to Ready For Testing
- Pull request https://github.com/theforeman/foreman/pull/1952 added
- Pull request deleted (
)
Updated by Dominic Cleal over 9 years ago
- Category set to Search
- Assignee set to Daniel Lobato Garcia
Updated by Daniel Lobato Garcia over 9 years ago
- Status changed from Ready For Testing to Closed
Updated by Daniel Lobato Garcia over 9 years ago
- Status changed from Closed to Rejected
Actions