[Done v1.3.7] EZpages bug 1.3.6 demonstratable - reproducable

[kiddo]

The prev and next buttons on TOC links seem broken in 1.3.6

I tried this with fresh installs on remote and local test servers.

Here are links to remote test server to demonstrate.

version 1.3.5: here, notice that the priv and next buttons point to the proper locations and work as expected:

http://lp-hm-1.net/zc1/index.php?mai...os=v&chapter=1

version 1.3.6: here, notice that the prev and next links do NOT point correctly.

http://pen.st/index.php?main_page=page&id=4&chapter=1

I may be not so smart and miss something obvious, but I have this same issue on all 1.3.6 I test on. (I hope my pasted links work)

Thank you for looking.

Kiddo

[kiddo]

Well, I took includes/modules/pages/page/header_php.php from a 1.3.5 package and dropped it into a 1.3.6 test rig and it --SEEMS-- to fix the TOC links problem, but without going through the code to find the bug I'm really hesitant to upgrade my production 1.3.5 installs and use this kind of fix.

This fix works on both my in house setup and on a remote test box but I have no idea what doing this may have broke elsewhere.

Kiddo

[ideasgirl]

True! happens on my testing site as well... let's wait for a fix.

[kiddo]

Thanks IdeaGirl for verifying bug.

I'm not exactly a php wiz but I may spend some time on this over the weekend and will post a fix if I can work it out. The broken code is under 200 lines so shouldn't be too bad.

Kim

[Ajeh]

/me looks around for the fix to this that she posted last week or so ...

[Ajeh]

It all comes back to me now ...

Change the file:
/includes/modules/pages/page/header_php.php

Change the line:

  if ($value == $id) { 

To read:

  if ($value == $ezpage_id) { 

All better ...

[kiddo]

Thank you Oba-san Ajeh !

Your Zen-Fu is very powerful :)

Kim

[ideasgirl]

Why a "Sticky: Confirmed bugs and workarounds in v1.3.6" isn't created so we can follow the real bugs on that spot like with 1.3.5?

[Ajeh]

We will be putting one up in due time ... thanks ...