Quantcast

GTInspector duplicates overriden extensions

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

GTInspector duplicates overriden extensions

Peter Uhnak
Hi,

when I override a method with a GTInspector (because I need to change it),
then inspector contains the extension twice.

It seems Inspector scans the whole hierarchy of the inspected object for methods, but doesn't deduplicate them.

Peter
_______________________________________________
Moose-dev mailing list
[hidden email]
https://www.list.inf.unibe.ch/listinfo/moose-dev
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: GTInspector duplicates overriden extensions

Andrei Chis
Hi,

Usually when we want to specialize a view in a subclass we override the method and do not add the gtInspectorPresentationOrder: annotation. But also removing duplicationd would make sense.

Cheers,
Andrei


On Feb 25, 2017 4:36 PM, "Peter Uhnak" <[hidden email]> wrote:
Hi,

when I override a method with a GTInspector (because I need to change it),
then inspector contains the extension twice.

It seems Inspector scans the whole hierarchy of the inspected object for methods, but doesn't deduplicate them.

Peter
_______________________________________________
Moose-dev mailing list
[hidden email]
https://www.list.inf.unibe.ch/listinfo/moose-dev


_______________________________________________
Moose-dev mailing list
[hidden email]
https://www.list.inf.unibe.ch/listinfo/moose-dev
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: GTInspector duplicates overriden extensions

Peter Uhnak
On Sat, Feb 25, 2017 at 04:44:15PM +0100, Andrei Chis wrote:
> Hi,
>
> Usually when we want to specialize a view in a subclass we override the
> method and do not add the gtInspectorPresentationOrder: annotation. But
> also removing duplicationd would make sense.

Ah, I didn't think of just removing the pragma. :)

Thanks!

Peter
_______________________________________________
Moose-dev mailing list
[hidden email]
https://www.list.inf.unibe.ch/listinfo/moose-dev
Loading...