LexikFormFilterBundle icon indicating copy to clipboard operation
LexikFormFilterBundle copied to clipboard

fix ltrim with left parts match

Open gilles-g opened this issue 4 years ago • 6 comments

gilles-g avatar Oct 21 '21 12:10 gilles-g

Hi @rtreffler-goldcore,

I open this PR in favor of the #314

We use a substr instead, I think we are good to merge

Thx for your feedback! it's really a silent bug that I had never encountered in 5 years: p

gilles-g avatar Oct 21 '21 12:10 gilles-g

Hi @rtreffler-goldcore,

I open this PR in favor of the #314

We use a substr instead, I think we are good to merge

Thx for your feedback! it's really a silent bug that I had never encountered in 5 years: p

thanks, I appreciate your quick action, I believe that one is worth being ported for v5.* so people don't live on forks

rtreffler-goldcore avatar Oct 21 '21 12:10 rtreffler-goldcore

I can 't maintain old release, because the code in the master branch has a big BC break with doctrine/dbal...

Maybe I could fix the v6., but v5. is too old

gilles-g avatar Oct 21 '21 13:10 gilles-g

I can 't maintain old release, because the code in the master branch has a big BC break with doctrine/dbal...

Maybe I could fix the v6., but v5. is too old

imho this change can be cherrypicked safely, this file exists and is almost the same (just different import for Doctrine's Type(s) - https://github.com/lexik/LexikFormFilterBundle/blob/v5.0.10/Event/Subscriber/DoctrineORMSubscriber.php)

I'm soon gonna have anther fix (I hope) for you in that file, so I would love to have it in v5.0.11 presumably.

rtreffler-goldcore avatar Oct 21 '21 15:10 rtreffler-goldcore

We don't have branches for each version, so I can't create a new tag from the master branch.

There is two solutions:

  1. migrate to the last version of the bundle (and update doctrime/orm and dbal in your project)
  2. or use an different alias in your dql

gilles-g avatar Oct 22 '21 07:10 gilles-g

We don't have branches for each version, so I can't create a new tag from the master branch.

There is two solutions:

  1. migrate to the last version of the bundle (and update doctrime/orm and dbal in your project)
  2. or use an different alias in your dql

Sorry if I'm too pushy, maybe my understanding is wrong here. But can't you branch a release/5.0.11 from v5.0.10 tag, cherrypick the change and create a v5.0.11 tag from that point ?

Thanks for you patience with me and explanation.

rtreffler-goldcore avatar Oct 22 '21 07:10 rtreffler-goldcore