I have no use for a copyright line, but I really want to display a little unobtrusive "Powered by [link]Drupal[/link]" line where the copyright line is located in this theme. There's a setting for editing the name of the copyright holder, however it's unnecessarily narrow in that it doesn't allow me to change the "Copyright (c) [year]" part.

I think the setting would be more useful if it covered the whole line instead.

P.S.: The spelling really is "copyright", as in who's got the rights to copy the contents. The incorrect spelling "copywrite" could be corrected in user-visible strings as well and if the variable is repurposed in an update to fit the whole line, maybe it can be renamed to the correct spelling as well - in fact, you could have an easy migration path by defaulting a 'copyright' variable to the following expression:

t('Copyright !c !year, !name', array('!c' => '©', '!year' => date('Y'), '!name' => theme_get_setting('copywrite_holder')))

but using theme_get_setting('copyright') instead if it's not empty.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mheinke’s picture

ill put a patch together for this.

thanks!

mheinke’s picture

FileSize
3.21 KB

test this patch

mheinke’s picture

FileSize
3.84 KB
jpetso’s picture

Thanks, I'll have a look later. (No time right now, but I'll get back to you.) On a quick glance, I saw that there's a 'copright_information' field in there somewhere, that's likely to be a typo, but I haven't had the time to figure out whether or how it impacts the functionality of this patch.

mheinke’s picture

FileSize
3.84 KB
mheinke’s picture

FileSize
238.09 KB
26.14 KB

tested with simplytest.me (2.x branch + latest patch.) here are screenshots

mheinke’s picture

this patch has been applied to the 2.x-dev branch and will be in the next release

mheinke’s picture

Status: Active » Patch (to be ported)
jpetso’s picture

Sorry for the late response, went travelling for a few weeks. The new patch works great. Thanks!

mheinke’s picture

no worries! thanks for confirming!

mheinke’s picture

let me know if there are any other features you want :)

mheinke’s picture

Status: Patch (to be ported) » Closed (fixed)