You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@felix.apache.org by fm...@apache.org on 2012/10/25 16:26:34 UTC
svn commit: r1402163 - in
/felix/trunk/metatype/src/main/java/org/apache/felix/metatype/internal:
MetaTypeProviderTracker.java ServiceMetaTypeInformation.java
Author: fmeschbe
Date: Thu Oct 25 14:26:34 2012
New Revision: 1402163
URL: http://svn.apache.org/viewvc?rev=1402163&view=rev
Log:
FELIX-3736 Properly handle MetaTypeProvider service modifications
Modified:
felix/trunk/metatype/src/main/java/org/apache/felix/metatype/internal/MetaTypeProviderTracker.java
felix/trunk/metatype/src/main/java/org/apache/felix/metatype/internal/ServiceMetaTypeInformation.java
Modified: felix/trunk/metatype/src/main/java/org/apache/felix/metatype/internal/MetaTypeProviderTracker.java
URL: http://svn.apache.org/viewvc/felix/trunk/metatype/src/main/java/org/apache/felix/metatype/internal/MetaTypeProviderTracker.java?rev=1402163&r1=1402162&r2=1402163&view=diff
==============================================================================
--- felix/trunk/metatype/src/main/java/org/apache/felix/metatype/internal/MetaTypeProviderTracker.java (original)
+++ felix/trunk/metatype/src/main/java/org/apache/felix/metatype/internal/MetaTypeProviderTracker.java Thu Oct 25 14:26:34 2012
@@ -19,6 +19,8 @@
package org.apache.felix.metatype.internal;
+import java.util.Arrays;
+
import org.osgi.framework.BundleContext;
import org.osgi.framework.ServiceReference;
import org.osgi.service.metatype.MetaTypeProvider;
@@ -46,47 +48,92 @@ public class MetaTypeProviderTracker ext
return null;
}
- MetaTypeProvider mtp = ( MetaTypeProvider ) super.addingService( reference );
-
- final String[] pids = ServiceMetaTypeInformation.getStringPlus( reference, MetaTypeProvider.METATYPE_PID );
- if ( pids != null )
- {
- mti.addSingletonMetaTypeProvider( pids, mtp );
- }
+ final MetaTypeProvider mtp = ( MetaTypeProvider ) this.context.getService( reference );
+ final RegistrationPropertyHolder rph = new RegistrationPropertyHolder( reference, mtp );
- final String[] factoryPids = ServiceMetaTypeInformation.getStringPlus( reference,
- MetaTypeProvider.METATYPE_FACTORY_PID );
- if ( factoryPids != null )
- {
- mti.addFactoryMetaTypeProvider( factoryPids, mtp );
- }
+ rph.addMetaTypeProvider( this.mti );
- return mtp;
+ return rph;
}
public void modifiedService( ServiceReference reference, Object service )
{
- // TODO This is tricky: we must know the registration before the update !!
- super.modifiedService( reference, service );
+ RegistrationPropertyHolder rph = ( RegistrationPropertyHolder ) service;
+ rph.update( this.mti, reference );
}
public void removedService( ServiceReference reference, Object service )
{
- final String[] pids = ServiceMetaTypeInformation.getStringPlus( reference, MetaTypeProvider.METATYPE_PID );
- if ( pids != null )
+ RegistrationPropertyHolder rph = ( RegistrationPropertyHolder ) service;
+ rph.removeMetaTypeProvider( this.mti );
+ this.context.ungetService( reference );
+ }
+
+ private static class RegistrationPropertyHolder
+ {
+ private String[] pids;
+ private String[] factoryPids;
+ private final MetaTypeProvider provider;
+
+
+ RegistrationPropertyHolder( final ServiceReference reference, final MetaTypeProvider provider )
{
- mti.removeSingletonMetaTypeProvider( pids );
+ this.pids = ServiceMetaTypeInformation.getStringPlus( reference, MetaTypeProvider.METATYPE_PID );
+ this.factoryPids = ServiceMetaTypeInformation.getStringPlus( reference,
+ MetaTypeProvider.METATYPE_FACTORY_PID );
+ this.provider = provider;
}
- final String[] factoryPids = ServiceMetaTypeInformation.getStringPlus( reference,
- MetaTypeProvider.METATYPE_FACTORY_PID );
- if ( factoryPids != null )
+
+ MetaTypeProvider getProvider()
{
- mti.removeFactoryMetaTypeProvider( factoryPids );
+ return provider;
}
- super.removedService( reference, service );
+
+ void addMetaTypeProvider( final MetaTypeInformationImpl mti )
+ {
+ if ( pids != null )
+ {
+ mti.addSingletonMetaTypeProvider( pids, provider );
+ }
+
+ if ( factoryPids != null )
+ {
+ mti.addFactoryMetaTypeProvider( factoryPids, provider );
+ }
+ }
+
+
+ void removeMetaTypeProvider( final MetaTypeInformationImpl mti )
+ {
+ if ( pids != null )
+ {
+ mti.removeSingletonMetaTypeProvider( pids );
+ }
+
+ if ( factoryPids != null )
+ {
+ mti.removeFactoryMetaTypeProvider( factoryPids );
+ }
+ }
+
+
+ void update( final MetaTypeInformationImpl mti, final ServiceReference reference )
+ {
+ String[] pids = ServiceMetaTypeInformation.getStringPlus( reference, MetaTypeProvider.METATYPE_PID );
+ String[] factoryPids = ServiceMetaTypeInformation.getStringPlus( reference,
+ MetaTypeProvider.METATYPE_FACTORY_PID );
+
+ if ( !Arrays.equals( pids, this.pids ) || !Arrays.equals( factoryPids, this.factoryPids ) )
+ {
+ removeMetaTypeProvider( mti );
+ this.pids = pids;
+ this.factoryPids = factoryPids;
+ addMetaTypeProvider( mti );
+ }
+ }
}
}
Modified: felix/trunk/metatype/src/main/java/org/apache/felix/metatype/internal/ServiceMetaTypeInformation.java
URL: http://svn.apache.org/viewvc/felix/trunk/metatype/src/main/java/org/apache/felix/metatype/internal/ServiceMetaTypeInformation.java?rev=1402163&r1=1402162&r2=1402163&view=diff
==============================================================================
--- felix/trunk/metatype/src/main/java/org/apache/felix/metatype/internal/ServiceMetaTypeInformation.java (original)
+++ felix/trunk/metatype/src/main/java/org/apache/felix/metatype/internal/ServiceMetaTypeInformation.java Thu Oct 25 14:26:34 2012
@@ -302,6 +302,11 @@ public class ServiceMetaTypeInformation
res = null;
}
+ if ( res != null )
+ {
+ Arrays.sort( res );
+ }
+
return res;
}