Bugzilla::Extension should export a web_dir attribute to get full path to the extension's web directory

RESOLVED FIXED in Bugzilla 4.2

Status

()

--
enhancement
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: dkl, Assigned: dkl)

Tracking

unspecified
Bugzilla 4.2
Bug Flags:
approval +

Details

(Whiteboard: [bmo4.0-resolved])

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

8 years ago
Created attachment 521028 [details] [diff] [review]
Patch to add web_dir attribute to Bugzilla::Extension (v1)
Attachment #521028 - Flags: review?(mkanat)

Comment 1

8 years ago
Comment on attachment 521028 [details] [diff] [review]
Patch to add web_dir attribute to Bugzilla::Extension (v1)

Looks good to me! :-)
Attachment #521028 - Flags: review?(mkanat) → review+

Updated

8 years ago
Severity: normal → enhancement
Flags: approval+
Target Milestone: --- → Bugzilla 4.2
(Assignee)

Comment 2

8 years ago
Created attachment 521031 [details] [diff] [review]
Patch to add web_dir attribute to Bugzilla::Extension (v2)

Forgot to add POD documentation to Bugzilla::Extension for web_dir. Please review the docs.

Dave
Attachment #521028 - Attachment is obsolete: true
Attachment #521031 - Flags: review?(mkanat)
(Assignee)

Updated

8 years ago
Attachment #521028 - Flags: review+
(Assignee)

Updated

8 years ago
Flags: approval+ → approval?

Comment 3

8 years ago
Comment on attachment 521031 [details] [diff] [review]
Patch to add web_dir attribute to Bugzilla::Extension (v2)

Cool. Comma after "in" and before "such", on checkin.
Attachment #521031 - Flags: review?(mkanat) → review+

Updated

8 years ago
Flags: approval? → approval+
(Assignee)

Comment 4

8 years ago
Thanks Max.

trunk:
Committing to: bzr+ssh://dlawrence%40mozilla.com@bzr.mozilla.org/bugzilla/trunk
modified Bugzilla/Install/Util.pm
modified Bugzilla/Extension.pm
Committed revision 7757.
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
(Assignee)

Updated

8 years ago
Whiteboard: [bmo4.0-resolved]
Blocks: 223988
You need to log in before you can comment on or make changes to this bug.