Modify

Opened 15 months ago

Last modified 14 months ago

#11214 new enhancement

Rename db_cleanup to admin_db_cleanup

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

Description

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 (4)

comment:1 Changed 15 months ago by hasienda

In 13324:

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

comment:2 Changed 15 months ago by rjollos

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

comment:3 follow-up: Changed 14 months 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 14 months 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.

Add Comment

Modify Ticket

Action
as new .
Author


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

 
Note: See TracTickets for help on using tickets.