[jira] [Commented] (FELIX-5278) Error in method managedInjectedObject of InstanceManager

2016-06-08 Thread Clement Escoffier (JIRA)

[ 
https://issues.apache.org/jira/browse/FELIX-5278?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15320489#comment-15320489
 ] 

Clement Escoffier commented on FELIX-5278:
--

BTW there is a way to inject an existing "service object" in an iPOJO 
container. I can't remember the name of the property. You may be able to use 
this mechanism.

> Error in method managedInjectedObject of InstanceManager
> 
>
> Key: FELIX-5278
> URL: https://issues.apache.org/jira/browse/FELIX-5278
> Project: Felix
>  Issue Type: Bug
>  Components: iPOJO
>Affects Versions: ipojo-runtime-1.12.1
> Environment: Ubuntu 
>Reporter: Aygalinc Colin
>  Labels: patch
>   Original Estimate: 1h
>  Remaining Estimate: 1h
>
> In InstanceManager.java, the method managedInjectedObject at line 1016 can 
> throw an java.lang.NoSuchMethodException in case of extension of the 
> InstanceManager.
> I recommend to change the line 1016:
> Method setIM = m_clazz.getDeclaredMethod("_setInstanceManager", new 
> Class[]{this.getClass()});
> by :
> Method setIM = m_clazz.getDeclaredMethod("_setInstanceManager", new new 
> Class[]{InstanceManager.class});
> because iPOJO manipulation always produces a _setInstanceManager method with 
> an InstanceManager.class attribute.
>  



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (FELIX-5278) Error in method managedInjectedObject of InstanceManager

2016-06-08 Thread Clement Escoffier (JIRA)

[ 
https://issues.apache.org/jira/browse/FELIX-5278?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15320478#comment-15320478
 ] 

Clement Escoffier commented on FELIX-5278:
--

The instance manager is not really made to be extended, it heavily depends on 
the bytecode manipulation. 

In this context, it's would make more sense to extend iPOJO by providing a new 
"component type type", such as composite.

> Error in method managedInjectedObject of InstanceManager
> 
>
> Key: FELIX-5278
> URL: https://issues.apache.org/jira/browse/FELIX-5278
> Project: Felix
>  Issue Type: Bug
>  Components: iPOJO
>Affects Versions: ipojo-runtime-1.12.1
> Environment: Ubuntu 
>Reporter: Aygalinc Colin
>  Labels: patch
>   Original Estimate: 1h
>  Remaining Estimate: 1h
>
> In InstanceManager.java, the method managedInjectedObject at line 1016 can 
> throw an java.lang.NoSuchMethodException in case of extension of the 
> InstanceManager.
> I recommend to change the line 1016:
> Method setIM = m_clazz.getDeclaredMethod("_setInstanceManager", new 
> Class[]{this.getClass()});
> by :
> Method setIM = m_clazz.getDeclaredMethod("_setInstanceManager", new new 
> Class[]{InstanceManager.class});
> because iPOJO manipulation always produces a _setInstanceManager method with 
> an InstanceManager.class attribute.
>  



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (FELIX-5278) Error in method managedInjectedObject of InstanceManager

2016-06-08 Thread Aygalinc Colin (JIRA)

[ 
https://issues.apache.org/jira/browse/FELIX-5278?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15320373#comment-15320373
 ] 

Aygalinc Colin commented on FELIX-5278:
---

It throws the exception in this case:
- A CustomInstanceManager extends InstanceManager.
- This manager creates the POJO before calling start method.
- In the first call of the start method, exception is thrown.

As example in createObject, at line 746 or 757, iPOJO uses 
InstanceManager.class to find the appropriate constructor and i think the same 
mechanism must be used in managedInjectedObject because of the invariant 
behavior of iPOJO bytecode manipulation.

> Error in method managedInjectedObject of InstanceManager
> 
>
> Key: FELIX-5278
> URL: https://issues.apache.org/jira/browse/FELIX-5278
> Project: Felix
>  Issue Type: Bug
>  Components: iPOJO
>Affects Versions: ipojo-runtime-1.12.1
> Environment: Ubuntu 
>Reporter: Aygalinc Colin
>  Labels: patch
>   Original Estimate: 1h
>  Remaining Estimate: 1h
>
> In InstanceManager.java, the method managedInjectedObject at line 1016 can 
> throw an java.lang.NoSuchMethodException in case of extension of the 
> InstanceManager.
> I recommend to change the line 1016:
> Method setIM = m_clazz.getDeclaredMethod("_setInstanceManager", new 
> Class[]{this.getClass()});
> by :
> Method setIM = m_clazz.getDeclaredMethod("_setInstanceManager", new new 
> Class[]{InstanceManager.class});
> because iPOJO manipulation always produces a _setInstanceManager method with 
> an InstanceManager.class attribute.
>  



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (FELIX-5278) Error in method managedInjectedObject of InstanceManager

2016-06-08 Thread Clement Escoffier (JIRA)

[ 
https://issues.apache.org/jira/browse/FELIX-5278?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15320327#comment-15320327
 ] 

Clement Escoffier commented on FELIX-5278:
--

can throw ? or throws ?

> Error in method managedInjectedObject of InstanceManager
> 
>
> Key: FELIX-5278
> URL: https://issues.apache.org/jira/browse/FELIX-5278
> Project: Felix
>  Issue Type: Bug
>  Components: iPOJO
>Affects Versions: ipojo-runtime-1.12.1
> Environment: Ubuntu 
>Reporter: Aygalinc Colin
>  Labels: patch
>   Original Estimate: 1h
>  Remaining Estimate: 1h
>
> In InstanceManager.java, the method managedInjectedObject at line 1016 can 
> throw an java.lang.NoSuchMethodException in case of extension of the 
> InstanceManager.
> I recommend to change the line 1016:
> Method setIM = m_clazz.getDeclaredMethod("_setInstanceManager", new 
> Class[]{this.getClass()});
> by :
> Method setIM = m_clazz.getDeclaredMethod("_setInstanceManager", new new 
> Class[]{InstanceManager.class});
> because iPOJO manipulation always produces a _setInstanceManager method with 
> an InstanceManager.class attribute.
>  



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)