If you edit a comment in Drupal 8 the title of the page is "Comment permalink"

comment_title.png

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.

Files: 
CommentFileSizeAuthor
#13 comment_edit-2085435-13.patch1.89 KBdawehner
PASSED: [[SimpleTest]]: [MySQL] 58,821 pass(es).
[ View ]
#13 interdiff.txt1.02 KBdawehner
#7 comment-perma-2085435.2.interdiff.txt1014 byteslarowlan
#7 comment-perma-2085435.2.patch1.75 KBlarowlan
PASSED: [[SimpleTest]]: [MySQL] 58,967 pass(es).
[ View ]
#3 comment-perma-2085435.fail_.patch896 byteslarowlan
FAILED: [[SimpleTest]]: [MySQL] 59,387 pass(es), 25 fail(s), and 7 exception(s).
[ View ]
#3 comment-perma-2085435.pass_.patch1.84 KBlarowlan
FAILED: [[SimpleTest]]: [MySQL] 59,248 pass(es), 1 fail(s), and 0 exception(s).
[ View ]
comment_title.png33.41 KBalexpott

Comments

Component:configuration system» comment.module

Assigned:Unassigned» larowlan

Assigned:larowlan» Unassigned
Status:Active» Needs review
StatusFileSize
new1.84 KB
FAILED: [[SimpleTest]]: [MySQL] 59,248 pass(es), 1 fail(s), and 0 exception(s).
[ View ]
new896 bytes
FAILED: [[SimpleTest]]: [MySQL] 59,387 pass(es), 25 fail(s), and 7 exception(s).
[ View ]

going for red/green

Status:Needs review» Needs work
Issue tags:-Novice, -WSSCI Conversion

The last submitted patch, comment-perma-2085435.pass_.patch, failed testing.

Status:Needs work» Needs review
Issue tags:+Novice, +WSSCI Conversion

#3: comment-perma-2085435.fail_.patch queued for re-testing.

Random fails on the 'fails' test.

StatusFileSize
new1.75 KB
PASSED: [[SimpleTest]]: [MySQL] 58,967 pass(es).
[ View ]
new1014 bytes

Fixes for preview

Status:Needs review» Reviewed & tested by the community

RTBC if green.

Status:Reviewed & tested by the community» Needs work
Issue tags:-Novice, -WSSCI Conversion

The last submitted patch, comment-perma-2085435.2.patch, failed testing.

Status:Needs work» Needs review
Issue tags:+Novice, +WSSCI Conversion

#7: comment-perma-2085435.2.patch queued for re-testing.

Status:Needs review» Reviewed & tested by the community

Back to RTBC.

Status:Reviewed & tested by the community» Needs review

+++ b/core/modules/comment/lib/Drupal/comment/Tests/CommentInterfaceTest.php
@@ -68,6 +68,9 @@ function testCommentInterface() {
     // Test changing the comment author to "Anonymous".
     $this->drupalGet('comment/' . $comment->id() . '/edit');
+    $this->assertTitle(t('Edit comment @title | Drupal', array(
+      '@title' => $comment->subject->value,
+    )));

Hm. 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.

StatusFileSize
new1.02 KB
new1.89 KB
PASSED: [[SimpleTest]]: [MySQL] 58,821 pass(es).
[ View ]

This test function tests quite a bunch of different cases, so what about just moving the comment a little bit around?

Issue tags:-WSSCI Conversion

Tag fix, WSSCI -> WSCCI.

Issue tags:+WSCCI-conversion

Sorry for the noise, didn't check autocomplete.

Status:Needs review» Reviewed & tested by the community

Putting back to RTBC after #13 let's see what @webchick thinks about the patch now.

Status:Reviewed & tested by the community» Fixed

Cool, that works for me!

Committed and pushed to 8.x. Thanks!

Automatically closed -- issue fixed for 2 weeks with no activity.