Opened 4 years ago

Last modified 6 weeks ago

#11214 new enhancement

Rename db_cleanup to admin_db_cleanup

Reported by: rjollos Owned by:
Priority: normal Component: AccountManagerPlugin
Severity: normal Keywords:
Cc: Trac Release:


All of the templates for the admin realm are prefixed with admin_, with the exception of db_cleanup.html. Would it make sense to rename db_cleanup.html to admin_db_cleanup.html?

Attachments (0)

Change History (5)

comment:1 Changed 4 years ago by hasienda

In 13324:

AccountManagerPlugin: Finally unify admin panel template names after [13054], refs #11214.

comment:2 Changed 4 years ago by rjollos

Thanks, it makes organizing the template map in ThemeEnginePlugin a bit easier.

comment:3 follow-up: Changed 3 years ago by jun66j5

I think that it should be a unique template name, e.g. admin_accounts_db_cleanup.html, acct_mgr_admin_db_cleanup.html, etc....

comment:4 in reply to: ↑ 3 Changed 3 years ago by rjollos

Replying to jun66j5:

I think that it should be a unique template name, ...

Seems like a good best practice, to prefix templates with the plugin name. I'll keep it in mind for other plugins.

comment:5 Changed 6 weeks ago by rjollos

  • Owner hasienda deleted

Add Comment

Modify Ticket

as new The ticket will remain with no owner.

E-mail address and user name can be saved in the Preferences.

Note: See TracTickets for help on using tickets.