You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sling.apache.org by "Carsten Ziegeler (JIRA)" <ji...@apache.org> on 2009/12/24 10:11:29 UTC

[jira] Resolved: (SLING-1236) Scala Scripting: Make all types of the script arguments visible on the Scala side

     [ https://issues.apache.org/jira/browse/SLING-1236?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Carsten Ziegeler resolved SLING-1236.
-------------------------------------

    Resolution: Fixed

Thanks for your patch, Michael - I've applied it in revision: 893712

> Scala Scripting: Make all types of the script arguments visible on the Scala side
> ---------------------------------------------------------------------------------
>
>                 Key: SLING-1236
>                 URL: https://issues.apache.org/jira/browse/SLING-1236
>             Project: Sling
>          Issue Type: Improvement
>          Components: Scripting
>            Reporter: Michael Dürig
>            Assignee: Carsten Ziegeler
>            Priority: Minor
>         Attachments: SLING-1236.patch
>
>
> Currently the Scala scripting engine inserts a preamble for the bindings passed to a Script. The preamble contains a declaration of the following form for each value in the Bindings instance:
>   val request: SlingHttpServletRequest = bindings.get("request").asInstanceOf[SlingHttpServletRequest]
> The types are hard coded in the ScalaScriptEngine.TypeHints class. For unknown types a declaration of type AnyRef is inserted into the preamble. 
> I suggest to implement a run time type discovery algorithm which discovers the accessible type from a given instance and inserts the respective declarations into the preamble. That way the static coupling between the known values in Bindings and its types would go away. Furthermore the values would be of a type appearing to be of all the visible types on the Scala side.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.