You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@isis.apache.org by "Dan Haywood (JIRA)" <ji...@apache.org> on 2011/04/23 08:43:05 UTC
[jira] [Created] (ISIS-85) Convert HelpFacetDecorator into a
HelpFacetFactory
Convert HelpFacetDecorator into a HelpFacetFactory
--------------------------------------------------
Key: ISIS-85
URL: https://issues.apache.org/jira/browse/ISIS-85
Project: Isis
Issue Type: Improvement
Reporter: Dan Haywood
Looking at the code, it seems that we never create HelpFacets through FacetFactory. So neither the HelpFacetDecorator nor the I18NFacetDecorator looks like they will ever install help, because both require there to be an underlying HelpFacet to decorate.
Moreover, there is a duplication between the I18 facet decorator and the help decorator, in that they both install help facets.
I think a good first step would be to make the HelpFacetDecorator into a regular HelpFacetFactory. At least then the role of i18nFacetDecorator will be well-defined.
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Updated] (ISIS-85) Convert HelpFacetDecorator into a
HelpFacetFactory
Posted by "Dan Haywood (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/ISIS-85?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Dan Haywood updated ISIS-85:
----------------------------
Priority: Minor (was: Major)
> Convert HelpFacetDecorator into a HelpFacetFactory
> --------------------------------------------------
>
> Key: ISIS-85
> URL: https://issues.apache.org/jira/browse/ISIS-85
> Project: Isis
> Issue Type: Improvement
> Reporter: Dan Haywood
> Priority: Minor
>
> Looking at the code, it seems that we never create HelpFacets through FacetFactory. So neither the HelpFacetDecorator nor the I18NFacetDecorator looks like they will ever install help, because both require there to be an underlying HelpFacet to decorate.
> Moreover, there is a duplication between the I18 facet decorator and the help decorator, in that they both install help facets.
> I think a good first step would be to make the HelpFacetDecorator into a regular HelpFacetFactory. At least then the role of i18nFacetDecorator will be well-defined.
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Updated] (ISIS-85) Convert HelpFacetDecorator into a
HelpFacetFactory
Posted by "Dan Haywood (Updated) (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/ISIS-85?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Dan Haywood updated ISIS-85:
----------------------------
Component/s: Core: MetaModel
Affects Version/s: 0.1.2-incubating
> Convert HelpFacetDecorator into a HelpFacetFactory
> --------------------------------------------------
>
> Key: ISIS-85
> URL: https://issues.apache.org/jira/browse/ISIS-85
> Project: Isis
> Issue Type: Improvement
> Components: Core: MetaModel
> Affects Versions: 0.1.2-incubating
> Reporter: Dan Haywood
> Priority: Minor
>
> Looking at the code, it seems that we never create HelpFacets through FacetFactory. So neither the HelpFacetDecorator nor the I18NFacetDecorator looks like they will ever install help, because both require there to be an underlying HelpFacet to decorate.
> Moreover, there is a duplication between the I18 facet decorator and the help decorator, in that they both install help facets.
> I think a good first step would be to make the HelpFacetDecorator into a regular HelpFacetFactory. At least then the role of i18nFacetDecorator will be well-defined.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira