[Helma-user] Suggestion: onPersist function call

Kris Leite kleite at imcsoftware.com
Wed Apr 4 07:43:16 CEST 2007


Hi Hannes,

Attached is a patch for the 'onPersist' function call.  In version 1.6 I 
found that
the 'onInit' was moved from the Node class to NodeManager.  So I put the
'invokeOnPersist' in the NodeManager class to be consistent.  I also 
removed
the check for 'dbmap' in the 'onInit' section since that would only 
allow the routine
to be activated for only database configuration.  The documentation for 
'onInit'
indicated it was to be activated for both embedded and database persistence
storage.

Thanks,
Kris


Hannes Wallnoefer wrote:
> Hi Kris,
>
> 2007/4/3, Kris Leite <kleite at imcsoftware.com>:
>   
>> I had a need to do some changes on a HopObject before it was
>> written into storage.  So I made a customized version Helma 1.5.3
>> to call 'onPersist' function when a HopObject was being written
>> into the embedded or relational database.
>>
>> It would be nice if this was added as a new feature for Helma 1.6.
>>
>> The way I implemented it was very similar to the 'onInit()' function
>> by making a copy of the "invokeOnInit" function in Node.java
>> and changing the names to 'OnPersist' then adding a call to
>> 'invokeOnPersist' in the NodeManager methods 'insertNode' and
>> 'updateNode'.
>>
>> I will be happy to provide the patches if you like.
>>     
>
> Yes, please send a patch.
>
> hannes
>
>   
>> Thanks,
>> Kris
>>
>>
>> _______________________________________________
>> Helma-user mailing list
>> Helma-user at helma.org
>> http://helma.org/mailman/listinfo/helma-user
>>
>>     
> _______________________________________________
> Helma-user mailing list
> Helma-user at helma.org
> http://helma.org/mailman/listinfo/helma-user
>
>   
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: onPersist.patch
Url: http://helma.org/pipermail/helma-user/attachments/20070403/601f55fe/attachment.ksh 


More information about the Helma-user mailing list