Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 333575 - Report links contain superfluous &= in the URL, causing them to ignore following parameters
Summary: Report links contain superfluous &= in the URL, causing them to ignore follow...
Status: RESOLVED FIXED
Alias: None
Product: Community
Classification: Eclipse Foundation
Component: Bugzilla (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P3 major (vote)
Target Milestone: ---   Edit
Assignee: Eclipse Webmaster CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-01-05 10:25 EST by Pierre Candela CLA
Modified: 2011-03-10 14:36 EST (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Pierre Candela CLA 2011-01-05 10:25:17 EST
I created several tabular reports that I bookmarked in my browser (Firefox 3.6):
ex: http://tinyurl.com/35vlkjs

When I click on any link in the table, the "Bugzilla is pondering your search / Please stand by ..." page is displayed for a long while (at least 10 minutes) then I get this error:
- "Problem loading page / The connection was reset"

I never had this problem before. Can the recent Bugzilla upgrade be the cause?

Thanks for your help,
Pierre
Comment 1 Denis Roy CLA 2011-01-05 10:45:33 EST
Can you paste an actual URL?  I don't do URL shorteners.
Comment 4 Denis Roy CLA 2011-01-07 15:43:21 EST
The common patterm I'm seeing is that, in the URL, there is a blank parameter that is causing Bugzilla to ignore all the remaining parameters.

For instance, one link on your report page ends like this:

&value0-0-0=&votes=&=&component=wst.xsl&target_milestone=3.1 RC2

-------------------^^

That &= is not good.  Removing it causes the link to work.
Comment 6 Denis Roy CLA 2011-01-07 16:14:57 EST
> Here is a report URL

The report URL itself is fine .. the links it contain have an extra &=
Comment 7 Denis Roy CLA 2011-01-07 16:19:11 EST
I looked at the report template, and I found what was causing the empty &=.  I don't understand why this is happening, since this is unmodified bugzilla, but by removing the specific piece of code this becomes fixed.

--- ../../default/reports/report-table.html.tmpl        2011-01-02 08:11:23.000000000 -0500
+++ report-table.html.tmpl      2011-01-07 16:15:20.000000000 -0500
@@ -39,8 +39,7 @@
   [% urlbase = BLOCK %]buglist.cgi?[% buglistbase FILTER html %]
   [% "&$tbl_vals" IF tbl_vals %][% END %]
 [% ELSE %]
-  [% urlbase = BLOCK %]buglist.cgi?[% buglistbase FILTER html %]&
-  [% tbl_field FILTER url_quote %]=[% tbl FILTER url_quote %][% END %]
+  [% urlbase = BLOCK %]buglist.cgi?[% buglistbase FILTER html %][% END %]
 [% END %]
 
 [% IF tbl_field %]



I'll need to compare our local version of that template to Bugzilla's CVS.  In the meanwhile, the links inside the report should now work as expected (ie, clicking on "24" should return the correct list of 24 bugs).
Comment 8 Konstantin Komissarchik CLA 2011-01-07 16:26:14 EST
Thanks for fixing this. I can confirm that this is now working as expected.
Comment 9 Denis Roy CLA 2011-03-10 14:36:44 EST
As I close this, I scratch my head.