Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
The message advising that a Drupal update is available sometimes is shown 2 or 3 times:
There is a security update available for your version of Drupal. To ensure the security of your server, you should update immediately! See the available updates page for more information and to install your missing updates.
There is a security update available for your version of Drupal. To ensure the security of your server, you should update immediately! See the available updates page for more information and to install your missing updates.
Comment | File | Size | Author |
---|---|---|---|
#4 | double-notification-pre-patch.png | 31.5 KB | haydeniv |
#4 | single-notification-after-patch.png | 27.2 KB | haydeniv |
#4 | update_module-duplicate_notifications-1817680-4.patch | 1.14 KB | haydeniv |
screenshot update notification issue.png | 19.25 KB | Pere Orga |
Comments
Comment #1
Anonymous (not verified) CreditAttribution: Anonymous commentedMoving to D8.
Comment #2
jonhattanAlready fixed. See #1428032: Default drupal_set_message $repeat to FALSE instead of TRUE (followup)
This is the change record: http://drupal.org/node/1827234
Comment #3
Anonymous (not verified) CreditAttribution: Anonymous commentedSo I see from #1428032-19: Default drupal_set_message $repeat to FALSE instead of TRUE (followup) that it isn't committed to D7. The admin messages need to declare the $repeat parameter as FALSE for D7.
Comment #4
haydeniv CreditAttribution: haydeniv commentedThis has bothered me for a while now. Simple fix. Sorry don't know how to write a test for this or if it needs one.
Here are some manual steps to test:
I have uploaded a couple screenshots to demonstrate the patch.
Comment #5
Anonymous (not verified) CreditAttribution: Anonymous commentedLooks good to me. As for a test you would need to get the values of the message array and test that there was only one entry for the notification. I don't think it is worth a test at this point, but if there is it would need to be a new issue for D8.
Comment #6
David_Rothstein CreditAttribution: David_Rothstein commentedCommitted to 7.x - thanks! http://drupalcode.org/project/drupal.git/commit/2b86b82
Yeah, I agree, it's probably not worth the effort to write a test for this one.