Closed Bug 891236 Opened 11 years ago Closed 11 years ago

Allow Redmine See Also URLs

Categories

(Bugzilla :: Extensions, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
Bugzilla 5.0

People

(Reporter: mail, Assigned: mail)

References

Details

Attachments

(1 file)

Our Ruby friends need some love when it comes to supporting Redmine URLs in the 'See Also' field. This bug addresses that.
Attached patch v1 patchSplinter Review
Attachment #772497 - Flags: review?(justdave)
Attachment #772497 - Flags: review?(justdave)
Attachment #772497 - Flags: review?
Attachment #772497 - Flags: review+
Attachment #772497 - Flags: review?
a+ for trunk only.
Flags: approval+
Target Milestone: --- → Bugzilla 5.0
Committing to: bzr+ssh://bzr.mozilla.org/bugzilla/trunk/                       
modified extensions/MoreBugUrl/Extension.pm
modified extensions/MoreBugUrl/template/en/default/hook/global/user-error-bug_url_invalid_tracker.html.tmpl
Committed revision 8708.                                                       

Committing to: bzr+ssh://bzr.mozilla.org/bugzilla/trunk/                       
added extensions/MoreBugUrl/lib/Redmine.pm
Committed revision 8709.
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Comment on attachment 772497 [details] [diff] [review]
v1 patch

>=== added file 'extensions/MoreBugUrl/lib/Redmine.pm'

>+package Bugzilla::Extension::MoreBugUrl::Redmine;
>+use strict;

This module lacks |use 5.10.1|.

t/002goodperl.t ...... 1/848 
#   Failed test 'extensions/MoreBugUrl/lib/Redmine.pm DOES NOT require Perl 5.10.1 --WARNING'
#   at t/002goodperl.t line 105.
# Looks like you failed 1 test of 848.


>+use base qw(Bugzilla::BugUrl);

It must also |use parent| instead of |use base|.


Committing to: bzr+ssh://lpsolit%40gmail.com@bzr.mozilla.org/bugzilla/trunk/
modified extensions/MoreBugUrl/lib/Redmine.pm
Committed revision 8760.
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: