When trying to delete the thread, It shows the message like this:

Are you sure you want to delete the thread between you and the following users: admin, admin2?

where 'admin' is the current user. The message should be like this:

Are you sure you want to delete the thread between you and the following users: admin2?

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

anmolgoyal74 created an issue. See original summary.

anmolgoyal74’s picture

Title: Deleyion message needs to exclude current user? » Deletion message needs to exclude current user?
kadamk’s picture

I have made the changes as per the requirement. Request you to check and confirm.

phjou’s picture

Is it me or the indentation seems weird? I didn't apply the patch so I might be wrong.

kadamk’s picture

I have modified the indentation. Request you to check and confirm.

  • anmolgoyal74 committed 432eedb on 8.x-2.x authored by kadamk
    Issue #3099376 by anmolgoyal74: Deletion message needs to exclude...
anmolgoyal74’s picture

Status: Active » Needs review

Fixed the indentation, and updated the logic to match current user id to member id.
Thanks.

anmolgoyal74’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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