RE: [Springide-eclip-developer] Spring IDE Extensions - Search: Feedback

[Watkins, D. <dav...@fa...>]

Hi Torsten,
I've implemented suggestion 1 (a mistake on my behalf.  For item 2 I'm
actually using IBean's in the search result view.  I tried adding the
interface and method you suggested but they don't get called.  Instead
I've hacked another else clause onto the bottom of BeansView::show(...)
which can construct a BeansViewLocation from an IBean.  I haven't
checked this in yet as I wanted to run it past you. =20

A potentially neater solution to this is to make BeansViewLocation an
interface and make Bean into an IAdaptable class (I seem to remember we
can do this by registering with an AdapterManager somewhere so we do't
have to 'pollute' the bean class too much).  Just a thought!

Cheers,
  dw

> -----Original Message-----
> From: spr...@li...=20
> [mailto:spr...@li...]
>  On Behalf Of Torsten Juergeleit
> Sent: 24 October 2004 22:23
> To: Spring IDE Developer List
> Subject: [Springide-eclip-developer] Spring IDE Extensions -=20
> Search: Feedback
>=20
> Dave,
>=20
> the search facility of Spring IDE Extensions is very good.
> I have a few suggestions:
>=20
> 1. selecting "Search Again" in the search result page with a=20
> successful search adds duplicates to the list
>=20
> 2. selecting "Show In / Spring Beans" in the search result=20
> page for a node in the result tree view doesn't navigate to=20
> the corresponding node in the Spring Beans view
>=20
> For 2. BeanSearchResultPage has to implement IShowInSource=20
> and return in getShowInContext() an instance of ShowInContext=20
> which holds an input object of BeansViewLocation (prepared=20
> with project name, config name, bean name [if available] and=20
> property name [if available]) and a selection object of=20
> StructuredSelection (IResource of the selected node=20
> -> project or [config] file). The input object is recognized by Spring
> Beans view and the selection object i recognized by Eclipse's=20
> resource navigator or JDT's package explorer.
>=20
> For samples please refer to BeansView.getShowInContext() or=20
> the Beans UI action ShowInView.run().
>=20
> Ups, to recognize the input object first instead of the=20
> selection (if provided both) I have to modify=20
> BeansView.show(). Ok, I will fix this now. You can find the=20
> fix in CVS head.
>=20
> Cheers,
> Torsten
>=20
>=20
>=20
> -------------------------------------------------------
> This SF.net email is sponsored by: IT Product Guide on=20
> ITManagersJournal Use IT products in your business? Tell us=20
> what you think of them. Give us Your Opinions, Get Free=20
> ThinkGeek Gift Certificates! Click to find out more=20
> http://productguide.itmanagersjournal.com/guidepromo.tmpl
> _______________________________________________
> Springide-eclip-developer mailing list
> Spr...@li...
> https://lists.sourceforge.net/lists/listinfo/springide-eclip-developer
>=20