View Issue Details

IDProjectCategoryView StatusLast Update
0007064OXID ERP InterfaceOXID ERP Interface - subpublic2024-07-03 08:47
Reporterdx_bhesse Assigned To 
PrioritylowSeveritytweakReproducibilityalways
Status resolvedResolutionfixed 
OSWindows 10OS VersionBuild 18363.535 
Product Version3.1.0 
Fixed in Version4.1.0 
Summary0007064: Target-Namespace is generated with the wrong url separator under windows
DescriptionIn oxerpservice.php (line ~86):
$targetNamespace = !empty($targetNamespace) ? $targetNamespace : $sShopURL . 'modules' . DIRECTORY_SEPARATOR . 'erp' . DIRECTORY_SEPARATOR . 'OXERPService';

DIRECTORY_SEPARATOR is \ under windows instead of / and will generate invalid namespace urls - which will prevent SoapServer and SoapClient to be unable to parse the generated *.wsdl.

Suggested fix: Replace DIRECTORY_SEPARATOR with '/'
Workaround: Manually set the namespace by setting $this->erpTargetNamespace in config.inc.php

(There probably is no official windows support for oxid, but keeping it functional would be handy)
Steps To ReproduceOnly precondition is running oxid on a windows system and any regular call to oxerpservice.php will fail
TagsNo tags attached.

Activities

SvenBrunk

2024-06-26 11:40

manager   ~0017120

It is just plain wrong to use the platform independent DIRECTORY_SEPARATOR ( https://www.php.net/manual/en/dir.constants.php ) in URLs as those are always platform independent and always use /

SvenBrunk

2024-06-26 12:02

manager   ~0017126

https://github.com/OXID-eSales/erp/pull/28

SvenBrunk

2024-07-03 08:47

manager   ~0017151

Merged into b-7.1, release 4.1.0 of ERP module pending