evoteam team mailing list archive

[Daniel Hahler <launchpad@xxxxxxxxxx>]

So it makes sense for two at least.
You have not said, if it makes sense for you, too - but I'm going to close this as Invalid, until proven wrong.. ;)

** Changed in: b2evolution
       Status: Incomplete => Invalid

-- 
ItemListLight with a NULL limit doesn't return a unlimited query
https://bugs.launchpad.net/bugs/422352
You received this bug notification because you are a member of Evoteam,
which is subscribed to b2evolution.

Status in b2evolution: Invalid

Bug description:
I tought that using a ItemListLight with a NULL limit would issue a query with no SQL LIMIT clause.

But, ItemListLight inherits from DataObjectList2, that extends Results, that have a line like this:

		$this->limit = is_null($limit) ? $UserSettings->get('results_per_page') : $limit;

To users that are not logged in, the limit is 20, that is setterd in the ItemListLight itself, even if $limit on constructor is null.

Is this a bug? To me, a ItemListLight with a NULL limit should have... no limit :)