[ 
https://issues.jboss.org/browse/RF-12593?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12823450#comment-12823450
 ] 

Lukáš Fryč commented on RF-12593:
---------------------------------

As we discussed offline,

the original intention of {{toScriptArgs}} is different:
* it suppose to generate function parameters

so the fact it can return empty string is given from the fact, that it is 
usually intended to be placed directly to method invocations, such as:

{code}
new RichFaces.ui.Push(#{toScriptArgs(clientId, options)});
{code}

but not assigning a value, such as:

{code}
var options = #{toScriptArgs(options)};
{code}

We will address this issue in RF-13725.
                
> RenderKitUtils.toScriptArgs() returns empty string when no not-null parameter 
> provided
> --------------------------------------------------------------------------------------
>
>                 Key: RF-12593
>                 URL: https://issues.jboss.org/browse/RF-12593
>             Project: RichFaces
>          Issue Type: Bug
>      Security Level: Public(Everyone can see) 
>          Components: core
>    Affects Versions: 4.3.0.M2
>            Reporter: Lukáš Fryč
>            Assignee: Lukáš Fryč
>            Priority: Minor
>             Fix For: 5.0.0.Alpha2
>
>   Original Estimate: 45 minutes
>  Remaining Estimate: 45 minutes
>
> The {{toScriptArgs(Object object)}} should always return JavaScript object:
> {code:javascript}
> { ... }
> {code}
> In case null object or no not-null parameter provided, empty object should be 
> returned:
> {code:javascript}
> {}
> {code}
> Currently, it returns empty string {{""}} which is not handy if we expect 
> JavaScript object as a return value.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

_______________________________________________
richfaces-issues mailing list
[email protected]
https://lists.jboss.org/mailman/listinfo/richfaces-issues

Reply via email to