-
Notifications
You must be signed in to change notification settings - Fork 103
Description
Using community fork mage-os does not work with the plugin, and results in error : 'Vendor dir is corrupt or missing in the current Magento 2 installation path', when setting the mage-os path in plugin configuration.
Steps to reproduce the behavior:
- Install mage-os fork. Ref https://mage-os.org/
- Setup PHPStorm project to the above install
- Install / activate this module
- setup this module and point folder to root folder
- See this error
Expected behavior (*)
Acceptance of the mage-os vendor folder
Having had a quick look, I can see the issue is related to fact that mage-os has moved vendor folder away from /vendor/magento to /vendor/mage-os (a logical move)
This module uses hardcoded paths to /vendor/magento to check for setups
Please complete the following information: (*)
- OS: Ubuntu
- PhpStorm/Intellij version: 2023.3.4
- Plugin Version: 5.1.1
My suggestion si that the base magento folder should be a configurable value, and paths should be built using that, and not hadcoded to /vendor/magento
a workaround is to add a symlink from old magento folder to mage-os, via composer, so it is generated if vendor is wiped.
"scripts": {
"post-install-cmd": [
"mkdir -p ./vendor/magento && ln -s ./vendor/mage-os/framework ./vendor/magento/"
],
}
Activity
m2-assistant commentedon Mar 1, 2024
Hi @ProxiBlue. Thank you for your report.
To speed up processing of this issue, make sure that you provided sufficient information.
Add a comment to assign the issue:
@magento I am working on this
Join Magento Community Engineering Slack and ask your questions in #github channel.
SilinMykola commentedon Feb 16, 2025
@magento I am working on this
VitaliyBoyko commentedon Feb 16, 2025
@SilinMykola it is supposed to be fixed here #2081. Could you just check and close if so?
SilinMykola commentedon Feb 16, 2025
@VitaliyBoyko, I checked the issue locally, and everything is working fine. We can close the issue.
VitaliyBoyko commentedon Feb 16, 2025
@SilinMykola thanks.