You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by lu...@apache.org on 2008/06/04 20:53:58 UTC

svn commit: r663341 - in /myfaces/tomahawk/trunk/core/src: main/java/org/apache/myfaces/component/html/ext/ main/java/org/apache/myfaces/renderkit/html/ext/ test/java/org/apache/myfaces/component/html/ext/ test/java/org/apache/myfaces/test/utils/

Author: lu4242
Date: Wed Jun  4 11:53:57 2008
New Revision: 663341

URL: http://svn.apache.org/viewvc?rev=663341&view=rev
Log:
TOMAHAWK-1023 HtmlInputHidden fails unit test when using RI

Added:
    myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java   (with props)
Modified:
    myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/component/html/ext/AbstractHtmlInputHidden.java
    myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/component/html/ext/HtmlInputHiddenTest.java
    myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/test/utils/TestUtils.java

Modified: myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/component/html/ext/AbstractHtmlInputHidden.java
URL: http://svn.apache.org/viewvc/myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/component/html/ext/AbstractHtmlInputHidden.java?rev=663341&r1=663340&r2=663341&view=diff
==============================================================================
--- myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/component/html/ext/AbstractHtmlInputHidden.java (original)
+++ myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/component/html/ext/AbstractHtmlInputHidden.java Wed Jun  4 11:53:57 2008
@@ -42,7 +42,7 @@
     implements ForceIdAware
 {
     public static final String COMPONENT_TYPE = "org.apache.myfaces.HtmlInputHidden";
-    public static final String DEFAULT_RENDERER_TYPE = "javax.faces.Hidden";
+    public static final String DEFAULT_RENDERER_TYPE = "org.apache.myfaces.Hidden";
 
     public AbstractHtmlInputHidden()
     {

Added: myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java
URL: http://svn.apache.org/viewvc/myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java?rev=663341&view=auto
==============================================================================
--- myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java (added)
+++ myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java Wed Jun  4 11:53:57 2008
@@ -0,0 +1,90 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+package org.apache.myfaces.renderkit.html.ext;
+
+import java.io.IOException;
+
+import javax.faces.component.UIComponent;
+import javax.faces.component.UIInput;
+import javax.faces.component.UIOutput;
+import javax.faces.context.FacesContext;
+import javax.faces.context.ResponseWriter;
+import javax.faces.convert.ConverterException;
+
+import org.apache.myfaces.shared_tomahawk.renderkit.JSFAttr;
+import org.apache.myfaces.shared_tomahawk.renderkit.RendererUtils;
+import org.apache.myfaces.shared_tomahawk.renderkit.html.HTML;
+import org.apache.myfaces.shared_tomahawk.renderkit.html.HtmlRenderer;
+import org.apache.myfaces.shared_tomahawk.renderkit.html.HtmlRendererUtils;
+
+
+/**
+ * @JSFRenderer
+ *   renderKitId="HTML_BASIC"
+ *   family="javax.faces.Input"
+ *   type="org.apache.myfaces.Hidden"
+ *   
+ * @author Thomas Spiegl (latest modification by $Author$)
+ * @author Anton Koinov
+ * @version $Revision$ $Date$
+ */
+public class HtmlHiddenRenderer
+extends HtmlRenderer
+{
+    public void encodeEnd(FacesContext facesContext, UIComponent uiComponent)
+        throws IOException
+    {
+        RendererUtils.checkParamValidity(facesContext, uiComponent, UIInput.class);
+
+        ResponseWriter writer = facesContext.getResponseWriter();
+
+        writer.startElement(HTML.INPUT_ELEM, uiComponent);
+        writer.writeAttribute(HTML.TYPE_ATTR, HTML.INPUT_TYPE_HIDDEN, null);
+
+        String clientId = uiComponent.getClientId(facesContext);
+        writer.writeAttribute(HTML.ID_ATTR, clientId, null);
+        writer.writeAttribute(HTML.NAME_ATTR, clientId, null);
+
+        String value = RendererUtils.getStringValue(facesContext, uiComponent);
+        if (value != null)
+        {
+            writer.writeAttribute(HTML.VALUE_ATTR, value, JSFAttr.VALUE_ATTR);
+        }
+
+        writer.endElement(HTML.INPUT_ELEM);
+    }
+
+    public Object getConvertedValue(FacesContext facesContext, UIComponent uiComponent, Object submittedValue) throws ConverterException
+    {
+        RendererUtils.checkParamValidity(facesContext, uiComponent, UIOutput.class);
+        return RendererUtils.getConvertedUIOutputValue(facesContext,
+                                                       (UIOutput)uiComponent,
+                                                       submittedValue);
+    }
+
+
+    public void decode(FacesContext facesContext, UIComponent component)
+     {
+         RendererUtils.checkParamValidity(facesContext,component,UIInput.class);
+
+         HtmlRendererUtils.decodeUIInput(facesContext, component);
+
+     }
+
+}

Propchange: myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java
------------------------------------------------------------------------------
    svn:eol-style = native

Propchange: myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java
------------------------------------------------------------------------------
    svn:keywords = Date Author Id Revision HeadURL

Modified: myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/component/html/ext/HtmlInputHiddenTest.java
URL: http://svn.apache.org/viewvc/myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/component/html/ext/HtmlInputHiddenTest.java?rev=663341&r1=663340&r2=663341&view=diff
==============================================================================
--- myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/component/html/ext/HtmlInputHiddenTest.java (original)
+++ myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/component/html/ext/HtmlInputHiddenTest.java Wed Jun  4 11:53:57 2008
@@ -48,7 +48,9 @@
     {
         // Define the component
         UIComponent component = new HtmlInputHidden();
-        component.setParent(new HtmlForm());
+        component.setId("TestComponent");
+        HtmlForm form = new HtmlForm();
+        form.getChildren().add(component);
 
         // Render the component
         try

Modified: myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/test/utils/TestUtils.java
URL: http://svn.apache.org/viewvc/myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/test/utils/TestUtils.java?rev=663341&r1=663340&r2=663341&view=diff
==============================================================================
--- myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/test/utils/TestUtils.java (original)
+++ myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/test/utils/TestUtils.java Wed Jun  4 11:53:57 2008
@@ -73,6 +73,9 @@
         addRenderer(facesContext, "javax.faces.Input", "javax.faces.Hidden",
                 "org.apache.myfaces.renderkit.html.HtmlHiddenRenderer");
 
+        addRenderer(facesContext, "javax.faces.Input", "org.apache.myfaces.Hidden",
+                "org.apache.myfaces.renderkit.html.ext.HtmlHiddenRenderer");
+        
         addRenderer(facesContext, "javax.faces.Graphic", "javax.faces.Image",
                 "org.apache.myfaces.renderkit.html.HtmlImageRenderer");
 



Re: HtmlHiddenRenderer.java uses Java 5 annotations. was (Re: svn commit: r663341 -... )

Posted by Leonardo Uribe <lu...@gmail.com>.
On Wed, Jun 4, 2008 at 2:39 PM, Paul Spencer <pa...@apache.org> wrote:

> Leonardo,
>
> HtmlHiddenRenderer.java uses Java 5 annotations.  Tomahawk must work in
> Java 1.4
>

I have checked the code but there are using doclets, not annotations, so
this is fine ;) .

Thanks for pointing this stuff.

regards

Leonardo Uribe


>
> Paul Spencer
>
> lu4242@apache.org wrote:
>
>> Author: lu4242
>> Date: Wed Jun  4 11:53:57 2008
>> New Revision: 663341
>>
>> URL: http://svn.apache.org/viewvc?rev=663341&view=rev
>> Log:
>> TOMAHAWK-1023 HtmlInputHidden fails unit test when using RI
>>
>> Added:
>>
>>  myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java
>>   (with props)
>> Modified:
>>
>>  myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/component/html/ext/AbstractHtmlInputHidden.java
>>
>>  myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/component/html/ext/HtmlInputHiddenTest.java
>>
>>  myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/test/utils/TestUtils.java
>>
>> Modified:
>> myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/component/html/ext/AbstractHtmlInputHidden.java
>> URL:
>> http://svn.apache.org/viewvc/myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/component/html/ext/AbstractHtmlInputHidden.java?rev=663341&r1=663340&r2=663341&view=diff
>>
>> ==============================================================================
>> ---
>> myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/component/html/ext/AbstractHtmlInputHidden.java
>> (original)
>> +++
>> myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/component/html/ext/AbstractHtmlInputHidden.java
>> Wed Jun  4 11:53:57 2008
>> @@ -42,7 +42,7 @@
>>     implements ForceIdAware
>>  {
>>     public static final String COMPONENT_TYPE =
>> "org.apache.myfaces.HtmlInputHidden";
>> -    public static final String DEFAULT_RENDERER_TYPE =
>> "javax.faces.Hidden";
>> +    public static final String DEFAULT_RENDERER_TYPE =
>> "org.apache.myfaces.Hidden";
>>       public AbstractHtmlInputHidden()
>>     {
>>
>> Added:
>> myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java
>> URL:
>> http://svn.apache.org/viewvc/myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java?rev=663341&view=auto
>>
>> ==============================================================================
>> ---
>> myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java
>> (added)
>> +++
>> myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java
>> Wed Jun  4 11:53:57 2008
>> @@ -0,0 +1,90 @@
>> +/*
>> + * Licensed to the Apache Software Foundation (ASF) under one
>> + * or more contributor license agreements.  See the NOTICE file
>> + * distributed with this work for additional information
>> + * regarding copyright ownership.  The ASF licenses this file
>> + * to you under the Apache License, Version 2.0 (the
>> + * "License"); you may not use this file except in compliance
>> + * with the License.  You may obtain a copy of the License at
>> + *
>> + *   http://www.apache.org/licenses/LICENSE-2.0
>> + *
>> + * Unless required by applicable law or agreed to in writing,
>> + * software distributed under the License is distributed on an
>> + * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
>> + * KIND, either express or implied.  See the License for the
>> + * specific language governing permissions and limitations
>> + * under the License.
>> + */
>> +package org.apache.myfaces.renderkit.html.ext;
>> +
>> +import java.io.IOException;
>> +
>> +import javax.faces.component.UIComponent;
>> +import javax.faces.component.UIInput;
>> +import javax.faces.component.UIOutput;
>> +import javax.faces.context.FacesContext;
>> +import javax.faces.context.ResponseWriter;
>> +import javax.faces.convert.ConverterException;
>> +
>> +import org.apache.myfaces.shared_tomahawk.renderkit.JSFAttr;
>> +import org.apache.myfaces.shared_tomahawk.renderkit.RendererUtils;
>> +import org.apache.myfaces.shared_tomahawk.renderkit.html.HTML;
>> +import org.apache.myfaces.shared_tomahawk.renderkit.html.HtmlRenderer;
>> +import
>> org.apache.myfaces.shared_tomahawk.renderkit.html.HtmlRendererUtils;
>> +
>> +
>> +/**
>> + * @JSFRenderer
>> + *   renderKitId="HTML_BASIC"
>> + *   family="javax.faces.Input"
>> + *   type="org.apache.myfaces.Hidden"
>> + *   + * @author Thomas Spiegl (latest modification by $Author$)
>> + * @author Anton Koinov
>> + * @version $Revision$ $Date$
>> + */
>> +public class HtmlHiddenRenderer
>> +extends HtmlRenderer
>> +{
>> +    public void encodeEnd(FacesContext facesContext, UIComponent
>> uiComponent)
>> +        throws IOException
>> +    {
>> +        RendererUtils.checkParamValidity(facesContext, uiComponent,
>> UIInput.class);
>> +
>> +        ResponseWriter writer = facesContext.getResponseWriter();
>> +
>> +        writer.startElement(HTML.INPUT_ELEM, uiComponent);
>> +        writer.writeAttribute(HTML.TYPE_ATTR, HTML.INPUT_TYPE_HIDDEN,
>> null);
>> +
>> +        String clientId = uiComponent.getClientId(facesContext);
>> +        writer.writeAttribute(HTML.ID_ATTR, clientId, null);
>> +        writer.writeAttribute(HTML.NAME_ATTR, clientId, null);
>> +
>> +        String value = RendererUtils.getStringValue(facesContext,
>> uiComponent);
>> +        if (value != null)
>> +        {
>> +            writer.writeAttribute(HTML.VALUE_ATTR, value,
>> JSFAttr.VALUE_ATTR);
>> +        }
>> +
>> +        writer.endElement(HTML.INPUT_ELEM);
>> +    }
>> +
>> +    public Object getConvertedValue(FacesContext facesContext,
>> UIComponent uiComponent, Object submittedValue) throws ConverterException
>> +    {
>> +        RendererUtils.checkParamValidity(facesContext, uiComponent,
>> UIOutput.class);
>> +        return RendererUtils.getConvertedUIOutputValue(facesContext,
>> +
>> (UIOutput)uiComponent,
>> +                                                       submittedValue);
>> +    }
>> +
>> +
>> +    public void decode(FacesContext facesContext, UIComponent component)
>> +     {
>> +
>> RendererUtils.checkParamValidity(facesContext,component,UIInput.class);
>> +
>> +         HtmlRendererUtils.decodeUIInput(facesContext, component);
>> +
>> +     }
>> +
>> +}
>>
>> Propchange:
>> myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java
>>
>> ------------------------------------------------------------------------------
>>    svn:eol-style = native
>>
>> Propchange:
>> myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java
>>
>> ------------------------------------------------------------------------------
>>    svn:keywords = Date Author Id Revision HeadURL
>>
>> Modified:
>> myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/component/html/ext/HtmlInputHiddenTest.java
>> URL:
>> http://svn.apache.org/viewvc/myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/component/html/ext/HtmlInputHiddenTest.java?rev=663341&r1=663340&r2=663341&view=diff
>>
>> ==============================================================================
>> ---
>> myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/component/html/ext/HtmlInputHiddenTest.java
>> (original)
>> +++
>> myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/component/html/ext/HtmlInputHiddenTest.java
>> Wed Jun  4 11:53:57 2008
>> @@ -48,7 +48,9 @@
>>     {
>>         // Define the component
>>         UIComponent component = new HtmlInputHidden();
>> -        component.setParent(new HtmlForm());
>> +        component.setId("TestComponent");
>> +        HtmlForm form = new HtmlForm();
>> +        form.getChildren().add(component);
>>           // Render the component
>>         try
>>
>> Modified:
>> myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/test/utils/TestUtils.java
>> URL:
>> http://svn.apache.org/viewvc/myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/test/utils/TestUtils.java?rev=663341&r1=663340&r2=663341&view=diff
>>
>> ==============================================================================
>> ---
>> myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/test/utils/TestUtils.java
>> (original)
>> +++
>> myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/test/utils/TestUtils.java
>> Wed Jun  4 11:53:57 2008
>> @@ -73,6 +73,9 @@
>>         addRenderer(facesContext, "javax.faces.Input",
>> "javax.faces.Hidden",
>>                 "org.apache.myfaces.renderkit.html.HtmlHiddenRenderer");
>>  +        addRenderer(facesContext, "javax.faces.Input",
>> "org.apache.myfaces.Hidden",
>> +
>>  "org.apache.myfaces.renderkit.html.ext.HtmlHiddenRenderer");
>> +                 addRenderer(facesContext, "javax.faces.Graphic",
>> "javax.faces.Image",
>>                 "org.apache.myfaces.renderkit.html.HtmlImageRenderer");
>>
>>
>>
>>
>

HtmlHiddenRenderer.java uses Java 5 annotations. was (Re: svn commit: r663341 -... )

Posted by Paul Spencer <pa...@apache.org>.
Leonardo,

HtmlHiddenRenderer.java uses Java 5 annotations.  Tomahawk must work in 
Java 1.4

Paul Spencer

lu4242@apache.org wrote:
> Author: lu4242
> Date: Wed Jun  4 11:53:57 2008
> New Revision: 663341
> 
> URL: http://svn.apache.org/viewvc?rev=663341&view=rev
> Log:
> TOMAHAWK-1023 HtmlInputHidden fails unit test when using RI
> 
> Added:
>     myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java   (with props)
> Modified:
>     myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/component/html/ext/AbstractHtmlInputHidden.java
>     myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/component/html/ext/HtmlInputHiddenTest.java
>     myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/test/utils/TestUtils.java
> 
> Modified: myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/component/html/ext/AbstractHtmlInputHidden.java
> URL: http://svn.apache.org/viewvc/myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/component/html/ext/AbstractHtmlInputHidden.java?rev=663341&r1=663340&r2=663341&view=diff
> ==============================================================================
> --- myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/component/html/ext/AbstractHtmlInputHidden.java (original)
> +++ myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/component/html/ext/AbstractHtmlInputHidden.java Wed Jun  4 11:53:57 2008
> @@ -42,7 +42,7 @@
>      implements ForceIdAware
>  {
>      public static final String COMPONENT_TYPE = "org.apache.myfaces.HtmlInputHidden";
> -    public static final String DEFAULT_RENDERER_TYPE = "javax.faces.Hidden";
> +    public static final String DEFAULT_RENDERER_TYPE = "org.apache.myfaces.Hidden";
>  
>      public AbstractHtmlInputHidden()
>      {
> 
> Added: myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java
> URL: http://svn.apache.org/viewvc/myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java?rev=663341&view=auto
> ==============================================================================
> --- myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java (added)
> +++ myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java Wed Jun  4 11:53:57 2008
> @@ -0,0 +1,90 @@
> +/*
> + * Licensed to the Apache Software Foundation (ASF) under one
> + * or more contributor license agreements.  See the NOTICE file
> + * distributed with this work for additional information
> + * regarding copyright ownership.  The ASF licenses this file
> + * to you under the Apache License, Version 2.0 (the
> + * "License"); you may not use this file except in compliance
> + * with the License.  You may obtain a copy of the License at
> + *
> + *   http://www.apache.org/licenses/LICENSE-2.0
> + *
> + * Unless required by applicable law or agreed to in writing,
> + * software distributed under the License is distributed on an
> + * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
> + * KIND, either express or implied.  See the License for the
> + * specific language governing permissions and limitations
> + * under the License.
> + */
> +package org.apache.myfaces.renderkit.html.ext;
> +
> +import java.io.IOException;
> +
> +import javax.faces.component.UIComponent;
> +import javax.faces.component.UIInput;
> +import javax.faces.component.UIOutput;
> +import javax.faces.context.FacesContext;
> +import javax.faces.context.ResponseWriter;
> +import javax.faces.convert.ConverterException;
> +
> +import org.apache.myfaces.shared_tomahawk.renderkit.JSFAttr;
> +import org.apache.myfaces.shared_tomahawk.renderkit.RendererUtils;
> +import org.apache.myfaces.shared_tomahawk.renderkit.html.HTML;
> +import org.apache.myfaces.shared_tomahawk.renderkit.html.HtmlRenderer;
> +import org.apache.myfaces.shared_tomahawk.renderkit.html.HtmlRendererUtils;
> +
> +
> +/**
> + * @JSFRenderer
> + *   renderKitId="HTML_BASIC"
> + *   family="javax.faces.Input"
> + *   type="org.apache.myfaces.Hidden"
> + *   
> + * @author Thomas Spiegl (latest modification by $Author$)
> + * @author Anton Koinov
> + * @version $Revision$ $Date$
> + */
> +public class HtmlHiddenRenderer
> +extends HtmlRenderer
> +{
> +    public void encodeEnd(FacesContext facesContext, UIComponent uiComponent)
> +        throws IOException
> +    {
> +        RendererUtils.checkParamValidity(facesContext, uiComponent, UIInput.class);
> +
> +        ResponseWriter writer = facesContext.getResponseWriter();
> +
> +        writer.startElement(HTML.INPUT_ELEM, uiComponent);
> +        writer.writeAttribute(HTML.TYPE_ATTR, HTML.INPUT_TYPE_HIDDEN, null);
> +
> +        String clientId = uiComponent.getClientId(facesContext);
> +        writer.writeAttribute(HTML.ID_ATTR, clientId, null);
> +        writer.writeAttribute(HTML.NAME_ATTR, clientId, null);
> +
> +        String value = RendererUtils.getStringValue(facesContext, uiComponent);
> +        if (value != null)
> +        {
> +            writer.writeAttribute(HTML.VALUE_ATTR, value, JSFAttr.VALUE_ATTR);
> +        }
> +
> +        writer.endElement(HTML.INPUT_ELEM);
> +    }
> +
> +    public Object getConvertedValue(FacesContext facesContext, UIComponent uiComponent, Object submittedValue) throws ConverterException
> +    {
> +        RendererUtils.checkParamValidity(facesContext, uiComponent, UIOutput.class);
> +        return RendererUtils.getConvertedUIOutputValue(facesContext,
> +                                                       (UIOutput)uiComponent,
> +                                                       submittedValue);
> +    }
> +
> +
> +    public void decode(FacesContext facesContext, UIComponent component)
> +     {
> +         RendererUtils.checkParamValidity(facesContext,component,UIInput.class);
> +
> +         HtmlRendererUtils.decodeUIInput(facesContext, component);
> +
> +     }
> +
> +}
> 
> Propchange: myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java
> ------------------------------------------------------------------------------
>     svn:eol-style = native
> 
> Propchange: myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java
> ------------------------------------------------------------------------------
>     svn:keywords = Date Author Id Revision HeadURL
> 
> Modified: myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/component/html/ext/HtmlInputHiddenTest.java
> URL: http://svn.apache.org/viewvc/myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/component/html/ext/HtmlInputHiddenTest.java?rev=663341&r1=663340&r2=663341&view=diff
> ==============================================================================
> --- myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/component/html/ext/HtmlInputHiddenTest.java (original)
> +++ myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/component/html/ext/HtmlInputHiddenTest.java Wed Jun  4 11:53:57 2008
> @@ -48,7 +48,9 @@
>      {
>          // Define the component
>          UIComponent component = new HtmlInputHidden();
> -        component.setParent(new HtmlForm());
> +        component.setId("TestComponent");
> +        HtmlForm form = new HtmlForm();
> +        form.getChildren().add(component);
>  
>          // Render the component
>          try
> 
> Modified: myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/test/utils/TestUtils.java
> URL: http://svn.apache.org/viewvc/myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/test/utils/TestUtils.java?rev=663341&r1=663340&r2=663341&view=diff
> ==============================================================================
> --- myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/test/utils/TestUtils.java (original)
> +++ myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/test/utils/TestUtils.java Wed Jun  4 11:53:57 2008
> @@ -73,6 +73,9 @@
>          addRenderer(facesContext, "javax.faces.Input", "javax.faces.Hidden",
>                  "org.apache.myfaces.renderkit.html.HtmlHiddenRenderer");
>  
> +        addRenderer(facesContext, "javax.faces.Input", "org.apache.myfaces.Hidden",
> +                "org.apache.myfaces.renderkit.html.ext.HtmlHiddenRenderer");
> +        
>          addRenderer(facesContext, "javax.faces.Graphic", "javax.faces.Image",
>                  "org.apache.myfaces.renderkit.html.HtmlImageRenderer");
>  
> 
> 
> 


Re: svn commit: r663341 - in /myfaces/tomahawk/trunk/core/src: main/java/org/apache/myfaces/component/html/ext/ main/java/org/apache/myfaces/renderkit/html/ext/ test/java/org/apache/myfaces/component/html/ext/ test/java/org/apache/myfaces/test/utils/

Posted by Leonardo Uribe <lu...@gmail.com>.
Yes, but let it as is do not cause any problem.

On Wed, Jun 4, 2008 at 3:01 PM, Paul Spencer <pa...@mindspring.com>
wrote:

> Leonardo,
> Should HtmlHiddenRenderer.java be removed when we update to Shale 1.1?
>
> Paul Spencer
>
>
> Leonardo Uribe wrote:
>
>> On Wed, Jun 4, 2008 at 2:28 PM, Paul Spencer <pa...@apache.org> wrote:
>>
>>  Leonardo,
>>> Was HtmlHiddenRenderer.java add just to make the test pass, or is it
>>> need to when Tomahawk is run with the RI?
>>>
>>>
>> Really it is not necessary to tomahawk runs with the RI, but it is
>> necessary
>> to make the test pass, because TestUtils.addDefaultRenderers()  add this
>> as
>> javax.faces.Hidden renderer:
>>
>>        addRenderer(facesContext, "javax.faces.Input",
>> "javax.faces.Hidden",
>>
>>                "org.apache.myfaces.renderkit.html.HtmlHiddenRenderer");
>>
>> If the test can read the faces-config.xml file there is no problem, but in
>> that case the class referred on jsf ri does not exists.
>>
>>
>>  Paul Spencer
>>>
>>>
>>>
>>> lu4242@apache.org wrote:
>>>
>>>  Author: lu4242
>>>> Date: Wed Jun  4 11:53:57 2008
>>>> New Revision: 663341
>>>>
>>>> URL: http://svn.apache.org/viewvc?rev=663341&view=rev
>>>> Log:
>>>> TOMAHAWK-1023 HtmlInputHidden fails unit test when using RI
>>>>
>>>> Added:
>>>>
>>>>
>>>>  myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java
>>>>  (with props)
>>>> Modified:
>>>>
>>>>
>>>>  myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/component/html/ext/AbstractHtmlInputHidden.java
>>>>
>>>>
>>>>  myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/component/html/ext/HtmlInputHiddenTest.java
>>>>
>>>>
>>>>  myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/test/utils/TestUtils.java
>>>>
>>>> Modified:
>>>>
>>>> myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/component/html/ext/AbstractHtmlInputHidden.java
>>>> URL:
>>>>
>>>> http://svn.apache.org/viewvc/myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/component/html/ext/AbstractHtmlInputHidden.java?rev=663341&r1=663340&r2=663341&view=diff
>>>>
>>>>
>>>> ==============================================================================
>>>> ---
>>>>
>>>> myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/component/html/ext/AbstractHtmlInputHidden.java
>>>> (original)
>>>> +++
>>>>
>>>> myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/component/html/ext/AbstractHtmlInputHidden.java
>>>> Wed Jun  4 11:53:57 2008
>>>> @@ -42,7 +42,7 @@
>>>>    implements ForceIdAware
>>>>  {
>>>>    public static final String COMPONENT_TYPE =
>>>> "org.apache.myfaces.HtmlInputHidden";
>>>> -    public static final String DEFAULT_RENDERER_TYPE =
>>>> "javax.faces.Hidden";
>>>> +    public static final String DEFAULT_RENDERER_TYPE =
>>>> "org.apache.myfaces.Hidden";
>>>>      public AbstractHtmlInputHidden()
>>>>    {
>>>>
>>>> Added:
>>>>
>>>> myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java
>>>> URL:
>>>>
>>>> http://svn.apache.org/viewvc/myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java?rev=663341&view=auto
>>>>
>>>>
>>>> ==============================================================================
>>>> ---
>>>>
>>>> myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java
>>>> (added)
>>>> +++
>>>>
>>>> myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java
>>>> Wed Jun  4 11:53:57 2008
>>>> @@ -0,0 +1,90 @@
>>>> +/*
>>>> + * Licensed to the Apache Software Foundation (ASF) under one
>>>> + * or more contributor license agreements.  See the NOTICE file
>>>> + * distributed with this work for additional information
>>>> + * regarding copyright ownership.  The ASF licenses this file
>>>> + * to you under the Apache License, Version 2.0 (the
>>>> + * "License"); you may not use this file except in compliance
>>>> + * with the License.  You may obtain a copy of the License at
>>>> + *
>>>> + *   http://www.apache.org/licenses/LICENSE-2.0
>>>> + *
>>>> + * Unless required by applicable law or agreed to in writing,
>>>> + * software distributed under the License is distributed on an
>>>> + * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
>>>> + * KIND, either express or implied.  See the License for the
>>>> + * specific language governing permissions and limitations
>>>> + * under the License.
>>>> + */
>>>> +package org.apache.myfaces.renderkit.html.ext;
>>>> +
>>>> +import java.io.IOException;
>>>> +
>>>> +import javax.faces.component.UIComponent;
>>>> +import javax.faces.component.UIInput;
>>>> +import javax.faces.component.UIOutput;
>>>> +import javax.faces.context.FacesContext;
>>>> +import javax.faces.context.ResponseWriter;
>>>> +import javax.faces.convert.ConverterException;
>>>> +
>>>> +import org.apache.myfaces.shared_tomahawk.renderkit.JSFAttr;
>>>> +import org.apache.myfaces.shared_tomahawk.renderkit.RendererUtils;
>>>> +import org.apache.myfaces.shared_tomahawk.renderkit.html.HTML;
>>>> +import org.apache.myfaces.shared_tomahawk.renderkit.html.HtmlRenderer;
>>>> +import
>>>> org.apache.myfaces.shared_tomahawk.renderkit.html.HtmlRendererUtils;
>>>> +
>>>> +
>>>> +/**
>>>> + * @JSFRenderer
>>>> + *   renderKitId="HTML_BASIC"
>>>> + *   family="javax.faces.Input"
>>>> + *   type="org.apache.myfaces.Hidden"
>>>> + *   + * @author Thomas Spiegl (latest modification by $Author$)
>>>> + * @author Anton Koinov
>>>> + * @version $Revision$ $Date$
>>>> + */
>>>> +public class HtmlHiddenRenderer
>>>> +extends HtmlRenderer
>>>> +{
>>>> +    public void encodeEnd(FacesContext facesContext, UIComponent
>>>> uiComponent)
>>>> +        throws IOException
>>>> +    {
>>>> +        RendererUtils.checkParamValidity(facesContext, uiComponent,
>>>> UIInput.class);
>>>> +
>>>> +        ResponseWriter writer = facesContext.getResponseWriter();
>>>> +
>>>> +        writer.startElement(HTML.INPUT_ELEM, uiComponent);
>>>> +        writer.writeAttribute(HTML.TYPE_ATTR, HTML.INPUT_TYPE_HIDDEN,
>>>> null);
>>>> +
>>>> +        String clientId = uiComponent.getClientId(facesContext);
>>>> +        writer.writeAttribute(HTML.ID_ATTR, clientId, null);
>>>> +        writer.writeAttribute(HTML.NAME_ATTR, clientId, null);
>>>> +
>>>> +        String value = RendererUtils.getStringValue(facesContext,
>>>> uiComponent);
>>>> +        if (value != null)
>>>> +        {
>>>> +            writer.writeAttribute(HTML.VALUE_ATTR, value,
>>>> JSFAttr.VALUE_ATTR);
>>>> +        }
>>>> +
>>>> +        writer.endElement(HTML.INPUT_ELEM);
>>>> +    }
>>>> +
>>>> +    public Object getConvertedValue(FacesContext facesContext,
>>>> UIComponent uiComponent, Object submittedValue) throws
>>>> ConverterException
>>>> +    {
>>>> +        RendererUtils.checkParamValidity(facesContext, uiComponent,
>>>> UIOutput.class);
>>>> +        return RendererUtils.getConvertedUIOutputValue(facesContext,
>>>> +
>>>> (UIOutput)uiComponent,
>>>> +                                                       submittedValue);
>>>> +    }
>>>> +
>>>> +
>>>> +    public void decode(FacesContext facesContext, UIComponent
>>>> component)
>>>> +     {
>>>> +
>>>> RendererUtils.checkParamValidity(facesContext,component,UIInput.class);
>>>> +
>>>> +         HtmlRendererUtils.decodeUIInput(facesContext, component);
>>>> +
>>>> +     }
>>>> +
>>>> +}
>>>>
>>>> Propchange:
>>>>
>>>> myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java
>>>>
>>>>
>>>> ------------------------------------------------------------------------------
>>>>   svn:eol-style = native
>>>>
>>>> Propchange:
>>>>
>>>> myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java
>>>>
>>>>
>>>> ------------------------------------------------------------------------------
>>>>   svn:keywords = Date Author Id Revision HeadURL
>>>>
>>>> Modified:
>>>>
>>>> myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/component/html/ext/HtmlInputHiddenTest.java
>>>> URL:
>>>>
>>>> http://svn.apache.org/viewvc/myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/component/html/ext/HtmlInputHiddenTest.java?rev=663341&r1=663340&r2=663341&view=diff
>>>>
>>>>
>>>> ==============================================================================
>>>> ---
>>>>
>>>> myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/component/html/ext/HtmlInputHiddenTest.java
>>>> (original)
>>>> +++
>>>>
>>>> myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/component/html/ext/HtmlInputHiddenTest.java
>>>> Wed Jun  4 11:53:57 2008
>>>> @@ -48,7 +48,9 @@
>>>>    {
>>>>        // Define the component
>>>>        UIComponent component = new HtmlInputHidden();
>>>> -        component.setParent(new HtmlForm());
>>>> +        component.setId("TestComponent");
>>>> +        HtmlForm form = new HtmlForm();
>>>> +        form.getChildren().add(component);
>>>>          // Render the component
>>>>        try
>>>>
>>>> Modified:
>>>>
>>>> myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/test/utils/TestUtils.java
>>>> URL:
>>>>
>>>> http://svn.apache.org/viewvc/myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/test/utils/TestUtils.java?rev=663341&r1=663340&r2=663341&view=diff
>>>>
>>>>
>>>> ==============================================================================
>>>> ---
>>>>
>>>> myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/test/utils/TestUtils.java
>>>> (original)
>>>> +++
>>>>
>>>> myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/test/utils/TestUtils.java
>>>> Wed Jun  4 11:53:57 2008
>>>> @@ -73,6 +73,9 @@
>>>>        addRenderer(facesContext, "javax.faces.Input",
>>>> "javax.faces.Hidden",
>>>>                "org.apache.myfaces.renderkit.html.HtmlHiddenRenderer");
>>>>  +        addRenderer(facesContext, "javax.faces.Input",
>>>> "org.apache.myfaces.Hidden",
>>>> +
>>>>  "org.apache.myfaces.renderkit.html.ext.HtmlHiddenRenderer");
>>>> +                 addRenderer(facesContext, "javax.faces.Graphic",
>>>> "javax.faces.Image",
>>>>                "org.apache.myfaces.renderkit.html.HtmlImageRenderer");
>>>>
>>>>
>>>>
>>>>
>>>>
>>>
>>
>

Re: svn commit: r663341 - in /myfaces/tomahawk/trunk/core/src: main/java/org/apache/myfaces/component/html/ext/ main/java/org/apache/myfaces/renderkit/html/ext/ test/java/org/apache/myfaces/component/html/ext/ test/java/org/apache/myfaces/test/utils/

Posted by Paul Spencer <pa...@mindspring.com>.
Leonardo,
Should HtmlHiddenRenderer.java be removed when we update to Shale 1.1?

Paul Spencer

Leonardo Uribe wrote:
> On Wed, Jun 4, 2008 at 2:28 PM, Paul Spencer <pa...@apache.org> wrote:
> 
>> Leonardo,
>> Was HtmlHiddenRenderer.java add just to make the test pass, or is it
>> need to when Tomahawk is run with the RI?
>>
> 
> Really it is not necessary to tomahawk runs with the RI, but it is necessary
> to make the test pass, because TestUtils.addDefaultRenderers()  add this as
> javax.faces.Hidden renderer:
> 
>         addRenderer(facesContext, "javax.faces.Input", "javax.faces.Hidden",
> 
>                 "org.apache.myfaces.renderkit.html.HtmlHiddenRenderer");
> 
> If the test can read the faces-config.xml file there is no problem, but in
> that case the class referred on jsf ri does not exists.
> 
> 
>> Paul Spencer
>>
>>
>>
>> lu4242@apache.org wrote:
>>
>>> Author: lu4242
>>> Date: Wed Jun  4 11:53:57 2008
>>> New Revision: 663341
>>>
>>> URL: http://svn.apache.org/viewvc?rev=663341&view=rev
>>> Log:
>>> TOMAHAWK-1023 HtmlInputHidden fails unit test when using RI
>>>
>>> Added:
>>>
>>>  myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java
>>>   (with props)
>>> Modified:
>>>
>>>  myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/component/html/ext/AbstractHtmlInputHidden.java
>>>
>>>  myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/component/html/ext/HtmlInputHiddenTest.java
>>>
>>>  myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/test/utils/TestUtils.java
>>>
>>> Modified:
>>> myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/component/html/ext/AbstractHtmlInputHidden.java
>>> URL:
>>> http://svn.apache.org/viewvc/myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/component/html/ext/AbstractHtmlInputHidden.java?rev=663341&r1=663340&r2=663341&view=diff
>>>
>>> ==============================================================================
>>> ---
>>> myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/component/html/ext/AbstractHtmlInputHidden.java
>>> (original)
>>> +++
>>> myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/component/html/ext/AbstractHtmlInputHidden.java
>>> Wed Jun  4 11:53:57 2008
>>> @@ -42,7 +42,7 @@
>>>     implements ForceIdAware
>>>  {
>>>     public static final String COMPONENT_TYPE =
>>> "org.apache.myfaces.HtmlInputHidden";
>>> -    public static final String DEFAULT_RENDERER_TYPE =
>>> "javax.faces.Hidden";
>>> +    public static final String DEFAULT_RENDERER_TYPE =
>>> "org.apache.myfaces.Hidden";
>>>       public AbstractHtmlInputHidden()
>>>     {
>>>
>>> Added:
>>> myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java
>>> URL:
>>> http://svn.apache.org/viewvc/myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java?rev=663341&view=auto
>>>
>>> ==============================================================================
>>> ---
>>> myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java
>>> (added)
>>> +++
>>> myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java
>>> Wed Jun  4 11:53:57 2008
>>> @@ -0,0 +1,90 @@
>>> +/*
>>> + * Licensed to the Apache Software Foundation (ASF) under one
>>> + * or more contributor license agreements.  See the NOTICE file
>>> + * distributed with this work for additional information
>>> + * regarding copyright ownership.  The ASF licenses this file
>>> + * to you under the Apache License, Version 2.0 (the
>>> + * "License"); you may not use this file except in compliance
>>> + * with the License.  You may obtain a copy of the License at
>>> + *
>>> + *   http://www.apache.org/licenses/LICENSE-2.0
>>> + *
>>> + * Unless required by applicable law or agreed to in writing,
>>> + * software distributed under the License is distributed on an
>>> + * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
>>> + * KIND, either express or implied.  See the License for the
>>> + * specific language governing permissions and limitations
>>> + * under the License.
>>> + */
>>> +package org.apache.myfaces.renderkit.html.ext;
>>> +
>>> +import java.io.IOException;
>>> +
>>> +import javax.faces.component.UIComponent;
>>> +import javax.faces.component.UIInput;
>>> +import javax.faces.component.UIOutput;
>>> +import javax.faces.context.FacesContext;
>>> +import javax.faces.context.ResponseWriter;
>>> +import javax.faces.convert.ConverterException;
>>> +
>>> +import org.apache.myfaces.shared_tomahawk.renderkit.JSFAttr;
>>> +import org.apache.myfaces.shared_tomahawk.renderkit.RendererUtils;
>>> +import org.apache.myfaces.shared_tomahawk.renderkit.html.HTML;
>>> +import org.apache.myfaces.shared_tomahawk.renderkit.html.HtmlRenderer;
>>> +import
>>> org.apache.myfaces.shared_tomahawk.renderkit.html.HtmlRendererUtils;
>>> +
>>> +
>>> +/**
>>> + * @JSFRenderer
>>> + *   renderKitId="HTML_BASIC"
>>> + *   family="javax.faces.Input"
>>> + *   type="org.apache.myfaces.Hidden"
>>> + *   + * @author Thomas Spiegl (latest modification by $Author$)
>>> + * @author Anton Koinov
>>> + * @version $Revision$ $Date$
>>> + */
>>> +public class HtmlHiddenRenderer
>>> +extends HtmlRenderer
>>> +{
>>> +    public void encodeEnd(FacesContext facesContext, UIComponent
>>> uiComponent)
>>> +        throws IOException
>>> +    {
>>> +        RendererUtils.checkParamValidity(facesContext, uiComponent,
>>> UIInput.class);
>>> +
>>> +        ResponseWriter writer = facesContext.getResponseWriter();
>>> +
>>> +        writer.startElement(HTML.INPUT_ELEM, uiComponent);
>>> +        writer.writeAttribute(HTML.TYPE_ATTR, HTML.INPUT_TYPE_HIDDEN,
>>> null);
>>> +
>>> +        String clientId = uiComponent.getClientId(facesContext);
>>> +        writer.writeAttribute(HTML.ID_ATTR, clientId, null);
>>> +        writer.writeAttribute(HTML.NAME_ATTR, clientId, null);
>>> +
>>> +        String value = RendererUtils.getStringValue(facesContext,
>>> uiComponent);
>>> +        if (value != null)
>>> +        {
>>> +            writer.writeAttribute(HTML.VALUE_ATTR, value,
>>> JSFAttr.VALUE_ATTR);
>>> +        }
>>> +
>>> +        writer.endElement(HTML.INPUT_ELEM);
>>> +    }
>>> +
>>> +    public Object getConvertedValue(FacesContext facesContext,
>>> UIComponent uiComponent, Object submittedValue) throws ConverterException
>>> +    {
>>> +        RendererUtils.checkParamValidity(facesContext, uiComponent,
>>> UIOutput.class);
>>> +        return RendererUtils.getConvertedUIOutputValue(facesContext,
>>> +
>>> (UIOutput)uiComponent,
>>> +                                                       submittedValue);
>>> +    }
>>> +
>>> +
>>> +    public void decode(FacesContext facesContext, UIComponent component)
>>> +     {
>>> +
>>> RendererUtils.checkParamValidity(facesContext,component,UIInput.class);
>>> +
>>> +         HtmlRendererUtils.decodeUIInput(facesContext, component);
>>> +
>>> +     }
>>> +
>>> +}
>>>
>>> Propchange:
>>> myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java
>>>
>>> ------------------------------------------------------------------------------
>>>    svn:eol-style = native
>>>
>>> Propchange:
>>> myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java
>>>
>>> ------------------------------------------------------------------------------
>>>    svn:keywords = Date Author Id Revision HeadURL
>>>
>>> Modified:
>>> myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/component/html/ext/HtmlInputHiddenTest.java
>>> URL:
>>> http://svn.apache.org/viewvc/myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/component/html/ext/HtmlInputHiddenTest.java?rev=663341&r1=663340&r2=663341&view=diff
>>>
>>> ==============================================================================
>>> ---
>>> myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/component/html/ext/HtmlInputHiddenTest.java
>>> (original)
>>> +++
>>> myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/component/html/ext/HtmlInputHiddenTest.java
>>> Wed Jun  4 11:53:57 2008
>>> @@ -48,7 +48,9 @@
>>>     {
>>>         // Define the component
>>>         UIComponent component = new HtmlInputHidden();
>>> -        component.setParent(new HtmlForm());
>>> +        component.setId("TestComponent");
>>> +        HtmlForm form = new HtmlForm();
>>> +        form.getChildren().add(component);
>>>           // Render the component
>>>         try
>>>
>>> Modified:
>>> myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/test/utils/TestUtils.java
>>> URL:
>>> http://svn.apache.org/viewvc/myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/test/utils/TestUtils.java?rev=663341&r1=663340&r2=663341&view=diff
>>>
>>> ==============================================================================
>>> ---
>>> myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/test/utils/TestUtils.java
>>> (original)
>>> +++
>>> myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/test/utils/TestUtils.java
>>> Wed Jun  4 11:53:57 2008
>>> @@ -73,6 +73,9 @@
>>>         addRenderer(facesContext, "javax.faces.Input",
>>> "javax.faces.Hidden",
>>>                 "org.apache.myfaces.renderkit.html.HtmlHiddenRenderer");
>>>  +        addRenderer(facesContext, "javax.faces.Input",
>>> "org.apache.myfaces.Hidden",
>>> +
>>>  "org.apache.myfaces.renderkit.html.ext.HtmlHiddenRenderer");
>>> +                 addRenderer(facesContext, "javax.faces.Graphic",
>>> "javax.faces.Image",
>>>                 "org.apache.myfaces.renderkit.html.HtmlImageRenderer");
>>>
>>>
>>>
>>>
>>
> 


Re: svn commit: r663341 - in /myfaces/tomahawk/trunk/core/src: main/java/org/apache/myfaces/component/html/ext/ main/java/org/apache/myfaces/renderkit/html/ext/ test/java/org/apache/myfaces/component/html/ext/ test/java/org/apache/myfaces/test/utils/

Posted by Leonardo Uribe <lu...@gmail.com>.
On Wed, Jun 4, 2008 at 2:28 PM, Paul Spencer <pa...@apache.org> wrote:

> Leonardo,
> Was HtmlHiddenRenderer.java add just to make the test pass, or is it
> need to when Tomahawk is run with the RI?
>

Really it is not necessary to tomahawk runs with the RI, but it is necessary
to make the test pass, because TestUtils.addDefaultRenderers()  add this as
javax.faces.Hidden renderer:

        addRenderer(facesContext, "javax.faces.Input", "javax.faces.Hidden",

                "org.apache.myfaces.renderkit.html.HtmlHiddenRenderer");

If the test can read the faces-config.xml file there is no problem, but in
that case the class referred on jsf ri does not exists.


>
> Paul Spencer
>
>
>
> lu4242@apache.org wrote:
>
>> Author: lu4242
>> Date: Wed Jun  4 11:53:57 2008
>> New Revision: 663341
>>
>> URL: http://svn.apache.org/viewvc?rev=663341&view=rev
>> Log:
>> TOMAHAWK-1023 HtmlInputHidden fails unit test when using RI
>>
>> Added:
>>
>>  myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java
>>   (with props)
>> Modified:
>>
>>  myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/component/html/ext/AbstractHtmlInputHidden.java
>>
>>  myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/component/html/ext/HtmlInputHiddenTest.java
>>
>>  myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/test/utils/TestUtils.java
>>
>> Modified:
>> myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/component/html/ext/AbstractHtmlInputHidden.java
>> URL:
>> http://svn.apache.org/viewvc/myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/component/html/ext/AbstractHtmlInputHidden.java?rev=663341&r1=663340&r2=663341&view=diff
>>
>> ==============================================================================
>> ---
>> myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/component/html/ext/AbstractHtmlInputHidden.java
>> (original)
>> +++
>> myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/component/html/ext/AbstractHtmlInputHidden.java
>> Wed Jun  4 11:53:57 2008
>> @@ -42,7 +42,7 @@
>>     implements ForceIdAware
>>  {
>>     public static final String COMPONENT_TYPE =
>> "org.apache.myfaces.HtmlInputHidden";
>> -    public static final String DEFAULT_RENDERER_TYPE =
>> "javax.faces.Hidden";
>> +    public static final String DEFAULT_RENDERER_TYPE =
>> "org.apache.myfaces.Hidden";
>>       public AbstractHtmlInputHidden()
>>     {
>>
>> Added:
>> myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java
>> URL:
>> http://svn.apache.org/viewvc/myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java?rev=663341&view=auto
>>
>> ==============================================================================
>> ---
>> myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java
>> (added)
>> +++
>> myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java
>> Wed Jun  4 11:53:57 2008
>> @@ -0,0 +1,90 @@
>> +/*
>> + * Licensed to the Apache Software Foundation (ASF) under one
>> + * or more contributor license agreements.  See the NOTICE file
>> + * distributed with this work for additional information
>> + * regarding copyright ownership.  The ASF licenses this file
>> + * to you under the Apache License, Version 2.0 (the
>> + * "License"); you may not use this file except in compliance
>> + * with the License.  You may obtain a copy of the License at
>> + *
>> + *   http://www.apache.org/licenses/LICENSE-2.0
>> + *
>> + * Unless required by applicable law or agreed to in writing,
>> + * software distributed under the License is distributed on an
>> + * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
>> + * KIND, either express or implied.  See the License for the
>> + * specific language governing permissions and limitations
>> + * under the License.
>> + */
>> +package org.apache.myfaces.renderkit.html.ext;
>> +
>> +import java.io.IOException;
>> +
>> +import javax.faces.component.UIComponent;
>> +import javax.faces.component.UIInput;
>> +import javax.faces.component.UIOutput;
>> +import javax.faces.context.FacesContext;
>> +import javax.faces.context.ResponseWriter;
>> +import javax.faces.convert.ConverterException;
>> +
>> +import org.apache.myfaces.shared_tomahawk.renderkit.JSFAttr;
>> +import org.apache.myfaces.shared_tomahawk.renderkit.RendererUtils;
>> +import org.apache.myfaces.shared_tomahawk.renderkit.html.HTML;
>> +import org.apache.myfaces.shared_tomahawk.renderkit.html.HtmlRenderer;
>> +import
>> org.apache.myfaces.shared_tomahawk.renderkit.html.HtmlRendererUtils;
>> +
>> +
>> +/**
>> + * @JSFRenderer
>> + *   renderKitId="HTML_BASIC"
>> + *   family="javax.faces.Input"
>> + *   type="org.apache.myfaces.Hidden"
>> + *   + * @author Thomas Spiegl (latest modification by $Author$)
>> + * @author Anton Koinov
>> + * @version $Revision$ $Date$
>> + */
>> +public class HtmlHiddenRenderer
>> +extends HtmlRenderer
>> +{
>> +    public void encodeEnd(FacesContext facesContext, UIComponent
>> uiComponent)
>> +        throws IOException
>> +    {
>> +        RendererUtils.checkParamValidity(facesContext, uiComponent,
>> UIInput.class);
>> +
>> +        ResponseWriter writer = facesContext.getResponseWriter();
>> +
>> +        writer.startElement(HTML.INPUT_ELEM, uiComponent);
>> +        writer.writeAttribute(HTML.TYPE_ATTR, HTML.INPUT_TYPE_HIDDEN,
>> null);
>> +
>> +        String clientId = uiComponent.getClientId(facesContext);
>> +        writer.writeAttribute(HTML.ID_ATTR, clientId, null);
>> +        writer.writeAttribute(HTML.NAME_ATTR, clientId, null);
>> +
>> +        String value = RendererUtils.getStringValue(facesContext,
>> uiComponent);
>> +        if (value != null)
>> +        {
>> +            writer.writeAttribute(HTML.VALUE_ATTR, value,
>> JSFAttr.VALUE_ATTR);
>> +        }
>> +
>> +        writer.endElement(HTML.INPUT_ELEM);
>> +    }
>> +
>> +    public Object getConvertedValue(FacesContext facesContext,
>> UIComponent uiComponent, Object submittedValue) throws ConverterException
>> +    {
>> +        RendererUtils.checkParamValidity(facesContext, uiComponent,
>> UIOutput.class);
>> +        return RendererUtils.getConvertedUIOutputValue(facesContext,
>> +
>> (UIOutput)uiComponent,
>> +                                                       submittedValue);
>> +    }
>> +
>> +
>> +    public void decode(FacesContext facesContext, UIComponent component)
>> +     {
>> +
>> RendererUtils.checkParamValidity(facesContext,component,UIInput.class);
>> +
>> +         HtmlRendererUtils.decodeUIInput(facesContext, component);
>> +
>> +     }
>> +
>> +}
>>
>> Propchange:
>> myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java
>>
>> ------------------------------------------------------------------------------
>>    svn:eol-style = native
>>
>> Propchange:
>> myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java
>>
>> ------------------------------------------------------------------------------
>>    svn:keywords = Date Author Id Revision HeadURL
>>
>> Modified:
>> myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/component/html/ext/HtmlInputHiddenTest.java
>> URL:
>> http://svn.apache.org/viewvc/myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/component/html/ext/HtmlInputHiddenTest.java?rev=663341&r1=663340&r2=663341&view=diff
>>
>> ==============================================================================
>> ---
>> myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/component/html/ext/HtmlInputHiddenTest.java
>> (original)
>> +++
>> myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/component/html/ext/HtmlInputHiddenTest.java
>> Wed Jun  4 11:53:57 2008
>> @@ -48,7 +48,9 @@
>>     {
>>         // Define the component
>>         UIComponent component = new HtmlInputHidden();
>> -        component.setParent(new HtmlForm());
>> +        component.setId("TestComponent");
>> +        HtmlForm form = new HtmlForm();
>> +        form.getChildren().add(component);
>>           // Render the component
>>         try
>>
>> Modified:
>> myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/test/utils/TestUtils.java
>> URL:
>> http://svn.apache.org/viewvc/myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/test/utils/TestUtils.java?rev=663341&r1=663340&r2=663341&view=diff
>>
>> ==============================================================================
>> ---
>> myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/test/utils/TestUtils.java
>> (original)
>> +++
>> myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/test/utils/TestUtils.java
>> Wed Jun  4 11:53:57 2008
>> @@ -73,6 +73,9 @@
>>         addRenderer(facesContext, "javax.faces.Input",
>> "javax.faces.Hidden",
>>                 "org.apache.myfaces.renderkit.html.HtmlHiddenRenderer");
>>  +        addRenderer(facesContext, "javax.faces.Input",
>> "org.apache.myfaces.Hidden",
>> +
>>  "org.apache.myfaces.renderkit.html.ext.HtmlHiddenRenderer");
>> +                 addRenderer(facesContext, "javax.faces.Graphic",
>> "javax.faces.Image",
>>                 "org.apache.myfaces.renderkit.html.HtmlImageRenderer");
>>
>>
>>
>>
>
>

Re: svn commit: r663341 - in /myfaces/tomahawk/trunk/core/src: main/java/org/apache/myfaces/component/html/ext/ main/java/org/apache/myfaces/renderkit/html/ext/ test/java/org/apache/myfaces/component/html/ext/ test/java/org/apache/myfaces/test/utils/

Posted by Paul Spencer <pa...@apache.org>.
Leonardo,
Was HtmlHiddenRenderer.java add just to make the test pass, or is it
need to when Tomahawk is run with the RI?

Paul Spencer



lu4242@apache.org wrote:
> Author: lu4242
> Date: Wed Jun  4 11:53:57 2008
> New Revision: 663341
> 
> URL: http://svn.apache.org/viewvc?rev=663341&view=rev
> Log:
> TOMAHAWK-1023 HtmlInputHidden fails unit test when using RI
> 
> Added:
>     myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java   (with props)
> Modified:
>     myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/component/html/ext/AbstractHtmlInputHidden.java
>     myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/component/html/ext/HtmlInputHiddenTest.java
>     myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/test/utils/TestUtils.java
> 
> Modified: myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/component/html/ext/AbstractHtmlInputHidden.java
> URL: http://svn.apache.org/viewvc/myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/component/html/ext/AbstractHtmlInputHidden.java?rev=663341&r1=663340&r2=663341&view=diff
> ==============================================================================
> --- myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/component/html/ext/AbstractHtmlInputHidden.java (original)
> +++ myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/component/html/ext/AbstractHtmlInputHidden.java Wed Jun  4 11:53:57 2008
> @@ -42,7 +42,7 @@
>      implements ForceIdAware
>  {
>      public static final String COMPONENT_TYPE = "org.apache.myfaces.HtmlInputHidden";
> -    public static final String DEFAULT_RENDERER_TYPE = "javax.faces.Hidden";
> +    public static final String DEFAULT_RENDERER_TYPE = "org.apache.myfaces.Hidden";
>  
>      public AbstractHtmlInputHidden()
>      {
> 
> Added: myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java
> URL: http://svn.apache.org/viewvc/myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java?rev=663341&view=auto
> ==============================================================================
> --- myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java (added)
> +++ myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java Wed Jun  4 11:53:57 2008
> @@ -0,0 +1,90 @@
> +/*
> + * Licensed to the Apache Software Foundation (ASF) under one
> + * or more contributor license agreements.  See the NOTICE file
> + * distributed with this work for additional information
> + * regarding copyright ownership.  The ASF licenses this file
> + * to you under the Apache License, Version 2.0 (the
> + * "License"); you may not use this file except in compliance
> + * with the License.  You may obtain a copy of the License at
> + *
> + *   http://www.apache.org/licenses/LICENSE-2.0
> + *
> + * Unless required by applicable law or agreed to in writing,
> + * software distributed under the License is distributed on an
> + * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
> + * KIND, either express or implied.  See the License for the
> + * specific language governing permissions and limitations
> + * under the License.
> + */
> +package org.apache.myfaces.renderkit.html.ext;
> +
> +import java.io.IOException;
> +
> +import javax.faces.component.UIComponent;
> +import javax.faces.component.UIInput;
> +import javax.faces.component.UIOutput;
> +import javax.faces.context.FacesContext;
> +import javax.faces.context.ResponseWriter;
> +import javax.faces.convert.ConverterException;
> +
> +import org.apache.myfaces.shared_tomahawk.renderkit.JSFAttr;
> +import org.apache.myfaces.shared_tomahawk.renderkit.RendererUtils;
> +import org.apache.myfaces.shared_tomahawk.renderkit.html.HTML;
> +import org.apache.myfaces.shared_tomahawk.renderkit.html.HtmlRenderer;
> +import org.apache.myfaces.shared_tomahawk.renderkit.html.HtmlRendererUtils;
> +
> +
> +/**
> + * @JSFRenderer
> + *   renderKitId="HTML_BASIC"
> + *   family="javax.faces.Input"
> + *   type="org.apache.myfaces.Hidden"
> + *   
> + * @author Thomas Spiegl (latest modification by $Author$)
> + * @author Anton Koinov
> + * @version $Revision$ $Date$
> + */
> +public class HtmlHiddenRenderer
> +extends HtmlRenderer
> +{
> +    public void encodeEnd(FacesContext facesContext, UIComponent uiComponent)
> +        throws IOException
> +    {
> +        RendererUtils.checkParamValidity(facesContext, uiComponent, UIInput.class);
> +
> +        ResponseWriter writer = facesContext.getResponseWriter();
> +
> +        writer.startElement(HTML.INPUT_ELEM, uiComponent);
> +        writer.writeAttribute(HTML.TYPE_ATTR, HTML.INPUT_TYPE_HIDDEN, null);
> +
> +        String clientId = uiComponent.getClientId(facesContext);
> +        writer.writeAttribute(HTML.ID_ATTR, clientId, null);
> +        writer.writeAttribute(HTML.NAME_ATTR, clientId, null);
> +
> +        String value = RendererUtils.getStringValue(facesContext, uiComponent);
> +        if (value != null)
> +        {
> +            writer.writeAttribute(HTML.VALUE_ATTR, value, JSFAttr.VALUE_ATTR);
> +        }
> +
> +        writer.endElement(HTML.INPUT_ELEM);
> +    }
> +
> +    public Object getConvertedValue(FacesContext facesContext, UIComponent uiComponent, Object submittedValue) throws ConverterException
> +    {
> +        RendererUtils.checkParamValidity(facesContext, uiComponent, UIOutput.class);
> +        return RendererUtils.getConvertedUIOutputValue(facesContext,
> +                                                       (UIOutput)uiComponent,
> +                                                       submittedValue);
> +    }
> +
> +
> +    public void decode(FacesContext facesContext, UIComponent component)
> +     {
> +         RendererUtils.checkParamValidity(facesContext,component,UIInput.class);
> +
> +         HtmlRendererUtils.decodeUIInput(facesContext, component);
> +
> +     }
> +
> +}
> 
> Propchange: myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java
> ------------------------------------------------------------------------------
>     svn:eol-style = native
> 
> Propchange: myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java
> ------------------------------------------------------------------------------
>     svn:keywords = Date Author Id Revision HeadURL
> 
> Modified: myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/component/html/ext/HtmlInputHiddenTest.java
> URL: http://svn.apache.org/viewvc/myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/component/html/ext/HtmlInputHiddenTest.java?rev=663341&r1=663340&r2=663341&view=diff
> ==============================================================================
> --- myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/component/html/ext/HtmlInputHiddenTest.java (original)
> +++ myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/component/html/ext/HtmlInputHiddenTest.java Wed Jun  4 11:53:57 2008
> @@ -48,7 +48,9 @@
>      {
>          // Define the component
>          UIComponent component = new HtmlInputHidden();
> -        component.setParent(new HtmlForm());
> +        component.setId("TestComponent");
> +        HtmlForm form = new HtmlForm();
> +        form.getChildren().add(component);
>  
>          // Render the component
>          try
> 
> Modified: myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/test/utils/TestUtils.java
> URL: http://svn.apache.org/viewvc/myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/test/utils/TestUtils.java?rev=663341&r1=663340&r2=663341&view=diff
> ==============================================================================
> --- myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/test/utils/TestUtils.java (original)
> +++ myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/test/utils/TestUtils.java Wed Jun  4 11:53:57 2008
> @@ -73,6 +73,9 @@
>          addRenderer(facesContext, "javax.faces.Input", "javax.faces.Hidden",
>                  "org.apache.myfaces.renderkit.html.HtmlHiddenRenderer");
>  
> +        addRenderer(facesContext, "javax.faces.Input", "org.apache.myfaces.Hidden",
> +                "org.apache.myfaces.renderkit.html.ext.HtmlHiddenRenderer");
> +        
>          addRenderer(facesContext, "javax.faces.Graphic", "javax.faces.Image",
>                  "org.apache.myfaces.renderkit.html.HtmlImageRenderer");
>  
> 
> 
> 



Re: svn commit: r663341 - in /myfaces/tomahawk/trunk/core/src: main/java/org/apache/myfaces/component/html/ext/ main/java/org/apache/myfaces/renderkit/html/ext/ test/java/org/apache/myfaces/component/html/ext/ test/java/org/apache/myfaces/test/utils/

Posted by Paul Spencer <pa...@mindspring.com>.
Leonardo,
Was HtmlHiddenRenderer.java add just to make the test pass, or is it 
need to when Tomahawk is run with the RI?

Paul Spencer



lu4242@apache.org wrote:
> Author: lu4242
> Date: Wed Jun  4 11:53:57 2008
> New Revision: 663341
> 
> URL: http://svn.apache.org/viewvc?rev=663341&view=rev
> Log:
> TOMAHAWK-1023 HtmlInputHidden fails unit test when using RI
> 
> Added:
>     myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java   (with props)
> Modified:
>     myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/component/html/ext/AbstractHtmlInputHidden.java
>     myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/component/html/ext/HtmlInputHiddenTest.java
>     myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/test/utils/TestUtils.java
> 
> Modified: myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/component/html/ext/AbstractHtmlInputHidden.java
> URL: http://svn.apache.org/viewvc/myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/component/html/ext/AbstractHtmlInputHidden.java?rev=663341&r1=663340&r2=663341&view=diff
> ==============================================================================
> --- myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/component/html/ext/AbstractHtmlInputHidden.java (original)
> +++ myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/component/html/ext/AbstractHtmlInputHidden.java Wed Jun  4 11:53:57 2008
> @@ -42,7 +42,7 @@
>      implements ForceIdAware
>  {
>      public static final String COMPONENT_TYPE = "org.apache.myfaces.HtmlInputHidden";
> -    public static final String DEFAULT_RENDERER_TYPE = "javax.faces.Hidden";
> +    public static final String DEFAULT_RENDERER_TYPE = "org.apache.myfaces.Hidden";
>  
>      public AbstractHtmlInputHidden()
>      {
> 
> Added: myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java
> URL: http://svn.apache.org/viewvc/myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java?rev=663341&view=auto
> ==============================================================================
> --- myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java (added)
> +++ myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java Wed Jun  4 11:53:57 2008
> @@ -0,0 +1,90 @@
> +/*
> + * Licensed to the Apache Software Foundation (ASF) under one
> + * or more contributor license agreements.  See the NOTICE file
> + * distributed with this work for additional information
> + * regarding copyright ownership.  The ASF licenses this file
> + * to you under the Apache License, Version 2.0 (the
> + * "License"); you may not use this file except in compliance
> + * with the License.  You may obtain a copy of the License at
> + *
> + *   http://www.apache.org/licenses/LICENSE-2.0
> + *
> + * Unless required by applicable law or agreed to in writing,
> + * software distributed under the License is distributed on an
> + * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
> + * KIND, either express or implied.  See the License for the
> + * specific language governing permissions and limitations
> + * under the License.
> + */
> +package org.apache.myfaces.renderkit.html.ext;
> +
> +import java.io.IOException;
> +
> +import javax.faces.component.UIComponent;
> +import javax.faces.component.UIInput;
> +import javax.faces.component.UIOutput;
> +import javax.faces.context.FacesContext;
> +import javax.faces.context.ResponseWriter;
> +import javax.faces.convert.ConverterException;
> +
> +import org.apache.myfaces.shared_tomahawk.renderkit.JSFAttr;
> +import org.apache.myfaces.shared_tomahawk.renderkit.RendererUtils;
> +import org.apache.myfaces.shared_tomahawk.renderkit.html.HTML;
> +import org.apache.myfaces.shared_tomahawk.renderkit.html.HtmlRenderer;
> +import org.apache.myfaces.shared_tomahawk.renderkit.html.HtmlRendererUtils;
> +
> +
> +/**
> + * @JSFRenderer
> + *   renderKitId="HTML_BASIC"
> + *   family="javax.faces.Input"
> + *   type="org.apache.myfaces.Hidden"
> + *   
> + * @author Thomas Spiegl (latest modification by $Author$)
> + * @author Anton Koinov
> + * @version $Revision$ $Date$
> + */
> +public class HtmlHiddenRenderer
> +extends HtmlRenderer
> +{
> +    public void encodeEnd(FacesContext facesContext, UIComponent uiComponent)
> +        throws IOException
> +    {
> +        RendererUtils.checkParamValidity(facesContext, uiComponent, UIInput.class);
> +
> +        ResponseWriter writer = facesContext.getResponseWriter();
> +
> +        writer.startElement(HTML.INPUT_ELEM, uiComponent);
> +        writer.writeAttribute(HTML.TYPE_ATTR, HTML.INPUT_TYPE_HIDDEN, null);
> +
> +        String clientId = uiComponent.getClientId(facesContext);
> +        writer.writeAttribute(HTML.ID_ATTR, clientId, null);
> +        writer.writeAttribute(HTML.NAME_ATTR, clientId, null);
> +
> +        String value = RendererUtils.getStringValue(facesContext, uiComponent);
> +        if (value != null)
> +        {
> +            writer.writeAttribute(HTML.VALUE_ATTR, value, JSFAttr.VALUE_ATTR);
> +        }
> +
> +        writer.endElement(HTML.INPUT_ELEM);
> +    }
> +
> +    public Object getConvertedValue(FacesContext facesContext, UIComponent uiComponent, Object submittedValue) throws ConverterException
> +    {
> +        RendererUtils.checkParamValidity(facesContext, uiComponent, UIOutput.class);
> +        return RendererUtils.getConvertedUIOutputValue(facesContext,
> +                                                       (UIOutput)uiComponent,
> +                                                       submittedValue);
> +    }
> +
> +
> +    public void decode(FacesContext facesContext, UIComponent component)
> +     {
> +         RendererUtils.checkParamValidity(facesContext,component,UIInput.class);
> +
> +         HtmlRendererUtils.decodeUIInput(facesContext, component);
> +
> +     }
> +
> +}
> 
> Propchange: myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java
> ------------------------------------------------------------------------------
>     svn:eol-style = native
> 
> Propchange: myfaces/tomahawk/trunk/core/src/main/java/org/apache/myfaces/renderkit/html/ext/HtmlHiddenRenderer.java
> ------------------------------------------------------------------------------
>     svn:keywords = Date Author Id Revision HeadURL
> 
> Modified: myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/component/html/ext/HtmlInputHiddenTest.java
> URL: http://svn.apache.org/viewvc/myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/component/html/ext/HtmlInputHiddenTest.java?rev=663341&r1=663340&r2=663341&view=diff
> ==============================================================================
> --- myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/component/html/ext/HtmlInputHiddenTest.java (original)
> +++ myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/component/html/ext/HtmlInputHiddenTest.java Wed Jun  4 11:53:57 2008
> @@ -48,7 +48,9 @@
>      {
>          // Define the component
>          UIComponent component = new HtmlInputHidden();
> -        component.setParent(new HtmlForm());
> +        component.setId("TestComponent");
> +        HtmlForm form = new HtmlForm();
> +        form.getChildren().add(component);
>  
>          // Render the component
>          try
> 
> Modified: myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/test/utils/TestUtils.java
> URL: http://svn.apache.org/viewvc/myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/test/utils/TestUtils.java?rev=663341&r1=663340&r2=663341&view=diff
> ==============================================================================
> --- myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/test/utils/TestUtils.java (original)
> +++ myfaces/tomahawk/trunk/core/src/test/java/org/apache/myfaces/test/utils/TestUtils.java Wed Jun  4 11:53:57 2008
> @@ -73,6 +73,9 @@
>          addRenderer(facesContext, "javax.faces.Input", "javax.faces.Hidden",
>                  "org.apache.myfaces.renderkit.html.HtmlHiddenRenderer");
>  
> +        addRenderer(facesContext, "javax.faces.Input", "org.apache.myfaces.Hidden",
> +                "org.apache.myfaces.renderkit.html.ext.HtmlHiddenRenderer");
> +        
>          addRenderer(facesContext, "javax.faces.Graphic", "javax.faces.Image",
>                  "org.apache.myfaces.renderkit.html.HtmlImageRenderer");
>  
> 
> 
>