Markup bug in beta 14
Printed From: Codejock Forums
Category: Codejock Products
Forum Name: Toolkit Pro
Forum Description: Topics Related to Codejock Toolkit Pro
URL: http://forum.codejock.com/forum_posts.asp?TID=16220
Printed Date: 28 September 2024 at 7:54pm
Software Version: Web Wiz Forums 12.04 - http://www.webwizforums.com
Topic: Markup bug in beta 14
Posted By: adrien
Subject: Markup bug in beta 14
Date Posted: 18 February 2010 at 1:13am
Hi
We're using markup to create some buttons. We get the callback when the button is clicked, but for some reason the CJ framework messed with the tag we had on the element (inserted a semicolon into it). The tag names are using a URI format for an internal action so they can get longish (about 60chars).
This was introduced in 14.0 beta (original and latest).
-------------
http://www.wingate.com - http://www.wingate.com
|
Replies:
Posted By: adrien
Date Posted: 18 February 2010 at 2:36am
OK, I think I found the problem.
Using VSS to do a diff on XTPMarkupParser.cpp, there are only 2 diffs between 13.2.1 and 14.0.0 beta. IN
CXTPMarkupParser::TokenType CXTPMarkupParser::ScanHead()
line 205 if (c == '&')
c = ScanEntity();
was added.
ScanEntity returns ';' if not an entity.
We have ampersands in our tag values (since it's a URL with query string).
I don't think you should be trying to parse entities out of tag values? Probably not any values. Anything between quotes shouldn't be parsed?
-------------
http://www.wingate.com - http://www.wingate.com
|
Posted By: adrien
Date Posted: 18 February 2010 at 2:39am
yep, that's definitely it. It inserted the ';' about 31 chars after the &
-------------
http://www.wingate.com - http://www.wingate.com
|
Posted By: adrien
Date Posted: 18 February 2010 at 4:15am
OK, looks like my bad.
Looks like ampersands must be escaped.
-------------
http://www.wingate.com - http://www.wingate.com
|
Posted By: adrien
Date Posted: 18 February 2010 at 4:19am
looks like they must not be escaped in prior versions though.... hmmm
-------------
http://www.wingate.com - http://www.wingate.com
|
|