First step is just to investigate the problem. Note that this application serves as a test case for users porting existing ICEfaces 1.8 applications to ICEfaces 2.0, so we should prefer fixes in icefaces-compat to application changes.

Add /icefaces/* servlet mapping for FacesServlet.

Replace vanilla 'head' and 'body' tags with their corresponding h:* components. They are needed for resource injection.

Reference jsf.js resource.

Move beans in ICEfaces non-standard request scope into session scope. Thus same bean instances will be used between Ajax requests.

Serialize also the element that triggered the submit so that ice:* components can identify themselves.

Patch Mojarra JSF-RI Javascript code so that table elements like TD and TH are updated properly. See table-element-update.patch file.

Clicking repeatedly on "Bid" button and then "x" button makes the application to run out of memory pretty fast (about 10 clicks).

Table update patch reported to javaserverfaces issue tracker:

https://javaserverfaces.dev.java.net/issues/show_bug.cgi?id=1187

The main problem with the appearance of the application involves the use of the <center> tag, but is not due to the <center> tag per se.

When the panelGrid is used:

<ice:panelGrid columns="5" >
<center>
<ice:commandLink action="#

" />
</center></ice:panelGroup>
...

Move session bound beans in 'view' scope.

Improved patch that works properly for all table child elements.

Which browsers? Let's refine the patch a bit before we send it to Sun again – they've already applied the previous one. For instance, perhaps we can factor out parent.replaceChild().

Why trim spaces? Perhaps that should be done on the server (not necessarily a choice for us here, since this is in the reference implementation.)

compat/auctionMonitor starts working for me if I make a change like the following:

Index: compat/components/src/main/resources/META-INF/resources/icesubmit.js
===================================================================
— compat/components/src/main/resources/META-INF/resources/icesubmit.js (revision 19044)
+++ compat/components/src/main/resources/META-INF/resources/icesubmit.js (working copy)
@@ -14,7 +14,7 @@
return false;
}
}

• ice.submitForm(evt, form || formOf(component));
  + ice.submitEvent(evt, component, form || formOf(component));
  return false;
  }

Mircea, perhaps you have a local change of this nature that is not checked in? commandButton does not work if it is not the "component" submitting the form (the current code submits the form anonymously).

You're right I missed to commit icesubmit.js changes. The fix is committed now.

> Which browsers?

Firefox, for sure.

> Let's refine the patch a bit before we send it to Sun again – they've already applied the
> previous one. For instance, perhaps we can factor out parent.replaceChild().

The patch is almost similar with what we have in Trunk. With the first patch I completely forgot about the fact that browser are inserting intermediate elements. When I went back to Trunk code I remembered why I had to do the "while" loop.

OutOfMemoryError is gone since the state saving was fixed. See https://javaserverfaces.dev.java.net/issues/show_bug.cgi?id=1142 .

Fix table element update in IE. IE creates the "table" element only if "table" tag is part of the innerHTML markup. Also reverted the search in the array of table element names, native arrays are not indexed by cell values. See patch file: table-element-update_ie-fix.patch