View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0006839 | OXID eShop (all versions) | 6. ------ Setup ------- | public | 2018-06-07 15:12 | 2023-11-15 15:35 |
| Reporter | Helmut L. | Assigned To | |||
| Priority | normal | Severity | feature | Reproducibility | always |
| Status | closed | Resolution | reopened | ||
| Product Version | 6.0.2 | ||||
| Summary | 0006839: source-path parameter not fully working | ||||
| Description | If you set a source-path in the composer.json file, the shop is installed into that folder instead of the "source" folder. This already works, but there are some files in the shop in which the folder name is explicitly set to "source" despite the shop being installed into another folder. This results in a fatal error when you try to open the shop. | ||||
| Steps To Reproduce | 1. run composer in an empty directory to get a composer.json file: composer create-project --no-dev --no-install --remove-vcs oxid-esales/oxideshop-project . dev-b-6.0-ee 2. set the source path: composer config extra.oxideshop.source-path htdocs 3. install the shop: composer install --no-dev 4. try to open the shop | ||||
| Additional Information | So far I've found this hard coded folder name in two files: oxid-esales/oxideshop-ce/source/bootstrap.php, line 11 oxideshop-facts/src/Config/ConfigFile.php, line 58 | ||||
| Tags | No tags attached. | ||||
| Theme | Not defined | ||||
| Browser | Not defined | ||||
| PHP Version | Not defined | ||||
| Database Version | Not defined | ||||
|
|
The name of the source directory inside the shop directory is mandatory. |
|
|
If the folder name must be "source" why was the source-path parameter added to change it? |
|
|
Where did you get the parameter from? |
|
|
In the abstract package installer are constants for the possible options that can be set through the composer.json file. /vendor/oxid-esales/oxideshop-composer-plugin/src/Installer/Package/AbstractPackageInstaller.php, line 30: /** Used to decide what the shop source directory is. */ const EXTRA_PARAMETER_SOURCE_PATH = 'source-path'; |
|
|
It is still in discussion, but it is currently not intended to change the source directory via this parameter. |
|
|
We will follow this in the linked ticket. Closing this one. Please follow the development in the linked ticket |
