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.
in core/lib/Drupal/Core/Ajax/AjaxResponse.php and core/includes/ajax.inc there is a duplicate code comment, which is both unclear and references md5(), which we wouldn't use now per Drupal secure coding standards at http://drupal.org/node/845876
Comment | File | Size | Author |
---|---|---|---|
#5 | 1884840-5.patch | 821 bytes | Pix_ |
#1 | 1884840-1.patch | 1.75 KB | pwolanin |
Comments
Comment #1
pwolanin CreditAttribution: pwolanin commentedComment #2
jhodgdonOK, I'll get tihs committed. Thanks! Does it need backporting to 7?
Comment #3
jhodgdonCommitted to 8.x. I think it can probably be ported to 7.x by grepping for md5 to find the relevant comments.
Comment #4
pwolanin CreditAttribution: pwolanin commentedYes, looks like these are comments that were added in D7
Comment #5
Pix_ CreditAttribution: Pix_ commentedBackport for d7 attached
Comment #6
pwolanin CreditAttribution: pwolanin commentedlooks good
Comment #7
jhodgdonThanks - committed to 7.x.