[
https://issues.apache.org/jira/browse/SHINDIG-969?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12682475#action_12682475
]
John Hjelmstad commented on SHINDIG-969:
----------------------------------------
Hi Balaji:
In order to maintain existing style, could you remove the param from the ctor
(since it accepts null anyway) and add method .setGadgetFeatureRegistry()?
> Need convenience method to get all required features of a gadget.
> -----------------------------------------------------------------
>
> Key: SHINDIG-969
> URL: https://issues.apache.org/jira/browse/SHINDIG-969
> Project: Shindig
> Issue Type: Improvement
> Components: Java
> Affects Versions: trunk
> Reporter: Balaji Srinivasan
> Priority: Minor
> Fix For: trunk
>
> Attachments: patch.txt
>
>
> Right now if we want to find out all the required features for a gadget, we
> have to do the following
> gadgetFeatureRegistry.getFeatures(gadget.getSpec().getModulePrefs().getFeatures().keySet())
> This means that any component that needs to get to the list of required
> features, has to get access to the gadget feature registry.
> It would be nice to have a convenience method in Gadget that would give
> that...
> ie gadget.getAllFeatures();
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
