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.
Comment | File | Size | Author |
---|---|---|---|
#19 | remove_usage_of-2358995-19.patch | 4.02 KB | Alienpruts |
#10 | 2358995-10.patch | 4.51 KB | gaurav.pahuja |
Comments
Comment #1
JeroenTRemoved all usages of drupal_wrap_mail. Patch attached.
Comment #2
JeroenTGo testbot!
Comment #3
JeroenTCreated follow-up issue to remove the function itself: #2359071: Remove drupal_wrap_mail.
Comment #4
rpayanmminor changes
Comment #5
javivf CreditAttribution: javivf commentedComment #6
alexpottNot formatted correctly.
Should be a single line. For example the first one could be:
Comment #7
JeroenTComment #8
er.pushpinderrana CreditAttribution: er.pushpinderrana commentedRe-rolled.
Comment #9
rpayanm\Drupal\Core\Mail\MailFormatHelper::wrapMail()-space here-for
Comment #10
gaurav.pahuja CreditAttribution: gaurav.pahuja commentedAdded required space.
Comment #11
rpayanmLook fine for me :)
Comment #12
alexpottThis should be changed to just:
This is a special format for PHPUnit test method doc blocks that informs code coverage reports what is being tested. The docblock is essentially useless here.
Comment #13
alexpottNeeds works for #12 and since the replacement docblock in #10 breaks code standards by being multi-line.
Comment #14
JeroenTMade changes as suggested by alexpott in #2358995-12: Remove usage of drupal_wrap_mail(). Patch attached.
Comment #15
rpayanmFix comments length :)
Comment #18
rpayanmComment #19
Alienpruts CreditAttribution: Alienpruts commentedMy first attempt at a reroll.
Had to resolve one conflict in file HtmlToTextTest.php :
use Drupal\Component\Utility\String;
use Drupal\Component\Utility\Unicode;
use Drupal\Core\Mail\MailFormatHelper;
use Drupal\Core\Site\Settings;
use Drupal\simpletest\WebTestBase;
I hope I dit it correctly, if not : please advise on what I did wrong and what I should've done :)
PS : yes, i tested the patch against the 8.0.x branch with git apply --check :)
Comment #20
Wim LeersLooks good!
Comment #21
alexpottThis issue is a prioritized change (deprecated function removal) as per https://www.drupal.org/core/beta-changes and it's benefits outweigh any disruption. Committed 547a1a0 and pushed to 8.0.x. Thanks!
Minor fixes on commit.