Follow-up of: #2407405: Remove usage of drupal_clean_id_identifier.

Remove the deprecated function drupal_clean_id_identifier().

CommentFileSizeAuthor
#4 2407407-4.patch821 bytesvedpareek
remove_drupal_clean_id_identifier.patch859 bytesJeroenT
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

JeroenT’s picture

JeroenT
Primary language Dutch
Location 🇧🇪
CreditAttribution: JeroenT commented
Title: Remove drupl_clean_id_identifier. » Remove drupal_clean_id_identifier.
Related issues: +#2407405: Remove usage of drupal_clean_id_identifier.
JeroenT’s picture

JeroenT
Primary language Dutch
Location 🇧🇪
CreditAttribution: JeroenT commented
Status: Postponed » Needs review
LinL’s picture

LinL CreditAttribution: LinL commented
Issue summary: View changes
Status: Needs review » Needs work
Issue tags: +Needs reroll

No longer applies, tagging for reroll.

vedpareek’s picture

vedpareek CreditAttribution: vedpareek commented
Status: Needs work » Needs review
FileSize
2407407-4.patch821 bytes

Rerolled

pythagory’s picture

pythagory CreditAttribution: pythagory commented
Issue tags: -Needs reroll
dinarcon’s picture

dinarcon CreditAttribution: dinarcon commented
Status: Needs review » Reviewed & tested by the community

The patch applies correctly. Let's mark this as RTBC.

jwilson3’s picture

jwilson3
Primary language English
Location Ecuador
CreditAttribution: jwilson3 commented
Title: Remove drupal_clean_id_identifier. » Remove drupal_clean_id_identifier
Issue summary: View changes
alexpott’s picture

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

Removing deprecated code is allowed as per the beta evaluation. Committed 68af24d and pushed to 8.0.x. Thanks!

  • alexpott committed 68af24d on 8.0.x 
    Issue #2407407 by JeroenT, vedpareek: Remove drupal_clean_id_identifier

Status: Fixed » Closed (fixed)

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