[infradawn]

The following behaviour was reproduced in the demo store.

Add line 8 to shop:product_list partial to force 1 record per page:

7        $records_per_page = isset($records_per_page) ? $records_per_page : 3;
8        $records_per_page = 1;
9        $pagination = $products->paginate($page_index, $records_per_page);

Navigate to any category in the store Catalog, e.g. Photography and:

Fatal error: Call to a member function is_discounted() on a non-object in /home/lemony/sites/demo_new/modules/cms/classes/cms_controller.php(268) : eval()'d code on line 16

I don't currently use 1 record per page functionality, I was just testing my 'show per page' controls. But I can see that being able to scroll through a category 1 product at a time, for example with a full product description on each page, would be useful.

Thanks

iD


In case you get 2 emails, I added this bug report a second time this morning. Last nights site hiccup lost the first one :)

[Aleksey]

Hi. iD!

It is a quirk of the default ActiveRecord implementation. I updated the base class and now the find_all() method will always be returning Db_DataCollection. I tested it with 1 product per page pagination. The update will be published today.

Thanks

[infradawn]

Hi Aleksey,

Does that also 'fix' the need for this line in public function search() ?

if ($records < 2)
   $records = 2;

:)

iD

[Aleksey]

Hi, iD!

You are right. I will publish the update today. Thanks!