Changes between Version 1 and Version 2 of Ticket #14443, comment 3


Ignore:
Timestamp:
Dec 12, 2011, 2:15:30 AM (8 years ago)
Author:
bill
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #14443, comment 3

    v1 v2  
    77I thought to set it to root, or whatever tree does where persist == false.
    88
     9We should add unit tests for both these cases.   Previously we didn't have unit tests because it's difficult, having to clear and set cookies, but I guess all you really need to do is to clear the cookies originally, then run the Tree to a certain state, and the delete it and recreate it [after having modified the data store some].
     10
    911> BTW: what's the reason for the double negation if(!!nextNode) line 1010?
    1012
    11 I assume that's needed in case the array return from filter is zero-length.    Differentiating between [][0] and [false][0].
    12 We should add unit tests for both these cases.   Previously we didn't have unit tests because it's difficult, having to clear and set cookies, but I guess all you really need to do is to clear the cookies originally, then run the Tree to a certain state, and the delete it and recreate it [after having modified the data store some].
     13I assume that's needed in case the array return from filter is zero-length.    Differentiating between {{{[][0]}}} and {{{[false][0]}}}.
     14