When remove an attachment, remove also the comment entry

[Marco Rodrigues]

Anyone can verify what I'm saying at for example this bug - https://bugs.launchpad.net/ubuntu/+source/ipkungfu/+bug/126106

Check the 3rd comment.. I've removed the attachment, so why it's the entry of it there?

When remove a attachment, should remove also the comment associated with it.

Thanks!

[Marco Rodrigues (gothicx)]

Like mpt said, it should only remove the comment with no text inside it. So only if it has attachment with no text.

[Björn Tillenius (bjornt)]

The comment shouldn't be removed from the database, though. The code that displays the comments on the bug page should simply hide it if it's empty. This is to ensure that the permalinks of the comments don't change when removing an attachment.

[Graham Binns (gmb)]

Björn Tillenius wrote:
> The comment shouldn't be removed from the database, though. The code
> that displays the comments on the bug page should simply hide it if it's
> empty. This is to ensure that the permalinks of the comments don't
> change when removing an attachment.
>

I'll bear that in mind, thanks.

--
Graham

[Graham Binns (gmb)]

Fix in review.