prevnext_nav() takes at least 3 arguments (2 given)

[Olaf Meeuwissen <olaf@…>]

Just a heads up but ​trac:changeset:8597 breaks the /build/configuration page with a traceback. The relevant change is in trac.web.chrome.py. Basically, prevnext_nav() changed from

def prevnext_nav(req, label, uplabel=None):

to

def prevnext_nav(req, prev_label, next_label, up_label=None):

A fix would be pretty trivial if one does not have to address backward compatibility with Trac-0.11.x. But if you need to fix this so it works with that version as well as Trac's trunk, I don't see an easy fix.

Suggestions, anyone?

[osimons]

I don't test Bitten with Trac trunk (0.12dev), so I won't be looking into this quite yet.

Hint: You can use trac.util.arity(fn) to get the count of arguments for the function, and pass arguments accordingly.

[Olaf Meeuwissen <olaf@…>]

Thanks for the clue bat.

The prevnext_nav.diff​ is perhaps a bit ugly but it works for me. Now Bitten trunk​ works with both Trac ​trunk before (and by extension ​Trac-0.11) and ​after the prevnext_nav() changes.

[cmlenz]

Patch looks good to me, had to apply it to get Bitten running with Trac trunk, and it works as advertised.

[osimons]

Verified. Committed compat code in [801]. Will merge to 0.6.x.

[osimons]

...and closing.