View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0006422 | OXID eShop (all versions) | 4.01. Database handling | public | 2016-07-05 12:23 | 2016-07-14 16:43 |
Reporter | flowcontrol | Assigned To | |||
Priority | normal | Severity | block | Reproducibility | always |
Status | resolved | Resolution | fixed | ||
Platform | all | OS | all | OS Version | all |
Product Version | 4.10.0 / 5.3.0 | ||||
Fixed in Version | 4.10.1 / 5.3.1 | ||||
Summary | 0006422: Missing DIRECTORY_SEPARATOR in mysqli Driver leads to 500 | ||||
Description | PHP message: PHP Warning: require_once(/var/www/html/core/adodblite/adodbSQL_drivers/mysqli../../../interface/ResultSetInterface.php): failed to open stream: No such file or directory in /var/www/html/core/adodblite/adodbSQL_drivers/mysqli/mysqli_driver.inc on line 394 | ||||
Steps To Reproduce | - Update 5.2.9 EE to 5.3.0 - we already had mysqli as the mysql driver before that update - open shop in browser - 500 | ||||
Additional Information | Patch --- a/core/adodblite/adodbSQL_drivers/mysqli/mysqli_driver.inc +++ b/core/adodblite/adodbSQL_drivers/mysqli/mysqli_driver.inc @@ -391,7 +391,7 @@ class mysqli_driver_ADOConnection extends ADOConnection } } -require_once __DIR__ . '..' . DIRECTORY_SEPARATOR. '..' . DIRECTORY_SEPARATOR. '..' . DIRECTORY_SEPARATOR. 'interface' . DIRECTORY_SEPARATOR. 'ResultSetInterface.php'; +require_once __DIR__ . DIRECTORY_SEPARATOR . '..' . DIRECTORY_SEPARATOR. '..' . DIRECTORY_SEPARATOR. '..' . DIRECTORY_SEPARATOR. 'interface' . DIRECTORY_SEPARATOR. 'ResultSetInterface.php'; | ||||
Tags | No tags attached. | ||||
Theme | Not defined | ||||
Browser | Not defined | ||||
PHP Version | Not defined | ||||
Database Version | Not defined | ||||
|
https://github.com/OXID-eSales/oxideshop_ce/pull/426 |
|
I have same bug when upgrading right now. After I've fixed it, seeing that a bug report for that issue is already submitted. BTW: error is only in mysqli_driver.inc, not in mysql_driver.inc. |
|
Thank you for the pull request. Just merged to 5.3. |