Unused variables

Open on Drupal.org →

Created on 13 June 2023, over 1 year ago

Updated 30 June 2023, over 1 year ago

FILE: D:\xampp\htdocs\drupal_10\web\modules\contrib\backup_migrate\backup_migrate.module
----------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
----------------------------------------------------------------------------------------
112 | WARNING | Unused variable $sources.
----------------------------------------------------------------------------------------

FILE: D:\xampp\htdocs\drupal_10\web\modules\contrib\backup_migrate\src\Core\Filter\DBExcludeFilter.php
------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
------------------------------------------------------------------------------------------------------
68 | WARNING | Unused variable $source_key.
------------------------------------------------------------------------------------------------------

FILE: D:\xampp\htdocs\drupal_10\web\modules\contrib\backup_migrate\src\Core\Filter\MetadataWriter.php
-----------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
-----------------------------------------------------------------------------------------------------
80 | WARNING | Unused variable $source.
-----------------------------------------------------------------------------------------------------

FILE: D:\xampp\htdocs\drupal_10\web\modules\contrib\backup_migrate\src\Core\Filter\Notify.php
---------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
---------------------------------------------------------------------------------------------
86 | WARNING | Unused variable $body.
---------------------------------------------------------------------------------------------

FILE: D:\xampp\htdocs\drupal_10\web\modules\contrib\backup_migrate\src\Core\Plugin\PluginManager.php
----------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
----------------------------------------------------------------------------------------------------
191 | WARNING | Unused variable $key.
----------------------------------------------------------------------------------------------------

FILE: D:\xampp\htdocs\drupal_10\web\modules\contrib\backup_migrate\src\Core\Source\MySQLiSource.php
---------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 3 WARNINGS AFFECTING 3 LINES
---------------------------------------------------------------------------------------------------
48 | WARNING | Unused variable $connection.
217 | WARNING | Unused variable $generator.
405 | WARNING | Unused variable $key.
---------------------------------------------------------------------------------------------------

🐛 Bug report

Status

Version

5.0

Component

Code

Created by

🇮🇳India arti_parmar

Live updates comments and jobs are added and updated live.

Sign in to follow issues

Comments & Activities

Issue created by @arti_parmar
Issue was unassigned.
Status changed to Needs review over 1 year ago9:15am 13 June 2023
Open in Jenkins → Open on Drupal.org →
Core: 9.5.x + Environment: PHP 7.4 & MySQL 5.7
last update over 1 year ago
25 pass
Comment over 1 year ago →
🇮🇳India arti_parmar
Kindly review patch.
Comment over 1 year ago →
🇮🇳India chandrashekhar_srijan
I have implemented the patch on my local machine using Backup and Migrate V5.0.x. All unused variable is now removed. I am attaching the screenshot for the reference.
Comment over 1 year ago →
🇮🇳India Jaspreet-Kaur
Reviewed! Patch #2 worked for me. Thanks
Comment over 1 year ago →
🇮🇳India Akshay kashyap
@arti_parmar

Thanks for the work.

I have reviewed and tested the #2 patch. and it looks good remove All unused variables in the backup and migrate the module. Moving to RTBC
Status changed to RTBC over 1 year ago5:39am 30 June 2023
Comment over 1 year ago →
🇮🇳India Akshay kashyap
Open in Jenkins → Open on Drupal.org →
Core: 9.5.x + Environment: PHP 7.4 & MySQL 5.7
last update over 1 year ago
25 pass
@akshay-kashyap opened merge request.

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024