Sorry, I accidentally hit reply before typing anything.
Torsten, what you are trying to do is not incorrect and should work as you
expected it to. The reason why it didn't has less to do with XMLSupport per se
and more to do with its reliance on Pharo's TextConverter system. The problem
is faulty matching of the "encoding" attribute value to the appropriate
subclass of TextConverter. The code responsible for this in XMLSupport is:
converterClass :=
(Smalltalk
at: #TextConverter
ifAbsent: [^ self])
defaultConverterClassForEncoding:
anEncodingName asLowercase.
But as you can see, the matching is actually done by TextConverter and its
class-side #defaultConverterClassForEncoding: method, which works by sending
#encodingNames to all subclasses and testing the array returned to see if it
includes the specified encoding name. If you browse Latin1TextConverter, the
right class for the encoding you specified, and look at its #encodingNames
message, you will see the array it returns does not include "ISO-8859-1":
^ #('latin-1' 'latin1') copy.
Change it to this (note the lowercase):
^ #('latin-1' 'latin1' 'iso-8859-1') copy.
and everything now works.
So this is really a bug in TextConverter and its Latin1TextConverter subclass,
not XMLSupport. Also, the #allSubclassesDo: test in
#defaultConverterClassForEncoding: should probably be augmented with a
Dictionary cache to speed-up lookups for known encoding-converter pairs. Can
someone forward this message to whoever maintains TextConverter?
---- On Mon, 10 Jan 2011 04:38:23 -0800 Torsten Bergmann wrote ----
>Create a file "test.xml" with the following
>contents (german umlaut):
>
>
>
>
>
>After loading ConfigurationOfXML try to parse it:
>
> |fs|
> fs := FileStream fileNamed: 'test.xml'.
> XMLDOMParser parseDocumentFrom: fs.
>
>
>=> gives an error: 'Invalid utf8 input detected'
>=> it works if you remove the CDATA section
>
>Looks like UTF8TextConverter is used independent
>from the encoding of the XML...
>
>Bye
>T.
>--
>NEU: FreePhone - kostenlos mobil telefonieren und surfen!
>Jetzt informieren: http://www.gmx.net/de/go/freephone
>
>
