[richfaces-svn-commits] JBoss Rich Faces SVN: r5131 - in trunk/ui/listShuttle/src/main: java/org/richfaces/renderkit and 1 other directories.

[richfaces-svn-commits at lists.jboss.org]

Author: nbelaevski
Date: 2008-01-03 15:45:41 -0500 (Thu, 03 Jan 2008)
New Revision: 5131

Modified:
   trunk/ui/listShuttle/src/main/java/org/richfaces/component/UIListShuttle.java
   trunk/ui/listShuttle/src/main/java/org/richfaces/renderkit/ListShuttleRendererBase.java
   trunk/ui/listShuttle/src/main/templates/org/richfaces/htmlListShuttle.jspx
Log:
http://jira.jboss.com/jira/browse/RF-1819

Modified: trunk/ui/listShuttle/src/main/java/org/richfaces/component/UIListShuttle.java
===================================================================
--- trunk/ui/listShuttle/src/main/java/org/richfaces/component/UIListShuttle.java	2008-01-03 17:16:29 UTC (rev 5130)
+++ trunk/ui/listShuttle/src/main/java/org/richfaces/component/UIListShuttle.java	2008-01-03 20:45:41 UTC (rev 5131)
@@ -8,13 +8,11 @@
 import java.io.Serializable;
 import java.util.ArrayList;
 import java.util.Collection;
-import java.util.HashSet;
 import java.util.Iterator;
 import java.util.Map;
 import java.util.Set;
 import java.util.Map.Entry;
 
-import javax.faces.FacesException;
 import javax.faces.application.FacesMessage;
 import javax.faces.component.UIInput;
 import javax.faces.context.FacesContext;
@@ -563,6 +561,10 @@
 			final ArrayList targetList = new ArrayList();
 
 			try {
+				
+				Object key = getRowKey();
+				captureOrigValue(context);
+				
 				walk(context, new DataVisitor() {
 
 					public void process(FacesContext context, Object rowKey,
@@ -578,6 +580,10 @@
 						}
 					}
 				}, null);
+
+				setRowKey(key);
+				restoreOrigValue(context);
+
 			} catch (IOException e) {
 				throw new ConverterException(e.getLocalizedMessage(), e);
 			}

Modified: trunk/ui/listShuttle/src/main/java/org/richfaces/renderkit/ListShuttleRendererBase.java
===================================================================
--- trunk/ui/listShuttle/src/main/java/org/richfaces/renderkit/ListShuttleRendererBase.java	2008-01-03 17:16:29 UTC (rev 5130)
+++ trunk/ui/listShuttle/src/main/java/org/richfaces/renderkit/ListShuttleRendererBase.java	2008-01-03 20:45:41 UTC (rev 5131)
@@ -5,7 +5,6 @@
 
 import java.io.IOException;
 import java.io.StringWriter;
-import java.util.HashMap;
 import java.util.HashSet;
 import java.util.Iterator;
 import java.util.LinkedHashMap;
@@ -115,7 +114,7 @@
 			if (source == listShuttleRowKey.isFacadeSource()) {
 				
 				ResponseWriter writer = context.getResponseWriter();
-				String clientId = holder.getTable().getClientId(context);
+				String clientId = table.getClientId(context);
 				writer.startElement(HTML.TR_ELEMENT, table);
 				writer.writeAttribute("id",  clientId, null);
 

Modified: trunk/ui/listShuttle/src/main/templates/org/richfaces/htmlListShuttle.jspx
===================================================================
--- trunk/ui/listShuttle/src/main/templates/org/richfaces/htmlListShuttle.jspx	2008-01-03 17:16:29 UTC (rev 5130)
+++ trunk/ui/listShuttle/src/main/templates/org/richfaces/htmlListShuttle.jspx	2008-01-03 20:45:41 UTC (rev 5131)
@@ -28,6 +28,7 @@
 	</h:scripts>
 	
 	<f:clientId var="clientId"/>
+	
 	<table id="#{clientId}" class="rich-list-shuttle #{component.attributes['styleClass']}"
 			cellspacing="0" cellpadding="0"
 			x:passThruWithExclusions="id, class, styleClass">
@@ -47,6 +48,8 @@
 						
 						Boolean switchByClick = (Boolean) component.getAttributes().get("switchByClick");
 						variables.setVariable("switchByClick", switchByClick);
+					
+						variables.setVariable("baseClientId", component.getBaseClientId(context));
 					]]>
 				</jsp:scriptlet>
 
@@ -121,7 +124,7 @@
 					</td>
 					<td>
 						<div class="rich-shuttle-target-items #{component.listClass}" >
-					    	<input type="hidden" value=":" style="display: none;" name="#{clientId}" />
+							<input type="hidden" value=":" style="display: none;" name="#{baseClientId}" />
 	
 					    	<input id="#{clientId}tlFocusKeeper" type="button" value="" style="width: 1px; position: absolute; left: -32767px;"/>