Author: abelevich Date: 2008-04-11 07:46:07 -0400 (Fri, 11 Apr 2008) New Revision: 7760 Modified: trunk/ui/inplaceInput/src/main/java/org/richfaces/component/UIInplaceInput.java trunk/ui/inplaceInput/src/main/java/org/richfaces/renderkit/InplaceInputBaseRenderer.java trunk/ui/inplaceInput/src/main/templates/inplaceinput.jspx Log: apply review TODO Modified: trunk/ui/inplaceInput/src/main/java/org/richfaces/component/UIInplaceInput.java =================================================================== --- trunk/ui/inplaceInput/src/main/java/org/richfaces/component/UIInplaceInput.java 2008-04-11 10:52:33 UTC (rev 7759) +++ trunk/ui/inplaceInput/src/main/java/org/richfaces/component/UIInplaceInput.java 2008-04-11 11:46:07 UTC (rev 7760) @@ -20,7 +20,7 @@ public static final String COMPONENT_TYPE = "org.richfaces.InplaceInput"; final static public String COMPONENT_FAMILY = "org.richfaces.InplaceInput"; - public static final String VALUE_SUFFIX = "value"; +// public static final String VALUE_SUFFIX = "value"; public abstract String getDefaultLabel(); public abstract void setDefaultLabel(String label); } \ No newline at end of file Modified: trunk/ui/inplaceInput/src/main/java/org/richfaces/renderkit/InplaceInputBaseRenderer.java =================================================================== --- trunk/ui/inplaceInput/src/main/java/org/richfaces/renderkit/InplaceInputBaseRenderer.java 2008-04-11 10:52:33 UTC (rev 7759) +++ trunk/ui/inplaceInput/src/main/java/org/richfaces/renderkit/InplaceInputBaseRenderer.java 2008-04-11 11:46:07 UTC (rev 7760) @@ -125,7 +125,6 @@ //TODO: //Should use clientId instead? - clientId = clientId + UIInplaceInput.VALUE_SUFFIX; Map <String, String> request = context.getExternalContext().getRequestParameterMap(); String newValue = (String)request.get(clientId); Modified: trunk/ui/inplaceInput/src/main/templates/inplaceinput.jspx =================================================================== --- trunk/ui/inplaceInput/src/main/templates/inplaceinput.jspx 2008-04-11 10:52:33 UTC (rev 7759) +++ trunk/ui/inplaceInput/src/main/templates/inplaceinput.jspx 2008-04-11 11:46:07 UTC (rev 7760) @@ -108,7 +108,7 @@ onmouseup='#{component.attributes["oninputmouseup"]}' /> - <input id='#{clientId}value' name='#{clientId}value' type='hidden' value='#{fieldValue}'/> + <input id='#{clientId}value' name='#{clientId}' type='hidden' value='#{fieldValue}'/> <div id="#{clientId}bar" class="rich-inplace-input-controls-set" style="display:none;"> <jsp:scriptlet> <![CDATA[