bpolaszek / bentools-pager

src/BenTools/Pager/Page.php

Braces +33 added lines, -28 removed lines

@@ -72,8 +72,9 @@ 
      * @return int
      */
     public function getOffset() {
-        if (!$this->offset)
-            $this->offset   =   ($this->getIndex() * $this->getPager()->getResultsPerPage());
+        if (!$this->offset) {
+                    $this->offset   =   ($this->getIndex() * $this->getPager()->getResultsPerPage());
+        }
         return $this->offset;
     }
 
@@ -82,8 +83,9 @@ 
      * @return int
      */
     public function getResultCount() {
-        if (!$this->resultCount)
-            $this->resultCount   =   $this->isLastPage() ? $this->getPager()->getTotalResultCount() - $this->getOffset() : $this->getPager()->getResultsPerPage();
+        if (!$this->resultCount) {
+                    $this->resultCount   =   $this->isLastPage() ? $this->getPager()->getTotalResultCount() - $this->getOffset() : $this->getPager()->getResultsPerPage();
+        }
         return $this->resultCount;
     }
 
@@ -92,8 +94,9 @@ 
      * @return bool
      */
     public function isFirstPage() {
-        if (!$this->isFirstPage)
-            $this->isFirstPage    =    ($this->getIteration() == 1);
+        if (!$this->isFirstPage) {
+                    $this->isFirstPage    =    ($this->getIteration() == 1);
+        }
         return $this->isFirstPage;
     }
 
@@ -102,8 +105,9 @@ 
      * @return bool
      */
     public function isLastPage() {
-        if (!$this->isLastPage)
-            $this->isLastPage    =    ($this->getIteration() == $this->pager->getNbPages());
+        if (!$this->isLastPage) {
+                    $this->isLastPage    =    ($this->getIteration() == $this->pager->getNbPages());
+        }
         return $this->isLastPage;
     }
 
@@ -112,8 +116,9 @@ 
      * @return bool
      */
     public function isCurrentPage() {
-        if (!$this->isCurrentPage)
-            $this->isCurrentPage    =    ($this->getIteration() == ($this->pager->getCurrentPageIteration()));
+        if (!$this->isCurrentPage) {
+                    $this->isCurrentPage    =    ($this->getIteration() == ($this->pager->getCurrentPageIteration()));
+        }
         return $this->isCurrentPage;
     }
 
@@ -122,8 +127,9 @@ 
      * @return bool
      */
     public function isPreviousPage() {
-        if (!$this->isPreviousPage)
-            $this->isPreviousPage    =    ($this->getIteration() == ($this->pager->getCurrentPageIteration() - 1));
+        if (!$this->isPreviousPage) {
+                    $this->isPreviousPage    =    ($this->getIteration() == ($this->pager->getCurrentPageIteration() - 1));
+        }
         return $this->isPreviousPage;
     }
 
@@ -132,8 +138,9 @@ 
      * @return bool
      */
     public function isNextPage() {
-        if (!$this->isNextPage)
-            $this->isNextPage    =    ($this->getIteration() == ($this->pager->getCurrentPageIteration() + 1));
+        if (!$this->isNextPage) {
+                    $this->isNextPage    =    ($this->getIteration() == ($this->pager->getCurrentPageIteration() + 1));
+        }
         return $this->isNextPage;
     }
 
@@ -168,18 +175,17 @@ 
     public function getUrl() {
 
         if ($this->pager->getRewriteString()) {
-            if (preg_match('#' . sprintf($this->pager->getRewriteString(), '([0-9]+)') . '#', (string) $this->pager->getUrl()))
-                $this->setUrl(new Url(preg_replace('#' . sprintf($this->pager->getRewriteString(), '([0-9]+)') . '#', sprintf($this->pager->getRewriteString(), $this->getIteration()), (string) $this->pager->getUrl())));
-            else {
+            if (preg_match('#' . sprintf($this->pager->getRewriteString(), '([0-9]+)') . '#', (string) $this->pager->getUrl())) {
+                            $this->setUrl(new Url(preg_replace('#' . sprintf($this->pager->getRewriteString(), '([0-9]+)') . '#', sprintf($this->pager->getRewriteString(), $this->getIteration()), (string) $this->pager->getUrl())));
+            } else {
                 $url        =   clone $this->getPager()->getUrl();
                 $this->setUrl($url->appendToPath(sprintf($this->getPager()->getRewriteString(), $this->getIteration())));
             }
 
+        } else {
+                    $this->setUrl(Url::NewInstance($this->pager->getUrl())->setParam($this->pager->getQueryParam(), $this->getIteration()));
         }
 
-        else
-            $this->setUrl(Url::NewInstance($this->pager->getUrl())->setParam($this->pager->getQueryParam(), $this->getIteration()));
-
         return $this->url;
     }
 
@@ -216,14 +222,13 @@ 
      * @return mixed|null
      */
     public function offsetGet($offset) {
-        if (is_callable([$this, 'get' . $offset]))
-            return call_user_func([$this, 'get' . $offset]);
-
-        elseif (is_callable([$this, $offset]))
-            return call_user_func([$this, $offset]);
-
-        else
-            return null;
+        if (is_callable([$this, 'get' . $offset])) {
+                    return call_user_func([$this, 'get' . $offset]);
+        } elseif (is_callable([$this, $offset])) {
+                    return call_user_func([$this, $offset]);
+        } else {
+                    return null;
+        }
     }
 
     /**

src/BenTools/Pager/Pager.php

Braces +55 added lines, -50 removed lines

@@ -112,17 +112,15 @@ 
 
     public function __construct($url = null, $resultsPerPage = null, $totalResultCount = null, $queryParam = self::DEFAULT_QUERY_PARAM, $rewriteString = null, $currentPageIteration = null, $delta = self::DEFAULT_DELTA) {
 
-        if ($url instanceof Url)
-            $this->setUrl($url);
-
-        elseif ($url)
-            $this->setUrl(new Url($url));
-
-        elseif (isset($_SERVER['REQUEST_URI']))
-            $this->setUrl(new Url($_SERVER['REQUEST_URI']));
-
-        else
-            $this->setUrl(new Url('/'));
+        if ($url instanceof Url) {
+                    $this->setUrl($url);
+        } elseif ($url) {
+                    $this->setUrl(new Url($url));
+        } elseif (isset($_SERVER['REQUEST_URI'])) {
+                    $this->setUrl(new Url($_SERVER['REQUEST_URI']));
+        } else {
+                    $this->setUrl(new Url('/'));
+        }
 
         $this   ->  setResultsPerPage($resultsPerPage)
                 ->  setTotalResultCount($totalResultCount)
@@ -184,10 +182,11 @@ 
      */
     public function getCurrentPageIteration() {
         if (!$this->currentPageIteration) {
-            if (!is_null($this->rewriteString) && preg_match('#' . sprintf($this->rewriteString, '([0-9]+)') . '#', (string) $this->url, $matches))
-                $this->currentPageIteration = (int) ((isset($matches[1]) && $matches[1] > 0) ? $matches[1] : 1);
-            else
-                $this->currentPageIteration = ($this->url->getParam($this->queryParam)) ? (int) $this->url->getParam($this->queryParam) : 1;
+            if (!is_null($this->rewriteString) && preg_match('#' . sprintf($this->rewriteString, '([0-9]+)') . '#', (string) $this->url, $matches)) {
+                            $this->currentPageIteration = (int) ((isset($matches[1]) && $matches[1] > 0) ? $matches[1] : 1);
+            } else {
+                            $this->currentPageIteration = ($this->url->getParam($this->queryParam)) ? (int) $this->url->getParam($this->queryParam) : 1;
+            }
         }
         return (int) $this->currentPageIteration;
 
@@ -234,10 +233,11 @@ 
      * @return Page Instance
      */
     public function getPreviousPage() {
-        if ($this->getCurrentPageIteration() - 1 >= 1)
-            return $this->getPageInstance($this->getCurrentPageIteration() - 1);
-        else
-            return $this->getFirstPage();
+        if ($this->getCurrentPageIteration() - 1 >= 1) {
+                    return $this->getPageInstance($this->getCurrentPageIteration() - 1);
+        } else {
+                    return $this->getFirstPage();
+        }
     }
 
     /**
@@ -246,10 +246,11 @@ 
      * @return Page Instance
      */
     public function getNextPage() {
-        if ($this->getCurrentPageIteration() + 1 <= $this->getNbPages())
-            return $this->getPageInstance($this->getCurrentPageIteration() + 1);
-        else
-            return $this->getLastPage();
+        if ($this->getCurrentPageIteration() + 1 <= $this->getNbPages()) {
+                    return $this->getPageInstance($this->getCurrentPageIteration() + 1);
+        } else {
+                    return $this->getLastPage();
+        }
     }
 
     /**
@@ -269,10 +270,12 @@ 
      */
     public function run() {
 
-        if (!$this->resultsPerPage)
-            throw new PagerException(get_called_class() . "::setResultsPerPage has to be called first.");
-        if (!is_integer($this->totalResultCount))
-            throw new PagerException(get_called_class() . "::setTotalResultCount has to be called first.");
+        if (!$this->resultsPerPage) {
+                    throw new PagerException(get_called_class() . "::setResultsPerPage has to be called first.");
+        }
+        if (!is_integer($this->totalResultCount)) {
+                    throw new PagerException(get_called_class() . "::setTotalResultCount has to be called first.");
+        }
 
         $this->pages        =   [];
 
@@ -293,27 +296,31 @@ 
 
                 ####    SUPERIOR DELTA        ####
                 # If the next page doesn't overtstep the last one
-                if ($this->getCurrentPageIteration() + $i <= $this->getNbPages())
-                    $this->pages[] = $this->getPageInstance($this->getCurrentPageIteration() + $i);
+                if ($this->getCurrentPageIteration() + $i <= $this->getNbPages()) {
+                                    $this->pages[] = $this->getPageInstance($this->getCurrentPageIteration() + $i);
+                }
 
                 # Otherwise, we put the remaining delta before, providing it doesn't overstep the 1st one
-                elseif ($this->getCurrentPageIteration() - $i >= 1)
-                    $this->pages[] = $this->getPageInstance($this->getCurrentPageIteration() - $i);
+                elseif ($this->getCurrentPageIteration() - $i >= 1) {
+                                    $this->pages[] = $this->getPageInstance($this->getCurrentPageIteration() - $i);
+                }
 
                 ####    INFERIOR DELTA        ####
                 # If the previous page doesn't overstep the 1st one
-                if ($this->getCurrentPageIteration() - $i >= 1)
-                    $this->pages[] = $this->getPageInstance($this->getCurrentPageIteration() - $i);
+                if ($this->getCurrentPageIteration() - $i >= 1) {
+                                    $this->pages[] = $this->getPageInstance($this->getCurrentPageIteration() - $i);
+                }
 
                 # Otherwise, we put the remaining delta before, providing it doesn't overstep the last one
-                elseif ($this->getCurrentPageIteration() + $i <= $this->getNbPages())
-                    $this->pages[] = $this->getPageInstance($this->getCurrentPageIteration() + $i);
+                elseif ($this->getCurrentPageIteration() + $i <= $this->getNbPages()) {
+                                    $this->pages[] = $this->getPageInstance($this->getCurrentPageIteration() + $i);
+                }
 
             }
-        }
-        else {
-            for ($i = 1; $i <= $this->getNbPages(); $i++)
-                $this->pages[]  =   $this->getPageInstance($i);
+        } else {
+            for ($i = 1; $i <= $this->getNbPages(); $i++) {
+                            $this->pages[]  =   $this->getPageInstance($i);
+            }
         }
         $this->pages = array_unique((array) $this->pages);
         sort($this->pages);
@@ -487,17 +494,15 @@ 
      */
     public function offsetGet($offset) {
 
-        if (is_numeric($offset))
-            return array_key_exists($offset, $this->getPages()) ? $this->getPages()[$offset] : null;
-
-        elseif (is_callable([$this, 'get' . $offset]))
-            return call_user_func([$this, 'get' . $offset]);
-
-        elseif (is_callable([$this, 'is' . $offset]))
-            return call_user_func([$this, 'is' . $offset]);
-
-        else
-            return null;
+        if (is_numeric($offset)) {
+                    return array_key_exists($offset, $this->getPages()) ? $this->getPages()[$offset] : null;
+        } elseif (is_callable([$this, 'get' . $offset])) {
+                    return call_user_func([$this, 'get' . $offset]);
+        } elseif (is_callable([$this, 'is' . $offset])) {
+                    return call_user_func([$this, 'is' . $offset]);
+        } else {
+                    return null;
+        }
     }
 
     /**