First of all I must say this is an excellent module.

The problem however is that many people who click on the link are not logged in. Right now this translates "me" into "0" which is invalid in most cases (e.g. editing anon user profile). What is needed instead is redirect to login form that redirects back to "me" page.

Attached is the patched file for 4.7. This is basically addition of a simple condition into me_relay.

CommentFileSizeAuthor
me.module.txt2.3 KBdkruglyak

Comments

ahoeben’s picture

ahoeben commented

The target module should really be handling the 'uid=0'/access denied case. It could be desirable to have the '0' case work as it does currently. Site admins could put in place aliases for cases when the '0' scenario is not properly handled.

However, I can see that an option to go through a login form could be a good addition as well. Thanks for the suggestion. Could you make a patch?

ahoeben’s picture

ahoeben commented

Applied to both 4.7.x and 5.x dev versions

ahoeben’s picture

ahoeben commented
Status: Reviewed & tested by the community » Fixed
Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)