Updated: Comment #N
1. Documentation in MailFactory references the old class names for devel's MailInterface implementations.
The second problem is that VariableLog is incorrectly named, it no longer uses the variable system.
1. Rename the first to Drupal\devel\DevelMailLog. The other one never existed, it was temporarly in the issue that originally added the interface but it got removed and the documentation stayed:. I'd suggest to use the full class name of VariableLog instead, with it's final name.
2. I'd suggest to name the class after what it does and not how. So, MailCollector. Rename class, the file and all references to it.
Write patch, review, commit. Does not need tests as there's no functional bug.
User interface changes
None except someone used VariableLog themself directly, but very unlikely at this point.
The doxygen of MailFactory::get contains class names which are not PSR-0: DevelMailLog and DrupalDevNullMailSend. The first should be renamed to Drupal\Core\Mail\VariableLog and the second to perhaps
Drupal\devel\Mail\Null as this class doesn't exist.
A followup should be filed to rename VariableLog to MailCollector.
PASSED: [[SimpleTest]]: [MySQL] 60,043 pass(es). View