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.
If you edit a comment in Drupal 8 the title of the page is "Comment permalink"
In Drupal 7 this used to be "Edit"
$items['comment/%comment/edit'] = array(
'title' => 'Edit',
'page callback' => 'comment_edit_page',
'page arguments' => array(1),
'access callback' => 'comment_access',
'access arguments' => array('edit', 1),
'type' => MENU_LOCAL_TASK,
'weight' => 0,
);
There is no hook_menu in Drupal 8 because the route has been converted.
Comment | File | Size | Author |
---|---|---|---|
#13 | comment_edit-2085435-13.patch | 1.89 KB | dawehner |
#13 | interdiff.txt | 1.02 KB | dawehner |
#7 | comment-perma-2085435.2.interdiff.txt | 1014 bytes | larowlan |
#7 | comment-perma-2085435.2.patch | 1.75 KB | larowlan |
#3 | comment-perma-2085435.fail_.patch | 896 bytes | larowlan |
Comments
Comment #1
larowlanComment #2
larowlanComment #3
larowlangoing for red/green
Comment #5
larowlan#3: comment-perma-2085435.fail_.patch queued for re-testing.
Comment #6
larowlanRandom fails on the 'fails' test.
Comment #7
larowlanFixes for preview
Comment #8
jibranRTBC if green.
Comment #10
larowlan#7: comment-perma-2085435.2.patch queued for re-testing.
Comment #11
jibranBack to RTBC.
Comment #12
webchickHm. I realize that we normally introduce tests for regressions, but here it seems a bit weird thing to test for. Also, it really doesn't make much sense under a comment titled "Test changing the comment author to anonymous."
IMO we should drop the test, or else abstract it somehow but I can't really imagine how.
Comment #13
dawehnerThis test function tests quite a bunch of different cases, so what about just moving the comment a little bit around?
Comment #14
star-szrTag fix, WSSCI -> WSCCI.
Comment #15
star-szrSorry for the noise, didn't check autocomplete.
Comment #16
jibranPutting back to RTBC after #13 let's see what @webchick thinks about the patch now.
Comment #17
webchickCool, that works for me!
Committed and pushed to 8.x. Thanks!