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.Problem/Motivation
The comments added to default.settings.php in bc60c92 (released in D7.50) regarding multibyte UTF-8 support are missing an asterisk. https://www.drupal.org/node/1354#general state that "Docblocks normally have * at the beginning of each line".
Proposed resolution
Add the missing asterisk at line 158 of sites/default/default.settings.php which reads "5.5.3 and up, as well as mysqlnd 5.0.9 and up).". While changes to this file normally get mentioned in core release notes, something this minor might not warrant such a mention.
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | drupal-coding-stardard-fixes-2766537-5.patch | 651 bytes | bhavikshah9 |
Comments
Comment #2
mforbes CreditAttribution: mforbes commentedComment #3
cilefen CreditAttribution: cilefen commentedComment #4
bhavikshah9 CreditAttribution: bhavikshah9 as a volunteer and at Tatvasoft commentedComment #5
bhavikshah9 CreditAttribution: bhavikshah9 at Tatvasoft commentedHere is the patch.
Comment #6
bhavikshah9 CreditAttribution: bhavikshah9 at Tatvasoft commentedComment #7
mforbes CreditAttribution: mforbes commentedThanks! Looks good but maybe someone else will check for RTBC.
Comment #8
David_Rothstein CreditAttribution: David_Rothstein as a volunteer commentedLooks good to me - thanks!
Comment #10
stefan.r CreditAttribution: stefan.r commentedCommitted and pushed to 7.x, thanks!