Skip to content

Bump Doctrine package versions#1109

Merged
NoResponseMate merged 1 commit intoSylius:1.14from
loic425:bump-doctrine-packages
Dec 19, 2025
Merged

Bump Doctrine package versions#1109
NoResponseMate merged 1 commit intoSylius:1.14from
loic425:bump-doctrine-packages

Conversation

@loic425
Copy link
Copy Markdown
Member

@loic425 loic425 commented Dec 8, 2025

Q A
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Related tickets fixes #1092
License MIT

Replace #1103

@loic425 loic425 force-pushed the bump-doctrine-packages branch from 3e659e8 to f931456 Compare December 8, 2025 16:03
@loic425 loic425 marked this pull request as draft December 8, 2025 16:03
@loic425 loic425 force-pushed the bump-doctrine-packages branch from f931456 to b48cd2f Compare December 8, 2025 20:25
@loic425 loic425 added the dependencies Pull requests that update a dependency file label Dec 18, 2025
@loic425 loic425 mentioned this pull request Dec 19, 2025
15 tasks
@loic425 loic425 force-pushed the bump-doctrine-packages branch from b48cd2f to 87e829f Compare December 19, 2025 11:47
/** @var ResourceInterface|MockObject $resourceMock */
$resourceMock = $this->createMock(ResourceInterface::class);
$requestConfigurationMock->expects($this->once())->method('getRepositoryMethod')->willReturn('findAll');
$requestConfigurationMock->expects($this->once())->method('getRepositoryMethod')->willReturn('find');
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we need a method with a single result.
By default, findOneBy is used (see the previous test in the same test class)

@loic425 loic425 force-pushed the bump-doctrine-packages branch from 87e829f to 20ef4ea Compare December 19, 2025 11:51
@loic425 loic425 marked this pull request as ready for review December 19, 2025 13:09
@NoResponseMate NoResponseMate merged commit a166c11 into Sylius:1.14 Dec 19, 2025
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Allow doctrine/doctrine-bundle ^3

2 participants