Bugs item #858888, was opened at 2003-12-12 14:06
Message generated for change (Comment added) made by fgiust
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=536613&aid=858888&group_id=73068

Category: main tag library
Group: v 1.0-b2
>Status: Closed
Resolution: None
Priority: 5
Submitted By: Paul McCulloch (paulmcculloch)
>Assigned to: fabrizio giustina (fgiust)
Summary: requestURI parameters are overwritten by current values

Initial Comment:
Parameters specifed via <table>'s requestURI attribute
don't override parameters with the same name in the
original URI. This means that the original URI values
are used when the user tries to sort or change page.

A fix is to change org.displaytag.util.Href.java (line 143)
in method addParameterMap to check that the parameter
doesn't already exist:

            if(!parameters.containsKey(key)){
                parameters.put(key, value);
            }

rather than the simple put().

This bug is related to 830652 (which I couldn't re-open).


Example:
Consider a Struts apllication. The user runs a complex
query with a URI such as:

http://host/app/QueryAction.do?method=runQuery

The results are put into session scope (as the query is
expensive) and the results are displayed using the
display tag:

<table ... requestURI="QueryAction.do?method=redraw" ...

When changing page the URL used should set the method
paramter to "redraw" but it currently sets it to
"runQuery".


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

>Comment By: fabrizio giustina (fgiust)
Date: 2004-01-02 17:56

Message:
Logged In: YES 
user_id=798060

fixed in CVS for release 1.0-b3

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

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


-------------------------------------------------------
This SF.net email is sponsored by: IBM Linux Tutorials.
Become an expert in LINUX or just sharpen your skills.  Sign up for IBM's
Free Linux Tutorials.  Learn everything from the bash shell to sys admin.
Click now! http://ads.osdn.com/?ad_id=1278&alloc_id=3371&op=click
_______________________________________________
displaytag-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/displaytag-devel

Reply via email to