*From: *gmur...@dev.java.net <mailto:gmur...@dev.java.net> *Date: *April 10, 2007 10:39:23 PM PDT *To: *cv...@ajax.dev.java.net <mailto:cv...@ajax.dev.java.net> *Subject: **CVS update: /ajax/ws/jmaki/shared/src/java/jmaki/runtime/jsf/AjaxWrapperRenderer.java*

*Reply-To: *cv...@ajax.dev.java.net <mailto:cv...@ajax.dev.java.net>

User: gmurray71 Date: 2007-04-11 05:39:23+0000 Log: Using a custom uuid genrerator that will create unique ids that are unique application wide. The default ids genrated by JSF are unique to a page and not to across pages meaning that if we inject content from one page into another the same widget can be given the same id and breaking the widgets. This was seen with the tabbed view that use the injector under the covers.

File Changes:

Directory: /ajax/ws/jmaki/shared/src/java/jmaki/runtime/jsf/ ============================================================

File [changed]: AjaxWrapperRenderer.java Url: https://ajax.dev.java.net/source/browse/ajax/ws/jmaki/shared/src/java/jmaki/runtime/jsf/AjaxWrapperRenderer.java?r1=1.2&r2=1.3 <https://ajax.dev.java.net/source/browse/ajax/ws/jmaki/shared/src/java/jmaki/runtime/jsf/AjaxWrapperRenderer.java?r1=1.2&r2=1.3>

Delta lines: +11 -2

-------------------- --- AjaxWrapperRenderer.java 2007-04-10 06:25:23+0000 1.2 +++ AjaxWrapperRenderer.java 2007-04-11 05:39:21+0000 1.3 @@ -51,7 +51,7 @@

private static Logger logger;

-// private static int nextId = 0; + private static int nextId = 0; // /** Creates a new instance of AjaxDynamicLabelRenderer */ public AjaxWrapperRenderer() { @@ -82,6 +82,10 @@

public void encodeEnd(FacesContext context, UIComponent component) throws IOException { Map headers = context.getExternalContext().getRequestHeaderMap(); + //gmuray71 : Rest the uuid after 50000 + if (nextId > 50000) { + nextId = 0; + } String renderHeader = (String) headers.get("com.sun.faces.avatar.partial");

// If this is an AJAX request, just encode our value @@ -104,7 +108,12 @@ AjaxCommonRenderer ajaxCommon = new AjaxCommonRenderer(ajx);

ajaxCommon.setName(wrapper.getName()); - ajaxCommon.setUuid(wrapper.getClientId(context)); + // gmurray71 + // unique ids are unique to a page and not to an session meaning that if we inject + // content from one page into another the same widget can be given the same id thus + // the same id appears 2x in the same page. + ajaxCommon.setUuid(wrapper.getName().replace('.', '_') + "_" + nextId++); + //ajaxCommon.setUuid(wrapper.getClientId(context)); ajaxCommon.setTemplate(wrapper.getTemplate()); ajaxCommon.setArgs(wrapper.getArgs());