Bug 469886

Hi Jeff,

I am not quite sure why do you need to send the shared_search url with the sharer_id to other people for? You can just send the people your shared search name and your name and they will be able to find it in their shared search by others section , or even you can view those searches in the footers of their bugzilla pages if they are in the group that you are sharing the search with. It is more user friendly to use that than to use the url , and after all that is the whole idea of the shared search feature.

Noura

I am trying to send out a simple URL instead of a very long URL.  I created a search today called "MRG NEEDINFO" and shared it with group redhat.  If I click on "Run" next to the search, it executes the following link:

https://bugzilla.redhat.com/buglist.cgi?cmdtype=runnamed&namedcmd=MRG%20NEEDINFO

but when I send that out to others, it does not work unless they go to Preferences->Saved Searches, hunt for it in the ever-increasing list of shared searches, and execute it from there.  But when they do, they see the sharer_id added at the end, and that URL can be sent and shared and clicked on from within e-mail or from wiki pages.

Hi Jeff,

How about you check the "Add to footer" checkbox for your shared search that way this shared search will appear in the footer of bugzilla to all the members of the redhat group and that way they can just click on its name in the footer and it will run the search for them ;)

Noura

Created attachment 322523 [details]
Patch add link to saved search table that can be shared (v1)

Adding a patch that adds a hyperlink to the shared search table that can be copied to others without putting it in their footer. The link is only displayed if the search is shared with a group and the person using the link would need to be in that group.

Please review
Dave

Thanks, Dave.  Does this link get added to the top table or the bottom table?  If it's the bottom table, I'll still have the same problem as a user doesn't see his own shared searches in that table.

Jeff , the link is added to the top table

Comment on attachment 322523 [details]
Patch add link to saved search table that can be shared (v1)

Hi Dave ,

I tested the patch and it works fine and looks good, I also had another idea of just making the url for the Run link in the top table to show the sharer id when copy link as or save link as if the case of shared searches something like that:

Index: template/en/default/account/prefs/saved-searches.html.tmpl
===================================================================
RCS file: /cvs/qa/rh_bugzilla_3/template/en/default/account/prefs/saved-searches.html.tmpl,v
retrieving revision 1.2
diff -p -u -r1.2 saved-searches.html.tmpl
--- template/en/default/account/prefs/saved-searches.html.tmpl  12 Feb 2008 19:44:57 -0000      1.2
+++ template/en/default/account/prefs/saved-searches.html.tmpl  5 Nov 2008 11:31:47 -0000
@@ -96,7 +96,15 @@
       <tr>
         <td>[% q.name FILTER html %]</td>
         <td>
-          <a href="buglist.cgi?cmdtype=runnamed&amp;namedcmd=[% q.name FILTER url_quote %]">Run</a>
+          [%# REDHAT EXTENSION START 469886 %]
+          [% IF q.shared_with_group.id %]
+            <a href="buglist.cgi?cmdtype=dorem&amp;remaction=run&amp;namedcmd=
+                     [% q.name FILTER url_quote %]&amp;sharer_id=
+                     [% user.id FILTER url_quote %]">Run</a>
+          [% ELSE %]
+            <a href="buglist.cgi?cmdtype=runnamed&amp;namedcmd=[% q.name FILTER url_quote %]">Run</a>
+          [% END %]
+          [%# REDHAT EXTENSION END 469886 %]
         </td>
         <td>
           <a href="query.cgi?[% q.edit_link FILTER html %]&amp;known_name=

That way we don't have to change the UI much. either ways work for me.

Thanks,
Noura

Noura, that would definitely work for me.  Thanks.

Committed to CVS. Will be on the next Bugzilla update (normally on Thursdays).