[FEAT] add filter by operator

[AbdelrahmanBl]

Dear sir I had been developed out of the box a method for filtering by operator. for the example:

QueryBuilder::for(User::class)
->allowedFilters([
    AllowedFilter::operator('salary', FilterOperator::GREATER_THAN),
])
->get();

Note: I realized that there is some old test cases not working.

[siarheipashkevich]

@AbdelrahmanBl is it possible to pass the operator through query parameters?

[AbdelrahmanBl]

It's really a great idea! Now you can search by a dynamic filter operator. for the example:

// users?filter[salary]=>=1000,<=3000
QueryBuilder::for(User::class)
->allowedFilters([
    AllowedFilter::operator('salary', FilterOperator::DYNAMIC),
])
->get();

[AbdelrahmanBl]

@freekmurze

[AbdelrahmanBl]

Hello sir! How are you? Can you check this feature? @freekmurze

[benswinburne]

Hello sir! How are you? Can you check this feature? @freekmurze

@AbdelrahmanBl Please stop deleting your previous comment and @ people. No doubt they're busy and they'll get to it when they get to it.

[AbdelrahmanBl]

Thank you for your response 🙏 and i won't contribute with spatie anymore for not annoying them.

[benswinburne]

I'm not from Spatie, just someone whose inbox you were spamming. Repeatedly @ing anyone isn't productive be it Spatie or anyone else.

I'd subscribed to the thread because i'm interested in the feature as I'm sure others are too. Closing it because someone suggest you be patient seems overkill, but it is your PR to do what you will with.

[AbdelrahmanBl]

I thought you were from spatie and no problem i will reopen it to benefit the others 😊

[AbdelrahmanBl]

@freekmurze kindly check this commit!

[cgarcia-lightit]

Hi @freekmurze,

This improvement could be considered to be released. I mean, it's a good feature, I don't understand why not implement the feature. Are there some pending considerations? Or Do you want to implement this feature in another way? It will be great to know it.

Thanks

[freekmurze]

Could you update the readme with clear examples on how to use this?

[cgarciagarcia]

Hi @AbdelrahmanBl ,

Could you generate the requested documentation? We are close to get merge this pr 😄

[AbdelrahmanBl]

@cgarciagarcia Okay i will implement that as soon as possible

[cgarciagarcia]

Hello @AbdelrahmanBl , I would like to know if there are posibilities to include the documententation that freekmurze requested to merge this PR. I like your feature and I think it's a good resolution approach.

[AlexVanderbist]

Thanks everyone for your input and hard work! I'll add docs for this feature and tag it as a new feature release.

[siarheipashkevich]

@AbdelrahmanBl thank you

[AbdelrahmanBl]

thanks guys, i'm very busy nowadays and that is the reason for not completing the doc

[cgarcia-lightit]

Thanks @AbdelrahmanBl for your work! and spatie team for your care.