On Wed, Jun 4, 2008 at 2:28 PM, Paul Spencer <[EMAIL PROTECTED]> 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
>
>
>
> [EMAIL PROTECTED] 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");
>>
>>
>>
>>
>
>