You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by ma...@apache.org on 2004/12/03 09:46:09 UTC

cvs commit: incubator-myfaces/src/myfaces/org/apache/myfaces/application ApplicationImpl.java

manolito    2004/12/03 00:46:09

  Modified:    src/myfaces/org/apache/myfaces/application
                        ApplicationImpl.java
  Log:
  MYFACES-45 / ApplicationImpl does not correctly traverse a Class' hierarchy to create a Converter
  
  Revision  Changes    Path
  1.35      +10 -6     incubator-myfaces/src/myfaces/org/apache/myfaces/application/ApplicationImpl.java
  
  Index: ApplicationImpl.java
  ===================================================================
  RCS file: /home/cvs/incubator-myfaces/src/myfaces/org/apache/myfaces/application/ApplicationImpl.java,v
  retrieving revision 1.34
  retrieving revision 1.35
  diff -u -r1.34 -r1.35
  --- ApplicationImpl.java	13 Oct 2004 11:50:59 -0000	1.34
  +++ ApplicationImpl.java	3 Dec 2004 08:46:09 -0000	1.35
  @@ -48,6 +48,9 @@
    * @author Thomas Spiegl
    * @version $Revision$ $Date$
    * $Log$
  + * Revision 1.35  2004/12/03 08:46:09  manolito
  + * MYFACES-45 / ApplicationImpl does not correctly traverse a Class' hierarchy to create a Converter
  + *
    * Revision 1.34  2004/10/13 11:50:59  matze
    * renamed packages to org.apache
    *
  @@ -488,12 +491,13 @@
               Class interfaces[] = targetClass.getInterfaces();
               if (interfaces != null)
               {
  -                for (int i = 0; i < interfaces.length; i++)
  +                for (int i = 0, len = interfaces.length; i < len; i++)
                   {
  -                    converterClass = (Class)_converterTypeMap.get(interfaces[i]);
  -                    if(converterClass != null)
  +                	// search all superinterfaces for a matching converter, create it
  +                    Converter converter = internalCreateConverter(interfaces[i]);
  +                    if (converter != null)
                       {
  -                        break;
  +                        return converter;
                       }
                   }
               }