Closed
Bug 394923
Opened 18 years ago
Closed 18 years ago
Bugzilla.pm should be subclassable
Categories
(Bugzilla :: Bugzilla-General, enhancement)
Tracking
()
RESOLVED
FIXED
Bugzilla 3.2
People
(Reporter: mkanat, Assigned: mkanat)
References
Details
Attachments
(1 file)
|
8.81 KB,
patch
|
gregaryh
:
review+
|
Details | Diff | Splinter Review |
Plugins really need to do things like:
package Testopia;
use base qw(Bugzilla);
But in order to do this, we have to remember to always use $class->method() inside of Bugzilla.pm when calling methods, instead of just doing method().
|
Assignee | |
Comment 1•18 years ago
|
||
Here, this should do it. I tested this briefly, but it probably needs either a careful code review or a lot of testing to make sure I didn't typo anywhere.
|
||
Comment 2•18 years ago
|
||
Comment on attachment 279651 [details] [diff] [review]
v1
Nit:
It would be good to include some documentation on examples of how to extend Bugzilla.
That can be its own bug though.
Attachment #279651 -
Flags: review?(ghendricks) → review+
|
|
Assignee | |
Updated•18 years ago
|
Flags: approval+
|
|
Assignee | |
Comment 3•18 years ago
|
||
Checking in Bugzilla.pm;
/cvsroot/mozilla/webtools/bugzilla/Bugzilla.pm,v <-- Bugzilla.pm
new revision: 1.60; previous revision: 1.59
done
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•