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.The documentation for drupal_parse_info_file tells developers to use variable_set and variable_get to create module settings.
Info files are NOT for placing arbitrary theme and module-specific settings. Use variable_get() and variable_set() for that. Info files are formatted as YAML. If the 'version' key is set to 'VERSION' in any info file, then the value will be substituted with the current version of Drupal core.
It should instead refer them to the Config class methods.
Info files are NOT for placing arbitrary theme and module-specific settings.
Use the Config::get() and Config::set()->save() for that. Info files are
formatted as YAML. If the 'version' key is set to 'VERSION' in any info file,
then the value will be substituted with the current version of Drupal core.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | bad_docs_on_parse_info_file-2001180-3.patch | 960 bytes | frob |
| #1 | bad_docs_on_parse_info_file-2001180-1.patch | 964 bytes | frob |
Comments
Comment #1
frobhere is a patch
Comment #2
jhodgdonLet's take "the" out of "Use the Config::get() and Config::set()->save() for that.". Thanks!
Comment #3
frobOkay.
Comment #4
jhodgdonLooks good! Assuming the test bot agrees (you needed to set the issue status to "needs review" when you submitted the new patch), I'll get this committed shortly. Thanks!
Comment #5
froboops, sorry
Comment #6
frobit passed
Comment #7
jhodgdonThanks again! Committed to 8.x.