Welcome to the Treehouse Community

Want to collaborate on code errors? Have bugs you need feedback on? Looking for an extra set of eyes on your latest project? Get support with fellow developers, designers, and programmers of all backgrounds and skill levels here with the Treehouse Community! While you're at it, check out some resources Treehouse students have shared here.

Looking to learn something new?

Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and join thousands of Treehouse students and alumni in the community today.

Start your free trial

PHP Integrating PHP with Databases Limiting Records in SQL Pagination Function

Thomas Morling
Thomas Morling
12,754 Points
Posted by Thomas Morling
Thomas Morling
12,754 Points

Not sure why this working. Pagination Challenge

Not sure why this doesn't work

<?php include "pagination.php";

/* add function here / function pagination($total_pages, $current_page, $section, $search) { / setting pagination variable / $pagination = "<div class=\"pagination\">"; $pagination .= "Pages: ";
for ($i = 1;$i <= $total_pages;$i++) { if ($i == $current_page) { $pagination .= " <span>$i</span>"; } else { $pagination .= " <a href='catalog.php?"; if (!empty($search)) { $pagination .= "s=".urlencode(htmlspecialchars($search)) . '&' ; } else if (!empty($section)) { $pagination .= "cat=".$section . '&'; } $pagination .= "pg=$i'>$i</a>"; } } $pagination .= "</div>"; return $pagination; } / displaying the pagination */ pagination();

index.php 
<?php
include "pagination.php";

/* add function here */
function pagination($total_pages, $current_page, $section, $search) { 
/* setting pagination variable */
$pagination = "<div class=\"pagination\">";
$pagination .= "Pages: ";  
for ($i = 1;$i <= $total_pages;$i++) {
    if ($i == $current_page) {
        $pagination .= " <span>$i</span>";
    } else {
        $pagination .= " <a href='catalog.php?";
        if (!empty($search)) {
            $pagination .= "s=".urlencode(htmlspecialchars($search)) . '&' ;
        } else if (!empty($section)) {
            $pagination .= "cat=".$section . '&';
        }
        $pagination .= "pg=$i'>$i</a>";
    }
}
$pagination .= "</div>";
return   $pagination;
}
/* displaying the pagination */
pagination();

1 Answer

Matthew Bilz
Matthew Bilz
15,829 Points

I just went through the challenge and I have 2 observations:

1) Make sure on your bottom line, you include the arguments needed to call the pagination function, so:

$pagination = pagination($total_pages, $current_page, $section, $search)

2) For some reason, the final line of the function still need to be an echo statement instead of a return statement to pass the challenge. I dunno about that one, but that's what passed the quiz! So instead of return $pagination:

echo $pagination