Modify

Opened 4 years ago

Last modified 7 months ago

#11214 new enhancement

Rename db_cleanup to admin_db_cleanup

Reported by: Ryan J Ollos Owned by:
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 (5)

comment:1 Changed 4 years ago by Steffen Hoffmann

In 13324:

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

comment:2 Changed 4 years ago by Ryan J Ollos

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

comment:3 Changed 4 years ago by Jun Omae

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 4 years ago by Ryan J Ollos

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 7 months ago by Ryan J Ollos

Owner: Steffen Hoffmann deleted

Modify Ticket

Change Properties
Set your email in Preferences
Action
as new The ticket will remain with no owner.

Add Comment


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

 
Note: See TracTickets for help on using tickets.