You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@myfaces.apache.org by bo...@apache.org on 2007/12/10 11:49:11 UTC

svn commit: r602857 - /myfaces/tobago/trunk/theme/scarborough/src/main/java/org/apache/myfaces/tobago/renderkit/html/scarborough/standard/tag/PageRenderer.java

Author: bommel
Date: Mon Dec 10 02:49:03 2007
New Revision: 602857

URL: http://svn.apache.org/viewvc?rev=602857&view=rev
Log:
(TOBAGO-562) List of Popup in UIPage should be a Set

Modified:
    myfaces/tobago/trunk/theme/scarborough/src/main/java/org/apache/myfaces/tobago/renderkit/html/scarborough/standard/tag/PageRenderer.java

Modified: myfaces/tobago/trunk/theme/scarborough/src/main/java/org/apache/myfaces/tobago/renderkit/html/scarborough/standard/tag/PageRenderer.java
URL: http://svn.apache.org/viewvc/myfaces/tobago/trunk/theme/scarborough/src/main/java/org/apache/myfaces/tobago/renderkit/html/scarborough/standard/tag/PageRenderer.java?rev=602857&r1=602856&r2=602857&view=diff
==============================================================================
--- myfaces/tobago/trunk/theme/scarborough/src/main/java/org/apache/myfaces/tobago/renderkit/html/scarborough/standard/tag/PageRenderer.java (original)
+++ myfaces/tobago/trunk/theme/scarborough/src/main/java/org/apache/myfaces/tobago/renderkit/html/scarborough/standard/tag/PageRenderer.java Mon Dec 10 02:49:03 2007
@@ -138,7 +138,10 @@
 
     // write popup components
     // beware of ConcurrentModificationException in cascating popups!
-    for (UIPopup popup: page.getPopups()) {
+    // no foreach
+    UIPopup[] popupArray = (UIPopup[]) page.getPopups().toArray();
+    for (int i = 0; i < popupArray.length; i++) {
+      UIComponent popup = popupArray[i];
       RenderUtil.encode(facesContext, popup);
     }