Re: [xwiki-devs] [xwiki-notifications] r28726 - platform/web/branches/xwiki-web-2.3/standard/src/main/webapp/templates
On 05/05/2010 02:28 PM, tmortagne (SVN) wrote:
Author: tmortagne Date: 2010-05-05 14:28:17 +0200 (Wed, 05 May 2010) New Revision: 28726
Modified: platform/web/branches/xwiki-web-2.3/standard/src/main/webapp/templates/commentsinline.vm platform/web/branches/xwiki-web-2.3/standard/src/main/webapp/templates/informationinline.vm Log: [misc] Fix XHTML validity
Modified: platform/web/branches/xwiki-web-2.3/standard/src/main/webapp/templates/commentsinline.vm =================================================================== --- platform/web/branches/xwiki-web-2.3/standard/src/main/webapp/templates/commentsinline.vm 2010-05-05 12:27:21 UTC (rev 28725) +++ platform/web/branches/xwiki-web-2.3/standard/src/main/webapp/templates/commentsinline.vm 2010-05-05 12:28:17 UTC (rev 28726) @@ -90,7 +90,7 @@ #foreach($name in $request.getParameterNames()) #set($queryString = "$!queryString&$name=$escapetool.url($request.get($name))") #end - $response.sendRedirect("$doc.getURL('view')?captchaAnswerWrong=1&$queryString"); + $response.sendRedirect("$doc.getURL('view', "captchaAnswerWrong=1&$queryString")");
Are you sure this works? Looks like nested quotes which makes Velocity a bit unhappy... Also the & is still unescaped. $response.sendRedirect($doc.getURL('view', "captchaAnswerWrong=1&$queryString"));
#end $response.setContentLength(0) #stop
-- Sergiu Dumitriu http://purl.org/net/sergiu/
On Wed, May 5, 2010 at 14:35, Sergiu Dumitriu <sergiu@xwiki.com> wrote:
On 05/05/2010 02:28 PM, tmortagne (SVN) wrote:
Author: tmortagne Date: 2010-05-05 14:28:17 +0200 (Wed, 05 May 2010) New Revision: 28726
Modified: platform/web/branches/xwiki-web-2.3/standard/src/main/webapp/templates/commentsinline.vm platform/web/branches/xwiki-web-2.3/standard/src/main/webapp/templates/informationinline.vm Log: [misc] Fix XHTML validity
Modified: platform/web/branches/xwiki-web-2.3/standard/src/main/webapp/templates/commentsinline.vm =================================================================== --- platform/web/branches/xwiki-web-2.3/standard/src/main/webapp/templates/commentsinline.vm 2010-05-05 12:27:21 UTC (rev 28725) +++ platform/web/branches/xwiki-web-2.3/standard/src/main/webapp/templates/commentsinline.vm 2010-05-05 12:28:17 UTC (rev 28726) @@ -90,7 +90,7 @@ #foreach($name in $request.getParameterNames()) #set($queryString = "$!queryString&$name=$escapetool.url($request.get($name))") #end - $response.sendRedirect("$doc.getURL('view')?captchaAnswerWrong=1&$queryString"); + $response.sendRedirect("$doc.getURL('view', "captchaAnswerWrong=1&$queryString")");
Are you sure this works? Looks like nested quotes which makes Velocity a
Indeed this is wrong i apply the same thing that if it was a <a href instead of sendRedirect call. I'm fixing it.
bit unhappy... Also the & is still unescaped.
Actually i tough getURL was doing the job when since when testing in a page i got proper escaping but it's probably escaped by XHTML renderer now that i'm thinking more about it.
$response.sendRedirect($doc.getURL('view', "captchaAnswerWrong=1&$queryString"));
#end $response.setContentLength(0) #stop
-- Sergiu Dumitriu http://purl.org/net/sergiu/ _______________________________________________ devs mailing list devs@xwiki.org http://lists.xwiki.org/mailman/listinfo/devs
-- Thomas Mortagne
On 05/05/2010 02:51 PM, Thomas Mortagne wrote:
bit unhappy... Also the& is still unescaped.
Actually i tough getURL was doing the job when since when testing in a page i got proper escaping but it's probably escaped by XHTML renderer now that i'm thinking more about it.
$response.sendRedirect($doc.getURL('view', "captchaAnswerWrong=1&$queryString"));
Actually no, sendRedirect is not HTML, so it should NOT be escaped. -- Sergiu Dumitriu http://purl.org/net/sergiu/
participants (2)
-
Sergiu Dumitriu -
Thomas Mortagne