I'm sorry.

I was reading that "hands off" too late...

regards,

Martin

On 2/28/06, Sean Schofield <[EMAIL PROTECTED]> wrote:
> I resolved all conflicts.  I'm also able to run mvn install on the
> trunk after tweaking a few of Martin's last minute commons classes
> that were added after the commons was copied to shared.  I will check
> in tomorrow if I do not hear any objections (we can always roll back
> if we get into trouble.)
>
> Sean
>
> On 2/27/06, Sean Schofield <[EMAIL PROTECTED]> wrote:
> > I took a look at the resulting jars and the poms (for core only.)
> > There might be a few changes to make later but this really looks good!
> >  Nice job Manfred!  I did an svn merge and there are only a few
> > conflicts to sort out.
> >
> > The conflicts are in poms (expected) and JspStateManagerImpl
> > (unexpected.)  I will sort out the conflcts and check in on the trunk
> > if there are no objections.  We'll also need to set up the shared
> > project on continuum (deactivate the commons one.)
> >
> > I think we should do a release of shared once everyone gets a chance
> > to check things out.  Then we release core (soon!)  I will merge the
> > tomahawk changes sometime soon if the core results look good.
> >
> > Sean
> >
> > On 2/27/06, Sean Schofield <[EMAIL PROTECTED]> wrote:
> > > Changes on the trunk are less problematic.  You are right we can use
> > > SVN to find this out but its easier to ask someone where to look for
> > > any major changes.  Thanks.
> > >
> > > Sean
> > >
> > > On 2/27/06, Dennis Byrne <[EMAIL PROTECTED]> wrote:
> > > > Here are two I remember, both to trunk.  I'm pretty sure your SVN 
> > > > client can give you this info for all commits though.
> > > >
> > > > http://mail-archives.apache.org/mod_mbox/myfaces-commits/200602.mbox/[EMAIL
> > > >  PROTECTED]
> > > >
> > > > http://mail-archives.apache.org/mod_mbox/myfaces-commits/200602.mbox/[EMAIL
> > > >  PROTECTED]
> > > >
> > > > Dennis Byrne
> > > >
> > > > >-----Original Message-----
> > > > >From: Sean Schofield [mailto:[EMAIL PROTECTED]
> > > > >Sent: Monday, February 27, 2006 11:33 AM
> > > > >To: 'MyFaces Development'
> > > > >Subject: Re: PLEASE: No major checkins
> > > > >
> > > > >Both b/c both need to merge.  Also no double checkins as Manfred
> > > > >suggested.  I think the merge will work even with the package
> > > > >renaming.  Please let us know what changes were done on *both* the
> > > > >branch and trunk if any (revision numbers please.)
> > > > >
> > > > >Sean
> > > > >
> > > > >On 2/27/06, Dennis Byrne <[EMAIL PROTECTED]> wrote:
> > > > >> I sympathize.
> > > > >>
> > > > >> But do you mean the branch, trunk or both?
> > > > >>
> > > > >> Dennis Byrne
> > > > >>
> > > > >> >-----Original Message-----
> > > > >> >From: Sean Schofield [mailto:[EMAIL PROTECTED]
> > > > >> >Sent: Monday, February 27, 2006 10:26 AM
> > > > >> >To: 'MyFaces Development'
> > > > >> >Subject: PLEASE: No major checkins
> > > > >> >
> > > > >> >Please .... Please ... Please try not to make major changes to
> > > > >> >anything these next few days until the refactoring is complete.  I 
> > > > >> >beg
> > > > >> >you.  This is already complex enough as it is.
> > > > >> >
> > > > >> >Sean
> > > > >> >
> > > > >> >On 2/26/06, Manfred Geiler <[EMAIL PROTECTED]> wrote:
> > > > >> >> Step 1 of refactoring is done.
> > > > >> >> That means: All commons classes have been copied to the shared 
> > > > >> >> module
> > > > >> >> and are now in package org.apache.myfaces.shared.*. This was done 
> > > > >> >> in
> > > > >> >> shared trunk(!) and core and tomahawk branches 1_1_2.
> > > > >> >>
> > > > >> >> Step 2 is pending (Hope I can finish it today):
> > > > >> >> That is: Deriving two "shared" libs with different namespace
> > > > >> >> (o.a.m.impl.shared and o.a.m.tomahawk.shared) and refactoring 
> > > > >> >> core and
> > > > >> >> tomahawk accordingly.
> > > > >> >>
> > > > >> >> So, if you have a serious bugfix for commons/shared, please fix in
> > > > >> >> both shared trunk and commons trunk, so that both nightly builds 
> > > > >> >> and
> > > > >> >> coming release will have this fix. Redundant code change sounds a 
> > > > >> >> bit
> > > > >> >> queerish, but there is no other solution at the moment. Right, 
> > > > >> >> Sean?
> > > > >> >>
> > > > >> >> After the release 1.1.2 we merge core and tomahawk branches down 
> > > > >> >> to
> > > > >> >> the trunk and must ensure somehow that meantime commons changes 
> > > > >> >> are
> > > > >> >> merged into shared. Don't know if that will be possible with SVN
> > > > >> >> trickery. So, if you have low priority commons changes, please
> > > > >> >> postpone until after the release.
> > > > >> >>
> > > > >> >> Thanks,
> > > > >> >> Manfred
> > > > >> >>
> > > > >> >>
> > > > >> >>
> > > > >> >> On 2/25/06, Martin Marinschek <[EMAIL PROTECTED]> wrote:
> > > > >> >> > ;)
> > > > >> >> >
> > > > >> >> > I think Manfred is working on a branch, so you shouldn't be in 
> > > > >> >> > danger
> > > > >> >> > while dancing.
> > > > >> >> >
> > > > >> >> > regards,
> > > > >> >> >
> > > > >> >> > Martin
> > > > >> >> >
> > > > >> >> > On 2/25/06, Dennis Byrne <[EMAIL PROTECTED]> wrote:
> > > > >> >> > > DJ Manfred,
> > > > >> >> > >
> > > > >> >> > > Are you done remixing commons, or should we postpone commits 
> > > > >> >> > > until the end of the weekend?
> > > > >> >> > >
> > > > >> >> > > Dennis Byrne
> > > > >> >> > >
> > > > >> >> > > >-----Original Message-----
> > > > >> >> > > >From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED]
> > > > >> >> > > >Sent: Friday, February 24, 2006 10:47 AM
> > > > >> >> > > >To: [email protected]
> > > > >> >> > > >Subject: svn commit: r380693 [9/9] - in /myfaces: 
> > > > >> >> > > >core/branches/1_1_2/assembly/ core/branches/1_1_2/impl/ 
> > > > >> >> > > >core/branches/1_1_2/impl/src/main/java/org/apache/myfaces/application/
> > > > >> >> > > > 
> > > > >> >> > > >core/branches/1_1_2/impl/src/main/java/org/apache/myfaces/application/jsp/
> > > > >> >> > > > core/bran...
> > > > >> >> > > >
> > > > >> >> > > >Modified: 
> > > > >> >> > > >myfaces/tomahawk/branches/1_1_2/sandbox/core/src/main/java/org/apache/myfaces/custom/script/Script.java
> > > > >> >> > > >URL: 
> > > > >> >> > > >http://svn.apache.org/viewcvs/myfaces/tomahawk/branches/1_1_2/sandbox/core/src/main/java/org/apache/myfaces/custom/script/Script.java?rev=380693&r1=380692&r2=380693&view=diff
> > > > >> >> > > >==============================================================================
> > > > >> >> > > >--- 
> > > > >> >> > > >myfaces/tomahawk/branches/1_1_2/sandbox/core/src/main/java/org/apache/myfaces/custom/script/Script.java
> > > > >> >> > > > (original)
> > > > >> >> > > >+++ 
> > > > >> >> > > >myfaces/tomahawk/branches/1_1_2/sandbox/core/src/main/java/org/apache/myfaces/custom/script/Script.java
> > > > >> >> > > > Fri Feb 24 07:47:18 2006
> > > > >> >> > > >@@ -19,8 +19,8 @@
> > > > >> >> > > > import javax.faces.context.FacesContext;
> > > > >> >> > > > import javax.faces.el.ValueBinding;
> > > > >> >> > > >
> > > > >> >> > > >-import org.apache.myfaces.renderkit.html.HTML;
> > > > >> >> > > >-import org.apache.myfaces.util._ComponentUtils;
> > > > >> >> > > >+import org.apache.myfaces.shared.renderkit.html.HTML;
> > > > >> >> > > >+import org.apache.myfaces.shared.util._ComponentUtils;
> > > > >> >> > > >
> > > > >> >> > > > /**
> > > > >> >> > > >  * @author Matthias Wessendorf (changed by $Author$)
> > > > >> >> > > >@@ -28,12 +28,12 @@
> > > > >> >> > > >  */
> > > > >> >> > > > public class Script extends UIOutput {
> > > > >> >> > > >
> > > > >> >> > > >-      public static final String COMPONENT_TYPE = 
> > > > >> >> > > >"org.apache.myfaces.Script";
> > > > >> >> > > >-      public static final String COMPONENT_FAMILY = 
> > > > >> >> > > >"javax.faces.Output";
> > > > >> >> > > >-      private static final String DEFAULT_RENDERER_TYPE = 
> > > > >> >> > > >"org.apache.myfaces.Script";
> > > > >> >> > > >-
> > > > >> >> > > >-      private String src = null;
> > > > >> >> > > >-      private String type = 
> > > > >> >> > > >HTML.SCRIPT_TYPE_TEXT_JAVASCRIPT;
> > > > >> >> > > >+    public static final String COMPONENT_TYPE = 
> > > > >> >> > > >"org.apache.myfaces.Script";
> > > > >> >> > > >+    public static final String COMPONENT_FAMILY = 
> > > > >> >> > > >"javax.faces.Output";
> > > > >> >> > > >+    private static final String DEFAULT_RENDERER_TYPE = 
> > > > >> >> > > >"org.apache.myfaces.Script";
> > > > >> >> > > >+
> > > > >> >> > > >+    private String src = null;
> > > > >> >> > > >+    private String type = HTML.SCRIPT_TYPE_TEXT_JAVASCRIPT;
> > > > >> >> > > >
> > > > >> >> > > >     // 
> > > > >> >> > > > ------------------------------------------------------------
> > > > >> >> > > >  Constructor
> > > > >> >> > > >     public Script() {
> > > > >> >> > > >@@ -50,28 +50,28 @@
> > > > >> >> > > >     }
> > > > >> >> > > >
> > > > >> >> > > >     //  
> > > > >> >> > > > ------------------------------------------------------------
> > > > >> >> > > >  getter/setter
> > > > >> >> > > >-      public String getSrc() {
> > > > >> >> > > >+    public String getSrc() {
> > > > >> >> > > >         if (src != null) return src;
> > > > >> >> > > >         ValueBinding vb = getValueBinding("src");
> > > > >> >> > > >         return vb != null ? 
> > > > >> >> > > > _ComponentUtils.getStringValue(getFacesContext(), vb) : 
> > > > >> >> > > > null;
> > > > >> >> > > >-      }
> > > > >> >> > > >+    }
> > > > >> >> > > >
> > > > >> >> > > >
> > > > >> >> > > >-      public void setSrc(String src) {
> > > > >> >> > > >-              this.src = src;
> > > > >> >> > > >-      }
> > > > >> >> > > >+    public void setSrc(String src) {
> > > > >> >> > > >+        this.src = src;
> > > > >> >> > > >+    }
> > > > >> >> > > >
> > > > >> >> > > >
> > > > >> >> > > >-      public String getType() {
> > > > >> >> > > >+    public String getType() {
> > > > >> >> > > >         if (type != null) return type;
> > > > >> >> > > >         ValueBinding vb = getValueBinding("type");
> > > > >> >> > > >         return vb != null ? 
> > > > >> >> > > > _ComponentUtils.getStringValue(getFacesContext(), vb) : 
> > > > >> >> > > > null;
> > > > >> >> > > >-      }
> > > > >> >> > > >+    }
> > > > >> >> > > >
> > > > >> >> > > >
> > > > >> >> > > >-      public void setType(String type) {
> > > > >> >> > > >-              this.type = type;
> > > > >> >> > > >-      }
> > > > >> >> > > >+    public void setType(String type) {
> > > > >> >> > > >+        this.type = type;
> > > > >> >> > > >+    }
> > > > >> >> > > >
> > > > >> >> > > >     //  
> > > > >> >> > > > ------------------------------------------------------------
> > > > >> >> > > >  StateHolder
> > > > >> >> > > >     public void restoreState(FacesContext context, Object 
> > > > >> >> > > > state) {
> > > > >> >> > > >@@ -93,5 +93,5 @@
> > > > >> >> > > >
> > > > >> >> > > >     }
> > > > >> >> > > >
> > > > >> >> > > >-
> > > > >> >> > > >+
> > > > >> >> > > > }
> > > > >> >> > > >
> > > > >> >> > > >Modified: 
> > > > >> >> > > >myfaces/tomahawk/branches/1_1_2/sandbox/core/src/main/java/org/apache/myfaces/custom/script/ScriptRenderer.java
> > > > >> >> > > >URL: 
> > > > >> >> > > >http://svn.apache.org/viewcvs/myfaces/tomahawk/branches/1_1_2/sandbox/core/src/main/java/org/apache/myfaces/custom/script/ScriptRenderer.java?rev=380693&r1=380692&r2=380693&view=diff
> > > > >> >> > > >==============================================================================
> > > > >> >> > > >--- 
> > > > >> >> > > >myfaces/tomahawk/branches/1_1_2/sandbox/core/src/main/java/org/apache/myfaces/custom/script/ScriptRenderer.java
> > > > >> >> > > > (original)
> > > > >> >> > > >+++ 
> > > > >> >> > > >myfaces/tomahawk/branches/1_1_2/sandbox/core/src/main/java/org/apache/myfaces/custom/script/ScriptRenderer.java
> > > > >> >> > > > Fri Feb 24 07:47:18 2006
> > > > >> >> > > >@@ -21,26 +21,26 @@
> > > > >> >> > > > import javax.faces.context.FacesContext;
> > > > >> >> > > > import javax.faces.context.ResponseWriter;
> > > > >> >> > > >
> > > > >> >> > > >-import org.apache.myfaces.renderkit.html.HTML;
> > > > >> >> > > >-import org.apache.myfaces.renderkit.html.HtmlRenderer;
> > > > >> >> > > >+import org.apache.myfaces.shared.renderkit.html.HTML;
> > > > >> >> > > >+import 
> > > > >> >> > > >org.apache.myfaces.shared.renderkit.html.HtmlRenderer;
> > > > >> >> > > >
> > > > >> >> > > > /**
> > > > >> >> > > >  * @author Matthias Wessendorf (changed by $Author$)
> > > > >> >> > > >  * @version $Revision$ $Date$
> > > > >> >> > > >  */
> > > > >> >> > > > public class ScriptRenderer extends HtmlRenderer {
> > > > >> >> > > >-
> > > > >> >> > > >-      public void encodeEnd(FacesContext context, 
> > > > >> >> > > >UIComponent component) throws IOException {
> > > > >> >> > > >-          if ((context == null) || (component == null)) {
> > > > >> >> > > >-              throw new NullPointerException();
> > > > >> >> > > >-          }
> > > > >> >> > > >-          Script script = (Script) component;
> > > > >> >> > > >-          ResponseWriter writer = 
> > > > >> >> > > >context.getResponseWriter();
> > > > >> >> > > >-
> > > > >> >> > > >+
> > > > >> >> > > >+    public void encodeEnd(FacesContext context, UIComponent 
> > > > >> >> > > >component) throws IOException {
> > > > >> >> > > >+        if ((context == null) || (component == null)) {
> > > > >> >> > > >+            throw new NullPointerException();
> > > > >> >> > > >+        }
> > > > >> >> > > >+        Script script = (Script) component;
> > > > >> >> > > >+        ResponseWriter writer = context.getResponseWriter();
> > > > >> >> > > >+
> > > > >> >> > > >         writer.startElement(HTML.SCRIPT_ELEM, component);
> > > > >> >> > > >         writer.writeAttribute(HTML.SRC_ATTR, 
> > > > >> >> > > > context.getExternalContext().getRequestContextPath()+script.getSrc(),
> > > > >> >> > > >  null);
> > > > >> >> > > >         writer.writeAttribute(HTML.TYPE_ATTR, 
> > > > >> >> > > > script.getType(), null);
> > > > >> >> > > >         writer.writeText("",null);
> > > > >> >> > > >-        writer.endElement(HTML.SCRIPT_ELEM);
> > > > >> >> > > >-      }
> > > > >> >> > > >+        writer.endElement(HTML.SCRIPT_ELEM);
> > > > >> >> > > >+    }
> > > > >> >> > > > }
> > > > >> >> > > >
> > > > >> >> > > >Modified: 
> > > > >> >> > > >myfaces/tomahawk/branches/1_1_2/sandbox/core/src/main/java/org/apache/myfaces/custom/script/ScriptTag.java
> > > > >> >> > > >URL: 
> > > > >> >> > > >http://svn.apache.org/viewcvs/myfaces/tomahawk/branches/1_1_2/sandbox/core/src/main/java/org/apache/myfaces/custom/script/ScriptTag.java?rev=380693&r1=380692&r2=380693&view=diff
> > > > >> >> > > >==============================================================================
> > > > >> >> > > >--- 
> > > > >> >> > > >myfaces/tomahawk/branches/1_1_2/sandbox/core/src/main/java/org/apache/myfaces/custom/script/ScriptTag.java
> > > > >> >> > > > (original)
> > > > >> >> > > >+++ 
> > > > >> >> > > >myfaces/tomahawk/branches/1_1_2/sandbox/core/src/main/java/org/apache/myfaces/custom/script/ScriptTag.java
> > > > >> >> > > > Fri Feb 24 07:47:18 2006
> > > > >> >> > > >@@ -17,8 +17,8 @@
> > > > >> >> > > >
> > > > >> >> > > > import javax.faces.component.UIComponent;
> > > > >> >> > > >
> > > > >> >> > > >-import org.apache.myfaces.renderkit.html.HTML;
> > > > >> >> > > >-import org.apache.myfaces.taglib.html.HtmlOutputTextTagBase;
> > > > >> >> > > >+import org.apache.myfaces.shared.renderkit.html.HTML;
> > > > >> >> > > >+import 
> > > > >> >> > > >org.apache.myfaces.shared.taglib.html.HtmlOutputTextTagBase;
> > > > >> >> > > >
> > > > >> >> > > > /**
> > > > >> >> > > >  * @author Matthias Wessendorf (changed by $Author$)
> > > > >> >> > > >
> > > > >> >> > > >Modified: 
> > > > >> >> > > >myfaces/tomahawk/branches/1_1_2/sandbox/core/src/main/java/org/apache/myfaces/custom/selectOneRow/SelectOneRowRenderer.java
> > > > >> >> > > >URL: 
> > > > >> >> > > >http://svn.apache.org/viewcvs/myfaces/tomahawk/branches/1_1_2/sandbox/core/src/main/java/org/apache/myfaces/custom/selectOneRow/SelectOneRowRenderer.java?rev=380693&r1=380692&r2=380693&view=diff
> > > > >> >> > > >==============================================================================
> > > > >> >> > > >--- 
> > > > >> >> > > >myfaces/tomahawk/branches/1_1_2/sandbox/core/src/main/java/org/apache/myfaces/custom/selectOneRow/SelectOneRowRenderer.java
> > > > >> >> > > > (original)
> > > > >> >> > > >+++ 
> > > > >> >> > > >myfaces/tomahawk/branches/1_1_2/sandbox/core/src/main/java/org/apache/myfaces/custom/selectOneRow/SelectOneRowRenderer.java
> > > > >> >> > > > Fri Feb 24 07:47:18 2006
> > > > >> >> > > >@@ -1,8 +1,8 @@
> > > > >> >> > > > package org.apache.myfaces.custom.selectOneRow;
> > > > >> >> > > >
> > > > >> >> > > >-import org.apache.myfaces.renderkit.html.HtmlRenderer;
> > > > >> >> > > >-import org.apache.myfaces.renderkit.html.HTML;
> > > > >> >> > > >-import org.apache.myfaces.renderkit.html.HtmlRendererUtils;
> > > > >> >> > > >+import 
> > > > >> >> > > >org.apache.myfaces.shared.renderkit.html.HtmlRenderer;
> > > > >> >> > > >+import org.apache.myfaces.shared.renderkit.html.HTML;
> > > > >> >> > > >+import 
> > > > >> >> > > >org.apache.myfaces.shared.renderkit.html.HtmlRendererUtils;
> > > > >> >> > > >
> > > > >> >> > > > import javax.faces.component.UIComponent;
> > > > >> >> > > > import javax.faces.component.UIData;
> > > > >> >> > > >
> > > > >> >> > > >Modified: 
> > > > >> >> > > >myfaces/tomahawk/branches/1_1_2/sandbox/core/src/main/java/org/apache/myfaces/custom/selectOneRow/SelectOneRowTag.java
> > > > >> >> > > >URL: 
> > > > >> >> > > >http://svn.apache.org/viewcvs/myfaces/tomahawk/branches/1_1_2/sandbox/core/src/main/java/org/apache/myfaces/custom/selectOneRow/SelectOneRowTag.java?rev=380693&r1=380692&r2=380693&view=diff
> > > > >> >> > > >==============================================================================
> > > > >> >> > > >--- 
> > > > >> >> > > >myfaces/tomahawk/branches/1_1_2/sandbox/core/src/main/java/org/apache/myfaces/custom/selectOneRow/SelectOneRowTag.java
> > > > >> >> > > > (original)
> > > > >> >> > > >+++ 
> > > > >> >> > > >myfaces/tomahawk/branches/1_1_2/sandbox/core/src/main/java/org/apache/myfaces/custom/selectOneRow/SelectOneRowTag.java
> > > > >> >> > > > Fri Feb 24 07:47:18 2006
> > > > >> >> > > >@@ -1,9 +1,9 @@
> > > > >> >> > > > package org.apache.myfaces.custom.selectOneRow;
> > > > >> >> > > >
> > > > >> >> > > >-import org.apache.myfaces.taglib.html.HtmlInputTagBase;
> > > > >> >> > > >+import 
> > > > >> >> > > >org.apache.myfaces.shared.taglib.html.HtmlInputTagBase;
> > > > >> >> > > >+
> > > > >> >> > > >+import org.apache.myfaces.shared.renderkit.html.HTML;
> > > > >> >> > > >
> > > > >> >> > > >-import org.apache.myfaces.renderkit.JSFAttr;
> > > > >> >> > > >-import org.apache.myfaces.renderkit.html.HTML;
> > > > >> >> > > > import javax.faces.application.Application;
> > > > >> >> > > > import javax.faces.component.UIComponent;
> > > > >> >> > > > import javax.faces.component.UIInput;
> > > > >> >> > > >
> > > > >> >> > > >Modified: 
> > > > >> >> > > >myfaces/tomahawk/branches/1_1_2/sandbox/core/src/main/java/org/apache/myfaces/custom/statechangednotifier/StateChangedNotifier.java
> > > > >> >> > > >URL: 
> > > > >> >> > > >http://svn.apache.org/viewcvs/myfaces/tomahawk/branches/1_1_2/sandbox/core/src/main/java/org/apache/myfaces/custom/statechangednotifier/StateChangedNotifier.java?rev=380693&r1=380692&r2=380693&view=diff
> > > > >> >> > > >==============================================================================
> > > > >> >> > > >--- 
> > > > >> >> > > >myfaces/tomahawk/branches/1_1_2/sandbox/core/src/main/java/org/apache/myfaces/custom/statechangednotifier/StateChangedNotifier.java
> > > > >> >> > > > (original)
> > > > >> >> > > >+++ 
> > > > >> >> > > >myfaces/tomahawk/branches/1_1_2/sandbox/core/src/main/java/org/apache/myfaces/custom/statechangednotifier/StateChangedNotifier.java
> > > > >> >> > > > Fri Feb 24 07:47:18 2006
> > > > >> >> > > >@@ -15,7 +15,7 @@
> > > > >> >> > > >  */
> > > > >> >> > > > package org.apache.myfaces.custom.statechangednotifier;
> > > > >> >> > > >
> > > > >> >> > > >-import org.apache.myfaces.util._ComponentUtils;
> > > > >> >> > > >+import org.apache.myfaces.shared.util._ComponentUtils;
> > > > >> >> > > >
> > > > >> >> > > > import javax.faces.component.html.HtmlInputHidden;
> > > > >> >> > > > import javax.faces.context.FacesContext;
> > > > >> >> > > >
> > > > >> >> > > >Modified: 
> > > > >> >> > > >myfaces/tomahawk/branches/1_1_2/sandbox/core/src/main/java/org/apache/myfaces/custom/statechangednotifier/StateChangedNotifierRenderer.java
> > > > >> >> > > >URL: 
> > > > >> >> > > >http://svn.apache.org/viewcvs/myfaces/tomahawk/branches/1_1_2/sandbox/core/src/main/java/org/apache/myfaces/custom/statechangednotifier/StateChangedNotifierRenderer.java?rev=380693&r1=380692&r2=380693&view=diff
> > > > >> >> > > >==============================================================================
> > > > >> >> > > >--- 
> > > > >> >> > > >myfaces/tomahawk/branches/1_1_2/sandbox/core/src/main/java/org/apache/myfaces/custom/statechangednotifier/StateChangedNotifierRenderer.java
> > > > >> >> > > > (original)
> > > > >> >> > > >+++ 
> > > > >> >> > > >myfaces/tomahawk/branches/1_1_2/sandbox/core/src/main/java/org/apache/myfaces/custom/statechangednotifier/StateChangedNotifierRenderer.java
> > > > >> >> > > > Fri Feb 24 07:47:18 2006
> > > > >> >> > > >@@ -16,10 +16,10 @@
> > > > >> >> > > > package org.apache.myfaces.custom.statechangednotifier;
> > > > >> >> > > >
> > > > >> >> > > > import org.apache.myfaces.renderkit.html.HtmlHiddenRenderer;
> > > > >> >> > > >-import org.apache.myfaces.renderkit.html.util.AddResource;
> > > > >> >> > > >-import 
> > > > >> >> > > >org.apache.myfaces.renderkit.html.util.AddResourceFactory;
> > > > >> >> > > >-import org.apache.myfaces.renderkit.RendererUtils;
> > > > >> >> > > >-import org.apache.myfaces.renderkit.JSFAttr;
> > > > >> >> > > >+import 
> > > > >> >> > > >org.apache.myfaces.shared.renderkit.html.util.AddResource;
> > > > >> >> > > >+import 
> > > > >> >> > > >org.apache.myfaces.shared.renderkit.html.util.AddResourceFactory;
> > > > >> >> > > >+import org.apache.myfaces.shared.renderkit.RendererUtils;
> > > > >> >> > > >+import org.apache.myfaces.shared.renderkit.JSFAttr;
> > > > >> >> > > > import org.apache.myfaces.custom.dojo.DojoUtils;
> > > > >> >> > > > import org.apache.myfaces.custom.dojo.DojoConfig;
> > > > >> >> > > >
> > > > >> >> > > >@@ -36,7 +36,8 @@
> > > > >> >> > > > {
> > > > >> >> > > >     public void encodeEnd(FacesContext facesContext, 
> > > > >> >> > > > UIComponent uiComponent) throws IOException
> > > > >> >> > > >     {
> > > > >> >> > > >-        RendererUtils.checkParamValidity(facesContext, 
> > > > >> >> > > >uiComponent, StateChangedNotifier.class);
> > > > >> >> > > >+        RendererUtils.checkParamValidity(facesContext, 
> > > > >> >> > > >uiComponent,
> > > > >> >> > > >+                StateChangedNotifier.class);
> > > > >> >> > > >
> > > > >> >> > > >         StateChangedNotifier notifier = 
> > > > >> >> > > > (StateChangedNotifier) uiComponent;
> > > > >> >> > > >
> > > > >> >> > > >@@ -52,8 +53,8 @@
> > > > >> >> > > >         //DojoUtils.addRequire(facesContext, 
> > > > >> >> > > > "dojo.xml.Parse");
> > > > >> >> > > >
> > > > >> >> > > >         AddResource addResource = 
> > > > >> >> > > > AddResourceFactory.getInstance(facesContext);
> > > > >> >> > > >-        addResource.addJavaScriptAtPosition(facesContext, 
> > > > >> >> > > >AddResource.HEADER_BEGIN, StateChangedNotifierRenderer.class,
> > > > >> >> > > >-                "stateChangedNotifier.js");
> > > > >> >> > > >+        addResource.addJavaScriptAtPosition(facesContext, 
> > > > >> >> > > >AddResource.HEADER_BEGIN,
> > > > >> >> > > >+                StateChangedNotifierRenderer.class, 
> > > > >> >> > > >"stateChangedNotifier.js");
> > > > >> >> > > >
> > > > >> >> > > >         encodeJavascript(facesContext, notifier);
> > > > >> >> > > >
> > > > >> >> > > >@@ -65,39 +66,37 @@
> > > > >> >> > > >         String notifierClientId = 
> > > > >> >> > > > notifier.getClientId(facesContext);
> > > > >> >> > > >
> > > > >> >> > > >         String replacedClientId = 
> > > > >> >> > > > notifierClientId.replaceAll(":", "_");
> > > > >> >> > > >-        String initFunctionName = "init_" + 
> > > > >> >> > > >replacedClientId;
> > > > >> >> > > >+        String initFunctionName = "init_"+replacedClientId;
> > > > >> >> > > >
> > > > >> >> > > >         UIForm form = getParentForm(notifier);
> > > > >> >> > > >         String formId = form.getClientId(facesContext);
> > > > >> >> > > >
> > > > >> >> > > >-        String notifierVar = replacedClientId + "Notifier";
> > > > >> >> > > >+        String notifierVar = replacedClientId+"Notifier";
> > > > >> >> > > >
> > > > >> >> > > >         StringBuffer sb = new StringBuffer();
> > > > >> >> > > >         sb.append("<!--\n");
> > > > >> >> > > >-        sb.append("var " + notifierVar + " = null;\n");
> > > > >> >> > > >-
> > > > >> >> > > >-        sb.append("function " + initFunctionName + "() 
> > > > >> >> > > >{\n");
> > > > >> >> > > >-        sb.append(notifierVar + " = new 
> > > > >> >> > > >org_apache_myfaces_StateChangedNotifier('" + notifierVar + 
> > > > >> >> > > >"','" + formId
> > > > >> >> > > >-                + "','" + notifierClientId + "','" + 
> > > > >> >> > > >notifier.getConfirmationMessage() + "',");
> > > > >> >> > > >+        sb.append("dojo.addOnLoad(window, 
> > > > >> >> > > >'"+initFunctionName+"');\n");
> > > > >> >> > > >+        sb.append("function "+initFunctionName+ "() {\n");
> > > > >> >> > > >+        sb.append(notifierVar+" = new 
> > > > >> >> > > >org.apache.myfaces.StateChangedNotifier('"+notifierVar+"','"+formId+"','"+notifierClientId+"','"+notifier.getConfirmationMessage()+"',");
> > > > >> >> > > >
> > > > >> >> > > >         String excludedCommandIds = 
> > > > >> >> > > > notifier.getExcludedIds();
> > > > >> >> > > >         if (excludedCommandIds != null)
> > > > >> >> > > >         {
> > > > >> >> > > >-            sb.append("'" + excludedCommandIds + "');\n");
> > > > >> >> > > >+            sb.append("'"+excludedCommandIds+"');\n");
> > > > >> >> > > >         }
> > > > >> >> > > >         else
> > > > >> >> > > >         {
> > > > >> >> > > >             sb.append("'');\n");
> > > > >> >> > > >         }
> > > > >> >> > > >
> > > > >> >> > > >-        sb.append(replacedClientId + 
> > > > >> >> > > >"Notifier.prepareNotifier();\n");
> > > > >> >> > > >+        
> > > > >> >> > > >sb.append(replacedClientId+"Notifier.prepareNotifier();\n");
> > > > >> >> > > >
> > > > >> >> > > >         sb.append("}\n");
> > > > >> >> > > >-        sb.append("dojo.addOnLoad(window, '" + 
> > > > >> >> > > >initFunctionName + "');\n");
> > > > >> >> > > >         sb.append("-->\n");
> > > > >> >> > > >
> > > > >> >> > > >         AddResource addResource = 
> > > > >> >> > > > AddResourceFactory.getInstance(facesContext);
> > > > >> >> > > >-        addResource.addInlineScriptAtPosition(facesContext, 
> > > > >> >> > > >AddResource.HEADER_BEGIN, sb.toString());
> > > > >> >> > > >+        addResource.addInlineScriptAtPosition(facesContext, 
> > > > >> >> > > >AddResource.HEADER_BEGIN,
> > > > >> >> > > >+                sb.toString());
> > > > >> >> > > >     }
> > > > >> >> > > >
> > > > >> >> > > >     /**
> > > > >> >> > > >@@ -106,12 +105,10 @@
> > > > >> >> > > >      * @param component
> > > > >> >> > > >      * @return UIForm
> > > > >> >> > > >      */
> > > > >> >> > > >-    private UIForm getParentForm(UIComponent component)
> > > > >> >> > > >-    {
> > > > >> >> > > >+    private UIForm getParentForm(UIComponent component) {
> > > > >> >> > > >         // See if we are in a form
> > > > >> >> > > >         UIComponent parent = component.getParent();
> > > > >> >> > > >-        while (parent != null && !(parent instanceof 
> > > > >> >> > > >UIForm))
> > > > >> >> > > >-        {
> > > > >> >> > > >+        while (parent != null && !(parent instanceof 
> > > > >> >> > > >UIForm)) {
> > > > >> >> > > >             parent = parent.getParent();
> > > > >> >> > > >         }
> > > > >> >> > > >         return (UIForm) parent;
> > > > >> >> > > >
> > > > >> >> > > >Modified: 
> > > > >> >> > > >myfaces/tomahawk/branches/1_1_2/sandbox/core/src/main/java/org/apache/myfaces/custom/subform/SubForm.java
> > > > >> >> > > >URL: 
> > > > >> >> > > >http://svn.apache.org/viewcvs/myfaces/tomahawk/branches/1_1_2/sandbox/core/src/main/java/org/apache/myfaces/custom/subform/SubForm.java?rev=380693&r1=380692&r2=380693&view=diff
> > > > >> >> > > >==============================================================================
> > > > >> >> > > >--- 
> > > > >> >> > > >myfaces/tomahawk/branches/1_1_2/sandbox/core/src/main/java/org/apache/myfaces/custom/subform/SubForm.java
> > > > >> >> > > > (original)
> > > > >> >> > > >+++ 
> > > > >> >> > > >myfaces/tomahawk/branches/1_1_2/sandbox/core/src/main/java/org/apache/myfaces/custom/subform/SubForm.java
> > > > >> >> > > > Fri Feb 24 07:47:18 2006
> > > > >> >> > > >@@ -1,6 +1,6 @@
> > > > >> >> > > > package org.apache.myfaces.custom.subform;
> > > > >> >> > > >
> > > > >> >> > > >-import org.apache.myfaces.renderkit.RendererUtils;
> > > > >> >> > > >+import org.apache.myfaces.shared.renderkit.RendererUtils;
> > > > >> >> > > >
> > > > >> >> > > > import javax.faces.component.NamingContainer;
> > > > >> >> > > > import javax.faces.component.UIComponent;
> > > > >> >> > > >
> > > > >> >> > > >Modified: 
> > > > >> >> > > >myfaces/tomahawk/branches/1_1_2/sandbox/core/src/main/java/org/apache/myfaces/custom/subform/SubFormRenderer.java
> > > > >> >> > > >URL: 
> > > > >> >> > > >http://svn.apache.org/viewcvs/myfaces/tomahawk/branches/1_1_2/sandbox/core/src/main/java/org/apache/myfaces/custom/subform/SubFormRenderer.java?rev=380693&r1=380692&r2=380693&view=diff
> > > > >> >> > > >==============================================================================
> > > > >> >> > > >--- 
> > > > >> >> > > >myfaces/tomahawk/branches/1_1_2/sandbox/core/src/main/java/org/apache/myfaces/custom/subform/SubFormRenderer.java
> > > > >> >> > > > (original)
> > > > >> >> > > >+++ 
> > > > >> >> > > >myfaces/tomahawk/branches/1_1_2/sandbox/core/src/main/java/org/apache/myfaces/custom/subform/SubFormRenderer.java
> > > > >> >> > > > Fri Feb 24 07:47:18 2006
> > > > >> >> > > >@@ -1,6 +1,6 @@
> > > > >> >> > > > package org.apache.myfaces.custom.subform;
> > > > >> >> > > >
> > > > >> >> > > >-import org.apache.myfaces.renderkit.html.HtmlRenderer;
> > > > >> >> > > >+import 
> > > > >> >> > > >org.apache.myfaces.shared.renderkit.html.HtmlRenderer;
> > > > >> >> > > >
> > > > >> >> > > > /**
> > > > >> >> > > >  * @author Gerald Muellan
> > > > >> >> > > >@@ -9,5 +9,5 @@
> > > > >> >> > > >  */
> > > > >> >> > > > public class SubFormRenderer extends HtmlRenderer
> > > > >> >> > > > {
> > > > >> >> > > >-
> > > > >> >> > > >+
> > > > >> >> > > > }
> > > > >> >> > > >
> > > > >> >> > > >Modified: 
> > > > >> >> > > >myfaces/tomahawk/branches/1_1_2/sandbox/core/src/main/java/org/apache/myfaces/custom/suggest/InputSuggestRenderer.java
> > > > >> >> > > >URL: 
> > > > >> >> > > >http://svn.apache.org/viewcvs/myfaces/tomahawk/branches/1_1_2/sandbox/core/src/main/java/org/apache/myfaces/custom/suggest/InputSuggestRenderer.java?rev=380693&r1=380692&r2=380693&view=diff
> > > > >> >> > > >==============================================================================
> > > > >> >> > > >--- 
> > > > >> >> > > >myfaces/tomahawk/branches/1_1_2/sandbox/core/src/main/java/org/apache/myfaces/custom/suggest/InputSuggestRenderer.java
> > > > >> >> > > > (original)
> > > > >> >> > > >+++ 
> > > > >> >> > > >myfaces/tomahawk/branches/1_1_2/sandbox/core/src/main/java/org/apache/myfaces/custom/suggest/InputSuggestRenderer.java
> > > > >> >> > > > Fri Feb 24 07:47:18 2006
> > > > >> >> > > >@@ -29,11 +29,11 @@
> > > > >> >> > > > import javax.faces.render.Renderer;
> > > > >> >> > > >
> > > > >> >> > > > import 
> > > > >> >> > > > org.apache.myfaces.component.html.ext.HtmlInputHidden;
> > > > >> >> > > >-import org.apache.myfaces.renderkit.html.util.AddResource;
> > > > >> >> > > >-import 
> > > > >> >> > > >org.apache.myfaces.renderkit.html.util.AddResourceFactory;
> > > > >> >> > > >-import org.apache.myfaces.renderkit.JSFAttr;
> > > > >> >> > > >-import org.apache.myfaces.renderkit.html.HTML;
> > > > >> >> > > >-import org.apache.myfaces.renderkit.html.HtmlRendererUtils;
> > > > >> >> > > >+import 
> > > > >> >> > > >org.apache.myfaces.shared.renderkit.html.util.AddResource;
> > > > >> >> > > >+import 
> > > > >> >> > > >org.apache.myfaces.shared.renderkit.html.util.AddResourceFactory;
> > > > >> >> > > >+import org.apache.myfaces.shared.renderkit.html.HTML;
> > > > >> >> > > >+import org.apache.myfaces.shared.renderkit.JSFAttr;
> > > > >> >> > > >+import 
> > > > >> >> > > >org.apache.myfaces.shared.renderkit.html.HtmlRendererUtils;
> > > > >> >> > > >
> > > > >> >> > > > /**
> > > > >> >> > > >  * Basic HTML Renderer for the inputSuggest component.
> > > > >> >> > > >@@ -77,7 +77,7 @@
> > > > >> >> > > >     }
> > > > >> >> > > >
> > > > >> >> > > >     public void encodeBegin(FacesContext context, 
> > > > >> >> > > > UIComponent component) throws
> > > > >> >> > > >-        IOException
> > > > >> >> > > >+                                                            
> > > > >> >> > > >             IOException
> > > > >> >> > > >     {
> > > > >> >> > > >
> > > > >> >> > > >         if (!component.isRendered())
> > > > >> >> > > >@@ -120,7 +120,7 @@
> > > > >> >> > > >
> > > > >> >> > > >     private void renderInputField(ResponseWriter out, 
> > > > >> >> > > > String text,
> > > > >> >> > > >                                   String clientId, 
> > > > >> >> > > > UIComponent component) throws
> > > > >> >> > > >-        IOException
> > > > >> >> > > >+                                                            
> > > > >> >> > > >              IOException
> > > > >> >> > > >     {
> > > > >> >> > > >
> > > > >> >> > > >         out.startElement("input", component);
> > > > >> >> > > >@@ -139,7 +139,7 @@
> > > > >> >> > > >         component.getAttributes().put("autocomplete","off");
> > > > >> >> > > >
> > > > >> >> > > >         HtmlRendererUtils.renderHTMLAttributes(out,
> > > > >> >> > > >-                component, 
> > > > >> >> > > >HTML.INPUT_PASSTHROUGH_ATTRIBUTES_WITHOUT_DISABLED);
> > > > >> >> > > >+                                               component, 
> > > > >> >> > > >HTML.INPUT_PASSTHROUGH_ATTRIBUTES_WITHOUT_DISABLED);
> > > > >> >> > > >
> > > > >> >> > > >         if((component instanceof HtmlInputText) && 
> > > > >> >> > > > ((HtmlInputText) component).isDisabled())
> > > > >> >> > > >         {
> > > > >> >> > > >@@ -271,7 +271,7 @@
> > > > >> >> > > >      * @throws IOException
> > > > >> >> > > >      */
> > > > >> >> > > >     private void encodeStyles(UIComponent component, 
> > > > >> >> > > > FacesContext context) throws
> > > > >> >> > > >-        IOException
> > > > >> >> > > >+                                                            
> > > > >> >> > > >               IOException
> > > > >> >> > > >     {
> > > > >> >> > > >         ResponseWriter out = context.getResponseWriter();
> > > > >> >> > > >
> > > > >> >> > > >@@ -297,7 +297,7 @@
> > > > >> >> > > >      * @throws IOException
> > > > >> >> > > >      */
> > > > >> >> > > >     private void encodeJavascript(UIComponent component, 
> > > > >> >> > > > FacesContext context) throws
> > > > >> >> > > >-        IOException
> > > > >> >> > > >+                                                            
> > > > >> >> > > >                   IOException
> > > > >> >> > > >     {
> > > > >> >> > > >         ResponseWriter out = context.getResponseWriter();
> > > > >> >> > > >
> > > > >> >> > > >
> > > > >> >> > > >Modified: 
> > > > >> >> > > >myfaces/tomahawk/branches/1_1_2/sandbox/core/src/main/java/org/apache/myfaces/custom/suggest/InputSuggestTag.java
> > > > >> >> > > >URL: 
> > > > >> >> > > >http://svn.apache.org/viewcvs/myfaces/tomahawk/branches/1_1_2/sandbox/core/src/main/java/org/apache/myfaces/custom/suggest/InputSuggestTag.java?rev=380693&r1=380692&r2=380693&view=diff
> > > > >> >> > > >==============================================================================
> > > > >> >> > > >--- 
> > > > >> >> > > >myfaces/tomahawk/branches/1_1_2/sandbox/core/src/main/java/org/apache/myfaces/custom/suggest/InputSuggestTag.java
> > > > >> >> > > > (original)
> > > > >> >> > > >+++ 
> > > > >> >> > > >myfaces/tomahawk/branches/1_1_2/sandbox/core/src/main/java/org/apache/myfaces/custom/suggest/InputSuggestTag.java
> > > > >> >> > > > Fri Feb 24 07:47:18 2006
> > > > >> >> > > >@@ -17,7 +17,7 @@
> > > > >> >> > > >
> > > > >> >> > > > import javax.faces.component.UIComponent;
> > > > >> >> > > >
> > > > >> >> > > >-import org.apache.myfaces.taglib.html.HtmlInputTagBase;
> > > > >> >> > > >+import 
> > > > >> >> > > >org.apache.myfaces.shared.taglib.html.HtmlInputTagBase;
> > > > >> >> > > >
> > > > >> >> > > > /**
> > > > >> >> > > >  * @author Sean Schofield
> > > > >> >> > > >
> > > > >> >> > > >Modified: 
> > > > >> >> > > >myfaces/tomahawk/branches/1_1_2/sandbox/core/src/main/java/org/apache/myfaces/custom/urlvalidator/UrlValidator.java
> > > > >> >> > > >URL: 
> > > > >> >> > > >http://svn.apache.org/viewcvs/myfaces/tomahawk/branches/1_1_2/sandbox/core/src/main/java/org/apache/myfaces/custom/urlvalidator/UrlValidator.java?rev=380693&r1=380692&r2=380693&view=diff
> > > > >> >> > > >==============================================================================
> > > > >> >> > > >--- 
> > > > >> >> > > >myfaces/tomahawk/branches/1_1_2/sandbox/core/src/main/java/org/apache/myfaces/custom/urlvalidator/UrlValidator.java
> > > > >> >> > > > (original)
> > > > >> >> > > >+++ 
> > > > >> >> > > >myfaces/tomahawk/branches/1_1_2/sandbox/core/src/main/java/org/apache/myfaces/custom/urlvalidator/UrlValidator.java
> > > > >> >> > > > Fri Feb 24 07:47:18 2006
> > > > >> >> > > >@@ -16,7 +16,7 @@
> > > > >> >> > > >
> > > > >> >> > > > package org.apache.myfaces.custom.urlvalidator;
> > > > >> >> > > >
> > > > >> >> > > >-import org.apache.myfaces.util.MessageUtils;
> > > > >> >> > > >+import org.apache.myfaces.shared.util.MessageUtils;
> > > > >> >> > > >
> > > > >> >> > > > import org.apache.commons.validator.GenericValidator;
> > > > >> >> > > >
> > > > >> >> > > >
> > > > >> >> > > >Modified: 
> > > > >> >> > > >myfaces/tomahawk/branches/1_1_2/sandbox/examples/src/main/webapp/WEB-INF/web.xml
> > > > >> >> > > >URL: 
> > > > >> >> > > >http://svn.apache.org/viewcvs/myfaces/tomahawk/branches/1_1_2/sandbox/examples/src/main/webapp/WEB-INF/web.xml?rev=380693&r1=380692&r2=380693&view=diff
> > > > >> >> > > >==============================================================================
> > > > >> >> > > >--- 
> > > > >> >> > > >myfaces/tomahawk/branches/1_1_2/sandbox/examples/src/main/webapp/WEB-INF/web.xml
> > > > >> >> > > > (original)
> > > > >> >> > > >+++ 
> > > > >> >> > > >myfaces/tomahawk/branches/1_1_2/sandbox/examples/src/main/webapp/WEB-INF/web.xml
> > > > >> >> > > > Fri Feb 24 07:47:18 2006
> > > > >> >> > > >@@ -89,7 +89,7 @@
> > > > >> >> > > >   </servlet>
> > > > >> >> > > >   <servlet>
> > > > >> >> > > >     <servlet-name>SourceCodeServlet</servlet-name>
> > > > >> >> > > >-    
> > > > >> >> > > ><servlet-class>org.apache.myfaces.util.servlet.SourceCodeServlet</servlet-class>
> > > > >> >> > > >+    <servlet-class>SourceCodeServlet</servlet-class>
> > > > >> >> > > >   </servlet>
> > > > >> >> > > >   <servlet-mapping>
> > > > >> >> > > >     <servlet-name>Faces Servlet</servlet-name>
> > > > >> >> > > >
> > > > >> >> > > >
> > > > >> >> > > >
> > > > >> >> > >
> > > > >> >> > >
> > > > >> >> > >
> > > > >> >> >
> > > > >> >> >
> > > > >> >> > --
> > > > >> >> >
> > > > >> >> > http://www.irian.at
> > > > >> >> >
> > > > >> >> > Your JSF powerhouse -
> > > > >> >> > JSF Consulting, Development and
> > > > >> >> > Courses in English and German
> > > > >> >> >
> > > > >> >> > Professional Support for Apache MyFaces
> > > > >> >> >
> > > > >> >>
> > > > >> >
> > > > >>
> > > > >>
> > > > >>
> > > > >
> > > >
> > > >
> > > >
> > >
> >
>


--

http://www.irian.at

Your JSF powerhouse -
JSF Consulting, Development and
Courses in English and German

Professional Support for Apache MyFaces

Reply via email to