[ 
https://issues.apache.org/jira/browse/SLING-15?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12529544
 ] 

Felix Meschberger commented on SLING-15:
----------------------------------------

You are of course right and both solutions in the end are probably equivalent. 
But as I do not like having empty stuff lying around, I rather implement 
support to cope with missing stuff. There are two more arguments: Nodetype 
management is not well supported currently, so changing the node type would 
cause headaches and coping with possible null value in the ScriptedComponent 
makes it more stable.

> NPE if the scripted component hasn't sling:scripts node created
> ---------------------------------------------------------------
>
>                 Key: SLING-15
>                 URL: https://issues.apache.org/jira/browse/SLING-15
>             Project: Sling
>          Issue Type: Bug
>          Components: Scripting
>            Reporter: Edgar Poce
>            Assignee: Felix Meschberger
>            Priority: Minor
>             Fix For: 2.0.0
>
>
> if the ScriptedComponent node hasn't the child node "sling:scripts" the 
> request for nodes that refer to that scripted component can't be resolved, it 
> throws NPE at ScriptedComponent: 148 because the variable scripts is not 
> initialized.
> proposed fix: add "mandatory" and "autocreated" to sling:ScriptedComponent 
> Nodetype definition.

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

Reply via email to