792008 - please remove comment 33 from bug 755593

Status: RESOLVED FIXED

(Data & BI Services Team :: DB: MySQL, task)

Description

[:glob ✱]

(from Robert Kaiser (:[e-mail]) from bug 792007)
> https://bugzilla.mozilla.org/show_bug.cgi?id=755593#c33
> 
> The first expletive there wasn't caught by Profanivore.
> 
> Unfortunately, this is a somewhat high-profile bug, so quite a few people
> will read this. We should get rid of the expletive and also try to find out
> why Profanivore missed it.

can this comment please be removed.


thanks.

Comment 1

[:glob ✱]

it looks like the comment_id is 6640942.

can you please verify this is correct, then nuke it with:
  DELETE FROM longdescs WHERE comment_id=6640942

Comment 2

[Gervase Markham [:gerv]]

Do we do actual deletion? Or do we just set the text to ''? 

If we do think deletion doesn't cause data integrity problems, do we need an "allow_comment_deletion" Param, I wonder?

Gerv

Comment 3

[Reed Loden [:reed]]

(In reply to Gervase Markham [:gerv] from comment #2)
> Do we do actual deletion? Or do we just set the text to ''? 

We actually remove comments completely. It regularly occurs for spam comments, so it's nothing new.

> If we do think deletion doesn't cause data integrity problems, do we need an
> "allow_comment_deletion" Param, I wonder?

I could definitely see that being useful to others. File an upstream bug?

Comment 4

[Sheeri Cabral [:sheeri]]

Is that param allowing an administrator to delete comments? I think that's a fabulous idea, and something lots of folks would love. I don't think self-deletion (e.g. I delete a comment I made) is a good idea necessarily, there needs to be *some* amount of pain so people think before they type, but having to involve a DBA is overkill.

Comment 5

[Gervase Markham [:gerv]]

Bug 792080 filed.

Gerv

Comment 6

[Sheeri Cabral [:sheeri]]

mysql> update longdescs set thetext=REPLACE(thetext,'Fuck','****')   WHERE comment_id=6640942;
Query OK, 1 row affected (0.00 sec)
Rows matched: 1  Changed: 1  Warnings: 0

Updated the text to not have the swear word, because we're not intending to censor the whole post, just the one word.