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

Comments

andypost created an issue. See original summary.

andypost’s picture

Status: Active » Needs review
StatusFileSize
new2.69 KB
voleger’s picture

StatusFileSize
new2.86 KB
new295 bytes

Status: Needs review » Needs work

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

andypost’s picture

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

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

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.