[JIRA-OS] Commented: (WW-216) Bugs in URLTag

[email protected]
Newsgroups gmane.comp.java.open-symphony.cvs
Message-ID <[email protected]>
The following comment has been added to this issue:

     Author: Patrick Lightbody
    Created: Sun, 14 Sep 2003 7:08 PM
       Body:
I don't fully understand the issues you are bringing up. Maybe you could supply some test cases (you can use URLTagTest as a starting point).
---------------------------------------------------------------------
View the issue:

  http://jira.opensymphony.com/secure/ViewIssue.jspa?key=WW-216


Here is an overview of the issue:
---------------------------------------------------------------------
        Key: WW-216
    Summary: Bugs in URLTag
       Type: Bug

     Status: In Progress
   Priority: Major

    Project: WebWork
 Components: 
             Views
   Fix Fors:
             2.0-beta2
   Versions:
             2.0-beta2
             2.0-beta1

   Assignee: Patrick Lightbody
   Reporter: Grzegorz Mucha

    Created: Thu, 12 Jun 2003 7:08 AM
    Updated: Sun, 14 Sep 2003 7:04 PM

Description:
The url tag has some bugs. Notably:

When putting the value into url scope wrong variable is used (value is used, when result should be) - this results in loss of parameter values in the declared attribute.

Additionally, I'd suggest adding handling of empty parameter value. Currently the code is not doing anything - my suggestion is to use value of parameter from request. In all cases though, it appends the ampersand to url end, so it could look like this

this.action?p1=a&&p2

... and that's messy to me ;)


---------------------------------------------------------------------
JIRA INFORMATION:
This message is automatically generated by JIRA.

If you think it was sent incorrectly contact one of the administrators:
   http://jira.opensymphony.com/secure/Administrators.jspa

If you want more information on JIRA, or have a bug to report see:
   http://www.atlassian.com/software/jira



-------------------------------------------------------
This sf.net email is sponsored by:ThinkGeek
Welcome to geek heaven.
http://thinkgeek.com/sf
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.