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 2009/05/16 01:59:44 UTC
svn commit: r775399 -
/myfaces/shared/trunk_3.0.x/core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlImageRendererBase.java
Author: lu4242
Date: Fri May 15 23:59:44 2009
New Revision: 775399
URL: http://svn.apache.org/viewvc?rev=775399&view=rev
Log:
TOMAHAWK-1291 t:graphicImage doesnot generate XHTML complaint code (warn alt not found attribute on shared 3.0.x)
Modified:
myfaces/shared/trunk_3.0.x/core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlImageRendererBase.java
Modified: myfaces/shared/trunk_3.0.x/core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlImageRendererBase.java
URL: http://svn.apache.org/viewvc/myfaces/shared/trunk_3.0.x/core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlImageRendererBase.java?rev=775399&r1=775398&r2=775399&view=diff
==============================================================================
--- myfaces/shared/trunk_3.0.x/core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlImageRendererBase.java (original)
+++ myfaces/shared/trunk_3.0.x/core/src/main/java/org/apache/myfaces/shared/renderkit/html/HtmlImageRendererBase.java Fri May 15 23:59:44 2009
@@ -75,6 +75,14 @@
if (log.isWarnEnabled()) log.warn("Graphic with id " + uiComponent.getClientId(facesContext) + " has no value (url).");
}
+ /*
+ * Warn the user if the ALT attribute is missing.
+ */
+ if (uiComponent.getAttributes().get(HTML.ALT_ATTR) == null)
+ {
+ log.warn("ALT attribute is missing for : " + uiComponent.getId());
+ }
+
HtmlRendererUtils.renderHTMLAttributes(writer, uiComponent, HTML.IMG_PASSTHROUGH_ATTRIBUTES);
writer.endElement(org.apache.myfaces.shared.renderkit.html.HTML.IMG_ELEM);