Problem/Motivation

Follow-up to #3262805-14: Deprecate drupal_required_modules()

The function is deprecated for removal in 10.0.x

Proposed resolution

remove it with legacy test

Remaining tasks

review/commit

User interface changes

no

API changes

no

Data model changes

no

Release notes snippet

no

CommentFileSizeAuthor
#3 interdiff-3262931-2-3.txt295 bytesvoleger
#3 3262931-3.patch2.86 KBvoleger
#2 3262931-2.patch2.69 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 3262931-2.patch2.69 KB
voleger’s picture

voleger
he/his
Primary language Ukrainian
Location Ukraine, Rivne
commented
StatusFileSize
new 3262931-3.patch2.86 KB
new interdiff-3262931-2-3.txt295 bytes

Status: Needs review » Needs work

The last submitted patch, 3: 3262931-3.patch, failed testing. View results

andypost’s picture

andypost
he/him
Primary language Russian
commented
Status: Needs work » Reviewed & tested by the community

Unrelated test failure

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 3: 3262931-3.patch, failed testing. View results

andypost’s picture

andypost
he/him
Primary language Russian
commented
Status: Needs work » Reviewed & tested by the community

Retest went fine

  • catch committed 4846745 on 10.0.x 
    Issue #3262931 by voleger, andypost: Remove drupal_required_modules()...
catch’s picture

catch
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Wouldn't normally commit one function removal at a time, but since module.inc has already been cleared out once, there's not really a bigger scope to do this in.

Committed 4846745 and pushed to 10.0.x. Thanks!

Status: Fixed » Closed (fixed)

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