Mailing-List: contact dev-help@cocoon.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cocoon.apache.org
Message-ID: <40925E6A.1070505@reverycodes.com>
Date: Fri, 30 Apr 2004 10:10:50 -0400
From: Vadim Gritsenko <vadim@reverycodes.com>
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US;
 rv:1.7b) Gecko/20040421
MIME-Version: 1.0
To: dev@cocoon.apache.org
Subject: Re: cforms: events on widget instances
References: <1083057893.28238.122.camel@23.13 	yum.ot 	yum
 <408ED30F.9090001@apache.org>	 <409246D6.3090709@reverycodes.com>
 <1083333661.5429.13.camel@23.13 	yum.ot 	yum
In-Reply-To: <1083333661.5429.13.camel@23.13 	yum.ot 	yum
Content-Type: text/plain; charset=us-ascii; format=flowed
Content-Transfer-Encoding: 7bit

Bruno Dumon wrote:

>On Fri, 2004-04-30 at 14:30, Vadim Gritsenko wrote:
>  
>
>>Sylvain Wallez wrote:
>>
>>    
>>
>>>Bruno Dumon wrote:
>>>      
>>>
>>...
>>
>>    
>>
>>>>And now the practical side: example usage in flowscript:
>>>>
>>>>form.getWidget("test").addActionListener(newActionListener(function(event) 
>>>>{
>>>>       print("hi there");
>>>>}));
>>>>
>>>>where the newActionListener function is as follows:
>>>>
>>>>function newActionListener(listenerFunction) {
>>>>   var listener = {actionPerformed: listenerFunction };
>>>>   var adapter = new
>>>>JavaAdapter(Packages.org.apache.cocoon.forms.event.ActionListener,
>>>>listener);
>>>>   return adapter;
>>>>}
>>>>
>>>>Question: do we add these newActionListener and newValueChangedListener
>>>>functions to the (javascript) form object, or as separate functions next
>>>>to it? 
>>>>
>>>>        
>>>>
>>>It could also be an additional method of ScriptableWidget, which would 
>>>take care of the adaptation.
>>>      
>>>
>>Agreed. And syntax should become then simply:
>>
>>form.getWidget("test").addActionListener(function(event) {
>>       print("hi there");
>>});
>>    
>>
>
>Saw my message about the v3 api?
>  
>

Saw but have not read yet - just got cable internet setup yesterday ;-)


>there I've implemented it as:
>
>form.model.test.onActivate = function() { print("hi there"); }
>
>which is still more readable and (I think) flexible enough.
>

Does it support more than one listener? Not that I need it right away, 
but addActionListener implies more than one listener, where "onActivate 
=" implies just one (setter semantic).

Vadim