Ticket #710 (closed task: wontfix)

Opened 2 years ago

Last modified 1 month ago

Python module name should be more explicit

Reported by: dackze+trac@gmail.com Assigned to: mgood
Priority: normal Component: AccountManagerPlugin
Severity: minor Keywords:
Cc: Trac Release: 0.9

Description

Right now, the plugin's module name is "acct_mgr". It would make more sense for it to be named something like "tracaccountmanager", especially when installed into site-packages. It wouldn't be ambiguous and there probably wouldn't be any chance for it to conflict with other potential packages.

That, and PEP 8 says module names shouldn't contain underscores.

Attachments

package_rename_to_accountmanager-trunk-r2547.patch (4.0 kB) - added by jluna on 09/22/07 12:59:51.

Change History

09/11/06 11:39:39 changed by mgood

  • type changed from defect to task.

Installing into site-packages will still put it in "TracAccountManager?-version.egg", so it should be pretty clear. I'm not in a terrible rush to do this, since I've seen no real problems from the name, and changing it would break the config for current users.

09/22/07 12:33:27 changed by jluna

  • owner changed from mgood to jluna.
  • status changed from new to assigned.

I don't find it intuitive either.

I created a simple patch to rename the acct_mgr module to accountmanager. acct_mgr is left mapped to the accountmanager directory (see the Distutils package-dir specification).

Alternatively, without having to rely on Distutils magic, and retaining the ability to log a depreciation notice when acct_mgr is used...

class acct_mgr(Component):
    def __getattr__(self, attr):
        self.log.warning('Plugin \'acct_mgr\' was renamed to \'accountmanager\' and is subject to later removal.')
        return getattr(acountmanager, attr)

...added as an entry-point to plugins.trac.

09/22/07 12:35:17 changed by jluna

  • owner changed from jluna to mgood.
  • status changed from assigned to new.

09/22/07 12:59:51 changed by jluna

  • attachment package_rename_to_accountmanager-trunk-r2547.patch added.

10/29/08 18:41:18 changed by pacopablo

  • status changed from new to closed.
  • resolution set to wontfix.

I'm closing this as won't fix because most of account manager will be merged into core. Hopefully for 0.12, but most likely for 0.13. Doesn't make sense to jump through hoops for what will eventually become a non-issue.


Add/Change #710 (Python module name should be more explicit)




Change Properties
Action