AdamB7586 / pagination

src/Pagination.php

Braces +2 added lines, -4 removed lines

@@ -74,10 +74,8 @@
             self::$page = (self::$current > $show ? (self::$current - $show) : 1);
             if (self::$current < (self::$lastpage - $show)) {
                 self::$lastpage = ((self::$current <= $show) ? (self::$current + ($numpages - self::$current)) : (self::$current + $show));
-            }
-            else { self::$page = self::$current - ($numpages - ((ceil($records / $maxshown) - self::$current)) - 1); }
-        }
-        else { self::$page = 1; }
+            } else { self::$page = self::$current - ($numpages - ((ceil($records / $maxshown) - self::$current)) - 1); }
+        } else { self::$page = 1; }
     }
     
     /**

Spacing +6 added lines, -6 removed lines

@@ -17,8 +17,8 @@ 
      * @param string $class This should be the class or classes that you wish to give to the pagination object 
      * @return $this
      */
-    public function setPaginationClass($class){
-        if((!empty(trim($class)))){
+    public function setPaginationClass($class) {
+        if ((!empty(trim($class)))) {
             $this->pagerClass = $class;
         }
         return $this;
@@ -28,7 +28,7 @@ 
      * Returns the class to give to the pagination object
      * @return type
      */
-    public function getPaginationClass(){
+    public function getPaginationClass() {
         return $this->pagerClass;
     }
     
@@ -37,8 +37,8 @@ 
      * @param string $class This should be the class to assign on active elements
      * @return $this
      */
-    public function setActiveClass($class){
-        if((!empty(trim($class)))){
+    public function setActiveClass($class) {
+        if ((!empty(trim($class)))) {
             $this->liActiveClass = $class;
         }
         return $this;
@@ -48,7 +48,7 @@ 
      * Returns the class to assign to active li elements
      * @return string $class This should be the class to assign on active elements
      */
-    public function getActiveClass(){
+    public function getActiveClass() {
         return $this->liActiveClass;
     }
     

tests/PaginationTest.php

Spacing +8 added lines, -8 removed lines

@@ -4,7 +4,7 @@ 
 use Pager\Pagination;
 use PHPUnit\Framework\TestCase;
 
-class PaginationTest extends TestCase{
+class PaginationTest extends TestCase {
     
     protected $pagination;
     
@@ -12,7 +12,7 @@ 
         $this->pagination = new Pagination();
     }
     
-    public function tearDown(){
+    public function tearDown() {
         unset($this->pagination);
     }
     
@@ -26,7 +26,7 @@ 
      * @covers Pager\Pagination::postLinks
      * @covers Pager\Pagination::preLinks
      */
-    public function testCreatePager(){
+    public function testCreatePager() {
         $pager = $this->pagination->paging(106, '/test-page');
         $this->assertStringStartsWith("<ul", $pager);
         $this->assertStringEndsWith("ul>", $pager);
@@ -46,7 +46,7 @@ 
      * @covers Pager\Pagination::postLinks
      * @covers Pager\Pagination::preLinks
      */
-    public function testNoPagerNeeded(){
+    public function testNoPagerNeeded() {
         $this->assertFalse($this->pagination->paging(10, '/test-page'));
     }
     
@@ -60,7 +60,7 @@ 
      * @covers Pager\Pagination::postLinks
      * @covers Pager\Pagination::preLinks
      */
-    public function testPagerArrows(){
+    public function testPagerArrows() {
         $this->markTestIncomplete('Test not yet implemented');
     }
     
@@ -74,7 +74,7 @@ 
      * @covers Pager\Pagination::postLinks
      * @covers Pager\Pagination::preLinks
      */
-    public function testPagerWithQueryString(){
+    public function testPagerWithQueryString() {
         $this->markTestIncomplete('Test not yet implemented');
     }
     
@@ -82,7 +82,7 @@ 
      * @covers Pager\Pagination::setPaginationClass
      * @covers Pager\Pagination::getPaginationClass
      */
-    public function testSetPagerClass(){
+    public function testSetPagerClass() {
         $this->assertEquals('pagination', $this->pagination->getPaginationClass());
         $this->pagination->setPaginationClass('my-class');
         $this->assertNotEquals('pagination', $this->pagination->getPaginationClass());
@@ -95,7 +95,7 @@ 
      * @covers Pager\Pagination::setActiveClass
      * @covers Pager\Pagination::getActiveClass
      */
-    public function testSetActiveClass(){
+    public function testSetActiveClass() {
         $this->assertEquals('active', $this->pagination->getActiveClass());
         $this->pagination->setActiveClass('current');
         $this->assertNotEquals('active', $this->pagination->getActiveClass());