Closed Bug 143231 Opened 22 years ago Closed 22 years ago

Changing a bug with an empty buglist gives a warning

Categories

(Bugzilla :: Creating/Changing Bugs, defect)

2.15
defect
Not set
critical

Tracking

()

RESOLVED FIXED
Bugzilla 2.16

People

(Reporter: jouni, Assigned: jouni)

Details

(Keywords: regression)

Attachments

(1 file)

If I have just reset my cookies and have a totally empty buglist, changing a 
bug causes navigate.html.tmpl to elicit a warning:

Argument "" isn't numeric in numeric gt (>) at 
data/template/en/default/bug/navigate.html.tmpl line 19 (#1) (W numeric) The 
indicated string was fed as an argument to an operator that expected a numeric 
value instead. If you're fortunate the message will identify which operator 
was so unfortunate. [Thu May 9 16:25:57 2002] 

I'll attach a patch soon.
Attached patch PatchSplinter Review
Makes navigate.html.tmpl check for buglist's existence.
Keywords: patch, review
Keywords: regression
Target Milestone: --- → Bugzilla 2.16
Comment on attachment 82913 [details] [diff] [review]
Patch

2xr= justdave
Attachment #82913 - Flags: review+
whoever checks this in, don't forget to put on both the trunk and
BUGZILLA-2_16_BRANCH
Assignee: myk → jouni
Fixed on branch and trunk.

Checking in ./template/en/default/bug/navigate.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/bug/navigate.html.tmpl,v
 <--  navigate.html.tmpl
new revision: 1.1.2.1; previous revision: 1.1
done

Checking in ./template/en/default/bug/navigate.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/bug/navigate.html.tmpl,v
 <--  navigate.html.tmpl
new revision: 1.2; previous revision: 1.1
done

Gerv
Status: NEW → RESOLVED
Closed: 22 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

Created:
Updated:
Size: