View Issue Details
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0007180||OXID eShop (all versions)||2.3. Extensions (modules, themes)||public||2020-10-05 14:12||2020-11-12 15:29|
|Target Version||Fixed in Version||6.2.3|
|Summary||0007180: Saving module settings issue, if module id includes - or / character|
|Description||We have noticed a problem in Shop 6.2 while updating module settings. |
We are not able to save module settings in Shop admin, if there is specific characters (- or /) exist in module id. e.g. OXPS-MyModule
|Steps To Reproduce||You can simply follow these steps. |
1. Please develop any simple module with module id including any one of these characters (- or /) and having some settings.
2. Install and enable this module.
3. Now please try to update any setting field from shop admin. You will notice that page is getting refreshed but setting is not updated on form and DB.
|Tags||No tags attached.|
We stumbled upon the same issue, but with OXID 6.2.2 and PHP 7.4. The module we used had a slash in its module-id. To change module settings you need to deactivate the module, change the settings and activate the module again.
Note also, that there are a ton of modules out there using a slash or a "-" in the module-id and this worked - perfectly until. Even OXID ships a few modules like this:
I created a module with ID "oxs/test-module". Both characters are present, but I can still modify my setting successfully. As a result I am unable to reproduce the issue with the steps provided. Maybe there is another fact that interferes with the functionality. Did you test with a simple module as described and in a standard OXID eShop environment? If not, please do so.
We found out that it works after commenting out this line: https://github.com/OXID-eSales/oxideshop_ce/blob/v6.5.6/source/Internal/Container/Event/ConfigurationChangedEventSubscriber.php#L23
Nitin confirmed that, too. We reported that to some folks from the core team and they were able to reproduce the issue. The issue is not related to the fact, that the module-id has a "-" or a "/" in it, we were able to reproduce this with a module-id that contains only letters.
||Thanks for the further investigation and report to Core Team. I will close this issue now, since you're telling it is not connected to the described characteristics.|
||Bug was reproducible via the service yaml and it is now fixed with https://github.com/OXID-eSales/oxideshop_ce/commit/b3ac96a09c9c6df2630e81c6e37b07a9d16db008|