Skip to content

Fix service definition for the uploadable manager#522

Merged
stof merged 1 commit intostof:mainfrom
jonathan-stimactiv:main
Jan 23, 2026
Merged

Fix service definition for the uploadable manager#522
stof merged 1 commit intostof:mainfrom
jonathan-stimactiv:main

Conversation

@jonathan-stimactiv
Copy link
Copy Markdown
Contributor

After an upgrade of te bundle from v1.14.x to 1.15.2, i keep getting the following error on a cache:clear:
You have requested a non-existent service "stof_doctrine_extensions.uploadable_manager".

After digging a bit, i found a typo in the alias for the uploadableManager src/Resources/uploadable.php.
At line 31 the service is defined as stof_doctrine_extensions.uploadable.manager (with a dot), but line 39, the reference called is stof_doctrine_extensions.uploadable_manager (with an underscore).

Just after that, I run into another error due to the incorrect call of methods setCacheItemPool, setAnnotationReader and setDefaultFileInfoClass on the manager, which are only existant in the listener and has nothing to do with the manager.

A quick release would be very appreciated since Uploadable is not useable at all without these changes

@stof
Copy link
Copy Markdown
Owner

stof commented Jan 23, 2026

For the fix of the service id for the UploadableManager autowiring alias, I just merged #524 which does the same fix (because I missed the notification about your PR).

I rebased your branch to keep the change about removing invalid method calls.

@stof stof changed the title [BUG] Fix Dependency Injection for uploadable [BUG] Fix service definition for the uploadable manager Jan 23, 2026
@stof stof changed the title [BUG] Fix service definition for the uploadable manager Fix service definition for the uploadable manager Jan 23, 2026
@stof stof merged commit 0f464d3 into stof:main Jan 23, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants