View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0006451 | OXID eShop (all versions) | 4.09. SEO, SEO URL | public | 2016-07-18 10:07 | 2025-07-23 11:33 |
| Reporter | tjungcl | Assigned To | |||
| Priority | normal | Severity | minor | Reproducibility | always |
| Status | resolved | Resolution | unable to reproduce | ||
| Product Version | 4.10.0 / 5.3.0 | ||||
| Summary | 0006451: "noindex,follow" instead of "noindex,nofollow" on vendorlist details with currency set in url | ||||
| Description | http://demoshop.oxid-esales.com/professional-edition/Nach-Hersteller/Liquid-Force/Bindung-LIQUID-FORCE-TRANSIT-BOOT-2010.html?cur=2 If you change currency, the whole page should be noindex-nofollow (oxubase->noindex()). But in details->noindex() there is an early return with noindex-follow, if the listtype is vendorlist or manufacturerlist. | ||||
| Tags | SEO Rewrite | ||||
| Theme | Not defined | ||||
| Browser | Not defined | ||||
| PHP Version | Not defined | ||||
| Database Version | Not defined | ||||
|
|
There will always "noindex, follow" instead of "noindex, nofollow" regardless of the currency parameters. Moreover, the reasons is lacking, why here should be "noindex, nofollow" with change of the currency. We need more information. application/controllers/details.php::noIndex() |
|
|
"noindex,nofollow" is returned in oxubase->noindex() if any cur is given. (if ($this->getConfig()->getRequestParameter('cur')) "noindex,follow" is set in details->noindex() if listtype is vendorlist or manufacturerlist. the parent-function is not called in that case. So there is a rule, that every page with cur-param will be noindex,nofollow. And a rule in details.php that overrides that rule with a less restrictive return value. |
|
|
We cannot reproduce this. This early return is only used when the parameter 'listtype' was set on the request. $sListType = $this->getConfig()->getRequestParameter('listtype'); if ($sListType && ('vendor' == $sListType || 'manufacturer' == $sListType)) { return $this->_iViewIndexState = VIEW_INDEXSTATE_NOINDEXFOLLOW; } return parent::noIndex(); } In our tests the page was always NOINDEX, NOFOLLOW |
