drupal_realpath() was deprecated prior to the formal deprecation policy. We need to add a trigger_error() and a deprecation test.

CommentFileSizeAuthor
#2 3068022-2.patch1.32 KBkim.pepper
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

kim.pepper created an issue. See original summary.

kim.pepper’s picture

kim.pepper
Primary language English
Location 🏄‍♂️🇦🇺Sydney, Australia
CreditAttribution: kim.pepper at PreviousNext commented
Status: Active » Needs review
FileSize
3068022-2.patch1.32 KB

Adds a trigger_error and deprecation test.

Berdir’s picture

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

Looks good.

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch at Third and Grove commented
Status: Reviewed & tested by the community » Fixed

Committed 442a3f2 and pushed to 8.8.x. Thanks!

  • catch committed 442a3f2 on 8.8.x 
    Issue #3068022 by kim.pepper: Properly deprecate drupal_realpath()
catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch at Third and Grove commented

Status: Fixed » Closed (fixed)

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