Problem/Motivation

Properly deprecate check_url

Proposed resolution

Clean up deprecations to be of the format proposed in #3024461: Adopt consistent deprecation format for core and contrib deprecation messages
Add @trigger_error()

Remaining tasks

review, commit

User interface changes

None

API changes

None

Data model changes

None

Release notes snippet

N/a

CommentFileSizeAuthor
#3 3029336-3.patch2.7 KBandypost
#3 interdiff.txt2.39 KBandypost
#2 3029336-2.patch2.16 KBandypost

Comments

andypost created an issue. See original summary.

andypost’s picture

andypost
he/him
Primary language Russian
commented
Status: Active » Needs review
StatusFileSize
new 3029336-2.patch2.16 KB

Test case used to separate legacy testing

andypost’s picture

andypost
he/him
Primary language Russian
commented
StatusFileSize
new interdiff.txt2.39 KB
new 3029336-3.patch2.7 KB

polishing old message

andypost’s picture

andypost
he/him
Primary language Russian
commented
Related issues: +#3024461: Adopt consistent deprecation format for core and contrib deprecation messages
andypost’s picture

andypost
he/him
Primary language Russian
commented
Issue tags: -Kill includes @deprecated +Kill includes, +@deprecated
berdir’s picture

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

Seems pretty straight-forward.

alexpott’s picture

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

Nice. Committed 4b68814 and pushed to 8.7.x. Thanks!

  • alexpott committed 4b68814 on 8.7.x 
    Issue #3029336 by andypost: Properly deprecate check_url()

Status: Fixed » Closed (fixed)

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