[jbossws-dev] JSR181 Changes added in async changes

[Thomas Diesler]

IMO this is fuzzy logic and should be avoided.
There should at least be a comment that states the compelling reason to 
do this guess-work.

Jason T. Greene wrote:
> Heiko,
>
> I noticed the following change, which adds some non spec compliant
> behavior. I am not against it, but we probably should all discuss this
> further before we decide to do it. Do you have anything right now that
> depends on it?
>
> Also, it breaks the JSR181HandlerTestCase (although I will change the
> test in the mean time)
>
>           if (anWebService.endpointInterface().length() > 0)
>           {
>              seiName = anWebService.endpointInterface();
>              seiClass = udi.classLoader.loadClass(seiName);
>              anWebService =
> (WebService)seiClass.getAnnotation(WebService.class);
> +
>              if (anWebService == null)
>                 throw new WSException("Interface does not have a
> @WebService annotation: " + seiName);
>           }
> +         else
> +         {
>  
> +            WebService seiAnnotation = null;
> +
> +            for(Class potentialSEI : sepClass.getInterfaces())
> +            {
> +               if(potentialSEI.isAnnotationPresent(WebService.class))
> +               {
> +                  seiClass = potentialSEI;
> +                  seiName = seiClass.getName();
> +                  seiAnnotation =
> sepClass.getAnnotation(WebService.class);
> +                  break;
> +               }
> +            }
> +
> +            if(seiAnnotation!=null)
> +               anWebService = seiAnnotation;
> +         }
> +
>
> xxxxxxxxxxxxxxxxxxxxxxxxxxxx
> Jason T. Greene
> Senior Software Engineer
> JBoss, a division of Red Hat
> xxxxxxxxxxxxxxxxxxxxxxxxxxxx
>
>
> _______________________________________________
> jbossws-dev mailing list
> jbossws-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/jbossws-dev
>   

-- 
xxxxxxxxxxxxxxxxxxxxxxxxxxxx
Thomas Diesler
Web Service Lead
JBoss, a division of Red Hat
xxxxxxxxxxxxxxxxxxxxxxxxxxxx