Bugs item #897946, was opened at 2004-02-16 11:58
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=536613&aid=897946&group_id=73068

Category: None
Group: v 1.0-b2
Status: Open
Resolution: None
Priority: 5
Submitted By: Mike Moran (mikemoran)
Assigned to: Nobody/Anonymous (nobody)
Summary: requestURI attribute does not handle anchors ('#') properly

Initial Comment:
In the given example, the URIs that should be produced
for sorting are of the form "page.htm?query=farg#foo"
and not "page.htm#foo?query=farg" (the latter is what
is produced).

In a URI, the anchors should always go at the end. If
the query parameters are merely appended to the
displayURI value then they will be treated as part of
the anchor text, and not as the query part.

To be exact, the anchor is part of a URI-reference, as
described in rfc2396, section 4.

If it is the case that the requestURI cannot contain an
anchor then this should be made clear in the documentation.

Currently, the docs say: "When the [sic] present, links
for sorting, exports, and paging are formed by adding
any tag generated parameters to the value of requestURI
attribute."

This passage cannot simply mean that the query
parameters are appended to the String value, as I have
seen parameters already in the page such as "foo=bar"
get mixed in with displaytag generated parameters.

A suggested fix would be to look for the first
occurence of a '#' character in the URI, split it at
that point, apply the current algorithm to the first
section then append the anchor text (if any) on the end.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=536613&aid=897946&group_id=73068


-------------------------------------------------------
SF.Net is sponsored by: Speed Start Your Linux Apps Now.
Build and deploy apps & Web services for Linux with
a free DVD software kit from IBM. Click Now!
http://ads.osdn.com/?ad_id=1356&alloc_id=3438&op=click
_______________________________________________
displaytag-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/displaytag-devel

Reply via email to