[Typo3-shop] bug in 2.0.2.14 - OVERVIEW doesn't get linked to basket page

[Franz Holzinger]

Hello Rainer,
>>>to correct the columnsview, line 695 (v2.0.2.14) correct $icolcount
>>>< ... to icolcunt <= ..., otherwise the last <td></td> field isn't
>>>created and the page doesn't validate.
>>
>>I cannot follow you with this. Do you have an example configuration
>>where this error occurs?
> 
> suppose you have 3 products in item_list_display with displayBasketColumns = 
> 2. this will lead to 2 rows, the first on displaying 2 products, the second 
> row showing 1 product.
> as you know, these products get wrapped in <td></td>. the last row has only 
> <tr><td>mys prod</td></tr>, the last <td></td> is skipped due to $icolcount 
> < ... to $icolcount <= ... - the while condition drops out just 1 item too 
> soon =:o)
But this does not happen with the Bananguard Shop configuration at
http://fholzinger.com/index.php?id=39
which I have set to 2 columns showing 9 articles to test this scenario.
If the same code produces an error with other products data then I want 
to know why.

>>>also, i cannot confirm that the "PIDbasket is ignored in
>>>single_item_display" is solved with your latest version...
>>>unfortunately, i overwrote my corrected version with v2.0.2.15...
>>
>>I will check this later. This is no serious error. But if you have a
>>correction, mail it to me.
> 
> 
> well, it is a serious error if you need this feature, and the manual says 
> that it should work like this... i posted the correction for this in this 
> thread.
Where? Do you still have it?

>>Anyone who seriously wants to contribute will get access to the CVS.
>>There are already 4 persons with access to the CVS of tt_products.
> 
> ok, nice to hear. but i would consider testing, bug reporting and possibly 
> providing patches a serious contribution as well, don't you think so, too?
Yes. But I meant that I only want to give CVS access to persons who will 
really do something then. Just write me an email if you want to get this 
access. Before any update to CVS it will still be necessary to write me 
an email with the intended changes (and diff file) and wait for my 
placet (ok). Or even additionally discuss bigger changes before starting 
with them.


Greetings,

Franz