[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