Closed Bug 221160 Opened 21 years ago Closed 21 years ago

crash if illegal date specified in new charts date range

Categories

(Bugzilla :: Reporting/Charting, defect)

Product:
Bugzilla
Component:
Reporting/Charting
Version:
2.17.4
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
Bugzilla 2.18

People

(Reporter: kniht, Assigned: gerv)

Details

Attachments

(1 file)

Patch v.1
638 bytes, patch
kiko
: review+
Details | Diff | Splinter Review 
User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.4) Gecko/20030630 Galeon/1.3.8
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.4) Gecko/20030630 Galeon/1.3.8

In the new chart pages - after adding a dataset, if I fill in an invalid date in
the Date Range fields (such as 2003-10) then click on the Chart button it
generates an error.

Undefined subroutine &Bugzilla::Chart::ThrowUserError called at
Bugzilla/Chart.pm line 95.

It appears that a "&::" is needed in front of the ThrowUserError call on line 96.


Reproducible: Always

Steps to Reproduce:
Confirmed on the CVS tip.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Version: unspecified → 2.17.4
Attached patch Patch v.1 — — Splinter Review 
Patch as outlined in initial comment.

Gerv
Attachment #134656 - Flags: review?(jocuri)
Attachment #134656 - Flags: review?(jocuri) → review+
Flags: approval?
Flags: approval? → approval+
Target Milestone: --- → Bugzilla 2.18
Fixed.

Checking in Bugzilla/Chart.pm;
/cvsroot/mozilla/webtools/bugzilla/Bugzilla/Chart.pm,v  <--  Chart.pm
new revision: 1.2; previous revision: 1.1
done

Gerv
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: