I like the simplicity of this module. I was thinking to do something like this before finding it. One thing I'd like is the addition, maybe configurable, of an accompanying javascript that would replace the current year through javascript. Doing this it would allow cached versions to never display stale content. If it's fine by you I might give a patch for this a try.

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

hanoii created an issue. See original summary.

jamesoakley’s picture

What a great idea! There isn’t a log of scope to improve such a simple module, but that would be a really useful feature.

Yes please, if you submit a patch, I'll gladly review it and add this.

I'm unsure if this should be configurable, or just the new way of working. It's probably fine to move all sites over to use JS for this, but the placeholder (until the JS fires) should be the current year. Some search crawlers won't run JS, and it's important the versions of pages they crawl and index contain a valid copyright notice.

hanoii’s picture

Merge request opened. Simple patch, you are right, unnecessary for it to be configurable

  • jamesoakley committed 93aaae4c on 1.0.x authored by hanoii
    feat: #3592462 Add a configurable option to allow javascript to replace...
jamesoakley’s picture

Status: Active » Fixed

Thanks, nice and clean, and works as required. I'll tag a new release later.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

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