4747
4848namespace Platine \Pagination \Renderer ;
4949
50+ use Platine \Pagination \Page ;
5051use Platine \Pagination \Pagination ;
5152use Platine \Pagination \RendererInterface ;
5253
5354/**
54- * Class DefaultRenderer
55+ * @class DefaultRenderer
5556 * @package Platine\Pagination\Renderer
5657 */
5758class DefaultRenderer implements RendererInterface
@@ -68,27 +69,38 @@ public function render(Pagination $pagination): string
6869 $ html = '<ul class = "pagination"> ' ;
6970
7071 if ($ pagination ->hasPreviousPage ()) {
71- $ html .= '<li><a href = " ' . $ pagination ->getPreviousUrl () . '">« '
72- . $ pagination ->getPreviousText () . '</a></li> ' ;
72+ $ html .= sprintf (
73+ '<li><a href = "%s">« %s</a></li> ' ,
74+ $ pagination ->getPreviousUrl (),
75+ $ pagination ->getPreviousText ()
76+ );
7377 }
7478
75- /** @var array<\Platine\Pagination\ Page> $pages */
79+ /** @var Page[] $pages */
7680 $ pages = $ pagination ->getPages ();
7781
7882 foreach ($ pages as $ page ) {
7983 if ($ page ->getUrl () !== null ) {
80- $ html .= '<li ' . ($ page ->isCurrent () ? ' class = "active" ' : '' )
81- . '><a href = " ' . $ page ->getUrl () . '"> '
82- . $ page ->getNumber () . '</a></li> ' ;
84+ $ html .= sprintf (
85+ '<li%s><a href = "%s">%d</a></li> ' ,
86+ $ page ->isCurrent () ? ' class = "active" ' : '' ,
87+ $ page ->getUrl (),
88+ $ page ->getNumber ()
89+ );
8390 } else {
84- $ html .= '<li class = "disabled"><span> '
85- . $ page ->getNumber () . '</span></li> ' ;
91+ $ html .= sprintf (
92+ '<li class = "disabled"><span>%s</span></li> ' ,
93+ $ page ->getNumber ()
94+ );
8695 }
8796 }
8897
8998 if ($ pagination ->hasNextPage ()) {
90- $ html .= '<li><a href = " ' . $ pagination ->getNextUrl () . '"> '
91- . $ pagination ->getNextText () . ' »</a></li> ' ;
99+ $ html .= sprintf (
100+ '<li><a href = "%s">%s »</a></li> ' ,
101+ $ pagination ->getNextUrl (),
102+ $ pagination ->getNextText ()
103+ );
92104 }
93105
94106 $ html .= '</ul> ' ;
0 commit comments