Problem/Motivation

There's functions deprecated for removal in core 10 in core/includes/module.inc

Proposed resolution

As 10.0.x branch opens - remove deprecated functions and related tests

Remaining tasks

- patch/commit

User interface changes

no

API changes

no

Data model changes

no

Release notes snippet

no

CommentFileSizeAuthor
#7 3260781-7.patch3.51 KBandypost
#7 interdiff.txt4.78 KBandypost
#2 3260781-2.patch6.44 KBandypost

Comments

andypost created an issue. See original summary.

andypost’s picture

andypost
he/him
Primary language Russian
commented
Status: Active » Needs review
Related issues: +#3260615: Remove deprecated schema.inc
StatusFileSize
new 3260781-2.patch6.44 KB

Usage in schema.inc will be removed in #3260615: Remove deprecated schema.inc

Status: Needs review » Needs work

The last submitted patch, 2: 3260781-2.patch, failed testing. View results

andypost’s picture

andypost
he/him
Primary language Russian
commented
Title: Remove deprecated module.inc functions » [PP-1] Remove deprecated module.inc functions

Should wait for #3260615: Remove deprecated schema.inc

andypost’s picture

andypost
he/him
Primary language Russian
commented
Title: [PP-1] Remove deprecated module.inc functions » Remove deprecated module.inc functions

Blocker is in

andypost’s picture

andypost
he/him
Primary language Russian
commented
Status: Needs work » Needs review
andypost’s picture

andypost
he/him
Primary language Russian
commented
StatusFileSize
new interdiff.txt4.78 KB
new 3260781-7.patch3.51 KB

module_load_include deprecated for 11.0 so revert

andypost’s picture

andypost
he/him
Primary language Russian
commented
Related issues: +#3262807: Deprecate module_config_sort(), +#3262805: Deprecate drupal_required_modules(), +#2968232: Deprecate module_set_weight()
longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Status: Needs review » Reviewed & tested by the community

Looks OK to me.

alexpott’s picture

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

Committed 9e6fedb and pushed to 10.0.x. Thanks!

  • alexpott committed 9e6fedb on 10.0.x 
    Issue #3260781 by andypost: Remove deprecated module.inc functions

Status: Fixed » Closed (fixed)

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