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.

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

[Composer] Bumped minimum version of ezsystems/job-queue-bundle to 4.0.3 #74

Merged
merged 1 commit into from
Mar 24, 2023

Conversation

mnocon
Copy link
Contributor

@mnocon mnocon commented Mar 24, 2023

Failing tests:
https://github.com/ibexa/content/actions/runs/4505522541/jobs/7931339589 (3.3)
https://github.com/ibexa/commerce/actions/runs/4505528385/jobs/7931423854 (3.3)

The installation fails with:

Executing script cache:clear [KO]
 [KO]
Script cache:clear returned with error code 255
!!  
!!   // Clearing the cache for the behat environment with debug                     
!!   // true                                                                        
!!  
!!  PHP Fatal error:  Class JMS\JobQueueBundle\Entity\Listener\PersistentRelatedEntitiesCollection contains 2 abstract methods and must therefore be declared abstract or implement the remaining methods (Doctrine\Common\Collections\ReadableCollection::findFirst, Doctrine\Common\Collections\ReadableCollection::reduce) in /var/www/vendor/ezsystems/job-queue-bundle/JMS/JobQueueBundle/Entity/Listener/PersistentRelatedEntitiesCollection.php on line 22

This is the same error that was described in ezsystems/JMSJobQueueBundle#8 - and the release 4.0.3 was made to prevent it (https://github.com/ezsystems/JMSJobQueueBundle/releases/tag/v4.0.3)

In the linked failures Composer prefers to install higher version of doctrine/collections:

  - Locking doctrine/collections (2.1.2)

and lower version of ezsystems/job-queue-bundle:

  - Locking ezsystems/job-queue-bundle (v4.0.2)

Bumping the minimum version will prevent Composer from doing so.

@mnocon mnocon requested review from webhdx and a team March 24, 2023 09:38
@alongosz alongosz requested a review from a team March 24, 2023 09:39
@mnocon mnocon merged commit 459f75f into 3.3 Mar 24, 2023
@mnocon mnocon deleted the bump-jms-job-queue-bundle branch July 20, 2023 08:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants