I have multiple sites using this module. After upgrading to the latest version, the copyright block which previously said:

Copyright [site:name], [copyright_statement:dates]

changed to:

C

When I looked at the block configuration, the Text Format field was set to - Select -

I assume this was introduced by the recent patch to support addition/selection of a text filter for the copyright block content field, https://www.drupal.org/node/2370059

Haven't tested the latest module in a new site, but this change happened across the board to all sites already using the module when I upgraded.

Comments

millionleaves created an issue. See original summary.

millionleaves’s picture

Further investigation reveals that the copyright_block_text is not upgraded when upgrading to the new version of the module, e.g.

Module upgraded, Copyright Block displaying C (and nothing else):
s:308:"Copyright &copy; [site:name], [copyright_statement:dates] | Design by <a href="http://www.designcompany.com/" target="_blank">Design Company</a> | <a href="http://www.webcompany.com/website-packages" target="_blank">Local web development by Web Company</a> | <a href="/user">Log In</a>";

Module upgraded, and Copyright Block module settings manually fixed by editing block, Copyright block displaying correctly:
a:2:{s:5:"value";s:105:"<div>Copyright [site:name], [copyright_statement:dates] | Local Web Development by Web Company</div>";s:6:"format";s:9:"full_html";}

(please ignore the s:308 and s:105 values - I have anonymized the block text so these numbers will be wrong. Otherwise, these strings are copied directly from the DB of two sites running this module).

I presume the best fix would be to modify the module to update the copyright_block_text field during an upgrade to work with the new version of Copyright Block so it doesn't break (and perhaps retrospectively fix any broken installs such as mine)?

As part of that, I believe that setting the text format of the block for existing sites to Full HTML during the upgrade would be acceptable since that's how the text field functions in previous versions of the module.

opdavies’s picture

Assigned: Unassigned » opdavies

Hi millionleaves,

Thanks for reporting the issue, and for posting the follow-up information.

I'll test this out on a local copy and make any amendments to the module.

  • opdavies committed 6238750 on 7.x-2.x
    Issue #2563557 by millionleaves: New Filters feature appears to break...
opdavies’s picture

Assigned: opdavies » Unassigned
Status: Active » Fixed

I've added an update hook to update any old variables that are still set as strings, and change them into arrays with the correct format.

http://cgit.drupalcode.org/copyright_block/commit/?id=6238750

This has been tagged and released as 7.x-2.4.

Status: Fixed » Closed (fixed)

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