View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0001739 | OXID eShop (all versions) | 1.01. Products (product, categories, manufacturer, promotions etc.) | public | 2010-04-01 11:03 | 2012-12-07 14:15 |
Reporter | schuhe-schuhe | Assigned To | |||
Priority | high | Severity | major | Reproducibility | always |
Status | resolved | Resolution | fixed | ||
Product Version | 4.2.0 revision 23610 | ||||
Fixed in Version | 4.3.1 revision 27257 | ||||
Summary | 0001739: Changing Currency on manufacturer lists delivers empty pages | ||||
Description | When products are listed by manufacutrer, any change of currency leads do blanc manufacturer lists. | ||||
Tags | Manufacturer | ||||
Theme | |||||
Browser | All | ||||
PHP Version | any | ||||
Database Version | any | ||||
|
note: currency links are generated wrong. they dont include which manufacturer should be displayed. also after changing currency, there are two "cur" parameters in currency links |
|
old code views/oxcmp_cur.php: ... reset( $this->aCurrencies ); while ( list( , $oItem ) = each( $this->aCurrencies ) ) { $oItem->link = oxUtilsUrl::getInstance()->processUrl("{$sURL}&cur={$oItem->id}"); } ... new code: ... $oUrlUtils = oxUtilsUrl::getInstance(); $sUrl = $oUrlUtils->cleanUrl( $this->getParent()->getLink(), array( "cur" ) ); reset( $this->aCurrencies ); while ( list( , $oItem ) = each( $this->aCurrencies ) ) { $oItem->link = $oUrlUtils->processUrl( $sUrl, true, array( "cur" => $oItem->id ) ); } ... New method (implemented in SVN version, will come with next release) oxUtilsUrl::cleanUrl() removes any passed parameter from given URL. So in this case you must create a code, which removes "cur=X" parameter from url. |