|
From: Watkins, D. <dav...@fa...> - 2004-12-02 10:54:23
|
I'll take a look at the code but I think we need a 'universal=20 id' for inner beans and the like. The specific scenario I'm=20 thinking about is the problem markers. As you know they have a couple of attributes, including the bean id and the problem=20 type. In the case of inner beans the id is currently (?) the class of the surrounding 'outer' bean. This makes implementing quick fix suggestions on inner beans v. tricky! =20 Anyway, I'll take a look at the new stuff (the 'incomplete config set' stuff sounds good!). I'm hoping it'll allow me to simplify=20 some of the refactoring participant code as I shouldn't have to=20 walk a tree anymore. I'll let you know if I hit any problems. Cheers, dw > -----Original Message----- > From: spr...@li...=20 > [mailto:spr...@li...] > On Behalf Of Torsten Juergeleit > Sent: 02 December 2004 01:31 > To: Spring IDE Developer List > Subject: [Springide-eclip-developer] Do we really need=20 > support for separate lists for "normal" beans and inner beans=20 > in BeansConfig and BeansConfigSet? >=20 > While implementing the validation of bean references I added=20 > the ability to Bean to know it's outer bean if it's an inner bean. > Now I'm just wondering if we still need to maintain separate=20 > lists for the normal beans and inner beans in BeansConfig and=20 > BeansConfigSet. >=20 > This means that BeansConfig.getBeans() and=20 > BeansConfigSet.getBeans() return the aggregated list of=20 > normal and inner beans. The separate > getInnerBeans() methods are obsolete and will be removed from=20 > the interfaces. If anyone still needs separate lists then the=20 > aggregated list can be filtered via IBean.isInnerBean(). >=20 > What do you think? >=20 > Cheers, > Torsten >=20 >=20 >=20 > ------------------------------------------------------- > SF email is sponsored by - The IT Product Guide Read honest &=20 > candid reviews on hundreds of IT Products from real users. > Discover which products truly live up to the hype. Start reading now.=20 > http://productguide.itmanagersjournal.com/ > _______________________________________________ > Springide-eclip-developer mailing list > Spr...@li... > https://lists.sourceforge.net/lists/listinfo/springide-eclip-developer >=20 |