Navigation through an album seems to not work - at the top of the album there is a nice looking collection of tabs, labelled 1, 2, 3 etc. Trouble is, they all link to the SAME node which is just one of the pictures.. if I click on the thumbnail then I get to the correct picture, and now there are "Previous, First, Next and Last" tabs.. but they also all link to the same picture.

Check http://www.csse.uwa.edu.au/~gordon/drupal/?q=node/19 for example.

Comments

wpd’s picture

wpd commented

I can not recreate this with the latest version of drupal CVS and acidfree CVS.
What versions are you using?
Have customized your theme?

gfroyle’s picture

gfroyle commented

I am using 4.7rc2 and a non-customized FriendsElectric Theme...

gfroyle’s picture

gfroyle commented

..oops and Acidfree 4.7.0

wpd’s picture

wpd commented

You need to get the latest acidfree from CVS and drupal 4.7RC4.
I know there was changes to the pager in drupal since RC2. I am guessing that is your problem.

vhmauery’s picture

vhmauery commented
Title: Tab links within albums all link to same picture... » Pager links within albums all link to same picture...
Status: Active » Fixed

You had me entirely confused. Not tab link. Pager links. This is fixed in the latest tarball.

vhmauery’s picture

vhmauery commented
Status: Fixed » Closed (fixed)
anselmoluis’s picture

anselmoluis commented

The problem still remains.
No navigation in the acidfree albums. The same picture always.
Any suggestion?

vhmauery’s picture

vhmauery commented

Sorry, I just caught that too. It appears that one of my themes treats the pager differently than the other. It worked on the one I was doing development with and not the other. But I made a fix and now it works on both (and the default drupal theme). So in 12 hours or so it will be repackaged. Or if you don't want to wait, it is in CVS. If you don't want to bother with CVS, it is a one line fix.

--- branches/4.7.x/acidfree.module      (revision 325)
+++ branches/4.7.x/acidfree.module      (working copy)
@@ -2686,7 +2686,7 @@
 
     // Get page elements
     $pager = explode(',', $matches[2]);
-    if (count($pager) < ELEMENT_PAGER)
+    if (count($pager) < ELEMENT_PAGER+1)
         $pager = explode('%2C', $matches[2]);
     $offset = $pager[ELEMENT_PAGER];

Just add the +1 to that line.