I found the below issue while check through drupal-check tool.

drupal-check -ad web/modules/contrib/admin_toolbar
11/11 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%

------ -----------------------------------------------------------------------------------------------------------------------
Line admin_toolbar_tools/src/Plugin/Derivative/ExtraLinks.php
------ -----------------------------------------------------------------------------------------------------------------------
55 Constructor of class Drupal\admin_toolbar_tools\Plugin\Derivative\ExtraLinks has an unused parameter $base_plugin_id.
------ -----------------------------------------------------------------------------------------------------------------------

[ERROR] Found 1 error

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

vsujeetkumar created an issue. See original summary.

vsujeetkumar’s picture

Adding a patch that will remove the unused variable error for admin_toolbar_tools module.

vsujeetkumar’s picture

Status: Active » Needs review
vsujeetkumar’s picture

Assigned: vsujeetkumar » Unassigned
idebr’s picture

Status: Needs review » Needs work
+++ b/admin_toolbar_tools/src/Plugin/Derivative/ExtraLinks.php
@@ -53,6 +60,7 @@ class ExtraLinks extends DeriverBase implements ContainerDeriverInterface {
   public function __construct($base_plugin_id, EntityTypeManagerInterface $entity_type_manager, ModuleHandlerInterface $module_handler, RouteProviderInterface $route_provider, ThemeHandlerInterface $theme_handler) {
+    $this->basePluginId = $base_plugin_id;

Instead of an unused variable, we now have an unused class property.

How about we remove the variable from the ::create() return value instead?

idebr’s picture

Issue tags: -Drupal 9 compatibility

  • adriancid committed 8670706 on 8.x-2.x
    Issue #3109978 by vsujeetkumar, idebr, adriancid: Unused parameter $...
adriancid’s picture

Status: Needs work » Fixed

Thanks for the issue.

Status: Fixed » Closed (fixed)

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