Closed Bug 557806 Opened 15 years ago Closed 15 years ago

When setting the upgrade_notification parameter, an error should be thrown if you don't have all the required Perl modules installed

Categories

(Bugzilla :: Administration, task)

task
Not set
minor

Tracking

()

RESOLVED FIXED
Bugzilla 3.6

People

(Reporter: LpSolit, Assigned: LpSolit)

Details

Attachments

(1 file)

To download bugzilla-update.xml, you need to have XML::Twig and LWP::UserAgent installed. Else Bugzilla silently ignores this parameter, which confuses some admins. Bugzilla::Config::Common::check_notification() should make sure that these Perl modules are installed.
Attached patch patch, v1Splinter Review
Attachment #437559 - Flags: review?(bugzilla)
Comment on attachment 437559 [details] [diff] [review] patch, v1 r=glob
Attachment #437559 - Flags: review?(bugzilla) → review+
Flags: approval?
Flags: approval?
Flags: approval3.6+
Flags: approval+
Committing to: bzr+ssh://lpsolit%40gmail.com@bzr.mozilla.org/bugzilla/trunk/ modified Bugzilla/Config/Common.pm Committed revision 7119. Committing to: bzr+ssh://lpsolit%40gmail.com@bzr.mozilla.org/bugzilla/3.6/ modified Bugzilla/Config/Common.pm Committed revision 7077.
Status: ASSIGNED → RESOLVED
Closed: 15 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: