← Back to team overview

evoteam team mailing list archive

[Bug 422352] [NEW] ItemListLight with a NULL limit doesn't return a unlimited query

 

Public bug reported:

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

** Affects: b2evolution
     Importance: Undecided
         Status: New

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

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



Follow ups

References