Yii2: yii\data\Pagination pageSizeLimit & per-page=0

[tsvetiligo]

Я обнаружил такое поведение: если передать в значение pageSizeLimit массив с более чем двумя элементами, то pageSizeLimit не срабатывает вообще и есть возможность получить все записи ActiveDataProvider при указании per-page=0.
Не знаю много ли людей способно повторить это поведение, но именно так было в шаблоне который я использовал - chrisleekr/yii2-angular-boilerplate

[samdark]

Что именно вы предлагаете сделать?

[tsvetiligo]

При выставлении pageSizeLimit = false, pageSize становится равен defaultPageSize. А в случае некорректного значения pageSizeLimit типа [1,2,3], pageSize становится 0 и выводятся все записи. Я думаю что некорректное значение pageSizeLimit должно приравниваться к значению false.

[samdark]

А точно сейчас не так? https://github.com/yiisoft/yii2/blob/master/framework/data/Pagination.php#L238

_// в следующий раз попробуйте issue делать на английском, а то не все читать могут_

[tsvetiligo]

в https://github.com/yiisoft/yii2/blob/master/framework/data/Pagination.php#L238 лишнее _&& count($this->pageSizeLimit) === 2_
а тут - https://github.com/yiisoft/yii2/blob/master/framework/data/Pagination.php#L216 надо добавить _|| !isset($this->pageSizeLimit[0], $this->pageSizeLimit[1])_

[samdark]

Сделаете pull request?

[tsvetiligo]

Pull request is created.