94553 - process_bug.cgi allows too-short comments

Status: VERIFIED DUPLICATE of bug 67613

(Bugzilla :: Bugzilla-General, defect)

Description

[Myk Melez [:myk] [@mykmelez]]

For changes that require commentary, process_bug.cgi allows users to enter,
among other things, a single period "." as a comment.  This defeats the purpose
of the requirement.  process_bug.cgi should detect comments that do not contain
content and throw an error requesting the user to submit a real comment.

Comment 1

[Myk Melez [:myk] [@mykmelez]]

Attachment: patch v1: a first start

Comment 2

[Myk Melez [:myk] [@mykmelez]]

Attachment: patch v2: fixes syntax error

Comment 3

[Myk Melez [:myk] [@mykmelez]]

The patch is a small improvement over the current behavior, but it would still
allow someone to enter three periods as a comment.  I'm taking suggestions for
improvement.

Comment 4

[Myk Melez [:myk] [@mykmelez]]

Something funky happened.

Comment 5

[Myk Melez [:myk] [@mykmelez]]

*** This bug has been marked as a duplicate of 67613 ***

Comment 6

[timeless]

Yuck. verify duplicate of a bug that was verified wontfix.

I picked . because it results in no line of bugmail, but per asa i will 
probably discontinue such usage.

Personally I'd prefer that we allow some people to verify w/o comment but that 
seems unlikely.

Comment 7

[Asa Dotzler [:asa]]

myk, sorry for wasting your time because I can't make up my mind. 

Comment 8

[Dave Miller [:justdave]]

moving all closed Bugzilla bugs to the new Bugzilla product.
This batch is DUPLICATE/INVALID/WORKSFORME/WONTFIX
reassigning to default owner and QA in case of the bug being reopened.
Clearing milestones, since we really shouldn't have them on these types of
resolutions.  Sorry for the spam everyone...

Comment 9

[Markus Gerstel]

Verification got lost.

And now YOU suffer the consequences of this bug - or this mail would not have
been sent =)