Modify

Opened 19 months ago

Last modified 18 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 19 months ago by hasienda

In 13324:

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

comment:2 Changed 19 months ago by rjollos

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

comment:3 follow-up: Changed 19 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 18 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 The owner will remain hasienda.
Author


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

 
Note: See TracTickets for help on using tickets.