From 3d4f92dda59aa83885b01a54e46725dd9f1d07a4 Mon Sep 17 00:00:00 2001 From: yamamoto Date: Sat, 14 Feb 2026 00:05:47 +0900 Subject: [PATCH] fix: cast division results to integer for pagination calculations --- manager/actions/report/logging.static.php | 12 ++++++------ manager/includes/paginate.inc.php | 2 +- 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/manager/actions/report/logging.static.php b/manager/actions/report/logging.static.php index f1148546d..7d35b1a00 100644 --- a/manager/actions/report/logging.static.php +++ b/manager/actions/report/logging.static.php @@ -288,7 +288,7 @@ class="inputbox" style="width:100px" // Load up the 2 array in order to display result $array_paging = $p->getPagingArray(); $array_row_paging = $p->getPagingRowArray(); - $current_row = evo()->input_get('int_cur_position', 0) / $int_num_result; + $current_row = (int)(evo()->input_get('int_cur_position', 0) / $int_num_result); // Display the result as you like... echo sprintf('

%s %s', lang('paging_showing'), $array_paging['lower']); @@ -303,11 +303,11 @@ class="inputbox" style="width:100px" $paging .= $array_paging['previous_link'] . lang('paging_prev') . (isset($array_paging['previous_link']) ? " " : " "); $pagesfound = sizeof($array_row_paging); if ($pagesfound > 6) { - $paging .= $array_row_paging[$current_row - 2]; - $paging .= $array_row_paging[$current_row - 1]; - $paging .= $array_row_paging[$current_row]; - $paging .= $array_row_paging[$current_row + 1]; - $paging .= $array_row_paging[$current_row + 2]; + $start = max(0, min($current_row - 2, $pagesfound - 5)); + $end = min($pagesfound - 1, $start + 4); + for ($i = $start; $i <= $end; $i++) { + $paging .= $array_row_paging[$i]; + } } else { for ($i = 0; $i < $pagesfound; $i++) { $paging .= $array_row_paging[$i] . " "; diff --git a/manager/includes/paginate.inc.php b/manager/includes/paginate.inc.php index 5b7bf5539..e1022e330 100644 --- a/manager/includes/paginate.inc.php +++ b/manager/includes/paginate.inc.php @@ -119,7 +119,7 @@ public function getPagingRowArray() private function getNumberOfPage() { - return $this->int_nbr_row / $this->int_num_result; + return (int)ceil($this->int_nbr_row / $this->int_num_result); } private function getCurrentPage()