Limit for history article as config var

English forums
1

this might be an easy one:
in loadHistoryArticles() theres a hardcoded limit for 4 articles.

why not makin that configurable like with crossselling?

2

Hi,

I think a small module with

if (count($aHistoryArticles) > 7) {
             array_shift($aHistoryArticles);
         }

(if you need just 6 products) would make more sense in a manner of keeping the code lean and lank, wouldn’t it?

Regards

3

of course a module would do it, but you have to write it, enable it etc
nothing for lazy guys :wink:

imho i think hardcoded limits are quite unclean

and it would be just two more lines:
$myConfig = $this->getConfig();
$this->setSqlLimit( 0, $myConfig->getConfigParam( ‘iNrofLatestArticles’ ));
plus the admin form

as i said this might be an easy one

4

Hi,

at the end of the day, one shouldn’t find a hard coded value in a CMS :slight_smile:
https://bugs.oxid-esales.com/view.php?id=3113

Regards