Mailing-List: contact dev-help@cocoon.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cocoon.apache.org
Received-SPF: neutral (herse.apache.org: local policy)
Message-ID: <468E1570.8080904@apache.org>
Date: Fri, 06 Jul 2007 12:12:00 +0200
From: Grzegorz Kossakowski <gkossakowski@apache.org>
User-Agent: Thunderbird 2.0.0.4 (X11/20070604)
MIME-Version: 1.0
To: dev@cocoon.apache.org
Subject: Re: RT: map:call as generic non-redirecting controller code
References: <B7028BD0-7135-4F83-917E-A558A2344F1B@nukinetics.com>
 <4683DF1E.902@nada.kth.se>
 <884F6A4A-DA90-40FC-BFD8-351A0306CC45@nukinetics.com>
 <468B9869.8070206@nada.kth.se> <468BA762.9040405@apache.org>
 <BA5FC3D0-0D99-4ACE-99DF-475D2B9ECF12@nukinetics.com>
 <468D63B9.6060008@nada.kth.se>
 <A0C4E332-3351-4C60-B673-0F938F662D24@nukinetics.com>
In-Reply-To: <A0C4E332-3351-4C60-B673-0F938F662D24@nukinetics.com>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit

Ellis Pritchard pisze:
>> Do you mean that you would have flowscripts like:
>>
>> myFlowAction() {
>>   var result = calculateSomething();
>>   setReturnData(result);
>> }
>>
>> I would much prefer to just use the return value of the flowscript:
>>
>> myFlowAction() {
>>   return calculateSomething();
>> }
> 
> Agreed, the former is clumsy, but easier/less disruptive to implement, 
> the latter is much more natural and what I was originally aiming at.
> 
> Ok, I'll look out for further feedback, but otherwise submit a patch 
> using the direct return method for people to review.

Just to be sure, do you want to implement something like:
<map:match pattern="sth">
   <map:call function="prepareData"/>
   <map:generate src="..."/> <- some protocol to obtain the prepared data
   [...]
</map:match>

Such construct introduces new semantics for sitemap because data returned by <map:call> will be available _outside_ <map:call> element. Now 
it is important what is the scope where the data will be visible. Have you thought about it already?

-- 
Grzegorz Kossakowski
http://reflectingonthevicissitudes.wordpress.com/