Deryk Sinotte added a comment - 07/Nov/12 7:06 PM - edited Assignee: Mircea Toma (was: Ken Fyten) Re-opened and assigned to Mircea.

Deryk Sinotte added a comment - 06/Nov/12 11:56 PM - edited Attaching modified Mojarra 2.1.3 with both patches attached (jsf.js and UIComponentBase)

Mircea Toma added a comment - 05/Nov/12 11:39 PM - edited Attached the patched Mojarra 2.1.3 patch. The patch is based on the latest patch submitted under http://java.net/jira/browse/JAVASERVERFACES-2461 case.

Mircea Toma added a comment - 05/Nov/12 11:24 PM - edited Before getting into the issue we have with the patched Mojarra it should be noted that the test case needs to be modified first. The TestBean.clear method invoked when the "Clear" button is clicked doesn't really clear the bound values rendered by the input text components present in the page. The test page should be modified to use the TestBean.altClear method instead.

Deryk Sinotte added a comment - 02/Nov/12 11:19 PM - edited As near as I can figure at the moment, the problem does seem to exist in the patched version of jsf.js that we have in 2.1.3. In there is a clone attributes function that we modified. In that function, we altered how attributes are copied. Here is the section that deals with input style elements specifically: var isInputElement = target.nodeName.toLowerCase() === 'input'; var propertyNames = isInputElement ? coreElementProperties.concat(inputElementProperties) : coreElementProperties; for ( var iIndex = 0, iLength = propertyNames.length; iIndex < iLength; iIndex++) { var propertyName = propertyNames[iIndex]; var attributeName = propertyToAttribute(propertyName); if (sourceAttributeDetector(attributeName)) { var newValue = source[propertyName]; var oldValue = target[propertyName]; if (oldValue != newValue) { target[propertyName] = newValue; } } else if (targetAttributeDetector(attributeName)) { //setting property to '' seems to be the only cross-browser method for removing an attribute target[propertyName] = ''; } } From my testing, I don't see either the sourceAttributeDetector() or targetAttributeDetector() conditions being satisfied which would mean the attribute never gets updated. This aligns with the fact that the response coming back from the server is correct in that the value has been cleared. Since this code is similar to the code we passed to the Mojarra team for inclusion in JSF for 2.1.15, we should make sure we don't have the same problem there.

Arran Mccullough added a comment - 02/Nov/12 4:39 PM - edited Attached test case to reproduce issue.

Arran Mccullough added a comment - 02/Nov/12 4:37 PM - edited Another observation, if I set the value bound objects to null directly instead of the clear code, it still doesn't work with the patched jar. It does however work with the non-patched jar.