#2361757: Remove usage of mime_header_encode()
In this issue, let's remove the function itself.

CommentFileSizeAuthor
#5 2361761-5.patch1.18 KBer.pushpinderrana
#1 2361761-1.patch1.09 KBrpayanm
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

rpayanm’s picture

rpayanm
he/him
Primary language Spanish
CreditAttribution: rpayanm commented
Assigned: rpayanm » Unassigned
Status: Postponed » Needs review
Issue tags: +Novice, +@deprecated
FileSize
2361761-1.patch1.09 KB
aspilicious’s picture

aspilicious CreditAttribution: aspilicious commented
Status: Needs review » Reviewed & tested by the community
alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott commented
Status: Reviewed & tested by the community » Needs work

Can we remove mime_header_decode and mime_header_encode in the same patch. Thanks.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott commented
Title: Remove mime_header_encode. » Remove mime_header_encode and mime_header_decode
er.pushpinderrana’s picture

er.pushpinderrana CreditAttribution: er.pushpinderrana commented
Status: Needs work » Needs review
FileSize
2361761-5.patch1.18 KB

Remove both mime_header_decode and mime_header_encode in this patch.

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Status: Needs review » Reviewed & tested by the community

Alright

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott commented
Status: Reviewed & tested by the community » Fixed

This 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 f1158cf and pushed to 8.0.x. Thanks!

  • alexpott committed f1158cf on 8.0.x 
    Issue #2361761 by er.pushpinderrana, rpayanm: Remove mime_header_encode...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.