Closed
Bug 545715
Opened 16 years ago
Closed 16 years ago
New Hook: bugmail_relationships
Categories
(Bugzilla :: Extensions, enhancement)
Tracking
()
RESOLVED
FIXED
Bugzilla 4.0
People
(Reporter: mkanat, Assigned: mkanat)
References
Details
Attachments
(1 file)
6.61 KB,
patch
|
mkanat
:
review+
|
Details | Diff | Splinter Review |
Right now we have REL_NAMES and RELATIONSHIPS as two separate constants, but they both contain somewhat-duplicate data. They should be merged, and then we should have a hook to add additional relationships to the set.
Assignee | ||
Comment 1•16 years ago
|
||
Okay, while working on this, I realized that instead of having a hook, we should just store the list in the database, which would make things way simpler for extensions.
Summary: New Hook: bugmail_relationships → Store relationship id names in the database
Assignee | ||
Updated•16 years ago
|
Assignee: extensions → email-notifications
Component: Extensions → Email Notifications
Assignee | ||
Updated•16 years ago
|
Summary: Store relationship id names in the database → Store bugmail relationship id names in the database
Assignee | ||
Comment 2•16 years ago
|
||
When I started to do the "move them to the DB" work, I realized that it was overkill. I don't think that *that* many extensions are going to be adding new relationships. (The main reason that I wanted to move this into the DB was that the valid range of relationship ids is only from -127 to 127, so conflicts between extensions were too likely.)
So we'll just add a hook.
Assignee: email-notifications → extensions
Component: Email Notifications → Extensions
Summary: Store bugmail relationship id names in the database → New Hook: bugmail_relationships
Assignee | ||
Comment 3•16 years ago
|
||
Okay, here we go. This one's a little invasive, so it won't be backported.
Assignee | ||
Comment 4•16 years ago
|
||
Committing to: bzr+ssh://bzr.mozilla.org/bugzilla/trunk/
modified userprefs.cgi
modified Bugzilla/BugMail.pm
modified Bugzilla/Constants.pm
modified Bugzilla/Hook.pm
modified Bugzilla/User.pm
modified extensions/Example/Extension.pm
Committed revision 6988.
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Flags: approval+
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•