Add Change record to @deprecated in SafeMarkup.php

CommentFileSizeAuthor
#2 2873825_1.patch1.69 KBjosephcheek

Comments

josephcheek created an issue. See original summary.

josephcheek’s picture

josephcheek
Location Provo, UT
commented
Issue tags: +Baltimore2017
StatusFileSize
new 2873825_1.patch1.69 KB

patch attached

heddn’s picture

heddn
Primary language English
Location Illinois
commented
Title: Add Change record to @deprecated in SafeMarkup.php » Add Change record to @deprecated in SafeMarkup
sorabh.v6’s picture

sorabh.v6
He/His
Primary language Hindi
commented
Status: Active » Reviewed & tested by the community

@see have been added to SafeMarkup.php. I confirm that no other @deprecated without @see is left in SafeMarkup.php

I'm setting to RTBC. Thanks All

  • Gábor Hojtsy committed e55f12a on 8.3.x 
    Issue #2873825 by josephcheek: Add Change record to @deprecated in...
gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Version: 8.4.x-dev » 8.3.x-dev
Status: Reviewed & tested by the community » Fixed

Verified that all @deprecated are covered in the file and all of them are appropriately explained in the single URL that is added to all of them. Thanks, committed.

  • Gábor Hojtsy committed 19854be on 8.4.x 
    Issue #2873825 by josephcheek: Add Change record to @deprecated in...

Status: Fixed » Closed (fixed)

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

kay_v’s picture

kay_v commented
Parent issue: #2873705: Add change records to all @deprecated code »

Removing parent issue per conversation with @xjm at Drupalcon Nashville Mentored Sprint prep. Her recommendation to do so was based on a few points that made sense to all of us in the discussion, namely:
- so many child issues makes this parent unwieldy
- search filters will allow people needing to refind closed children