evoteam team mailing list archive

[Walter Cruz <walter.php@xxxxxxxxx>]

This is more or less what FP said on mailing list ;)

-- 
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: Incomplete

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 :)