Changes between Initial Version and Version 1 of Ticket #16785


Ignore:
Timestamp:
Mar 4, 2013, 3:02:35 AM (8 years ago)
Author:
bill
Comment:

So, even though the URL is different, it uses the same cookie? Even if it's the same URL though, I've seen errors because the Tree data has changed, and the previously selected node no longer exists.

I agree that persist should be false by default.

The code to persist selection was added in #14058, but now that you mention it, I don't see a purpose to it either. It was probably added more to remember the last focused node rather than the last selected node, but that's also misguided since Tree.focus() by default does not associate focus with selection.

About PathError, it's natural for set("path", ...) to throw an error / give an exception if the argument is invalid, but if persist:true saves the selection state, probably the error from set("path", ...) should be squelched.

So, I agree with you, and will implement your suggestion.

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #16785

    • Property Status changed from new to assigned
    • Property Milestone changed from tbd to 1.9
    • Property Summary changed from Diji/Tree throws invalid PathError to Tree: throws invalid PathError on startup
  • Ticket #16785 – Description

    initial v1  
    1 The dijit/Tree throws an invalid 'PathError' whenever two different trees use, or get, the same ID.
     1The dijit/Tree throws an invalid '!PathError' whenever two different trees use, or get, the same ID.
    22
    33The scenario: