Closed Bug 561170 Opened 14 years ago Closed 13 years ago

Various warnings thrown when running QA tests with Perl 5.12.0

Categories

(Bugzilla :: Bugzilla-General, defect)

defect
Not set
minor

Tracking

()

RESOLVED FIXED
Bugzilla 4.0

People

(Reporter: LpSolit, Assigned: LpSolit)

References

Details

Attachments

(1 file)

test_require_login.t
====================
long_list.cgi: Use of uninitialized value $buglist in split at /var/www/html/bugzilla_qa36/long_list.cgi line 32.
relogin.cgi: Use of uninitialized value $name in hash element at /opt/perl/lib/site_perl/5.12.0/i686-linux/Template/Context.pm line 100.
relogin.cgi: Use of uninitialized value $shortname in substitution (s///) at /opt/perl/lib/site_perl/5.12.0/i686-linux/Template/Context.pm line 116.
relogin.cgi: Use of uninitialized value $name in concatenation (.) or string at /opt/perl/lib/site_perl/5.12.0/i686-linux/Template/Context.pm line 168.

test_time_summary.t
===================
summarize_time.cgi: Use of uninitialized value in split at /var/www/html/bugzilla_qa36/summarize_time.cgi line 267.
summarize_time.cgi: Use of uninitialized value in numeric gt (>) at /var/www/html/bugzilla_qa36/summarize_time.cgi line 297.

webservice_bug_add_comment.t
============================
jsonrpc.cgi: Use of uninitialized value $arg in join or string at /opt/perl/lib/5.12.0/CGI/Carp.pm line 432.

I skipped all the warnings about SOAP::Lite. They represent 98+% of all warnings.
Bugzilla 3.6 is now restricted to security fixes only, and this bug got no traction for several months. We will retarget this bug once it has a patch ready for checkin.
Target Milestone: Bugzilla 3.6 → ---
Depends on: 670062, 670128
Attached patch patch, v1Splinter Review
This patch fixes warnings reported in long_list.cgi (4.0 only, this script is gone in 4.2) and summarize_time.cgi. Warnings related to relogin.cgi are fixed in bug 670128.
Assignee: general → LpSolit
Status: NEW → ASSIGNED
Attachment #544832 - Flags: review?(dkl)
Target Milestone: --- → Bugzilla 4.0
Comment on attachment 544832 [details] [diff] [review]
patch, v1

Review of attachment 544832 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good. r=dkl
Attachment #544832 - Flags: review?(dkl) → review+
Flags: approval?
Flags: approval4.0?
Thanks!
Flags: approval?
Flags: approval4.0?
Flags: approval4.0+
Flags: approval+
Committing to: bzr+ssh://lpsolit%40gmail.com@bzr.mozilla.org/bugzilla/trunk/
modified summarize_time.cgi
Committed revision 7856.

Committing to: bzr+ssh://lpsolit%40gmail.com@bzr.mozilla.org/bugzilla/4.0/
modified long_list.cgi
modified summarize_time.cgi
Committed revision 7616.
Status: ASSIGNED → RESOLVED
Closed: 13 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: