[#CONF-9723] remove comment notification wrongly claims that the comment author (or perhaps page author?) is the comment remover

Confluence

remove comment notification wrongly claims that the comment author (or perhaps page author?) is the comment remover

Created: 16/Oct/07 03:32 AM Updated: 27/Feb/08 09:41 PM

Component/s:

Notifications

Affects Version/s:

2.6.0

Fix Version/s:

2.7.2, 2.8

Time Tracking:

Not Specified

File Attachments:

None

Image Attachments:

1. Email notification says comment was removed by Ajay, who was neither the author nor the remover.jpg
(283 kB)

Issue Links:

Cloners

This issue is derived from:

~~CONF-8283~~ remove comment notification wrongly claims that the comment author is the comment remover

Duplicate

This issue is duplicated by:

~~CONF-9892~~ Notification mail for page-removal displays wrong user

Part

This issue is incorporated by:

~~CONF-8293~~ Removed content notifications should include "removed by" user

Reference

This issue relates to:

~~CONF-8293~~ Removed content notifications should include "removed by" user

This issue is related to:

~~CONF-8138~~ Notifications for file attachment removal states the file attachers name, not the file removers name

Participants:	Bob Swift, Don Willis [Atlassian], James Mortimer, Matt Ryall [Atlassian] and Rosie Jameson [Atlassian]
Since last comment:	46 weeks, 4 days ago
Internal Complexity:	5
Internal Value:	7
Resolution Date:	27/Feb/08 09:41 PM
Labels:

Description

« Hide

Clone of ~~CONF-8283~~ which was fixed in Confluence 2.5. Apparently occurring again.
when a comment is removed, we get an email notification such as:

Comment removed by Mr User

Where Mr User wrote the comment rather than removing it.

All

Comments

Work Log

Change History

Sort Order:

[ Permlink | « Hide ]

Rosie Jameson [Atlassian] added a comment - 06/Nov/07 10:00 PM - edited

Please see screenshot of an email notification which says the comment was removed by Ajay. In fact I (Rosie) was both the author and the remover of the comment.

Perhaps Ajay was the latest page editor at the time the comment was added? Or removed?
http://confluence.atlassian.com/pages/viewpreviouspageversions.action?pageId=10355145

[ Permlink | « Hide ]

Bob Swift added a comment - 07/Jan/08 01:31 PM - edited

Same problem on 2.7. I would recommend this issue gets set to MAJOR

[ Permlink | « Hide ]

James Mortimer added a comment - 23/Jan/08 09:47 AM

From ~~CONF-8283~~) and it was 'fixed' in 2.5 by not mentioning the 'user' or 'author' of the page or the comment because the 'remover' information is not available to the notification event.

the bug also occurs in 2.5.7

This is a fairly major source of pain / confusion for our users currently, and very upsetting to some people to recieve an email claiming they or a colleague did something they did not in fact do. I'm not sure if there are or are not legal implications of this, but there are definately social ones. But it doesn't result in application down time or crashes etc.

Perhaps if the username is the comment author or last modifier, you could simply changing the grammer from:

Comment removed by USERNAME

Comment by USERNAME has been removed.

and if you can do it, it would be most useful to know

Comment by USERNAME has been removed from page PAGENAME by USERNAME.

[ Permlink | « Hide ]

Bob Swift added a comment - 23/Jan/08 10:42 AM

I still recommend this issue gets set to MAJOR and at least the change in wording James suggests be done immediately even if a real fix cannot be done as quickly.

[ Permlink | « Hide ]

James Mortimer added a comment - 23/Jan/08 10:57 AM

I did not intend to disagree with Bob. I agree this is important to fix the wording immediately, since

changing the wording to remove the username again should be trivial (Agnes Ro already did it once), and
the visibility / impact of this 'bug' on users is quite obvious

[ Permlink | « Hide ]

Bob Swift added a comment - 23/Jan/08 11:00 AM

James, I didn't think you were disagreeing - the emphasis of my comment was that 2 weeks have gone by and this is still considered a minor issue.

[ Permlink | « Hide ]

Don Willis [Atlassian] added a comment - 17/Feb/08 05:42 PM

I've implemented this in 2.7.2 in the simple way of removing the false information, as was done in a prior bug fix.
In 2.8 I've implemented ~~CONF-8293~~ so that the removal information is back and contains the correct information.