Problem/Motivation
This is a followup to #2898947: Change "writeable" to "writable". Where that ticket covers the documentation, a follow-up was requested to cover changes to the method names. This is that follow-up.
Drupal core defines three functions with the less-used spelling 'writeable'...
The PHP standard library has a function named is_writable(), which Core uses 36 times; and an alias of that function named is_writeable(), which core does not use (as of commit 3834d60f1a).
Given that 'writable' already occurs more often than 'writeable', it makes sense for us change to 'writable' in this patch.
Proposed resolution
- Create new versions of the methods spelled "writable".
- Update the old versions to be wrappers of the new ones.
- Deprecate the old ones.
Remaining tasks
- Write a patch.
- Review by searching latest code with
grep -r 'writeable' /path/to/core
.
API changes
The following instances will be replaced/deprecated:
\Drupal\Component\PhpStorage\PhpStorageInterface::writeable()
\Drupal\Component\PhpStorage\FileStorage::writeable()
\Drupal\Component\PhpStorage\FileReadOnlyStorage::writeable()
Data model changes
N/A
Release notes snippet
N/A