The username is always shown as "Guest" for anonymous comments, despite the fact that the administrator has changed it. See attached images...

Comments

chris.smith’s picture

Status: Active » Needs review
StatusFileSize
new1.14 KB

This patch fixed my problems. Needs review.

WojtekWu’s picture

Title: Username is allways "Guest" » Username is allways "Anonymous"

I have the same problem.
But in my case the username is always shown as "Anonymous"

When I switch off that modul everything is alright.
I need some help. Please.

chris.smith’s picture

Try the patch and let us know if it works for you.

web360’s picture

I encountered the same issue, and can verify that the patch works.

LakY_cz’s picture

Thanks, it works

chris.smith’s picture

Status: Needs review » Reviewed & tested by the community

Can patch be incorporated in next release?

syakely’s picture

Patch worked for me too. Thanks.

Colin @ PCMarket’s picture

this patch needs to be applied as it fixed the same issue for me too

finne’s picture

the patch works for me too. Add to next release please.

dpfitzsi’s picture

Title: Username is allways "Anonymous" » Username is always "Anonymous"
Anonymous’s picture

patch works - thanks

dpfitzsi’s picture

Is there any update on when this patch will be applied to the module?

jduhls’s picture

Patch works

alan evans’s picture

Issue summary: View changes
Status: Reviewed & tested by the community » Postponed (maintainer needs more info)

I'm having trouble reproducing this issue. Can anyone provide detailed steps to reproduce?

Here's the STR I'm using and cannot find an issue:

  1. Install a bare D7 installation
  2. Checkout and enable comment_goodness 1.4
  3. Get an anonymous user to post a comment
  4. As an administrator, view the node containing the comment and click the "edit" link
  5. Change the username and submit the form
  6. ... ? ...

The last point is where I'm assuming things go wrong - is it the case that people are seeing the "anonymous" username here still, instead of the actual username that was set by the administrator?

The patch additionally needs to have the function header comment changed back to hook_preprocess_HOOK per https://api.drupal.org/api/drupal/modules%21system%21theme.api.php/funct...

I'm not against making this change if I can verify the actual problem, but at the moment I cannot. There is no obvious reason why the code in comment_goodness would be wrong, given that core uses the same variable here http://cgit.drupalcode.org/drupal/tree/modules/comment/comment.module?h=... . So I'm suspicious that (1) me being unable to reproduce this problem, and (2) that core uses the same pattern as the module might indicate that the problem lies elsewhere.

There may be some module hook ordering issue I guess, but would need more information to be able to determine that - at the moment I can't reproduce the issue (making many assumptions about what the actual issue is).

formatC'vt’s picture

strangest solution, theme username needs a user object (https://api.drupal.org/api/drupal/includes!theme.inc/function/theme_user...) in account key, not comment

hydra’s picture

Version: 7.x-1.4 » 7.x-1.x-dev
Status: Postponed (maintainer needs more info) » Reviewed & tested by the community
StatusFileSize
new570 bytes

To reproduce:
- Create a comment with an authenticated user
- View the comment, it say's it has been submitted by Anonymous instead of the users name

Patch from #1 works for me. The docu change is unnecessary. For me this is RTBC.

juc1’s picture

Patch from #1 works for me (I am posting here to remind myself for next time I install this module!)

juliakoelsch’s picture

Also works for me. Thank you!

chris.smith’s picture

Can we get this fix committed to the development branch and closed off.

pixiekat’s picture

If you're exposing comments as fields I also needed this fix

avpaderno’s picture

Status: Reviewed & tested by the community » Needs work

The module doesn't implement anymore comment_goodness_preprocess_comment(), nor does it use the code the patch is trying to alter.

avpaderno’s picture

Status: Needs work » Reviewed & tested by the community

I apologize: That was the master branch, not the 7.x-1.x branch.