On Tue, Oct 06, 2009 at 10:05:32AM +0200, Christian Boos wrote:
> 
> Hello,
> 
> While working on Environment.component_activated, I saw this code:
> 
>                 # Disable the pre-0.11 WebAdmin plugin
>                 # Please note that there's no recommendation to 
> uninstall the
>                 # plugin because doing so would obviously break the 
> backwards
>                 # compatibility that the new integration administration
>                 # interface tries to provide for old WebAdmin extensions
>                 if component_name.startswith('webadmin.'):
>                     self.log.info('The legacy TracWebAdmin plugin has been '
>                                   'automatically disabled, and the 
> integrated '
>                                   'administration interface will be used '
>                                   'instead.')
>                     return False
> 
> I'd like to get rid of the compatibility code mentioned in the comment 
> (IAdminPageProvider in trac/admin/web_ui.py), *and* recommend to 
> uninstall the WebAdmin plugin.
> 
> Would everyone be OK with that?
> 
> Likewise, I think we could start to deprecate the Clearsilver support, 
> and remove it in 0.13 (or even earlier if nobody complains).
> 
> -- Christian

I'll almost always choose keeping the code clean over sustaining backwards 
compatability, so +1 to both.

Jeff

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups "Trac 
Development" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to 
[email protected]
For more options, visit this group at 
http://groups.google.com/group/trac-dev?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to