Author: ivaynberg
Date: Tue May 27 10:25:30 2008
New Revision: 660618

URL: http://svn.apache.org/viewvc?rev=660618&view=rev
Log:
WICKET-1622

Modified:
    
wicket/branches/wicket-1.3.x/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/repeater/RefreshingView.java

Modified: 
wicket/branches/wicket-1.3.x/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/repeater/RefreshingView.java
URL: 
http://svn.apache.org/viewvc/wicket/branches/wicket-1.3.x/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/repeater/RefreshingView.java?rev=660618&r1=660617&r2=660618&view=diff
==============================================================================
--- 
wicket/branches/wicket-1.3.x/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/repeater/RefreshingView.java
 (original)
+++ 
wicket/branches/wicket-1.3.x/jdk-1.4/wicket/src/main/java/org/apache/wicket/markup/repeater/RefreshingView.java
 Tue May 27 10:25:30 2008
@@ -87,10 +87,20 @@
         */
        protected final void onPopulate()
        {
-
-               IItemFactory itemFactory = new IItemFactory()
+               Iterator models = getItemModels();
+               Iterator items = 
getItemReuseStrategy().getItems(newItemFactory(), models, getItems());
+               removeAll();
+               addItems(items);
+       }
+       
+       /**
+        * Create a new IItemFactory based upon the RefreshingView
+        * 
+        * @return An Item factory that delegates to the RefreshingView
+        */
+       protected IItemFactory newItemFactory() {
+               return new IItemFactory()
                {
-
                        public Item newItem(int index, IModel model)
                        {
                                String id = RefreshingView.this.newChildId();
@@ -98,13 +108,7 @@
                                RefreshingView.this.populateItem(item);
                                return item;
                        }
-
                };
-
-               Iterator models = getItemModels();
-               Iterator items = getItemReuseStrategy().getItems(itemFactory, 
models, getItems());
-               removeAll();
-               addItems(items);
        }
 
        /**


Reply via email to