I think there is a bug in AspectManager.addClassMetaData. The line

  | if (meta.matches(advisor, advisor.getClass()))
  | 

should actually be


  | if (meta.matches(advisor, ((ClassAdvisor)advisor).getClazz()))
  | 


I've rephrased the method implementation here.


  | 
  |  public void addClassMetaData(ClassMetaDataBinding meta)
  |     { 
  |         removeClassMetaData(meta.getName());
  |         synchronized (advisors)
  |             {
  |                 Iterator it = advisors.values().iterator();
  |                 while (it.hasNext())
  |                     {
  |                         Advisor advisor = (Advisor) it.next();
  |                         if (advisor instanceof ClassAdvisor) {
  |                             if (meta.matches(advisor, 
((ClassAdvisor)advisor).getClazz()))
  |                                 {
  |                                     meta.addAdvisor(advisor);
  |                                 }                                            
  |                       }
  |             }
  |         synchronized (classMetaData)
  |             {
  |                 classMetaData.put(meta.getName(), meta);
  |             }
  |     }
  | 

After this fix, I managed to get the meta-data that were set dynamically as 
illustrated in the previous post.

Would it be possible to roll this fix into the CVS Head

thanks
rajiv

View the original post : 
http://www.jboss.org/index.html?module=bb&op=viewtopic&p=3846044#3846044

Reply to the post : 
http://www.jboss.org/index.html?module=bb&op=posting&mode=reply&p=3846044


-------------------------------------------------------
SF.Net email is sponsored by Shop4tech.com-Lowest price on Blank Media
100pk Sonic DVD-R 4x for only $29 -100pk Sonic DVD+R for only $33
Save 50% off Retail on Ink & Toner - Free Shipping and Free Gift.
http://www.shop4tech.com/z/Inkjet_Cartridges/9_108_r285
_______________________________________________
JBoss-Development mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/jboss-development

Reply via email to