This project provides an AJAX pager for use by developers instead of the standard Drupal pager which involves full page reloads. This module is useful for developers who are using EntityFieldQuery to generate paginated displays because the other choice is to use Views. While using Views is not a terrible choice, its functionality is limited when used with alternate field storage backends (such as MongoDB).

This project contains 2 modules, one that provides the needed functionality and one that provides an example. To view the example, generate a bunch of nodes (500 is a good number) and activate the ajax_pagination_example module. Then navigate to /ajax-pagination-example which will display a paginated display of node teasers (10 at a time).

Project page: https://drupal.org/sandbox/bthompsonmindgrub/2165175

Git Repository: `git clone git.drupal.org:sandbox/bthompsonmindgrub/2165175.git`

Thanks!

Comments

PA robot’s picture

PA robot commented
Status: Needs review » Needs work

There are some errors reported by automated review tools, did you already check them? See http://pareview.sh/pareview/httpgitdrupalorgsandboxbthompsonmindgrub2165...

We are currently quite busy with all the project applications and we prefer projects with a review bonus. Please help reviewing and put yourself on the high priority list, then we will take a look at your project right away :-)

Also, you should get your friends, colleagues or other community members involved to review this application. Let them go through the review checklist and post a comment that sets this issue to "needs work" (they found some problems with the project) or "reviewed & tested by the community" (they found no major flaws).

I'm a robot and this is an automated message from Project Applications Scraper.

rvtraveller’s picture

rvtraveller commented

Re: Automated review -- README.txt is present in the repository and the majority of the CodeSniffer findings are with theme functions that are very similar to those found in Drupal Core. While I can certainly understand the argument for keeping lines to < 80 characters, in this case it makes sense to leave them as they are so they continue to match to the lines in Drupal core.

rvtraveller’s picture

rvtraveller commented
Status: Needs work » Needs review
grabimo’s picture

grabimo commented

Would ajax_pagination.js appear on all pages if it's included in the info. file? I have the similar issue with my module. I searched this site and found hook_page_build() for Drupal 7.x. Hope it may help you as well. Here is API page:

https://api.drupal.org/api/drupal/modules!system!system.api.php/function...

pachabhaiya’s picture

pachabhaiya commented
Status: Needs review » Needs work

Automated report using Coder module shows some minor errors (in ajax_pagination.module):

severity: minorreview: druplart_array_init
Line 13: Array initialization may be unnecessary if the array is assigned in all code paths. [druplart_array_init]
$items = array();

severity: minorreview: druplart_conditional_assignment
Line 367: Consider putting the assignment on a line above the conditional for better readability. [druplart_conditional_assignment]
if ($new_page = implode(',', pager_load_array($page_new[$element], $element, explode(',', $page)))) {

severity: minorreview: druplart_conditional_assignment
Line 375: Consider putting the assignment on a line above the conditional for better readability. [druplart_conditional_assignment]
if ($query_pager = pager_get_query_parameters()) {

1. You can remove empty array declaration: $items = array(); in line 13.
2. For error in line 367, use the following code instead:

$new_page = implode(',', pager_load_array($page_new[$element], $element, explode(',', $page)));
if ($new_page) {

3. For error in line 375, use the following code:

$query_pager = pager_get_query_parameters();
if ($query_pager) {

Besides this, pareview.sh automated test shows lots of errors, please correct those errors.

PA robot’s picture

PA robot commented
Status: Needs work » Closed (won't fix)

Closing due to lack of activity. Feel free to reopen if you are still working on this application (see also the project application workflow).

I'm a robot and this is an automated message from Project Applications Scraper.