Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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
Comment #2
millionleaves CreditAttribution: millionleaves commentedFurther 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 © [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.
Comment #3
opdaviesHi 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.
Comment #5
opdaviesI'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.