Author: ivaynberg
Date: Fri Dec  4 06:16:38 2009
New Revision: 887081

URL: http://svn.apache.org/viewvc?rev=887081&view=rev
Log:
WICKET-2953 wicket:message can cause infinite loop in render
Issue: WICKET-2953

Modified:
    
wicket/branches/wicket-1.4.x/wicket/src/main/java/org/apache/wicket/markup/resolver/WicketMessageResolver.java
    
wicket/branches/wicket-1.4.x/wicket/src/test/java/org/apache/wicket/markup/resolver/SimplePageExpectedResult_1.html
    
wicket/branches/wicket-1.4.x/wicket/src/test/java/org/apache/wicket/markup/resolver/SimplePage_1.html

Modified: 
wicket/branches/wicket-1.4.x/wicket/src/main/java/org/apache/wicket/markup/resolver/WicketMessageResolver.java
URL: 
http://svn.apache.org/viewvc/wicket/branches/wicket-1.4.x/wicket/src/main/java/org/apache/wicket/markup/resolver/WicketMessageResolver.java?rev=887081&r1=887080&r2=887081&view=diff
==============================================================================
--- 
wicket/branches/wicket-1.4.x/wicket/src/main/java/org/apache/wicket/markup/resolver/WicketMessageResolver.java
 (original)
+++ 
wicket/branches/wicket-1.4.x/wicket/src/main/java/org/apache/wicket/markup/resolver/WicketMessageResolver.java
 Fri Dec  4 06:16:38 2009
@@ -364,8 +364,13 @@
                                                        if (component != null)
                                                        {
                                                                
component.render(markupStream);
+                                                               
childTags.put(id, response.getBuffer());
                                                        }
-                                                       childTags.put(id, 
response.getBuffer());
+                                                       else
+                                                       {
+                                                               
markupStream.next();
+                                                       }
+
                                                }
                                                finally
                                                {

Modified: 
wicket/branches/wicket-1.4.x/wicket/src/test/java/org/apache/wicket/markup/resolver/SimplePageExpectedResult_1.html
URL: 
http://svn.apache.org/viewvc/wicket/branches/wicket-1.4.x/wicket/src/test/java/org/apache/wicket/markup/resolver/SimplePageExpectedResult_1.html?rev=887081&r1=887080&r2=887081&view=diff
==============================================================================
--- 
wicket/branches/wicket-1.4.x/wicket/src/test/java/org/apache/wicket/markup/resolver/SimplePageExpectedResult_1.html
 (original)
+++ 
wicket/branches/wicket-1.4.x/wicket/src/test/java/org/apache/wicket/markup/resolver/SimplePageExpectedResult_1.html
 Fri Dec  4 06:16:38 2009
@@ -4,6 +4,10 @@
   <wicket:message key="myKey">myValue</wicket:message>
   <wicket:message key="myKeyNotExsts"></wicket:message>
   <wicket:message key="myKey">myValue</wicket:message>
+
+  <!-- wicket message with some raw markup -->
+  <wicket:message key="myKey">myValue</wicket:message>
+
   <span wicket:id="message">here it goes</span>
   <span wicket:message="attr-name:i18n-key" attr-name="my i18n key">test 
2</span>
   <span wicket:id="message2" wicket:message="attr-name:i18n-key" attr-name="my 
i18n key">here it goes</span>

Modified: 
wicket/branches/wicket-1.4.x/wicket/src/test/java/org/apache/wicket/markup/resolver/SimplePage_1.html
URL: 
http://svn.apache.org/viewvc/wicket/branches/wicket-1.4.x/wicket/src/test/java/org/apache/wicket/markup/resolver/SimplePage_1.html?rev=887081&r1=887080&r2=887081&view=diff
==============================================================================
--- 
wicket/branches/wicket-1.4.x/wicket/src/test/java/org/apache/wicket/markup/resolver/SimplePage_1.html
 (original)
+++ 
wicket/branches/wicket-1.4.x/wicket/src/test/java/org/apache/wicket/markup/resolver/SimplePage_1.html
 Fri Dec  4 06:16:38 2009
@@ -4,6 +4,10 @@
   <wicket:message key="myKey">Default Text</wicket:message>
   <wicket:message key="myKeyNotExsts"/>
   <wicket:message key="myKey"/>
+
+  <!-- wicket message with some raw markup -->
+  <wicket:message key="myKey">blah <a href="blah">blah</a> 
blah</wicket:message>
+
   <span wicket:id="message">test</span>
   <span wicket:message="attr-name:i18n-key">test 2</span>
   <span wicket:id="message2" wicket:message="attr-name:i18n-key">test 2</span>


Reply via email to