Functions field_cache_clear() and field_info_cache_clear() does not use any arguments.

Detected few calls of these functions with argument in files date_tools/date_tools.wizard.inc and tests/date_field.test

CommentFileSizeAuthor
#2 remove_unused_argument-2769057-2.patch1.53 KBid.medion
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

id.medion created an issue. See original summary.

id.medion’s picture

id.medion
Primary language Ukrainian
Location Lutsk
CreditAttribution: id.medion commented
Assigned: id.medion » Unassigned
Status: Active » Needs review
FileSize
remove_unused_argument-2769057-2.patch1.53 KB

Attached patch for review.

Chris Matthews’s picture

Chris Matthews CreditAttribution: Chris Matthews as a volunteer commented

The 2 year old initial patch to date_tools.wizard.inc still applies cleanly to the latest 7.x-2.x-dev and seems like a reasonable fix to me.

Skipped patch 'tests/date_field.test'.
Checking patch date_tools/date_tools.wizard.inc...
Hunk #1 succeeded at 292 (offset -3 lines).
Applied patch date_tools/date_tools.wizard.inc cleanly.
steinmb’s picture

steinmb CreditAttribution: steinmb as a volunteer commented
Status: Needs review » Reviewed & tested by the community

Checked API and there shall be be non arguments for https://api.drupal.org/api/drupal/modules%21field%21field.info.inc/funct... and https://api.drupal.org/api/drupal/modules%21field%21field.module/functio...

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Priority: Normal » Minor
Status: Reviewed & tested by the community » Fixed
Parent issue: » #2867810: Plan for Date 7.x-2.11 release

Committed. Thanks.

Status: Fixed » Closed (fixed)

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