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.
Executing the coder ruleset.xml against default.settings.php generates errors due to the perl-style comments. The attached patch resolves those errors.
Comment | File | Size | Author |
---|---|---|---|
#23 | 2088089-22.patch | 8.92 KB | Akshay kashyap |
#22 | 2088089-22.patch | 9.53 KB | chaitanyadessai |
#3 | remove_perl_style-2088089-3.patch | 5.44 KB | deepakjoshi |
| |||
default.settings.php_.remove_perl-style_comments.d7-2.3.patch | 5.44 KB | chaunceyt | |
Issue fork drupal-2088089
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
Comment #1
mgiffordYou could try this for D8, but don't think there's any chance of making this change in D7.
Comment #2
deepakjoshi CreditAttribution: deepakjoshi as a volunteer commentedComment #3
deepakjoshi CreditAttribution: deepakjoshi as a volunteer commentedPlease review the changes.
Comment #5
deepakjoshi CreditAttribution: deepakjoshi as a volunteer commentedComment #6
droplet CreditAttribution: droplet commentedI bet it was coded for easy toggle on/off. We should make it multiple lines comment instead of block comment
Comment #7
droplet CreditAttribution: droplet commentedAlso # is better for readability I think
Comment #19
quietone CreditAttribution: quietone at PreviousNext commentedTagging for Coding standards. And un-assigning because it hasn't been worked on for 7 years.
Comment #22
chaitanyadessai CreditAttribution: chaitanyadessai at Specbee commentedRemoved perl-style comments from default.settings.php please review.
Comment #23
Akshay kashyap CreditAttribution: Akshay kashyap as a volunteer and at gai Technologies Pvt Ltd for gai Technologies Pvt Ltd commentedAdded a patch, Please review it.
Comment #24
catchThis is by design so that there's just a single character to remove to enable the setting, as pointed out by @droplet in 2016, and core's automated coding standards tools don't have an issue with the comments in settings.php