Is a beforeFind on a Model a bad idea?

I never found a consistent use for a beforeFind event, honestly If people feel it's useful then feel free to submit a PR.

For things like searching and filtering for an index page you can always chain methods together. Something like:

In controller:

PHP Code:

$users = $userModel->filter($this->request->getPost())->paginate(20); 


and the new method in the model might be something like:

PHP Code:

public function filter($params) 
{
    if(isset($params['search'] && ! empty($params['search'])) {
        $this->like('name', $params['search'].'%');
    }
    if(isset($params['city_id'] && ! empty($params['city_id'])) {
        $this->like('city_id', $params['city_id'].'%');
    }

    // Sorting
    $sortBy = isset($params['sort']) && trim($params['sort'] !== '')
        ? $params['sort']
        : 'first_name';
    $sortDir = isset($params['sort_dir']) && in_array($params['sort_dir'], ['asc', 'desc'])
        ? $params['sort_dir']
        : 'asc';
    $this->orderBy($sortBy, $sortDir);

    return $this;
} 


Support Development •  CodeIgniter 4 Foundations • Practical CodeIgniter 3  • CodeIgniter Tutorials