| @@ -74,10 +74,8 @@ | ||
| 74 | 74 | self::$page = (self::$current > $show ? (self::$current - $show) : 1); | 
| 75 | 75 |              if (self::$current < (self::$lastpage - $show)) { | 
| 76 | 76 | self::$lastpage = ((self::$current <= $show) ? (self::$current + ($numpages - self::$current)) : (self::$current + $show)); | 
| 77 | - } | |
| 78 | -            else { self::$page = self::$current - ($numpages - ((ceil($records / $maxshown) - self::$current)) - 1); } | |
| 79 | - } | |
| 80 | -        else { self::$page = 1; } | |
| 77 | +            } else { self::$page = self::$current - ($numpages - ((ceil($records / $maxshown) - self::$current)) - 1); } | |
| 78 | +        } else { self::$page = 1; } | |
| 81 | 79 | } | 
| 82 | 80 | |
| 83 | 81 | /** | 
| @@ -17,8 +17,8 @@ discard block | ||
| 17 | 17 | * @param string $class This should be the class or classes that you wish to give to the pagination object | 
| 18 | 18 | * @return $this | 
| 19 | 19 | */ | 
| 20 | -    public function setPaginationClass($class){ | |
| 21 | -        if((!empty(trim($class)))){ | |
| 20 | +    public function setPaginationClass($class) { | |
| 21 | +        if ((!empty(trim($class)))) { | |
| 22 | 22 | $this->pagerClass = $class; | 
| 23 | 23 | } | 
| 24 | 24 | return $this; | 
| @@ -28,7 +28,7 @@ discard block | ||
| 28 | 28 | * Returns the class to give to the pagination object | 
| 29 | 29 | * @return type | 
| 30 | 30 | */ | 
| 31 | -    public function getPaginationClass(){ | |
| 31 | +    public function getPaginationClass() { | |
| 32 | 32 | return $this->pagerClass; | 
| 33 | 33 | } | 
| 34 | 34 | |
| @@ -37,8 +37,8 @@ discard block | ||
| 37 | 37 | * @param string $class This should be the class to assign on active elements | 
| 38 | 38 | * @return $this | 
| 39 | 39 | */ | 
| 40 | -    public function setActiveClass($class){ | |
| 41 | -        if((!empty(trim($class)))){ | |
| 40 | +    public function setActiveClass($class) { | |
| 41 | +        if ((!empty(trim($class)))) { | |
| 42 | 42 | $this->liActiveClass = $class; | 
| 43 | 43 | } | 
| 44 | 44 | return $this; | 
| @@ -48,7 +48,7 @@ discard block | ||
| 48 | 48 | * Returns the class to assign to active li elements | 
| 49 | 49 | * @return string $class This should be the class to assign on active elements | 
| 50 | 50 | */ | 
| 51 | -    public function getActiveClass(){ | |
| 51 | +    public function getActiveClass() { | |
| 52 | 52 | return $this->liActiveClass; | 
| 53 | 53 | } | 
| 54 | 54 | |
| @@ -4,7 +4,7 @@ discard block | ||
| 4 | 4 | use Pager\Pagination; | 
| 5 | 5 | use PHPUnit\Framework\TestCase; | 
| 6 | 6 | |
| 7 | -class PaginationTest extends TestCase{ | |
| 7 | +class PaginationTest extends TestCase { | |
| 8 | 8 | |
| 9 | 9 | protected $pagination; | 
| 10 | 10 | |
| @@ -12,7 +12,7 @@ discard block | ||
| 12 | 12 | $this->pagination = new Pagination(); | 
| 13 | 13 | } | 
| 14 | 14 | |
| 15 | -    public function tearDown(){ | |
| 15 | +    public function tearDown() { | |
| 16 | 16 | unset($this->pagination); | 
| 17 | 17 | } | 
| 18 | 18 | |
| @@ -21,7 +21,7 @@ discard block | ||
| 21 | 21 | * @covers Pager\Pagination::buildLink | 
| 22 | 22 | * @covers Pager\Pagination::getPage | 
| 23 | 23 | */ | 
| 24 | -    public function testCreatePager(){ | |
| 24 | +    public function testCreatePager() { | |
| 25 | 25 | $this->pagination->paging(72, '/test-page'); | 
| 26 | 26 | } | 
| 27 | 27 | |
| @@ -30,7 +30,7 @@ discard block | ||
| 30 | 30 | * @covers Pager\Pagination::buildLink | 
| 31 | 31 | * @covers Pager\Pagination::getPage | 
| 32 | 32 | */ | 
| 33 | -    public function testNoPagerNeeded(){ | |
| 33 | +    public function testNoPagerNeeded() { | |
| 34 | 34 |          $this->assertEquals('', $this->pagination->paging(10, '/test-page')); | 
| 35 | 35 |          $this->markTestIncomplete('Test not yet implemented'); | 
| 36 | 36 | } | 
| @@ -44,7 +44,7 @@ discard block | ||
| 44 | 44 | * @covers Pager\Pagination::preLinks | 
| 45 | 45 | * @covers Pager\Pagination::postLinks | 
| 46 | 46 | */ | 
| 47 | -    public function testPagerWithArrows(){ | |
| 47 | +    public function testPagerWithArrows() { | |
| 48 | 48 |          $this->markTestIncomplete('Test not yet implemented'); | 
| 49 | 49 | } | 
| 50 | 50 | |
| @@ -55,7 +55,7 @@ discard block | ||
| 55 | 55 | * @covers Pager\Pagination::preLinks | 
| 56 | 56 | * @covers Pager\Pagination::postLinks | 
| 57 | 57 | */ | 
| 58 | -    public function testPagerWithQueryString(){ | |
| 58 | +    public function testPagerWithQueryString() { | |
| 59 | 59 |          $this->markTestIncomplete('Test not yet implemented'); | 
| 60 | 60 | } | 
| 61 | 61 | |
| @@ -63,7 +63,7 @@ discard block | ||
| 63 | 63 | * @covers Pager\Pagination::setPaginationClass | 
| 64 | 64 | * @covers Pager\Pagination::getPaginationClass | 
| 65 | 65 | */ | 
| 66 | -    public function testSetPagerClass(){ | |
| 66 | +    public function testSetPagerClass() { | |
| 67 | 67 |          $this->assertEquals('pagination', $this->pagination->getPaginationClass()); | 
| 68 | 68 |          $this->pagination->setPaginationClass('my-class'); | 
| 69 | 69 |          $this->assertNotEquals('pagination', $this->pagination->getPaginationClass()); | 
| @@ -76,7 +76,7 @@ discard block | ||
| 76 | 76 | * @covers Pager\Pagination::setPaginationClass | 
| 77 | 77 | * @covers Pager\Pagination::getPaginationClass | 
| 78 | 78 | */ | 
| 79 | -    public function testSetActiveClass(){ | |
| 79 | +    public function testSetActiveClass() { | |
| 80 | 80 |          $this->assertEquals('active', $this->pagination->getActiveClass()); | 
| 81 | 81 |          $this->pagination->setActiveClass('current'); | 
| 82 | 82 |          $this->assertNotEquals('active', $this->pagination->getActiveClass()); |