Closed Bug 545715 Opened 16 years ago Closed 16 years ago

New Hook: bugmail_relationships

Categories

(Bugzilla :: Extensions, enhancement)

3.5.3
enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
Bugzilla 4.0

People

(Reporter: mkanat, Assigned: mkanat)

References

Details

Attachments

(1 file)

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.
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: extensions → email-notifications
Component: Extensions → Email Notifications
Summary: Store relationship id names in the database → Store bugmail relationship id names in the database
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
Attached patch v1Splinter Review
Okay, here we go. This one's a little invasive, so it won't be backported.
Assignee: extensions → mkanat
Status: NEW → ASSIGNED
Attachment #426554 - Flags: review+
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
Blocks: 561745
No longer blocks: 561745
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: