For localization purpose, we need the parser to support general entities
declared in the external DTD file. (XML spec says an entity is internal if
the entity definition is an EntityValue; otherwise, it is an external one).
Also, the parser needs to provide a callback hook so that the locale
information could be used to locate and fetch the language-specific DTD.
Add URL to XUL localizability spec. Also assign QA contact.
Accepting bug and changing component to XML.
Catalin is working on this and from my talk with him yesterday, I think this
won't make it in time for M5. So, I'm moving this bug to M6, assigning this to
Catalin and putting myself on the cc list.
The status of it:
1. It takes absolute URL as the systemid only.
2. It need doctype declaration in the DTD file. This shouldn't be the case.
3. It does nou support co-existence of internal and external DTD subset yet.
I'll file separate bugs on them.
It's in, but the code is #ifdef'ed. Reassigning to Nisheeth so he can final test
& review and turn the flag on.
I tried testing the following XML and, with EXTERNAL_ENTITY_SUPPORT enabled,
nothing displays in viewer:
this text should display.
Ccing cata so that he can also take a look at this.
I thought the correct syntax was:
(please note the final "?")
Maybe that's the problem?
You are right about the syntax. But, an error message should show up in the
viewer. Currently, nothing shows up.
Oops, my bad. It seems that this behavior was occurring earlier also...
I'm in there testing more as I type...
My testing did not uncover new issues: everything I found already has bugs open
or was reported by James Clark.
Clearing status because basic XML does display fine with these changes enabled.
Changing summary to reflect that only enablement remains.
I'm setting the target milestone for the resolution of this bug to M8. We need
to address James Clarks' concerns as well as resolve bugs 8251, 8254, and 8255
before we turn this on for everybody. Adding those bugs to the bug dependency
I just turned this on in the builds of all platforms. Thanks a lot to James for
providing a patch to expat that adds this new feature. Many thanks to Tao for
integrating James' changes into the mozilla codebase.
Verified with jul-07-99 build.