[ofbiz-framework] branch trunk updated: Fixed: Propagate the theme in DataResourceWorker.renderDataResourceAsText() Improved: no functional change

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[ofbiz-framework] branch trunk updated: Fixed: Propagate the theme in DataResourceWorker.renderDataResourceAsText() Improved: no functional change

jleroux@apache.org
This is an automated email from the ASF dual-hosted git repository.

jleroux pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/ofbiz-framework.git


The following commit(s) were added to refs/heads/trunk by this push:
     new 2517a96  Fixed: Propagate the theme in DataResourceWorker.renderDataResourceAsText() Improved: no functional change
2517a96 is described below

commit 2517a9679218151cd45e5ea328d61957b8320434
Author: Jacques Le Roux <[hidden email]>
AuthorDate: Fri Mar 20 09:35:58 2020 +0100

    Fixed: Propagate the theme in DataResourceWorker.renderDataResourceAsText()
    Improved: no functional change
   
    (OFBIZ-9923)
   
    Seems that this has been fixed with OFBIZ-11466 "CommonTheme has a dependency on
    Flatgrey application.js"
   
    Thanks: Pierre Smits for the fix in OFBIZ-11466
---
 .../src/main/java/org/apache/ofbiz/content/data/DataResourceWorker.java | 2 --
 1 file changed, 2 deletions(-)

diff --git a/applications/content/src/main/java/org/apache/ofbiz/content/data/DataResourceWorker.java b/applications/content/src/main/java/org/apache/ofbiz/content/data/DataResourceWorker.java
index f87936a..b8f95c9 100644
--- a/applications/content/src/main/java/org/apache/ofbiz/content/data/DataResourceWorker.java
+++ b/applications/content/src/main/java/org/apache/ofbiz/content/data/DataResourceWorker.java
@@ -659,7 +659,6 @@ public class DataResourceWorker  implements org.apache.ofbiz.widget.content.Data
             locale = Locale.getDefault();
         }
 
-        //FIXME correctly propagate the theme, then fixes also the related FIXME below
         VisualTheme visualTheme = ThemeFactory.getVisualThemeFromId("COMMON");
         ModelTheme modelTheme = visualTheme.getModelTheme();
 
@@ -726,7 +725,6 @@ public class DataResourceWorker  implements org.apache.ofbiz.widget.content.Data
 
             } else if ("XSLT".equals(dataTemplateTypeId)) {
                 File targetFileLocation = new File(System.getProperty("ofbiz.home")+"/runtime/tempfiles/docbook.css");
-                // This is related with the other FIXME above: we need to correctly propagate the theme.
                 String defaultVisualThemeId = EntityUtilProperties.getPropertyValue("general", "VISUAL_THEME", delegator);
                 visualTheme = ThemeFactory.getVisualThemeFromId(defaultVisualThemeId);
                 modelTheme = visualTheme.getModelTheme();