View Issue Details

Jump to Notes
IDProjectCategoryView StatusDate SubmittedLast Update
0000848OXID eShop (all versions)4.09. SEO, SEO URLpublic2009-04-30 16:122012-12-10 13:35
ReporterGregor Assigned To 
PriorityhighSeveritycrashReproducibilityalways
Status resolvedResolutionfixed 
Product Version4.1.1 revision 18442 
Fixed in Version4.1.2 revision 18998 
Summary0000848: generating of category links fails
DescriptionAfter updating to 4.1.1. we have the following behaviour:

opening a category with several pages and clicking on the page number puts the page number behind the link; but it does not remove the number, if you click again.
This generates a link looking like this:
http://www.stempelmeer.de/Papier-Karten/9/12/7/2/6/12/6/6/12/7/5/10/2/10/7/10/5/3/5/3/4/4/8/6/4/12/3/4/2/7/2/6/11/2/2/2/11/8/12/4/

This value is stored in table OXSEO field OXSEOURL
and of course nothing is found
Additional InformationFor each paging a new value is stored in the SEO-Table.
As a quick measure I deleted the seo-entry in the table, but it is directly restored, if someone uses paging.
TagsNo tags attached.
Theme
BrowserAll
PHP Version5.2.6
Database Version5.0.33

Activities

Gregor

2009-05-05 17:46

reporter   ~0000844

05.05.2009:
dainius_bigelis wrote: could not reproduce this error.

Answer:
As a measure to avoid the problem for customers I increased the number of "products shown in article list" to 100. This prevents most of the categories from showing the paging.

Currently we have more than 100 products in the category http://www.stempelmeer.de/Embellishments/Knoepfe/ and I can reproduce this behaviour.

page 1: http://www.stempelmeer.de/Embellishments/Knoepfe/3/
page 2: http://www.stempelmeer.de/Embellishments/Knoepfe/3/2/
page 3: http://www.stempelmeer.de/Embellishments/Knoepfe/3/3/

Clicks on the page <<|1 2 3 |>> increases the number of page-indicators at the link http://www.stempelmeer.de/Embellishments/Knoepfe/3/2/3/2/

arvydas_vapsva

2009-05-06 14:06

reporter   ~0000857

Problem is in oxseoencoder::_loadFromDb() method:

instead:

    if ($sParams) {
        if ($blStrictParamsCheck) {
            $sQ .= " and oxparams = '{$sParams}'";
        } else {
            $sQ .= " order by oxparams = '{$sParams}' desc";
        }
    }

shoud be:

    $sParams = $sParams ? $sParams : '';
    if ( $sParams && $blStrictParamsCheck ) {
        $sQ .= " and oxparams = '{$sParams}'";
    } else {
        $sQ .= " order by oxparams = '{$sParams}' desc";
    }
    $sQ .= " limit 1";

or first record according to index will be returned.