Split off from #2873881: [meta] Add Change Record to @deprecated for common.inc because these types of issues need to scoped by change record not file.

To determine how to complete this issue see #2873705: Add change records to all @deprecated code

Related change record: https://www.drupal.org/node/2235461

Members fund testing for the Drupal project. Drupal Association Learn more

Comments

tedbow created an issue. See original summary.

sergiuteaca’s picture

Status: Active » Needs review
FileSize
915 bytes
tedbow’s picture

Status: Needs review » Reviewed & tested by the community

@sergiuteaca thanks!

I have looked at the related change record and this issues it references

  1. #2207743: Convert element_info() to a service
  2. #2226621: Remove usage of element_info(), element_child() and element_children(). Deprecate element_info_property().
  3. #2393331: Replace the calls to element_info() with the ElementInfo service.

I have confirm the only things deprecated in these patches where the 2 functions in this patch.

The patch looks good.

RTBC 🎉🙌

xjm credited JO0st.

xjm credited Janec.

xjm credited anyaabchiche.

xjm credited billywardrop.

xjm credited davemckain.

xjm credited dinarcon.

xjm credited jyraya.

xjm credited kwhite.

xjm credited mairi.

xjm credited sthomps5.

xjm’s picture

xjm’s picture

Status: Reviewed & tested by the community » Fixed

Committed to 8.5.x, and backported to 8.4.x as a documentation improvement. Thanks everyone!

xjm’s picture

Belatedly adding credit for @tedbow as a reviewer as well.

  • xjm committed d0e3796 on 8.5.x
    Issue #2920018 by sergiuteaca, billywardrop, anyaabchiche, Janec, JO0st...

  • xjm committed 0f9bf81 on 8.4.x
    Issue #2920018 by sergiuteaca, billywardrop, anyaabchiche, Janec, JO0st...

Status: Fixed » Closed (fixed)

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