refactor: use the config setting to disable preprocessing instead of having a tag that converts from preprocessed

Author: jdaugherty

grails-gsp/grails-sitemesh3/src/main/groovy/org/grails/plugins/sitemesh3/Sitemesh3GrailsPlugin.groovy

@@ -19,10 +19,11 @@
 
 package org.grails.plugins.sitemesh3
 
+import grails.core.DefaultGrailsApplication
 import grails.plugins.Plugin
 import org.grails.config.PropertySourcesConfig
+import org.grails.gsp.compiler.GroovyPageParser
 import org.grails.plugins.web.taglib.RenderSitemeshTagLib
-import org.grails.plugins.web.taglib.SitemeshTagLib
 import org.grails.web.config.http.GrailsFilters
 import org.grails.web.util.WebUtils
 import org.springframework.core.env.ConfigurableEnvironment
@@ -47,7 +48,6 @@ class Sitemesh3GrailsPlugin extends Plugin {
 
     def providedArtefacts = [
         RenderSitemeshTagLib,
-        SitemeshTagLib,
     ]
 
     static PropertySource getDefaultPropertySource(ConfigurableEnvironment configurableEnvironment, String defaultLayout) {
@@ -69,19 +69,23 @@ class Sitemesh3GrailsPlugin extends Plugin {
                 props.remove(it.key)
             }
         }
-        return new MapPropertySource("sitemesh3Properties", props)
+        return new MapPropertySource("defaultSitemesh3Properties", props)
     }
 
 
     Closure doWithSpring() {
         { ->
-            ConfigurableEnvironment configurableEnvironment = application.mainContext.environment
+            ConfigurableEnvironment configurableEnvironment = grailsApplication.mainContext.environment as ConfigurableEnvironment
             def propertySources = configurableEnvironment.getPropertySources()
             // https://gsp.grails.org/latest/guide/layouts.html
             // Default view should be application, but it is inefficient to add a rule for a page that may not exist.
             String defaultLayout = grailsApplication.getConfig().getProperty("grails.sitemesh.default.layout")
             propertySources.addFirst(getDefaultPropertySource(configurableEnvironment, defaultLayout))
-            application.config = new PropertySourcesConfig(propertySources)
+            propertySources.addFirst(new MapPropertySource('requiredSitemesh3Properties', [
+                    (GroovyPageParser.CONFIG_PROPERTY_GSP_GRAILS_LAYOUT_PREPROCESS): 'false'
+            ]))
+            (grailsApplication as DefaultGrailsApplication).config = new PropertySourcesConfig(propertySources)
+
             grailsLayoutHandlerMapping(GrailsLayoutHandlerMapping)
         }
     }

grails-gsp/grails-sitemesh3/src/main/groovy/org/grails/plugins/web/taglib/SitemeshTagLib.groovy

@@ -68,7 +68,6 @@
 import org.springframework.web.context.WebApplicationContext;
 import org.springframework.web.servlet.ViewResolver;
 
-import org.grails.plugins.web.taglib.SitemeshTagLib;
 import org.grails.plugins.web.taglib.RenderTagLib;
 
 import jakarta.servlet.ServletContext;
@@ -241,7 +240,7 @@ public GrailsApplication grailsApplication() {
 
     protected static class TagLibraryLookupRegistrar implements ImportBeanDefinitionRegistrar {
 
-        public static final Class<?>[] DEFAULT_TAGLIB_CLASSES=new Class<?>[] { SitemeshTagLib.class, RenderTagLib.class, RenderSitemeshTagLib.class };
+        public static final Class<?>[] DEFAULT_TAGLIB_CLASSES=new Class<?>[] { RenderTagLib.class, RenderSitemeshTagLib.class };
 
         @Override
         public void registerBeanDefinitions(AnnotationMetadata importingClassMetadata, BeanDefinitionRegistry registry) {

grails-gsp/spring-boot/src/main/java/grails/gsp/boot/GspAutoConfiguration.java

@@ -58,7 +58,6 @@ public LazyTagLibraryLookup() {
         }
         if(ClassUtils.isPresent("org.grails.plugins.sitemesh3.Sitemesh3GrailsPlugin", classLoader)) {
             try {
-                mockedClasses.add(Class.forName("org.grails.plugins.web.taglib.SitemeshTagLib"));
                 mockedClasses.add(Class.forName("org.grails.plugins.web.taglib.RenderSitemeshTagLib"));
             }
             catch(Exception ignored) {