Problem/Motivation

FILE: /home/paulocs/AmbienteTeste/Drupal 8.9/drupal-8.9.0/modules/contrib/colorbox/colorbox.theme.inc
-----------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 2 WARNINGS AFFECTING 2 LINES
-----------------------------------------------------------------------------------------------------
 28 | WARNING | Unused variable $gallery_token.
 41 | WARNING | Unused variable $entity_bundle.
-----------------------------------------------------------------------------------------------------

Proposed resolution

Remove unused variables.

$gallery_token was introduced in: https://git.drupalcode.org/project/colorbox/-/commit/b022668b3ddc70feb36cf9bbc8d61511e0af4d52 but it is not used any more.

$gallery_token was introduced in: https://www.drupal.org/project/colorbox/issues/2692235

CommentFileSizeAuthor
#2 3173192-2.patch743 bytespaulocs
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

paulocs created an issue. See original summary.

paulocs’s picture

paulocs
Primary language Portuguese, Brazil
Location Belo Horizonte
CreditAttribution: paulocs at CI&T commented
Assigned: paulocs » Unassigned
Status: Active » Needs review
FileSize
3173192-2.patch743 bytes

A patch for it.

Neslee Canil Pinto’s picture

Neslee Canil Pinto
he/him
Primary language English
CreditAttribution: Neslee Canil Pinto at Specbee commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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