diff --git a/portal/portal-tool/tool/pom.xml b/portal/portal-tool/tool/pom.xml index 0440daced264..15eeca26ba65 100644 --- a/portal/portal-tool/tool/pom.xml +++ b/portal/portal-tool/tool/pom.xml @@ -40,6 +40,10 @@ javax.servlet servlet-api + + org.apache.commons + commons-lang3 + diff --git a/portal/portal-tool/tool/src/java/org/sakaiproject/portal/tool/ToolPortal.java b/portal/portal-tool/tool/src/java/org/sakaiproject/portal/tool/ToolPortal.java index e6054241c098..927daf53989e 100644 --- a/portal/portal-tool/tool/src/java/org/sakaiproject/portal/tool/ToolPortal.java +++ b/portal/portal-tool/tool/src/java/org/sakaiproject/portal/tool/ToolPortal.java @@ -33,6 +33,9 @@ import org.slf4j.Logger; import org.slf4j.LoggerFactory; + +import org.apache.commons.lang3.StringUtils; + import org.sakaiproject.component.cover.ServerConfigurationService; import org.sakaiproject.exception.IdUnusedException; import org.sakaiproject.exception.PermissionException; @@ -308,21 +311,15 @@ protected void setupForward(HttpServletRequest req, HttpServletResponse res, if (site != null) { - String strMathJaxEnabled = site.getProperties().getProperty(MATHJAX_ENABLED); - if (strMathJaxEnabled != null) - { - String [] strMathJaxTools = strMathJaxEnabled.split(","); - List mathJaxTools = Arrays.asList(strMathJaxTools); - if (mathJaxTools != null) + String strMathJaxEnabledForSite = site.getProperties().getProperty(MATHJAX_ENABLED); + if (!StringUtils.isBlank(strMathJaxEnabledForSite)) + { + if (Boolean.valueOf(strMathJaxEnabledForSite)) { - String toolId = toolConfig.getTool().getId(); - if (toolId != null && mathJaxTools.contains(toolId)) - { - // this call to MathJax.Hub.Config seems to be needed for MathJax to work in IE - headJs += "\n"; - headJs += "\n"; - } - } + // this call to MathJax.Hub.Config seems to be needed for MathJax to work in IE + headJs += "\n"; + headJs += "\n"; + } } } }