[jbossseam-issues] [JBoss JIRA] Created: (JBSEAM-1719) UrlBuilder doesnt construct a correct url when the base url already has request params in place.
12:01 a.m.
UrlBuilder doesnt construct a correct url when the base url already has request params in
place.
------------------------------------------------------------------------------------------------
Key: JBSEAM-1719
URL: http://jira.jboss.com/jira/browse/JBSEAM-1719
Project: JBoss Seam
Issue Type: Bug
Components: Core
Affects Versions: 2.0.0.BETA1
Reporter: Chris Rudd
Given the following test case :
@Test
public void testBaseUrlAlreadyHasParams()
{
UrlBuilder url = new UrlBuilder("/someurl?arg1=a", "");
url.addParameter("foo", "bar");
String encodedUrl = url.getEncodedUrl();
Assert.assertEqual( encodedUrl, "/someurl?arg1=a&foo=bar" );
}
The following patch fixes the issue.
UrlBuilder.java line 62
protected String getParametersAsString()
{
String params = "";
for (String key : parameters.keySet())
{
params += "&" + key + "=" + parameters.get(key);
}
- if (!"".equals(params))
+ // repace the first "&" with a "?" if there are parameters and
the url doesnt already contain a "?"
+ if (!".equals(params) && (url!=null &&
!url.contains("?"))
{
params = "?" + params.substring(1);
}
return params;
}
NOTE: This condition occurs when using <s:link/> in a portlet, as the portlet
returns a url with parameters already added to the url.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.jboss.com/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
12:11 a.m.
New subject: [jbossseam-issues] [JBoss JIRA] Updated: (JBSEAM-1719) UrlBuilder doesnt construct a correct url when the base url already has request params in place.
[ http://jira.jboss.com/jira/browse/JBSEAM-1719?page=all ]
Chris Rudd updated JBSEAM-1719:
-------------------------------
Description:
Given the following test case :
@Test
public void testBaseUrlAlreadyHasParams()
{
UrlBuilder url = new UrlBuilder("/someurl?arg1=a", "");
url.addParameter("foo", "bar");
String encodedUrl = url.getEncodedUrl();
Assert.assertEqual( encodedUrl, "/someurl?arg1=a&foo=bar" );
}
The following patch fixes the issue.
UrlBuilder.java line 62
protected String getParametersAsString()
{
String params = "";
for (String key : parameters.keySet())
{
params += "&" + key + "=" + parameters.get(key);
}
- if (!"".equals(params))
+ // repace the first "&" with a "?" if there are parameters and
the url doesnt already contain a "?"
+ if (!".equals(params) && (url!=null &&
!url.contains("?")))
{
params = "?" + params.substring(1);
}
return params;
}
NOTE: This condition occurs when using <s:link/> in a portlet, as the portlet
returns a url with parameters already added to the url.
was:
Given the following test case :
@Test
public void testBaseUrlAlreadyHasParams()
{
UrlBuilder url = new UrlBuilder("/someurl?arg1=a", "");
url.addParameter("foo", "bar");
String encodedUrl = url.getEncodedUrl();
Assert.assertEqual( encodedUrl, "/someurl?arg1=a&foo=bar" );
}
The following patch fixes the issue.
UrlBuilder.java line 62
protected String getParametersAsString()
{
String params = "";
for (String key : parameters.keySet())
{
params += "&" + key + "=" + parameters.get(key);
}
- if (!"".equals(params))
+ // repace the first "&" with a "?" if there are parameters and
the url doesnt already contain a "?"
+ if (!".equals(params) && (url!=null &&
!url.contains("?"))
{
params = "?" + params.substring(1);
}
return params;
}
NOTE: This condition occurs when using <s:link/> in a portlet, as the portlet
returns a url with parameters already added to the url.
UrlBuilder doesnt construct a correct url when the base url already
has request params in place.
------------------------------------------------------------------------------------------------
Key: JBSEAM-1719
URL: http://jira.jboss.com/jira/browse/JBSEAM-1719
Project: JBoss Seam
Issue Type: Bug
Components: Core
Affects Versions: 2.0.0.BETA1
Reporter: Chris Rudd
Given the following test case :
@Test
public void testBaseUrlAlreadyHasParams()
{
UrlBuilder url = new UrlBuilder("/someurl?arg1=a", "");
url.addParameter("foo", "bar");
String encodedUrl = url.getEncodedUrl();
Assert.assertEqual( encodedUrl, "/someurl?arg1=a&foo=bar" );
}
The following patch fixes the issue.
UrlBuilder.java line 62
protected String getParametersAsString()
{
String params = "";
for (String key : parameters.keySet())
{
params += "&" + key + "=" + parameters.get(key);
}
- if (!"".equals(params))
+ // repace the first "&" with a "?" if there are parameters and
the url doesnt already contain a "?"
+ if (!".equals(params) && (url!=null &&
!url.contains("?")))
{
params = "?" + params.substring(1);
}
return params;
}
NOTE: This condition occurs when using <s:link/> in a portlet, as the portlet
returns a url with parameters already added to the url.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.jboss.com/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
4:45 p.m.
New subject: [jbossseam-issues] [JBoss JIRA] Assigned: (JBSEAM-1719) UrlBuilder doesnt construct a correct url when the base url already has request params in place.
[ http://jira.jboss.com/jira/browse/JBSEAM-1719?page=all ]
Pete Muir reassigned JBSEAM-1719:
---------------------------------
Assignee: Pete Muir
UrlBuilder doesnt construct a correct url when the base url already
has request params in place.
------------------------------------------------------------------------------------------------
Key: JBSEAM-1719
URL: http://jira.jboss.com/jira/browse/JBSEAM-1719
Project: JBoss Seam
Issue Type: Bug
Components: Core
Affects Versions: 2.0.0.BETA1
Reporter: Chris Rudd
Assigned To: Pete Muir
Given the following test case :
@Test
public void testBaseUrlAlreadyHasParams()
{
UrlBuilder url = new UrlBuilder("/someurl?arg1=a", "");
url.addParameter("foo", "bar");
String encodedUrl = url.getEncodedUrl();
Assert.assertEqual( encodedUrl, "/someurl?arg1=a&foo=bar" );
}
The following patch fixes the issue.
UrlBuilder.java line 62
protected String getParametersAsString()
{
String params = "";
for (String key : parameters.keySet())
{
params += "&" + key + "=" + parameters.get(key);
}
- if (!"".equals(params))
+ // repace the first "&" with a "?" if there are parameters and
the url doesnt already contain a "?"
+ if (!".equals(params) && (url!=null &&
!url.contains("?")))
{
params = "?" + params.substring(1);
}
return params;
}
NOTE: This condition occurs when using <s:link/> in a portlet, as the portlet
returns a url with parameters already added to the url.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.jboss.com/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
9:30 a.m.
New subject: [jbossseam-issues] [JBoss JIRA] Resolved: (JBSEAM-1719) UrlBuilder doesnt construct a correct url when the base url already has request params in place.
[ http://jira.jboss.com/jira/browse/JBSEAM-1719?page=all ]
Pete Muir resolved JBSEAM-1719.
-------------------------------
Fix Version/s: 2.0.0.CR1
Resolution: Done
Thanks!
UrlBuilder doesnt construct a correct url when the base url already
has request params in place.
------------------------------------------------------------------------------------------------
Key: JBSEAM-1719
URL: http://jira.jboss.com/jira/browse/JBSEAM-1719
Project: JBoss Seam
Issue Type: Bug
Components: Core
Affects Versions: 2.0.0.BETA1
Reporter: Chris Rudd
Assigned To: Pete Muir
Fix For: 2.0.0.CR1
Given the following test case :
@Test
public void testBaseUrlAlreadyHasParams()
{
UrlBuilder url = new UrlBuilder("/someurl?arg1=a", "");
url.addParameter("foo", "bar");
String encodedUrl = url.getEncodedUrl();
Assert.assertEqual( encodedUrl, "/someurl?arg1=a&foo=bar" );
}
The following patch fixes the issue.
UrlBuilder.java line 62
protected String getParametersAsString()
{
String params = "";
for (String key : parameters.keySet())
{
params += "&" + key + "=" + parameters.get(key);
}
- if (!"".equals(params))
+ // repace the first "&" with a "?" if there are parameters and
the url doesnt already contain a "?"
+ if (!".equals(params) && (url!=null &&
!url.contains("?")))
{
params = "?" + params.substring(1);
}
return params;
}
NOTE: This condition occurs when using <s:link/> in a portlet, as the portlet
returns a url with parameters already added to the url.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.jboss.com/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
5:07 a.m.
New subject: [jbossseam-issues] [JBoss JIRA] Closed: (JBSEAM-1719) UrlBuilder doesnt construct a correct url when the base url already has request params in place.
[ http://jira.jboss.com/jira/browse/JBSEAM-1719?page=all ]
Pete Muir closed JBSEAM-1719.
-----------------------------
UrlBuilder doesnt construct a correct url when the base url already
has request params in place.
------------------------------------------------------------------------------------------------
Key: JBSEAM-1719
URL: http://jira.jboss.com/jira/browse/JBSEAM-1719
Project: JBoss Seam
Issue Type: Bug
Components: Core
Affects Versions: 2.0.0.BETA1
Reporter: Chris Rudd
Assigned To: Pete Muir
Fix For: 2.0.0.CR1
Given the following test case :
@Test
public void testBaseUrlAlreadyHasParams()
{
UrlBuilder url = new UrlBuilder("/someurl?arg1=a", "");
url.addParameter("foo", "bar");
String encodedUrl = url.getEncodedUrl();
Assert.assertEqual( encodedUrl, "/someurl?arg1=a&foo=bar" );
}
The following patch fixes the issue.
UrlBuilder.java line 62
protected String getParametersAsString()
{
String params = "";
for (String key : parameters.keySet())
{
params += "&" + key + "=" + parameters.get(key);
}
- if (!"".equals(params))
+ // repace the first "&" with a "?" if there are parameters and
the url doesnt already contain a "?"
+ if (!".equals(params) && (url!=null &&
!url.contains("?")))
{
params = "?" + params.substring(1);
}
return params;
}
NOTE: This condition occurs when using <s:link/> in a portlet, as the portlet
returns a url with parameters already added to the url.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.jboss.com/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
6812
days inactive
6831
days old
4 comments
2 participants
participants (2)
-
Chris Rudd (JIRA) -
Pete Muir (JIRA)