I see that the "Flagging an item may trigger rules [...]" message has crept into the D5 version. But there's no Rules module for D5. So this message(s), in theme_flag_admin_page(), should go.

(D5 has "Workflow-NG" instead of Rules. But we decided not to support it.)

Comments

mooffie’s picture

mooffie commented

So this message(s), in theme_flag_admin_page(), should go.

Also should go the extra lines in flag_flag(). (You can see them in the link above.)

Note that nothing there is harmful, so fixing this isn't critical.

mooffie’s picture

mooffie commented

(Or... perhaps all this was intentional? Did you simply want D5's 'flag.module' and D6's to be "the same"? It's certainly a possibility.)

quicksketch’s picture

quicksketch
he/him
commented

No, it's totally an accident. :)

I removed the misleading line from #404038: Split out Admin Pages into flag.admin.inc, if that gets committed first.

quicksketch’s picture

quicksketch
he/him
commented
Status: Active » Fixed

Fixed in #404038: Split out Admin Pages into flag.admin.inc. Yes I know it wasn't polite to include changes in the middle of a file move. Modifications to the Actions text was the only thing I changed between versions. Trust me.

Status: Fixed » Closed (fixed)

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