Meta Issue: #2205673: [META] Remove all @deprecated functions marked "remove before 8.0"

CommentFileSizeAuthor
#5 remove_usage_of-2361809-5.patch4.2 KBnlisgo
#5 interdiff-2361809-3-5.txt1.18 KBnlisgo
#3 2361809-3.patch3.74 KBrpayanm
#1 2361809-1.patch3.54 KBrpayanm

Comments

rpayanm’s picture

rpayanm
he/him
Primary language Spanish
commented
Status: Active » Needs review
StatusFileSize
new 2361809-1.patch3.54 KB

Status: Needs review » Needs work

The last submitted patch, 1: 2361809-1.patch, failed testing.

rpayanm’s picture

rpayanm
he/him
Primary language Spanish
commented
Status: Needs work » Needs review
StatusFileSize
new 2361809-3.patch3.74 KB

Status: Needs review » Needs work

The last submitted patch, 3: 2361809-3.patch, failed testing.

nlisgo’s picture

nlisgo commented
Status: Needs work » Needs review
StatusFileSize
new interdiff-2361809-3-5.txt1.18 KB
new remove_usage_of-2361809-5.patch4.2 KB
rpayanm’s picture

rpayanm
he/him
Primary language Spanish
commented
Assigned: rpayanm » Unassigned
jeroent’s picture

jeroent
Primary language Dutch
Location 🇧🇪
commented
Status: Needs review » Reviewed & tested by the community

No occurrences left and patch looks ok to me.

alexpott’s picture

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

This issue is a prioritized change (removal of deprecated code) as per #2350615: [policy, no patch] What changes can be accepted during the Drupal 8 beta phase? and it's benefits outweigh any disruption. Committed f98cf1f and pushed to 8.0.x. Thanks!

  • alexpott committed f98cf1f on 8.0.x 
    Issue #2361809 by rpayanm, nlisgo: Remove usage of drupal_strtoupper().

Status: Fixed » Closed (fixed)

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