This module adds a pagination for main menu to your Drupal site.
This module can be used to traverse trough the pages of main menu using pagination. Users just need to click on the links to traverse into next page or into previous page of main menu navigation.

Sandbox link: http://drupal.org/sandbox/swarnadip2013/1988972
GIT repository link: http://drupalcode.org/sandbox/swarnadip2013/1988972.git/

NOTE: please refer to the "Commit for new module Main menu pager" in git repository.

Comments

swarnadip2013’s picture

Issue summary: View changes

Sandbox link has been added

PA robot’s picture

Status: Needs review » Needs work

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

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.

swarnadip2013’s picture

Title: Main menu pager » [D7] Main menu pager
Priority: Normal » Major
Status: Needs work » Needs review

Working on updated code according to issues specified in http://ventral.org/pareview/httpgitdrupalorgsandboxswarnadip20131988972git. When gets completed I will commit the code again and reply accordingly.

Dalay’s picture

Status: Needs review » Needs work

Hi, swarnadip2013.

Did everything according to the instructions in README.txt. After assigning the block into content region received errors only but not the new block:

  • html is broken;
  • Notice: Undefined index: admin/structure/menu/manage/main-menu in pagination->getLinkTitle() (line 73 of ...class.pagination.php);
  • Notice: Undefined index: node in pagination->getLinkTitle() (line 73 of ...class.pagination.php);
dclavain’s picture

Hi @swarnadip2013,

Automated Review

  1. Creates a branch specified, e.g. 7.x-1.x and works there. Delete master branch.
  2. Remove file LICENSE.txt.
  3. Remove version, project, and datestamp variables from .info file
  4. Remove main_menu_pager directory.
  5. Add comment from get_pager_items and main_menu_pager_theme about the purpose of the function.

Manual review

  1. I have installed your module and it gives me the following notice:

    Notice: Undefined index: node in pagination->getLinkTitle() (line 73 of ./sites/all/modules/contrib/main_menu_pager/class.pagination.php).

  2. Use the theme_item_list function instead of defining your own.
  3. I think that the module is too small to be able to evaluate.
swarnadip2013’s picture

Update code to fix problems

swarnadip2013’s picture

Update code to fix problems

PA robot’s picture

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.

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

PA robot’s picture

Issue summary: View changes

Update description with git repository link