[Laszlo-dev] For Review: Change 20081113-lou-C Summary:, lz.param: clarify return type in the reference, fix nav pane entries

André Bargull andre.bargull at udo.edu
Fri Nov 14 07:15:36 PST 2008


For example something like:

<!--- Returns value.
     @return *: the value to use for this parameter -->
<method name="getValue">
     return this.value;
</method>

And I've just spotted a minor spelling mistake:
Value to use for "parameter".
> -        <!--- Value to use for paramter. A getValue() method can be declared
> +        <!--- Value to use for paramter. A <code>getValue()</code> method can be declared



On 11/14/2008 4:00 PM, Lou Iorio wrote:
> Ah, so what should this be?
> 
>         <!--- Returns value.
>             @return String: the string representing the value. -->
>         <method name="getValue">
>           return this.value;
>         </method>
> 
> 
> On Nov 13, 2008, at 6:33 PM, André Bargull wrote:
> 
>> Not approved, because lz.param#getValue() may return any type, not 
>> just strings (the value-attribute is an expression not a string, too!).
>> The limitation for String does only apply to LzParam aka lz.Param aka 
>> lz.params, but not to lz.param! (hey, I didn't choose those names...)
>> (Ghost used LzParam and they wondered why it just returns strings in 
>> swf9, and they said the documentation does not cover this aspect, but 
>> they used the wrong reference: lz.param instead of LzParam.)
>>
>>> Change 20081113-lou-C by lou at loumac.local <mailto:lou at loumac.local> 
>>> on 2008-11-13 11:42:04 AST
>>>     in /Users/lou/src/svn/openlaszlo/trunk
>>>     for http://svn.openlaszlo.org/openlaszlo/trunk
>>>
>>> Summary: lz.param: clarify return type in the reference, fix nav pane 
>>>  entries
>>>
>>> Bugs Fixed: LPP-7327, LPP-7323
>>>
>>> Technical Reviewer: (pending)
>>> QA Reviewer: (pending)
>>> Doc Reviewer: Tucker
>>>
>>> Details: document that getValue() returns String
>>> move lz.param to RCP category
>>> add lz.Param to Data category
>>>
>>> Tests: visual verify
>>>
>>> Files:
>>> M      docs/src/nav/toc.xml
>>> M      lps/components/rpc/rpc.lzx
>>>
>>>
>>> Changeset: 
>>> http://svn.openlaszlo.org/openlaszlo/patches/20081113-lou-C.tar
>>>  
>>
> 



More information about the Laszlo-dev mailing list