Closed Bug 343702 Opened 19 years ago Closed 19 years ago

editflagtypes.cgi: Argument "" isn't numeric in numeric gt (>) at template/en/default/request/email.txt.tmpl line 155.

Categories

(Bugzilla :: Bugzilla-General, defect)

2.23
defect
Not set
minor

Tracking

()

RESOLVED FIXED
Bugzilla 2.22

People

(Reporter: LpSolit, Assigned: LpSolit)

Details

Attachments

(1 file)

When notifying a user about a flag being changed, request/email.txt.tmpl is called, which contains: [% IF Bugzilla.cgi.param("comment").length > 0 %] When changing the inclusion and exclusion lists in editflagtypes.cgi, no comment field exists and the following error is generated in the web server log: editflagtypes.cgi: Argument "" isn't numeric in numeric gt (>) at template/en/default/request/email.txt.tmpl line 155. So first have to check whether this field exists before looking at its length. This problem was introduced during deFORMication in 2.20, see bug 225818.
Attached patch patch, v1Splinter Review
Another one-liner.
Attachment #228243 - Flags: review?(myk)
Attachment #228243 - Flags: review?(myk) → review+
Flags: approval+
Flags: approval2.22+
tip: Checking in template/en/default/request/email.txt.tmpl; /cvsroot/mozilla/webtools/bugzilla/template/en/default/request/email.txt.tmpl,v <-- email.txt.tmpl new revision: 1.13; previous revision: 1.12 done 2.22: Checking in template/en/default/request/email.txt.tmpl; /cvsroot/mozilla/webtools/bugzilla/template/en/default/request/email.txt.tmpl,v <-- email.txt.tmpl new revision: 1.11.2.1; previous revision: 1.11 done
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: