I've just checked in branch forum-skin-links to add a Forum navbar entry before Tickets for all skins, pointing at /forum if the user has suitable capabilities.

I think I've got the logic as simple as possible, since anycap doesn't appear to check for the anonymous user's capabilities as well as the logged-in user's capabilities, and I don't see an anonanycap type of command in TH1.

anoncap checks whether all capabilities match for the anonymous user category, whereas anycap checks whether any capability matches for a specific user. I want something halfway between: check whether anonymous has any of several capabilities.

What the checkin does instead is

  if {[anycap 23456] || [anoncap 2] || [anoncap 3]} {

It works; it's just awkward.

I see, apparently I misremembered. Apologies!

Is there a good reason this branch isn't merged yet? As I said, I think it's sane, but since it's a taste thing, I don't want to merge it unilaterally. If it's unacceptable, I'd like to know why so I can learn to make better commits.

I made my forumPost commit directly on trunk, since I think it's uncontroversial, but the next logical step is to change the skins to start using it. I'm willing to do that work; I think it should be done on a branch, since it'll need to be tested before being merged to trunk, but I'm now hesitant to check something else in that might also be ignored.

I think it would be fine to go ahead and merge that branch.

Done, thanks!