215962 - Time Tracking: field => "field" not wrapped in {}, causing bad params to ThrowUserError in process_bug.cgi

Status: RESOLVED FIXED

(Bugzilla :: Creating/Changing Bugs, defect)

Description

[Erik Selberg]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.3) Gecko/20030312
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.3) Gecko/20030312

Line 187 of process_bug.cgi contains "field => $field" which should be wrapped
in {}, i.e. "{field => $field}". As it stands, ThrowUserError gets the wrong params.

Reproducible: Always

Steps to Reproduce:
1. Edit a bug with time constraints
2. Specify a zero / negative number (I used zero)
3. See weird error

Actual Results:  
Saw error

Expected Results:  
Displayed proper error

Comment 1

[Erik Selberg]

Attachment: Patch to fix bug

Just a quick patch to make life easier. Granted, we're talking all of two
characters here.

Comment 2

[Dave Miller [:justdave]]

doh.

Comment 3

[Dave Miller [:justdave]]

Checking in process_bug.cgi;
/cvsroot/mozilla/webtools/bugzilla/process_bug.cgi,v  <--  process_bug.cgi
new revision: 1.189; previous revision: 1.188
done