Comment Admin doesn't show name/email for logged-in user comments

[pjayme]

Comments made by logged in users don't show the user's name or email address in the admin screen. This makes it hard to determine who has made comments across the site.

It looks like changing the $summary_fields on Comment should resolve this - there's already a getAuthorName() method that can be used, but we'd need an equivalent getAuthorEmail() method.

[robbieaverill]

Hey @pjayme, can you please provide the versions of SilverStripe and comments that you're using?

It sounds like you can implement the changes you want in your own project using the $summary_fields method you suggested for now at least. At best you could use configuration to define related fields via dot notation, e.g. 'Author.FirstName' => 'Author name', 'Author.Email' => 'Author email' for example. This could be applied with a DataExtension or via YAML configuration.

[pjayme]

SilverStripe: 4.1
Comments: 3.1.2

Alright cool, will go for DataExtension approach.

Cheers

[robbieaverill]

Comments made by logged in users don't show the user's name or email address in the admin screen.

If this is the case, it's definitely a bug. Should look like this (anonymously posted comments):

[pjayme]

Yup, this is definitely the case, it doesn't display for logged in users - we are using it for an intranet so all our users will be logged in by default. We wouldn't get anonymous comments, it is also reinforced using the require_login: true config.

[pjayme]

Have done something like this on my CommentExtension.php for the time being:

<?php

namespace MySite\Extensions;

use SilverStripe\ORM\DataExtension;

class CommentExtension extends DataExtension
{
    /**
     * @var array
     */
    private static $summary_fields = [
        'Author.FirstName' => 'Author name',
        'Author.Email' => 'Author email',
    ];

    public function updateSummaryFields(&$fields)
    {
        $fields['Name'] = '';
        $fields['Email'] = '';
    }
}

[pjayme]

Also another thing which might be a UX enhancement but the Comment label for its column isn't the most suitable as some of those entries are Replies so its quite tricky for a user to see which is a Comment and which is a Reply to a comment.

Perhaps the label should be Comment/Replies?

[robbieaverill]

Fixed with #273, will be available in the next CWP recipe release