Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.Utterly minor, but I thought I'd post it since I noticed it...
I couldn't stand it anymore and just set comments to be read-only over at #79018: protect Drupal core .txt files
However, when you view the issue, there's still a quick-link to "Add new comment" even though there's no comment form on the page or any way to add a new comment.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | 236459_pi_add_comment_3.patch | 3.9 KB | aclight |
| #2 | add_comment_links.patch | 3.65 KB | hunmonk |
| #1 | 236459_pi_add_comment_1.patch | 3.19 KB | aclight |
Comments
Comment #1
aclight commentedHere's a patch that fixes the problem. I also noticed that if comments were disabled for an issue (instead of just set to be read only), the Most recent comment/First unread comment link was still showing up, so this patch fixes that problem as well.
Comment #2
hunmonk commentedi believe the link for most recent file attachment needs to go in there as well. attached patch corrects this, and has been tested as working for all node comment settings.
Comment #3
aclight commentedYep, good catch. Code looks good and this works as intended. I've rerolled the patch to eliminate the offset, but otherwise didn't change anything.
Comment #4
hunmonk commentedcommitted to 5.x-2.x and 5.x-3.x, deployed on d.o et. all.
Comment #5
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.