X-Cart forums

[xtech]

when I put this for ($i = 24; 576 >= $i; $i = $i + 24) {
$perPageValues[] = $i;
}
it is displaying 24,48,72 and 96 only.Have there any upper limit set to 96?
If we display from 24 to 576 then will it create any issue?

Thanks,
xtech

[PhilJ]

Well, if you have people viewing that many products at once, a) it may lock up their browser (especially if on a mobile device) b) it may slow your store right down.

[xtech]

Thank you for your logic.So what is best?

I have again revert back to

for ($i = 24; 96 >= $i; $i = $i + 24) {
$perPageValues[] = $i;
}

Is it ok now?
Thanks,
xtech

[PhilJ]

Yes, although even 96 per page seems a lot to me...

[xtech]

Originally Posted by PhilJ

Yes, although even 96 per page seems a lot to me...

What is best practice?

[PhilJ]

What is best practice?

Personally, I'd limit to around 50 per page max. Any further questions on this matter, please take to a ticket, thanks.

[elmirage001]

Untill Phil posts...

In x-cart v4.6.6 it looks like the security update affected 2 reBOOT files which will have to be manually patched.

\skin\reboot\modules\Customer_Reviews\vote_reviews .tpl
\skin\reboot\customer\main\address_box.tpl

Compare the original x-cart version vs the security version to see what changes took place and then manually patch the reBOOT version of the files.

Use a file comparison tool like Beyond Compare http://www.scootersoftware.com/

[PhilJ]

In x-cart v4.6.6 it looks like the security update affected 2 reBOOT files which will have to be manually patched.

I have the patched files available for reBOOT 3.41 / 3.50. So if anyone wants them, PM me and I'll send you details.

In addition, reBOOT will be updated for v4.7.2 shortly.

[elmirage001]

Hi Phil, Thank you very much for sending the file and also for adding the additional reBOOT file that needed the 'escape' !

Very much appreciated!!

Paul

[PhilJ]

reBOOT v3.51 released (X-Cart v4.7.2). Get in touch for updated files. Demo