[ https://issues.jboss.org/browse/RF-12593?page=com.atlassian.jira.plugin.s... ] Lukáš Fryč edited comment on RF-12593 at 10/18/13 8:43 AM: ----------------------------------------------------------- The issue is that {{toScriptArgs}} suppose to return not only collections, as can be seen in the test: https://github.com/richfaces/richfaces/blob/master/framework/src/test/jav... In this case, it would be much better if it return: * empty hash {{{}}} when empty map provided * empty array {{[]}} when empty collection or array provided * {{null}} if null provided This is mostly the behavior which it already has: https://github.com/richfaces/richfaces/blob/master/framework/src/test/jav... The only case is return of {{""}} instead of {{null}}. ---- While {{null}} can sometimes cause {{variable is undefined}} issue, returning empty value is much more dangerous, since it leads to the code which is not compilable in runtime at all. E.g.: https://github.com/richfaces/richfaces/blob/master/framework/src/main/jav... In this code, when no options are provided, the renderer code is: {code} var options = ; {code} which is syntactically wrong. was (Author: lfryc): The issue is that {{toScriptArgs}} suppose to return not only collections, as can be seen in the test: https://github.com/richfaces/richfaces/blob/master/framework/src/test/jav... In this case, it would be much better if it return: * empty hash {{{}}} when empty map provided * empty array {{[]}} when empty collection or array provided * {{null}} if null provided -- 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