[Laszlo-dev] For Review: Change 20070904-maxcarlson-N, Summary: Update explicit replicator when items are added or deleted

[P T Withington]

On 2007-09-04, at 17:57 EDT, André Bargull wrote:

> I guess I would change this line, because the latter version is in  
> my mind more obvious.
> this._ondatadel = new LzDelegate(this, '_updateChildren',  
> this._pointer.p, 'onDocumentChange')
> => this._ondatadel = new LzDelegate(this, '_updateChildren',  
> this.dataset, 'onDocumentChange')

Sounds reasonable.

> And maybe you should add a manual clean-up for "_ondatadel" to  
> avoid any memory leaks.

Sounds important!

> André
>
>
> Max Carlson wrote:
>> Change 20070904-maxcarlson-N by maxcarlson at plastik on 2007-09-04   
>> 13:51:25 PDT
>>      in /Users/maxcarlson/openlaszlo/wafflecone
>>      for http://svn.openlaszlo.org/openlaszlo/branches/wafflecone
>>
>> Summary: Update explicit replicator when items are added or deleted
>>
>> New Features:
>>
>> Bugs Fixed: LPP-4612 - Explicit replicator doesn't update when  
>> items  are added or deleted
>>
>> Technical Reviewer: pkang
>> QA Reviewer: ptw
>> Doc Reviewer: (pending)
>>
>> Documentation:
>>
>> Release Notes:
>>
>> Details: test/explicit-replicators/replicator.lzx - Incorporate  
>> tests  from LPP-4612.
>>
>> lps/components/utils/replicator/replicator.lzx - Store reference  
>> to  dataset pointer.  Register _updateChildren() for  
>> onDocumentChange  events.  Add _updateChildren() method to receive  
>> document change  events and rerun the xpath.
>>
>>
>> Tests: See LPP-4612.  Also updated test/explicit-replicators/  
>> replicator.lzx.
>>
>> Files:
>> M      test/explicit-replicators/replicator.lzx
>> M      lps/components/utils/replicator/replicator.lzx
>>
>> Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20070904-  
>> maxcarlson-N.tar
>
> -- 
> Official OpenLaszlo-Committer ^_^