When a user account is edited the message for $op=update is returned as:
%username's Flickr page has been deleted."

This is not correct. I have updated the message so that it is:
%username's Flickr page has been $op ed."

Patch is attached.

CommentFileSizeAuthor
#3 171077.patch1 KBandrewlevine
flickr_0.patch729 bytesemmajane

Comments

drewish’s picture

drewish commented
Version: 5.x-1.1 » 5.x-1.x-dev
Status: Active » Needs work

i think you're correct that the message is in correct but i'm reluctant to just include variable directly... it won't be localized correctly.

emmajane’s picture

emmajane commented

Whatever works best for all users is what should be done. The feedback for that entire section might need to be re-thought. It doesn't seem consistent in when/how it delivers messages...

andrewlevine’s picture

andrewlevine commented
Status: Needs work » Needs review
StatusFileSize
new 171077.patch1 KB

The original intent of the message was to make sure the user knew when their Flickr page was removed. I didn't think any other message was needed because Drupal already sets the message "The changes have been saved."

The proposed patch doesn't really make much sense because the message is only set when the flickr identifier field is blank. However, this did make me realize a bug. If the user didn't have a flickr page and they left the field blank, the "page deleted" message will still come up. My attached patch fixes that.

I haven't tested this.

andrewlevine’s picture

andrewlevine commented
Title: Poorly worded "update" message » delete message when user leaves Identifier blank
Status: Needs review » Reviewed & tested by the community

tested and ready, drewish can you committ?

drewish’s picture

drewish commented
Status: Reviewed & tested by the community » Fixed

committed to DRUPAL-5

Anonymous’s picture

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

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