View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0001126 | OXID eShop (all versions) | 1. ----- eShop frontend ----- | public | 2009-07-22 17:02 | 2012-12-10 14:17 |
| Reporter | rs | Assigned To | |||
| Priority | normal | Severity | minor | Reproducibility | always |
| Status | resolved | Resolution | fixed | ||
| Summary | 0001126: preg_replace in getRequestUrl | ||||
| Description | I write an own module for your e-shop. Unfortunately the preg_replace in getRequestUrl damages the parameter sysid ($_GET["sysid"]) because of sid in sysid. use: $sRequest = preg_replace( '/(\&|(\?))sid=[^&]*(&)?/', '$2', $sRequest); instead of $sRequest = preg_replace( '/((\&)?sid=[^&]*(&)?)/', '', $sRequest ); It would be nice if you can fix this. | ||||
| Tags | Subshops | ||||
| Theme | |||||
| Browser | |||||
| PHP Version | |||||
| Database Version | |||||
|
|
Ah... there is also a little bug in my preg_replace It works only for addresses starting with sid: ?sid=0&bla=1&blu=2 and ending with sid ?bla=1&blu=2&sid=0 But not for ?bla=1&sid=0&blu=2 Here a possible solution: $sRequest = preg_replace( '/(&sid=[^&]*|(\?)sid=[^&]*(&)?)/', '$2', $sRequest); |
|
|
Updated regexep for correct sid removing from uri |