ICEfaces
  1. ICEfaces
  2. ICE-7556

MyFaces 2: A warning is generated for all h:graphicImage tags that don't include an 'alt' attribute

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 3.0.RC1
    • Fix Version/s: 3.0
    • Component/s: Sample Apps
    • Labels:
      None
    • Environment:
      ICEfaces 3 MyFaces 2 ACE Showcase
    • Assignee Priority:
      P2
    • Affects:
      Sample App./Tutorial

      Description

      When running the ACE Showcase with MyFaces, any h:graphicImage that doesn't supply an 'alt' attribute generates the following warning:

      WARNING: Component UIGraphic j_id1378384490_1a3c8901:image has no attribute alt or attribute resolves to null. Path to component {Component-Path : [Class: javax.faces.component.UIViewRoot,ViewId: /showcase.xhtml][Class: javax.faces.component.html.HtmlBody,Id: _2068169349_6d256d7f][Class: org.icefaces.ace.component.panel.Panel,Id: j_id1803199580_452a22d8][Class: javax.faces.component.html.HtmlPanelGroup,Id: _1803199580_452a228d][Class: javax.faces.component.html.HtmlForm,Id: j_id1378384490_1a3c8901][Class: javax.faces.component.html.HtmlPanelGrid,Id: j_id1378384490_1a3c891b][Class: javax.faces.component.html.HtmlGraphicImage,Id: image]}

      This comes from the org.apache.myfaces.shared.renderkit.html.HtmlImageRendererBase:


      102 /*
      103 * Warn the user if the ALT attribute is missing.
      104 */
      105 if (uiComponent.getAttributes().get(HTML.ALT_ATTR) == null)
      106 {
      107 if(!facesContext.isProjectStage(ProjectStage.Development) && log.isLoggable(Level.WARNING))
      108 log.warning("Component UIGraphic " + uiComponent.getClientId(facesContext) + " has no attribute alt or attribute resolves to null. Path to component " + RendererUtils.getPathToComponent(uiComponent));
      109 }

      As you can see it only happens in Production Mode. This may actually be unintended (should be Development mode) and something we can petition them to change. However, I think it likely is intended since the 'alt' attribute is technically a required attribute as per the spec.

        Issue Links

          Activity

          Deryk Sinotte created issue -
          Deryk Sinotte made changes -
          Field Original Value New Value
          Estimated Complexity Low
          Salesforce Case []
          Assignee Priority P3
          Assignee Ken Fyten [ ken.fyten ]
          Priority Major [ 3 ] Minor [ 4 ]
          Ken Fyten made changes -
          Salesforce Case []
          Fix Version/s 3.0 [ 10241 ]
          Affects [Sample App./Tutorial]
          Assignee Priority P3 P2
          Assignee Ken Fyten [ ken.fyten ] Mark Collette [ mark.collette ]
          Mark Collette made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Duplicate [ 3 ]
          Mark Collette made changes -
          Link This issue duplicates ICE-7420 [ ICE-7420 ]
          Deryk Sinotte made changes -
          Resolution Duplicate [ 3 ]
          Status Resolved [ 5 ] Reopened [ 4 ]
          Assignee Mark Collette [ mark.collette ] Evgheni Sadovoi [ evgheni.sadovoi ]
          Evgheni Sadovoi made changes -
          Status Reopened [ 4 ] Resolved [ 5 ]
          Security Private [ 10001 ]
          Resolution Fixed [ 1 ]
          Ken Fyten made changes -
          Security Private [ 10001 ]
          Ken Fyten made changes -
          Status Resolved [ 5 ] Closed [ 6 ]

            People

            • Assignee:
              Evgheni Sadovoi
              Reporter:
              Deryk Sinotte
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: