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
From #2184387: Remove SchemaStorage. InstallStorage::getAllFolders() is a misleading method name:
getAllFolders() is a very confusing method name, assumed that it returns folders, but it returns an array keyed by configuration files with the folder as value.
Proposed resolution
Rename the method.
Remaining tasks
- Choose a new name.
- Rename the method in
InstallStorage
and its child classes. (Novice task)
API changes
InstallStorage::getAllFolders()
is renamed. (This method is protected; therefore, this is not a public API change and does not need a change record.)
Original report by @alexpott
Over in #2184387: Remove SchemaStorage @berdir remarked that getFolders is poorly named. I agree.
Comments
Comment #1
xjmUpdated the summary. Once we pick a name for the method, this will be a novice issue.
The title also indicates something about documentation standards needing updates, but that seems to be out of scope for an issue that renames an API function, and anyway it's not clear to me what documentation standards need an update?
Comment #2
sunFWIW, #2190723: Add a KeyValueStore\FileStorage to replace e.g. ConfigStorage removes that method entirely.