Attached patch performs a few things:

- The page separator string in the module's settings should be required, or Paging's code will fail elsewhere.
- Renamed variables in nodeapi('load') to make that code more understandable.
- Adds a missing break for nodeapi('view'). This one might fix some other reported bugs in the queue, dunno.

CommentFileSizeAuthor
paging-DRUPAL-5.cleanup.patch3.39 KBsun

Comments

Gurpartap Singh’s picture

Gurpartap Singh commented
Status: Needs review » Reviewed & tested by the community

Fixed in 6.x-dev. Need to commit to DRUPAL-5.

Gurpartap Singh’s picture

Gurpartap Singh commented
Status: Reviewed & tested by the community » Fixed

Fixed in 5.x branch. Thanks!

http://drupal.org/cvs?commit=166583

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.