Return builder object if $resetModel = false#738
Return builder object if $resetModel = false#738vrkansagara wants to merge 3 commits intoandersao:masterfrom
Conversation
Call to undefined function Prettus\Repository\Generators\str_contains()
|
Ping @andersao This would need to review on urgent bases. On of my project suffering from this PR. |
|
This is really user specific changes. Please use it by overwriting. If we need to do is need to implement it all classes. Thanks for pr @vrkansagara |
|
Can you provide me an example of how to override this ?
…On Fri, 30 Oct, 2020, 1:57 am Burak Sormageç, ***@***.***> wrote:
This is really user specific changes. Please use it by overwriting. If we
need to do is need to implement it all classes. Thanks for pr @vrkansagara
<https://github.com/vrkansagara>
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#738 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AC3TIU4W5FE6TUBMF7BDUZLSNHF23ANCNFSM4SZ3QSTQ>
.
|
|
Create new BaseRepository.php file in App/Repositories/BaseRepository.php and extends it main BaseRepository.php and overwrite any function you want to change. @vrkansagara |
|
This feature is useful without modification of abstraction and provide flexibility at run time. @bsormagec @andersao |
|
If you want to made it useful you need to change it every class has resetMode not only one method. |
|
agree @bsormagec , Let discuss here for other method as well. if other people agree upon this solutions I can add to other method as well. # # |
Returning the builder object give more power to integrate
l5-repositorywith other library.