The link to edit your social sites does not appear on the user profile when using "me" aliases http://drupal.org/project/me

What happens is the menu access calback calls follow_links_user_access() with 'me' as argument instead of the uid.
Tried to fix it nicely and provide a patch, but I didn't succeed in that.
Now I did this quick temp fix, within follow_links_user_access() I did this:

	if($uid = 'me') {
		$uid = $GLOBALS['user']->uid;
	}

I'm quite sure this is not the way to do it, but it works for me.

CommentFileSizeAuthor
#4 follow-1163654.patch3.09 KBrv0

Comments

q0rban’s picture

q0rban commented
Status: Active » Closed (won't fix)

Hi! Thanks for taking the time to write this up. I don't think this is anything I'm interested in supporting for Follow. Instead, you might want to open a feature request on the me module issue queue to work with Follow. :)

Just so you know, you'll need to have it be:

  if ($uid == 'me') {

not:

  if ($uid = 'me') {
rv0’s picture

rv0 commented

thanks for getting my typo ;)

Created an issue in the Me Aliases issue queue
http://drupal.org/node/1168158

nohup’s picture

nohup commented

q0rban,

me module works by substituting candidate paths (which have %user, %user_uid_optional or %user_category) with their me equivalent using hook_menu_alter.

Follow module uses 'user/%/follow' as a menu item, instead if it could use 'user/%user/follow', it would make follow work with me, without breaking anything when me module is not installed. The code for follow_links_form function will have to be changed a bit to handle the user object instead of uid.

I have created a patch to support this at #1168158: Support for Follow module

Thanks

rv0’s picture

rv0 commented
Status: Closed (won't fix) » Needs review
StatusFileSize
new follow-1163654.patch3.09 KB

thanks nohup
Your patch is great. It also needed a modification to the access function, which I made and rerolled in another patch

Changing status to "needs review" because I also think follow.module should use the 'user/%user/follow' path

Hope this doesn't break anything else, but it seems to work fine for me

q0rban’s picture

q0rban commented
Category: bug » feature
Status: Needs review » Closed (won't fix)

I haven't had many other requests for this, so I'm going to "won't fix" this issue. If there's others that want this, let me know.

rv0’s picture

rv0 commented
Status: Closed (won't fix) » Needs review

actually, using % instead of %user is kinda wrong (although it works), so this patch is very valid

astonvictor’s picture

astonvictor commented
Issue summary: View changes
Status: Needs review » Closed (outdated)

I'm closing it because the issue was created a long time ago without any further steps.

if you still need it then raise a new one.
thanks