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