FILE: D:\xampp\htdocs\drupal_10\web\modules\contrib\sitemap\src\Plugin\Block\SitemapSyndicateBlock.php
------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
------------------------------------------------------------------------------------------------------
109 | WARNING | Unused variable $config.
------------------------------------------------------------------------------------------------------

FILE: D:\xampp\htdocs\drupal_10\web\modules\contrib\sitemap\src\Plugin\Sitemap\Vocabulary.php
---------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
---------------------------------------------------------------------------------------------
531 | WARNING | Unused variable $route.
---------------------------------------------------------------------------------------------

CommentFileSizeAuthor
#2 3367619-2.patch1.44 KBarti_parmar

Issue fork sitemap-3367619

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

arti_parmar created an issue. See original summary.

arti_parmar’s picture

arti_parmar commented
Assigned: arti_parmar » Unassigned
Category: Task » Bug report
Status: Active » Needs review
StatusFileSize
new 3367619-2.patch1.44 KB

Kindly review patch.

mparker17 made their first commit to this issue’s fork.

mparker17’s picture

mparker17
he/him
Primary language English
Location UTC-5
commented
Title: Unused variables. » Remove unused variables

Update the title

mparker17 opened merge request !31

mparker17’s picture

mparker17
he/him
Primary language English
Location UTC-5
commented
Status: Needs review » Fixed

Tests seem to pass, and the code looks good; let's merge. Thanks @arti_parmar!

  • mparker17 committed eba095d4 on 8.x-2.x 
    Issue #3367619 by mparker17, arti_parmar: Remove unused variables
mparker17’s picture

mparker17
he/him
Primary language English
Location UTC-5
commented

Note that this change was released in sitemap-8.x-2.0-beta7.

Status: Fixed » Closed (fixed)

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