Cleanup

A couple things:

1. `find(:all)` has been deprecated for a while. You should just use `all` or `self.all` for the same thing.
2. `find(:all, condition: blah)` again, is deprecated. You should use `where(conditions_here)` instead.
3. Since you're using the same search value inside the query for `?`, you can use [placeholder conditions](http://guides.rubyonrails.org/active_record_querying.html#placeholder-conditions) to only have to specify the value once

This method is also a good use case for a [guard clause](http://thechrisoshow.com/2009/02/16/using-guard-clauses-in-your-ruby-code/).

Given that, this method could be rewritten like this:

```ruby
def self.search(search)
  return all unless search.present?

  where("name ILIKE :search OR location ILIKE :search", search: "%#{search}%")
end
```