116534 - the namespace declaration on the included stylesheet is not picked up

Status: VERIFIED FIXED

(Core :: XSLT, defect)

Description

[Simon Pepping]

document bug1a.xml, stylesheets bug1a.xsl, bug1a-incl.xsl:

   Error: template obs:test is not applied; instead the default
   template is applied.

   Probable cause: the namespace declaration on the included
   stylesheet is not picked up.

   Proof: When the namespace declaration

   xmlns:obs="http://salix.spepping.observations"

   is added to the stylesheet bug1a.xsl, the problem
   disappears.

   document bug1b.xml, stylesheets bug1b.xsl, bug1ba.xsl:

   Now there is a template matching /, and the template obs:test _is_
   applied, proving that the above error is connected with the
   treatment of the default / template.

Comment 1

[Simon Pepping]

Attachment: xml file for case a: show the problem

Comment 2

[Simon Pepping]

Attachment: xsl stylesheet for case a: show the problem

Comment 3

[Simon Pepping]

Attachment: included xsl stylesheet for case a: show the problem

Comment 4

[Simon Pepping]

Attachment: xml file for case b: remove the problem by have a template matching /

Comment 5

[Simon Pepping]

Attachment: stylesheet file for case b: remove the problem by have a template matching /

Comment 6

[Simon Pepping]

Attachment: included stylesheet file for case b: remove the problem by have a template matching /

Comment 7

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

this among most other namespace problems will be fixed once we have the new 
contexts.

Comment 8

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

er, actually, this is one namespaceproblem that *isn't* fixed there. It is 
however fixed in petervs output rewrite :-)

Comment 9

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

Sorry, i'm on crack, it's the contexts that are needed. Sorry for spam.

Oh, simon. Don't bother with checking more namespace problems. We have tons of 
known namespace bugs, but they should all get fixed once bug 96410 and bug 
96647 are fixed

Comment 10

[Peter Van der Beken [:peterv]]

Is this fixed?

Comment 11

[Axel Hecht]

yes, it is.

Comment 12

[Axel Hecht]

we didn't verify for a long time.
I really checked, so VERIFIED.