]> git.vanrenterghem.biz Git - git.ikiwiki.info.git/commit
reorder canedit checks during page creation to have best_loc first
authorJoey Hess <joey@gnu.kitenet.net>
Fri, 12 Feb 2010 01:13:30 +0000 (20:13 -0500)
committerJoey Hess <joey@gnu.kitenet.net>
Fri, 12 Feb 2010 01:13:30 +0000 (20:13 -0500)
commit7af18f2a1ec8b5ce4764813b0464112f517e806f
treecbb37ccbea33ee3321365d78e0446d986055947b
parent6f1d6236185148930a2c7694c9ae59877997d0d4
reorder canedit checks during page creation to have best_loc first

When creating a page, multiple locations are tested to see if they can be
edited. If all fail, one of the failure subs is called, to log the user in
to allow them to proceed with the edit. So far so good.

But, what if some pages fail for one reason, and some for another? This
occurs when httpauth_pagespec is used in conjunction with signinedit (and
openid or something). When the user is not signed in at all
The former will fail to edit a page because the user was not httpauthed.
The latter will fail to edit a different page, because the user was not
signed in. One of their failure methods gets to run first.

The page creation code always ran the failure method corresponding to the
topmost page location. So, when editing a foo/Discussion page, and with
httpauth_pagespec => "*!/Discussion", it ran the httpauth failure method,
which was exactly the wrong thing to do.

I fixed this by making it instead run the failure method for the *best*
page location. In the above example, that's foo/Discussion, so signinedit
runs, as desired, and we get the signin page.

This seems like it will be the right choice, or at least an acceptable
choice. If a user wants to use httpauth they can always choose it on the
signin page.
IkiWiki/Plugin/editpage.pm