[Mono-bugs] [Bug 77781][Nor] Changed - current() not working in xsl:sort

[bugzilla-daemon at bugzilla.ximian.com]

Please do not reply to this email- if you want to comment on the bug, go to the
URL shown below and enter your comments there.

Changed by atsushi at ximian.com.

http://bugzilla.ximian.com/show_bug.cgi?id=77781

--- shadow/77781	2006-03-14 16:10:42.000000000 -0500
+++ shadow/77781.tmp.17183	2006-03-15 06:00:03.000000000 -0500
@@ -11,13 +11,13 @@
 AssignedTo: atsushi at ximian.com                            
 ReportedBy: hugobendelin at hotmail.com               
 QAContact: mono-bugs at ximian.com
 TargetMilestone: ---
 URL: 
 Cc: 
-Summary: Xpath sort problem
+Summary: current() not working in xsl:sort
 
 Please fill in this template when reporting a bug, unless you know what you
 are doing.
 Description of Problem:
 
 I Have been trying to write a xsl file to sort and style a xml-document on
@@ -243,6 +243,18 @@
 <xsl:value-of select="$goals + $ass" /> (<xsl:value-of select="$goals"
 />, <xsl:value-of select="$ass" />).
   </xsl:for-each>
  </xsl:template>
 </xsl:stylesheet>
 
+
+------- Additional Comments From atsushi at ximian.com  2006-03-15 06:00 -------
+After digging into XSLT implementation I found that xsl:sort works if
+it does not contain current() function. Actually current mono approach
+is not sufficient to handle current() function inside xsl:sort since
+it is processed as XPathExpression.AddSort(), and the sort elements
+are evaluated without proper "current node" context. Thus it always
+evaluates current node context as its parent's.
+
+To make proper fix, I will have to put sort functionality besides
+XPathExpression.AddSort(). It might not be fixed soon, but hopefully
+within a few days.