The default message That item has already been flagged with @flag @num times. sometimes can sound confuse for the users. I'd like to be able to customize the message for each flag or not show the message at all.

CommentFileSizeAuthor
#2 flag_limit-customize_message-1516870-2.patch3.54 KBdnewkerk
#1 0001-Issue-1516870-Customize-message-for-flag-limit-reach.patch3.29 KBdanillonunes

Comments

danillonunes’s picture

danillonunes commented
Title: Customize message for flag limit reached » Customize message for "flag limit reached"
Status: Active » Needs review
StatusFileSize
new 0001-Issue-1516870-Customize-message-for-flag-limit-reach.patch3.29 KB

Here is the patch for 7.x-1.x branch.

I'm not a native English speaker, so someone please review for grammar issues. :)

dnewkerk’s picture

dnewkerk commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new flag_limit-customize_message-1516870-2.patch3.54 KB

Reviewed the patch and corrected the wording. Thanks for adding this feature danillonunes.

I'm attaching a new version of the patch with the corrections.

minoroffense’s picture

minoroffense commented
Status: Reviewed & tested by the community » Needs work

There's no means of removing dangerous input from the user in the messages. The message should be run through check_plain or at least filter_xss before being displayed.

I don't see anything in drupal_set_message which does this for you.