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

Comments

jeroent’s picture

jeroent
Primary language Dutch
Location 🇧🇪
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 🇧🇪
commented
Status: Postponed » Needs review
linl’s picture

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 commented
Status: Needs work » Needs review
StatusFileSize
new 2407407-4.patch821 bytes

Rerolled

pythagory’s picture

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

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
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 🇪🇺🌍
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.