73851 - ``natural order'' sort does not work in general case

Status: RESOLVED WONTFIX

(Core :: XUL, defect)

Description

[Chris Waterson]

Specifically, in nsXULOutlinerBuilder::CompareMatches()...

        // If the sort order is ``natural'', then see if the container
        // is an RDF sequence. If so, we'll try to use the ordinal
        // properties to determine order.
        //
        // XXX the problem with this is, it doesn't always get the
        // *real* container; e.g.,
        //
        //  <outlinerrow uri="?uri" />
        //
        //  <triple subject="?uri"
        //          predicate="http://home.netscape.com/NC-rdf#subheadings"
        //          object="?subheadings" />
        //
        //  <member container="?subheadings" child="?subheading" />
        //
        // In this case mContainerVar is bound to ?uri, not
        // ?subheadings. (The ``container'' in the template sense !=
        // container in the RDF sense.)

So basically, we don't get it right if the ``template container'' is not the
``RDF container''.

Comment 1

[Asa Dotzler [:asa]]

Bugs targeted at mozilla1.0 without the mozilla1.0 keyword moved to mozilla1.0.1 
(you can query for this string to delete spam or retrieve the list of bugs I've 
moved)

Comment 2

[Chris Waterson]

cc'ing sgehani. samir, is this the problem that you're seeing?

Comment 3

[Samir Gehani]

Well, the sidebar tabs are in a generic template, not in a tree, so I don't
think they exercise this code.

Comment 4

[Jonathan]

rg nsXULOutlinerBuilder finds nothing

Comment 5

[BugBot [:suhaib / :marco/ :calixte]]

The bug assignee didn't login in Bugzilla in the last 7 months.
:enndeakin, could you have a look please?
For more information, please visit auto_nag documentation.

Comment 6

[Neil Deakin]

No longer relevant.