Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge 2.20.x up into 3.3.x #11723

Merged
merged 17 commits into from
Nov 23, 2024
Merged

Merge 2.20.x up into 3.3.x #11723

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
439b4da
Is not correctly generated sql when changed/switched sqlFilter parame…
dbannik Oct 22, 2024
3ca9529
Merge pull request #11694 from dbannik/Bug-join-sql-when-change-sqlFi…
greg0ire Oct 23, 2024
9e2bfa8
Run tests against PostgreSQL 17 (#11697)
IndraGunawan Oct 25, 2024
ff3ccff
Add `isEmpty()` method to the Extra Lazy Associations tutorial
acasademont Nov 13, 2024
d9aa6ef
Merge pull request #11716 from acasademont/patch-1
greg0ire Nov 14, 2024
43ce0be
lazy and eager collection refresh
goetas Sep 26, 2022
7d1b24f
attempt a fix
goetas Sep 26, 2022
486e406
Merge pull request #10065 from goetas/lazy-eager-collection-refresh
greg0ire Nov 14, 2024
4fbce94
Bump codecov/codecov-action from 4 to 5
dependabot[bot] Nov 18, 2024
ba11851
Merge pull request #11718 from doctrine/dependabot/github_actions/2.2…
greg0ire Nov 18, 2024
bb5b2a3
Ignore deprecation about StaticReflectionService
greg0ire Nov 18, 2024
f140651
Merge pull request #11719 from greg0ire/ignore-deprecation
greg0ire Nov 18, 2024
346c498
Fix `Events::onFlush` and `PostFlush()` documentation: events are alw…
lyrixx Nov 7, 2024
58ad1d9
Merge pull request #11709 from lyrixx/fix-event-doc
greg0ire Nov 18, 2024
8422a41
unitofwork.rst: php => PHP
k00ni Jul 22, 2024
82e2c98
Merge pull request #11556 from k00ni/patch-1
greg0ire Nov 19, 2024
2ff998d
Merge remote-tracking branch 'origin/2.20.x' into 3.3.x
greg0ire Nov 23, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
attempt a fix
  • Loading branch information
@goetas
goetas committed Nov 14, 2024
commit 7d1b24f3b1fde6e12385c299ba6ea4b0da385031
4 changes: 2 additions & 2 deletions src/UnitOfWork.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2473,13 +2473,13 @@ private function doRefresh($entity, array &$visited, ?int $lockMode = null): voi
throw ORMInvalidArgumentException::entityNotManaged($entity);
}

$this->cascadeRefresh($entity, $visited, $lockMode);

$this->getEntityPersister($class->name)->refresh(
array_combine($class->getIdentifierFieldNames(), $this->entityIdentifiers[$oid]),
$entity,
$lockMode
);

$this->cascadeRefresh($entity, $visited, $lockMode);
}

/**
Expand Down
1 change: 1 addition & 0 deletions tests/Doctrine/Tests/ORM/Functional/Ticket/LazyEagerCollectionTest.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@ class LazyEagerCollectionTest extends OrmFunctionalTestCase
protected function setUp(): void
{
parent::setUp();

$this->createSchemaForModels(
LazyEagerCollectionUser::class,
LazyEagerCollectionAddress::class,
Expand Down
Loading