The url redirecting to user's notification settings is wrong. The default path is [site:url]/notification-settings instead of [site:url]/user/[message:user:uid]/notification-settings

I can fix it in the backoffice but the default path must be changed in the code.

[edit] : After some research, I found two lines where the link is wrong : commons_notify.features.inc lines 37 and 39

CommentFileSizeAuthor
#2 correct-user-notification-settings-url-2107967-2.patch2.97 KBdevin carlson

Comments

Flohw’s picture

Flohw commented

[to delete or hide]

devin carlson’s picture

devin carlson commented
Version: 7.x-3.3 » 7.x-3.x-dev
Status: Active » Needs review
StatusFileSize
new correct-user-notification-settings-url-2107967-2.patch2.97 KB

A patch to update the URL to the user's notification settings with an update function to revert the message feature.

ezra-g’s picture

ezra-g commented
Status: Needs review » Fixed

Looks good - Committed. Thanks for the bug report and the fix!

http://drupalcode.org/project/commons.git/commit/7d655fe

Status: Fixed » Closed (fixed)

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

Anonymous’s picture

(not verified) commented
Issue summary: View changes

More precision about the url