102571 - Proposed patch for 'Most Recently Visited' bug list query

Status: RESOLVED WONTFIX

(Bugzilla :: Extension Ideas, enhancement, P3)

Description

[Richard Walters]

This morning I cooked up some changes to Bugzilla to add a feature I call 
the 'Most Recently Visited' (MRV) List.  It adds a default query, similar to "My 
bugs", that when queried shows you a list of the last 20 bugs you have visited.  
As you visit other bugs or create new bugs, they are shifted through your MRV 
list, which is stored in your entry of the "profiles" table.

I'll be attaching a patch that can be applied to the latest version of Bugzilla, 
as of 2001-10-01 12:00 PDT.  My bugzilla setup is signifantly altered 
from the CVS source, and I've tested these changes only on my version, so 
there might be bugs with it working with the CVS source.  I welcome anyone to 
try this out and tell me if it works as-is or doesn't work.

I think this is a cool feature for folks who have a lot of bugs they're 
interested in, but forget the bug ID of the bug they were recently looking at, 
and don't want to find it through the full query page all over again.

Comment 1

[Richard Walters]

Attachment: Patch to apply to Bugzilla as of 2001-10-01 (patches 6 files)

Comment 2

[Richard Walters]

Hmm.. looks like I forgot to add code in the User Preferences to allow the user 
to "turn off" this feature.  I'll add that and then attach another patch.

Comment 3

[Richard Walters]

Attachment: Patch to add new feature to user prefs

Comment 4

[Gervase Markham [:gerv]]

rwalters: I find reading diff -c difficult; could you attach a diff -u with all
the files in? :-)

Thanks,

Gerv

Comment 5

[Richard Walters]

Attachment: Same as above two patches combined, but with diff -u

Comment 6

[Richard Walters]

By the way, there is a "feature" :) I found but haven't fixed yet - if you use 
the MRV query between when you upgrade Bugzilla to support MRV and when you make 
your next bug visit, your MRV list will be empty, causing the query to fetch 
_all_ the bugs in the _entire_ database.  This is because a query with no 
"bug_id" CGI parameters at all ends up having no limiting factor in the "where" 
portion of the SQL query.

Comment 7

[Gervase Markham [:gerv]]

rwalters: you need to check there are some bugs before running the query, and
fail gracefully if not :-)

I'd guess you need to look in buglist.cgi at the variable @mrvList, and its
initial value.

Gerv

Comment 8

[Dave Miller [:justdave]]

We are currently trying to wrap up Bugzilla 2.16.  We are now close enough to
release time that anything that wasn't already ranked at P1 isn't going to make
the cut.  Thus this is being retargetted at 2.18.  If you strongly disagree with
this retargetting, please comment, however, be aware that we only have about 2
weeks left to review and test anything at this point, and we intend to devote
this time to the remaining bugs that were designated as release blockers.

Comment 9

[Jouni Heikniemi]

Comment on attachment 52914 [details] [diff] [review]
Same as above two patches combined, but with diff -u

Needs templatization changes and some other work already mentioned in other
comments.

Comment 10

[Dave Miller [:justdave]]

enhancements without current patches are being pushed to 2.20

Comment 11

[Dave Miller [:justdave]]

Bugzilla 2.20 feature set is now frozen as of 15 Sept 2004.  Anything flagged
enhancement that hasn't already landed is being pushed out.  If this bug is
otherwise ready to land, we'll handle it on a case-by-case basis, please set the
blocking2.20 flag to '?' if you think it qualifies.

Comment 13

[:glob ✱]

this proposal is better suited to being implemented as an extension instead of in bugzilla core.

Comment 14

[Richard Walters]

LOL -- I would think that after 12.5 years, if Bugzilla/Mozilla wanted to implement this (either in the core or as an extension) it would have been done already.  Like, years ago.  Seriously.  I'm going to put this bug out of its misery.  Feel free to reopen it if you really care.

Comment 15

[Mark Côté [:mcote]]

So, actually, there is now a WebService API for it, thanks to the recent fix for bug 489028.  See http://www.bugzilla.org/docs/tip/en/html/api/Bugzilla/WebService/BugUserLastVisit.html.

I don't know if this qualifies as a fix for this bug, since it's not integrated into the UI, but it would be pretty easy to add now.