Add support for Ordered interface (spring configuration) about javamelody HOT 7 CLOSED

Which classe(s) of javamelody sould implement Ordered ? 
MonitoringSpringInterceptor, MonitoringSpringAdvisor, 
MonitoredWithInterfacePointcut, MonitoredWithAnnotationPointcut or 
SpringDataSourceBeanPostProcessor ?

What would be a suitable value to be returned by the implementation of 
getOrder() ? 0 ?

Thanks

Hi,
Note that I would want to make a release tomorrow 3rd of june.

I think that MonitoringSpringAdvisor should implement Ordered (and it already 
does with its super-class by the way). Is that what you need? And I have 
commited 0 for getOrder(). Is it ok?
Thanks

I thought it was MonitoringSpringInterceptor that needed it - for example se 
docs in 
http://www.jarvana.com/jarvana/view/org/springframework/spring/1.2.9/spring-1.2.
9-javadoc.jar!/org/springframework/aop/framework/ProxyFactoryBean.html.

The most important is that is configurable (settable) so that it can be 
configured by the end user. I think HIGHEST would be a sensible default. (A 
constant in the Ordered interface).

Hi,
It seems that all Advisors implements Ordered (and "order" is settable on 
Advisors).
Have you tried the newly released version 1.17.0 (where the order of the 
advisor is 0 by default) ?

Hi, Have you tried the newly released version 1.17.0 (where the order of the 
advisor is 0 by default and is settable) ?

Yes - but actually I just use aop:advisor directly in spring as I've extended 
the interceptor to do some additional stuff (logging times to a couple of slf4j 
loggers in a internal format).

Thanks!

So I suppose the issue is fixed. (If not, it can be reopened.)
Thanks