Closed (fixed)
Project:
Views Datasource
Version:
6.x-1.0-alpha2
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
9 Sep 2008 at 15:41 UTC
Updated:
24 Oct 2009 at 17:10 UTC
Jump to comment: Most recent file
Comments
Comment #1
mbutcher commentedEncoding the entities as numeric values instead of shorthand codes would also work and is cross-parser capable, since it doesn't depend on HTML-specific entities.
E.g. Instead of using amp in an entity, use #38. This would be a trivial change to views_xml.module. See the attached patch.
This can be used in conjunction with the patch above, or on its own (CDATA sections won't be necessary).
Comment #2
mbutcher commentedSince this has two patches attached now, I figured I'd better change the status.
Comment #3
dalinI like the idea but your patches need a bit of work. Please also patch for XHTML and RDFa. Please roll all patches into one file (preferably using cvs diff -upNR for better readability). Please do not comment out the code that you are replacing, just remove it.
This patch is racing against #286715: Invalid xml due to incorrectly encoded greater-than symbol but this is the better approach.
cheers
Comment #4
allisterbeharry commentedThis problem is actually specific to one entity - the apostrophe ('). there was a typo in the code: I had &pos; instead of '. Fixed in alpha3: http://drupal.org/node/598242
Comment #5
allisterbeharry commented