[ 
https://issues.apache.org/jira/browse/OFBIZ-2201?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12677103#action_12677103
 ] 

Bruno Busco commented on OFBIZ-2201:
------------------------------------

Sorry for not having replied soon to this.
I have tried to find a solution but not found a no-drawback one.

I think the solution to have the "ECOMMERCE" VisualThemeSet definition in the 
framework is the best one even if I do not like the idea of the esistence of an 
"ECOMMERCE" VisualThemeSet in a framework-only installation.

Unfortunately this is a must if we want the framework only installation have 
the whole themes folder deployed and only the application and specialpurpose 
folder removed.

If no objections I will go for this solution and submit a patch.

> Theme component should be loaded after applications component
> -------------------------------------------------------------
>
>                 Key: OFBIZ-2201
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-2201
>             Project: OFBiz
>          Issue Type: Improvement
>            Reporter: Bruno Busco
>            Assignee: Bruno Busco
>             Fix For: SVN trunk
>
>         Attachments: themeload.patch
>
>
> In the ecommerce application a specific (to the ecommerce application) 
> VisualThemeSet is defined:
> In EcommerceTypeData.xml:
>    <VisualThemeSet visualThemeSetId="ECOMMERCE" description="ECommerce Visual 
> Themes"/>
> The multiflex theme is defined as:
>    <VisualTheme visualThemeId="MULTIFLEX" visualThemeSetId="ECOMMERCE" 
> description="Alternative VisualTheme for Ecommerce"/>
> When doing "ant run-install" for the first time the 
> visualThemeSetId="ECOMMERCE" is not found when the VisualTheme entity gets 
> created.
> This is the reason I think themes should be loaded AFTER the applications 
> because, actually, they apply ON applications.

-- 
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