View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0003039 | OXID eShop (all versions) | 4.07. Source code, Test | public | 2011-07-08 11:52 | 2012-12-10 13:44 |
| Reporter | tomas_liubinas | Assigned To | |||
| Priority | high | Severity | major | Reproducibility | always |
| Status | resolved | Resolution | no change required | ||
| Summary | 0003039: oxConfig::appendUrl() method separates URL params using &_amp; | ||||
| Description | While researching strange paypal module behaviour we realised that oxConfig::appendUrl() method separates URL params using &_amp instead of single char &. The output of the method is used in oxConfig::getShopHomeUrl() which (I assume) could be used directly in templates for further reference. When we do it, &_amp; separator instead of single char & separator is a clearly wrong choice. But this is theoretical, in practice nobody complained about it and there must be a reason why &_amp; was implemnted anyway, that's why I am not sure if it's wrong or not. Needs to be researched. As a note I would like to mention, that the separator is only added in limited conditions (when resolving urls for different language for ssl or something like this) so the harmful effect of this behaviour might not be noticed under normal circumstances. P.S. I added an underscore within special char description as Mantis tends to screw them. | ||||
| Tags | No tags attached. | ||||
| Theme | Both | ||||
| Browser | All | ||||
| PHP Version | any | ||||
| Database Version | any | ||||
