Skip to content

conflicted doctrine/orm 2.20.7#4252

Merged
vitek-rostislav merged 1 commit into18.0from
mg-pin-orm
Oct 29, 2025
Merged

conflicted doctrine/orm 2.20.7#4252
vitek-rostislav merged 1 commit into18.0from
mg-pin-orm

Conversation

@grossmannmartin
Copy link
Copy Markdown
Member

@grossmannmartin grossmannmartin commented Oct 28, 2025

Description, the reason for the PR

  • calling $repository->findBy(['id' => []]) cause SQL error in doctrine/orm 2.20.7
  • this version is conflicted as it's not easy to safely identify problematic pieces of code

the conversation pn the fix continues in the original repository - doctrine/orm#12190

Fixes issues

...

License Agreement for contributions


🌐 Live Preview:

- calling `$repository->findBy(['id' => []])` cause SQL error in doctrine/orm 2.20.7
- this version is conflicted as it's not easy to safely identify problematic pieces of code
@grossmannmartin grossmannmartin added the Bug Something isn't working label Oct 28, 2025
@vitek-rostislav vitek-rostislav merged commit e0a378c into 18.0 Oct 29, 2025
35 checks passed
@vitek-rostislav vitek-rostislav deleted the mg-pin-orm branch October 29, 2025 10:52
vitek-rostislav added a commit that referenced this pull request Dec 23, 2025
- removed no longer needed conflict introduced in #4252
- see doctrine/orm#12254
ShopsysBot pushed a commit to shopsys/framework that referenced this pull request Dec 23, 2025
- removed no longer needed conflict introduced in shopsys/shopsys#4252
- see doctrine/orm#12254
ShopsysBot pushed a commit to shopsys/category-feed-luigis-box that referenced this pull request Dec 23, 2025
- removed no longer needed conflict introduced in shopsys/shopsys#4252
- see doctrine/orm#12254
ShopsysBot pushed a commit to shopsys/product-feed-zbozi that referenced this pull request Dec 23, 2025
- removed no longer needed conflict introduced in shopsys/shopsys#4252
- see doctrine/orm#12254
ShopsysBot pushed a commit to shopsys/product-feed-google that referenced this pull request Dec 23, 2025
- removed no longer needed conflict introduced in shopsys/shopsys#4252
- see doctrine/orm#12254
ShopsysBot pushed a commit to shopsys/product-feed-mergado that referenced this pull request Dec 23, 2025
- removed no longer needed conflict introduced in shopsys/shopsys#4252
- see doctrine/orm#12254
ShopsysBot pushed a commit to shopsys/product-feed-heureka that referenced this pull request Dec 23, 2025
- removed no longer needed conflict introduced in shopsys/shopsys#4252
- see doctrine/orm#12254
ShopsysBot pushed a commit to shopsys/product-feed-heureka-delivery that referenced this pull request Dec 23, 2025
- removed no longer needed conflict introduced in shopsys/shopsys#4252
- see doctrine/orm#12254
ShopsysBot pushed a commit to shopsys/product-feed-luigis-box that referenced this pull request Dec 23, 2025
- removed no longer needed conflict introduced in shopsys/shopsys#4252
- see doctrine/orm#12254
ShopsysBot pushed a commit to shopsys/article-feed-luigis-box that referenced this pull request Dec 23, 2025
- removed no longer needed conflict introduced in shopsys/shopsys#4252
- see doctrine/orm#12254
ShopsysBot pushed a commit to shopsys/brand-feed-luigis-box that referenced this pull request Dec 23, 2025
- removed no longer needed conflict introduced in shopsys/shopsys#4252
- see doctrine/orm#12254
ShopsysBot pushed a commit to shopsys/maker that referenced this pull request Dec 23, 2025
- removed no longer needed conflict introduced in shopsys/shopsys#4252
- see doctrine/orm#12254
ShopsysBot pushed a commit to shopsys/migrations that referenced this pull request Dec 23, 2025
- removed no longer needed conflict introduced in shopsys/shopsys#4252
- see doctrine/orm#12254
ShopsysBot pushed a commit to shopsys/project-base that referenced this pull request Dec 23, 2025
- removed no longer needed conflict introduced in shopsys/shopsys#4252
- see doctrine/orm#12254
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants