Re: DiscoveryView CQ (was Re: [ecf-dev] ECF status -- discovery)

[Scott Lewis <slewis@xxxxxxxxxxxxx>]

Hi Markus,

<stuff deleted>

Markus Alexander Kuppe wrote:

<stuff deleted>

On second thoughts, getting the code ready for CQ doesn't take that long
and we can work on the integration into the ECF UI afterwards. Thus I've
changed anything (hopefully) from com.versant to org.eclipse.ecf and
fixed all the copyright related parts. Hopefully I haven't broken all of
it.
  

Cool!  This is great to hear.  Thanks much.

Do you think you/your employer would be able to contribute to
ecf1.osusol.org CVS first, and allow us to work on it/prepare demos,
etc?  With the expectation of going through the IP process later?  Too
bad it's unlikely to be in Ganymede.
    

I received approval from my employer, so it's on ecf1.osuosl.org. Check
it out and maybe focus on IP related issues too so I can start the CQ
for it on Monday. :)
  

OK, I will take a look as soon as I can. 

Coincidently, I had a couple of hours today (Friday) and before I knew you had done this I did some work to simplify DiscoveryView.  It's not *much* simpler in structure and probably will provide a much easier way to integrate with your new contribution.  So that was fortuitous.

I also added an extension point:  org.eclipse.ecf.discovery.ui.serviceAccessHandler.  This extension point allows other plugins to insert menu items into the context menu for individual services listed in the discovery view.  See the interface:  IServiceAccessHandler and the documentation for the serviceAccessHandler extension point.  Here's the javadocs for IServiceAccessHandler:

http://www.eclipse.org/ecf/org.eclipse.ecf.docs/api/org/eclipse/ecf/discovery/ui/views/IServiceAccessHandler.html

Here's a trivial implementation class:

http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.ecf/plugins/org.eclipse.ecf.discovery.ui/src/org/eclipse/ecf/internal/discovery/ui/NullServiceAccessHandler.java?root=Technology_Project&view=log

And here's the markup for an extension:

   <extension
         point="org.eclipse.ecf.discovery.ui.serviceAccessHandler">
      <serviceAccessHandler
            class="org.eclipse.ecf.internal.discovery.ui.NullServiceAccessHandler">
      </serviceAccessHandler>
   </extension>

We can use this in the integration or not...depending upon whether it makes sense.

On behalf of the ECF community...thanks. 

I'll look forward to helping with the CQs for this contribution next week. 

Scott