Cancel model fetch

[mdsaicom]

Can i cancel the fetch of a model by using the afterFetch or beforeFetch events
public function beforeFetch() { if ($this->atrribute == 'example'){ //cancel fetch } }

[mjauvin]

Did you try returning false ?

[mdsaicom]

Yes, but returning the model anywere

[LukeTowers]

What exactly do you mean by "cancel the fetch"? What is the expected result of that? If you're trying to prevent access to a model in the afterFetch event just throw an ApplicationException

[mdsaicom]

I need not to return the model if it does not meet a condition, but the rest of the collection if I want to return it

[LukeTowers]

What? I don't understand what you mean.

[mdsaicom]

Sorry, I'm translating with google.
I have two records in the table and of the two records I only want to return those that meet a filter at the model level but not at the query level

[mjauvin]

You mean in the List controller?

[mjauvin]

If so, read this:
https://wintercms.com/docs/backend/lists#adding-filters

And this
https://wintercms.com/docs/backend/lists#list-filters

[mdsaicom]

public function beforeFetch()
{
$this->active = true;
Event::fire('user.check', [&$this->active]);
if ($this->active == false){
//CANCEL FETCH
}
}

[LukeTowers]

I'm still not sure what it is exactly that you're trying to do but I'm reasonably certain that is not the way to do it. Have you looked at https://wintercms.com/docs/events/event/backend.list.extendRecords?

[mdsaicom]

In frontend component this example works

$collection = MyModel::where('active', true)->get();

foreach ($collection as $index => $model) {
    Event::fire('my.event', [&$model]);
    if ($model->active == false){
        unset($collection[$index]);            
    }
}
return $collection;

-------------------------
    
Event::listen('my.event', function (&$model) {
    if ($model->mycustomfield > 0){
        $model->active = false;
    }
});

I need to do the same thing but in the beforeFetch event of the model

[LukeTowers]

See https://xyproblem.info/. Try asking for help with the desired end result you want rather than help with your current approach to solving that problem. I can tell you that firing an event in a loop is a pretty bad idea for performance reasons so that's already a huge negative to your proposed approach; and secondly attempting to exclude a model from a collection by doing something in the afterFetch event is not going to do what you want it too either.

[mdsaicom]

Ok. Thank you for your answers.
Finally, I'm going to choose to use collection overloading if you don't tell me otherwise.

/**
 * Create a new Collection instance.
 */
public function newCollection(array $models = [])
{
    return new CustomCollection($models);
}

[LukeTowers]

I'm pretty sure that's not the best way to go about it but since you haven't provided enough information on your use case for me to provide better alternatives if that works for you then go for it.