You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
required move from Model::paginate() line $page = $page >= 1 ? $page : (ctype_digit($_GET['page'] ?? '') && $_GET['page'] > 1 ? $_GET['page'] : 1); to Pager, because it pager business logic.
To Model::paginate() we must use $pager->getCurrentPage()
The text was updated successfully, but these errors were encountered:
MashinaMashina
added
the
bug
Verified issues on the current code behavior or pull requests that will fix them
label
Dec 28, 2019
MashinaMashina
changed the title
Bug: getCurrentPage try to return string, but must return int
Bug: Pager::getCurrentPage() try to return string, but must return int
Dec 28, 2019
MashinaMashina
changed the title
Bug: Pager::getCurrentPage() try to return string, but must return int
Bug: Move work with $_GET['page'] from Model to Pager
Dec 28, 2019
required move from Model::paginate() line
$page = $page >= 1 ? $page : (ctype_digit($_GET['page'] ?? '') && $_GET['page'] > 1 ? $_GET['page'] : 1);
to Pager, because it pager business logic.To Model::paginate() we must use $pager->getCurrentPage()
The text was updated successfully, but these errors were encountered: