Might it be better to use a naming convention something like custom-.tpl.php instead of just .tpl.php? If you were to use more than a couple of these, looking at a theme directory would be slightly confusing. Furthermore, naming the files like this makes it easy to recognise that they belong to this module, and groups the files together in a typical, alphanumerically sorted directory listing.

Comments

niklp’s picture

niklp commented

Grr filtered... the first sentence should have read:

Might it be better to use a naming convention something like custom-<name>.tpl.php instead of just <name>.tpl.php?

Setzler’s picture

Setzler commented

I concur.

irakli’s picture

irakli commented

I think this is a great idea. Furthermore, I think the solution should be even more flexible, where users can define the "prefix" for the template.

Some may like it to be custom-, some may like it to be blank, and some might like it to be the name of their girlfriend or a boyfriend :)

Thanks, I will try to implement this soon.

irakli’s picture

irakli commented
Assigned: Unassigned » irakli
niklp’s picture

niklp commented

Hm, not sure about the custom naming thing... it would make it more difficult for people examining the theme "from fresh" (ie, sigh unseen) to determine that custom page module was the cause of the "weird" filenames.

I really think it ought to be standardised. If there is room for debate on what that should be, let's open the floor up for discussion.

irakli’s picture

irakli commented

NikLP,

the default can be standardized. That should be good enough.

febbraro’s picture

febbraro commented
Assigned: irakli » febbraro
Status: Active » Fixed

This is in CVS waiting for a release.

I have allowed it to be customized, but it actually defaults to blank as to not break existing installations that already have custompage implementations using no prefix.

As a standard naming convention I would suggest always using 'custompage'

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.