Closed Bug 125515 Opened 24 years ago Closed 9 years ago

Allow admin to pick thier own bug match expression

Categories

(Webtools Graveyard :: Bonsai, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: jacob, Assigned: jacob)

Details

Attachments

(1 file)

Currently, on Bonsai's editparams.cgi page the only real option you have for matching bug number is the number of digits to require. When I first installed Bonsai, I didn't want this, I only wanted it linked if the word "bug" was before the number. At the time, I did a really ugly hack in order to support this. But I figured maybe some other people might like to be able to determine exactly what should be linked, so I just went back and made it a Param(). This also has the advantage of being generic enough to be included in the Bonsai source so I don't have to keep fighting with it when I update/make other patches :) This patch removes the two current params for dealing with buglinks and creates two now ones (bugs_pattern and bug_replace). The default of these new values cause Bonsai to behave identical to the old version, but have the ability to do a lot more.
Attached patch PatchSplinter Review
Keywords: patch, review
-> me
Assignee: tara → jake
Status: NEW → ASSIGNED
Attachment #69479 - Flags: review?(tara)
Comment on attachment 69479 [details] [diff] [review] Patch The replace string gets generated as plain text html code into the comment string. It looks like the regexp substitution is not escaped quite right. Check out this link for the results I'm seeing: http://landfill.bugzilla.org/bonsai-dev/showcheckins.cgi?person=baloo
Attachment #69479 - Flags: review?(tara) → review-
QA Contact: mattyt-bugzilla → bonsai
Bonsai was decommissioned, closing all remaining bugs "wontfix"
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → WONTFIX
Product: Webtools → Webtools Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: