Closed
Bug 533121
Opened 15 years ago
Closed 14 years ago
Support Google Code for see_also urls
Categories
(Bugzilla :: Creating/Changing Bugs, enhancement)
Tracking
()
RESOLVED
FIXED
Bugzilla 3.6
People
(Reporter: timeless, Assigned: reed)
References
(Depends on 1 open bug, )
Details
Attachments
(1 file)
1.75 KB,
patch
|
LpSolit
:
review+
|
Details | Diff | Splinter Review |
steps: 1. load bug 527382 2. paste http://code.google.com/p/gnome2-globalmenu/issues/detail?id=462 into the see also field 3. press enter expected results: http://code.google.com/p/gnome2-globalmenu/issues/detail?id=462 is added as a see also link. "Lots of projects use code.google.com" actual results: an error that says the url is not a valid bugzilla url.
Assignee | ||
Updated•15 years ago
|
Version: 3.5.2 → 3.4.4
Assignee | ||
Comment 1•15 years ago
|
||
http://googlecode.blogspot.com/2009/10/issue-tracker-data-api-for-project.html has details on Google Code's Issuer Tracker API.
Assignee | ||
Comment 2•15 years ago
|
||
Probably something like this, but I haven't tested any of this at all, and I'm not 100% sure, so not requesting review.
Assignee: create-and-change → reed
Status: NEW → ASSIGNED
Updated•15 years ago
|
Summary: Can't use code.google.com for see also urls → Support Google Code for see_also urls
Comment 3•15 years ago
|
||
Once the entire See Also system is implemented for the two systems we currently support, adding additional systems can be considered.
Depends on: bz-seealso
Assignee | ||
Updated•14 years ago
|
Attachment #416304 -
Flags: review?(LpSolit)
Comment 4•14 years ago
|
||
Comment on attachment 416304 [details] [diff] [review] patch - v1 >+ if ($uri->path =~ m|^p/([^/]+)/issues/detail$|) { $uri->path is of the form /p/PROJECT_NAME/issues/detail, so you must fix your regexp and add a / before p: m|^/p/([^/]+)/issues/detail$| Otherwise works fine. r=LpSolit
Attachment #416304 -
Flags: review?(LpSolit) → review+
Comment 6•14 years ago
|
||
Okay. You're making my life more difficult for improving the See Also system, though. (I would prefer to improve it and then add additional trackers, which would be a better software engineering practice.)
Flags: approval?
Flags: approval3.6?
Flags: approval3.6+
Flags: approval+
Target Milestone: --- → Bugzilla 3.6
Assignee | ||
Comment 7•14 years ago
|
||
Committing to: bzr+ssh://bzr.mozilla.org/bugzilla/trunk/ modified Bugzilla/Bug.pm Committed revision 7103. Committing to: bzr+ssh://bzr.mozilla.org/bugzilla/3.6/ modified Bugzilla/Bug.pm Committed revision 7061.
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 8•14 years ago
|
||
As per IRC, make regex for matching "code.google.com" case-insensitive. r=mkanat over IRC. Committing to: bzr+ssh://bzr.mozilla.org/bugzilla/trunk/ modified Bugzilla/Bug.pm Committed revision 7104. Committing to: bzr+ssh://bzr.mozilla.org/bugzilla/3.6/ modified Bugzilla/Bug.pm Committed revision 7063.
You need to log in
before you can comment on or make changes to this bug.
Description
•