Closed
Bug 110266
Opened 23 years ago
Closed 23 years ago
Namespace problem in TransforMiiX
Categories
(Core :: XSLT, defect)
Core
XSLT
Tracking
()
VERIFIED
FIXED
People
(Reporter: bugzilla3, Assigned: sicking)
References
Details
Attachments
(2 files, 3 obsolete files)
I'm having some trouble getting Transformiix to parse a document that uses
namespaces. The attached test case outputs "[]" instead of "[foo]"
See
http://groups.google.com/groups?hl=en&th=b162e03e2cdfa928&seekm=3BF3A3EF.5DF9450D%40numerik.uni-kiel.de&frame=off
for netnews discussion.
Reporter | ||
Comment 1•23 years ago
|
||
Reporter | ||
Comment 2•23 years ago
|
||
Comment 3•23 years ago
|
||
bug 102293 could fix this. We need to check a few landing gears for this.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Assignee | ||
Comment 4•23 years ago
|
||
we can't really fix this one fully until we resolve namespace during parsing,
which require the new contexts. We could however work around this and make it
work for the most common cases by setting mXPathParseContext to
xslDocument.documentElement at the top of ::findTemplate. It's a hack and will
not fully work, but it should cover the most common cases and i think that that
was in effect what we used to do.
We should at least get this in better shape for the next milestone, but until
then it might IMHO be better to concentrate our efforts on the RightThing.
Updated•23 years ago
|
Attachment #57916 -
Attachment mime type: text/xsl → text/xml
Assignee | ||
Comment 5•23 years ago
|
||
Since the plans for the contexts has become somewhat bigger i doubt that we'll
be able to get them in before 0.9.7. So this hack-around will make us do about
the same as we used to do, which should take care of most cases.
If we still aim the contexts for 0.9.7 then just ignore the patch
Comment 6•23 years ago
|
||
Comment on attachment 60326 [details] [diff] [review]
workaround hack
OK, contexts won't make it to 0.9.7.
Suggest a text to be put up on the project page, so
we can describe the horkage in detail.
Add a XXX wallpaper to the
comment, this is not a
solution.
Other than that, r=me
Attachment #60326 -
Flags: review+
Assignee | ||
Comment 7•23 years ago
|
||
added the comment in my local tree. I suggest we put something like
"Transformiix namespace handling currently has many problems, watch bug 96410
and bug 113611 for progress"
Assignee: kvisco → sicking
Comment 8•23 years ago
|
||
Comment on attachment 60326 [details] [diff] [review]
workaround hack
sr=jst
Attachment #60326 -
Flags: superreview+
Assignee | ||
Comment 9•23 years ago
|
||
checked in, we'll close this once namespacehandling actually works. Pike, do
you want this bug now?
Status: NEW → ASSIGNED
OS: Linux → All
Hardware: PC → All
Comment 10•23 years ago
|
||
Attachment #57916 -
Attachment is obsolete: true
Attachment #57917 -
Attachment is obsolete: true
Attachment #60326 -
Attachment is obsolete: true
Comment 11•23 years ago
|
||
Comment 12•23 years ago
|
||
Fixed.
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Comment 13•22 years ago
|
||
we didn't verify for a long time.
I really checked, so VERIFIED.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•