-
Notifications
You must be signed in to change notification settings - Fork 106
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
[Bug]: Class rebuild fails during pimcore install script #566
Comments
Are you sure, you are using the latest version of the demo? missing table should be created in bootstrap, see https://github.com/pimcore/demo/blob/11.x/dump/data-0-bootstrap.sql#L3183 |
I'm explicitly running the installer with the |
Confirming the issue and it's surprisingly more related to pimcore/pimcore#16157 rather than the demo install itself. Internally is a sort of "known issue", see https://github.com/pimcore/ee-pimcore/pull/19#issuecomment-1845337937 |
We were able to work around this issue by:
|
Thank you for the feedback. I've found a potential solution to this, by swapping the order like in pimcore/pimcore#16729 in combination with install-bundles option or (the step event
|
Ah wait, it's not intended to work like that, it's for the internal bundles ( |
@mattamon Do you see any contraindications on doing as pimcore/pimcore#16729? |
@kingjia90 no objections. I guess I was not fully aware of this. If you want to install an external bundle, you can always add it via the EventSubscriber like we do it here. Not sure about checking installable though. |
@wwidergoldbsc So i've merged pimcore/pimcore#16729 and with the suggestion to use the This issue is now considered as done, therefore closing for the moment, feel free to re-open (or even better a new one, but maybe not in the demo repository 😄 ) if the problem still occurs. Thank you again for reporting and testing |
@kingjia90 It only executes the even subscriber if the I'm happy to provide a PR for that change, just wanted to get your input first. |
Thank you for the feedback, glad to hear that it works now. Could be fine to move the event of that, but the current documentation is quite defined with the current implementations and could be tricky to refactor without BC breaks, what about a new event or so? |
Yeah ok. I had a closer look into the function and I noticed that you don't have to give an option there. Previously we had |
Pimcore version
11.1, 11.0.2
Steps to reproduce
Example tested on pimcore demo project:
run
bin/console doctrine:database:drop --force --no-interaction
run
bin/console doctrine:database:create --if-not-exists
run
php vendor/bin/pimcore-install --mysql-host-socket=db --mysql-username=pimcore --mysql-password=pimcore --mysql-database=pimcore --skip-database-data-dump
Actual Behavior
The installer will output
Even though the Pimcore install itself is successful, trying to install bundles manually after this will create and overwrite any class definitions related to the installed bundles, essentially discarding any changes made to bundle classes.
Expected Behavior
The text was updated successfully, but these errors were encountered: