Skip to content

Commit c7d672e

Browse files
committed
refactor: change method names more natural
1 parent 995d7b2 commit c7d672e

2 files changed

Lines changed: 9 additions & 9 deletions

File tree

system/Pager/PagerRenderer.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -407,7 +407,7 @@ public function getNextPage()
407407
*
408408
* @return integer
409409
*/
410-
public function getPageNumberFirst(): int
410+
public function getFirstPageNumber(): int
411411
{
412412
return $this->first;
413413
}
@@ -417,7 +417,7 @@ public function getPageNumberFirst(): int
417417
*
418418
* @return integer
419419
*/
420-
public function getPageNumberCurrent(): int
420+
public function getCurrentPageNumber(): int
421421
{
422422
return $this->current;
423423
}
@@ -427,7 +427,7 @@ public function getPageNumberCurrent(): int
427427
*
428428
* @return integer
429429
*/
430-
public function getPageNumberLast(): int
430+
public function getLastPageNumber(): int
431431
{
432432
return $this->last;
433433
}

tests/system/Pager/PagerRendererTest.php

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -573,9 +573,9 @@ public function testGetPageNumber()
573573
];
574574
$pager = new PagerRenderer($details);
575575

576-
$this->assertEquals(1, $pager->getPageNumberFirst());
577-
$this->assertEquals(3, $pager->getPageNumberCurrent());
578-
$this->assertEquals(10, $pager->getPageNumberLast());
576+
$this->assertEquals(1, $pager->getFirstPageNumber());
577+
$this->assertEquals(3, $pager->getCurrentPageNumber());
578+
$this->assertEquals(10, $pager->getLastPageNumber());
579579
}
580580

581581
public function testGetPageNumberSetSurroundCount()
@@ -591,8 +591,8 @@ public function testGetPageNumberSetSurroundCount()
591591
$pager = new PagerRenderer($details);
592592
$pager->setSurroundCount(2);
593593

594-
$this->assertEquals(3, $pager->getPageNumberFirst());
595-
$this->assertEquals(5, $pager->getPageNumberCurrent());
596-
$this->assertEquals(7, $pager->getPageNumberLast());
594+
$this->assertEquals(3, $pager->getFirstPageNumber());
595+
$this->assertEquals(5, $pager->getCurrentPageNumber());
596+
$this->assertEquals(7, $pager->getLastPageNumber());
597597
}
598598
}

0 commit comments

Comments
 (0)