Problem/Motivation

In core/modules/locale/locale.bulk.inc, $config variable never used.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

shetpooja04 created an issue. See original summary.

shetpooja04’s picture

Title: Unused variable $config in field module, locale.bulk.inc » Unused variable $config in locale module, locale.bulk.inc
shetpooja04’s picture

Commit ID: 91fc0d3e

Link: https://git.drupalcode.org/project/drupal/-/commit/91fc0d3ed91b263940b758588e416f78fd838d26

For Issue: https://www.drupal.org/project/drupal/issues/1905152 the changes were made

File: core/modules/locale/locale.bulk.inc

$config is never getting used

shetpooja04’s picture

Assigned: shetpooja04 » Unassigned
Status: Active » Needs review
jijojoseph_zyxware’s picture

Status: Needs review » Reviewed & tested by the community

I have reviewed the Patch #3. $config is never getting used. It is working fine without this variable.

@shetpooja04 Thanks for your patch.

RTBC!!!!!

  • catch committed 9b874df on 9.1.x
    Issue #3167600 by shetpooja04, jijojoseph_zyxware: Unused variable $...
catch’s picture

Status: Reviewed & tested by the community » Fixed

Committed 9b874df and pushed to 9.1.x. Thanks!

Status: Fixed » Closed (fixed)

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