[Yanel-dev] Sitetree has problems with special characters (patch included)

[Michael Wechner]

Hi

I think I have finally fixed the last bug re the changed implementation:

Sending        
src/contributions/resources/data-repo-sitetree/src/java/org/wyona/yanel/impl/resources/navigation/DataRepoSitetreeResource.java
Transmitting file data .
Committed revision 56946.

and the AJAX based navigation display works again as well

http://127.0.0.1:8080/yanel/from-scratch-realm/yanel/data-repository-sitetree.html

Cheers

Michael

On 2/8/11 10:37 AM, Michael Wechner wrote:
> Hi Cedric
>
> Thanks very much for this great patch. I have committed it now:
>
> Sending        
> src/contributions/resources/data-repo-sitetree/src/java/org/wyona/yanel/impl/resources/navigation/DataRepoSitetreeResource.java
> Transmitting file data .
> Committed revision 56610.
>
> whereas I have noticed another minor bug re display of the HTML, but 
> which is not
> related to your changes above (during review I first thought this 
> might be the case).
>
> I will try to fix it shortly.
>
> Cheers
>
> Michael
>
> On 1/25/11 1:35 PM, Cedric Staub wrote:
>> Hello
>>
>> As it turns out the data-repo-sitetree resource shipped with Yanel has
>> problems with special characters. The reason for this is very simple:
>> the resource uses a StringBuffer to construct the XML output. That means
>> special characters never get escaped.
>>
>> So I cleaned the whole thing up and migrated the resource to the W3C DOM
>> API (org.w3c.dom). All special characters are automatically escaped now.
>> As a result, the resource is much more robust than before and i18n-safe.
>>
>> See patch below.
>>
>> Cheers
>> Cedric
>