Merge branch '2.3.x' into 2.4.x

Author: lhotari

grails-databinding/src/test/groovy/org/grails/databinding/converters/CurrencyConversionSpec.groovy

@@ -18,7 +18,7 @@ class CurrencyConversionSpec extends Specification {
 
         then:
         bank.currency instanceof Currency
-        '$' == bank.currency.symbol
+        'USD' == bank.currency.currencyCode
     }
 }
 

grails-encoder/src/main/groovy/org/codehaus/groovy/grails/web/util/GrailsPrintWriter.java

@@ -94,14 +94,40 @@ protected Writer unwrapWriter(Writer writer) {
      * Provides Groovy << left shift operator, but intercepts call to make sure
      * nulls are converted to "" strings
      *
-     * @param value The value
+     * @param obj The value
      * @return Returns this object
      * @throws IOException
      */
-    public GrailsPrintWriter leftShift(Object value) throws IOException {
-        usageFlag = true;
-        if (value != null) {
-            InvokerHelper.write(this, value);
+    public GrailsPrintWriter leftShift(Object obj) throws IOException {
+        if (trouble || obj == null) {
+            usageFlag = true;
+            return this;
+        }
+
+        Class<?> clazz = obj.getClass();
+        if (clazz == String.class) {
+            write((String)obj);
+        }
+        else if (clazz == StreamCharBuffer.class) {
+            write((StreamCharBuffer)obj);
+        }
+        else if (clazz == GStringImpl.class) {
+            write((Writable)obj);
+        }
+        else if (obj instanceof Writable) {
+            write((Writable)obj);
+        }
+        else if (obj instanceof CharSequence) {
+            try {
+                usageFlag = true;
+                getOut().append((CharSequence)obj);
+            }
+            catch (IOException e) {
+                handleIOException(e);
+            }
+        }
+        else {        
+            InvokerHelper.write(this, obj);
         }
         return this;
     }
@@ -517,6 +543,11 @@ public void write(final Writable writable) {
     }
 
     protected void writeWritable(final Writable writable) {
+        if(writable.getClass() == StreamCharBuffer.class) {
+            write((StreamCharBuffer)writable);
+            return;
+        }
+        
         usageFlag = true;
         if (trouble)
             return;

grails-test-suite-persistence/src/test/groovy/org/codehaus/groovy/grails/orm/GrailsWebDataBinderSpec.groovy

@@ -1331,7 +1331,7 @@ class GrailsWebDataBinderSpec extends Specification {
 
         then:
         publisher.localCurrency instanceof Currency
-        '$' == publisher.localCurrency.symbol
+        'USD' == publisher.localCurrency.currencyCode
     }
 }
 

grails-web-gsp/src/main/groovy/org/codehaus/groovy/grails/web/pages/GroovyPageOutputStack.java

@@ -51,10 +51,11 @@ public static GroovyPageOutputStack currentStack(boolean allowCreate) {
     }
 
     public static GroovyPageOutputStack currentStack(GrailsWebRequest request, boolean allowCreate) {
-        if (allowCreate) {
-            return currentStack(request, allowCreate, null, allowCreate, false);
+        GroovyPageOutputStack outputStack = lookupStack(request);
+        if (outputStack == null && allowCreate) {
+            outputStack = currentStack(request, allowCreate, null, allowCreate, false);
         }
-        return lookupStack(request);
+        return outputStack;
     }
 
     public static GroovyPageOutputStack currentStack(boolean allowCreate, Writer topWriter, boolean autoSync, boolean pushTop) {
@@ -75,16 +76,16 @@ public static GroovyPageOutputStack currentStack(GroovyPageOutputStackAttributes
         }
 
         if (attributes.isAllowCreate()) {
-            if (attributes.getTopWriter() == null) {
-                attributes=new GroovyPageOutputStackAttributes.Builder(attributes).topWriter(lookupRequestWriter(attributes.getWebRequest())).build();
-            }
             return createNew(attributes);
         }
 
         return null;
     }
 
     private static final GroovyPageOutputStack createNew(GroovyPageOutputStackAttributes attributes) {
+        if (attributes.getTopWriter() == null) {
+            attributes=new GroovyPageOutputStackAttributes.Builder(attributes).topWriter(lookupRequestWriter(attributes.getWebRequest())).build();
+        }
         GroovyPageOutputStack instance = new GroovyPageOutputStack(attributes);
         attributes.getWebRequest().setAttribute(
                 ATTRIBUTE_NAME_OUTPUT_STACK, instance, RequestAttributes.SCOPE_REQUEST);