Fix issues reported by PHPStan

Created on 15 April 2024, 12 months ago
Updated 30 April 2024, 11 months ago

Problem/Motivation

See https://git.drupalcode.org/project/domain_menu_access/-/jobs/1330973

 ------ ----------------------------------------------------------------------------------------------------- 
  Line   modules/domain_menu_access_menu_block/src/Plugin/Block/DomainMenuAccessMenuMenuBlock.php             
 ------ ----------------------------------------------------------------------------------------------------- 
  20     Class                                                                                                
         Drupal\domain_menu_access_menu_block\Plugin\Block\DomainMenuAccessMenuMenuBlock                      
         extends unknown class Drupal\menu_block\Plugin\Block\MenuBlock.                                      
         💡 Learn more at https://phpstan.org/user-guide/discovering-symbols                                  
  20     Non-abstract class                                                                                   
         Drupal\domain_menu_access_menu_block\Plugin\Block\DomainMenuAccessMenuMenuBlock                      
         contains abstract method create() from interface                                                     
         Drupal\Core\Plugin\ContainerFactoryPluginInterface.                                                  
  26     Call to an undefined method                                                                          
         Drupal\domain_menu_access_menu_block\Plugin\Block\DomainMenuAccessMenuMenuBlock::getDerivativeId().  
  30     Drupal\domain_menu_access_menu_block\Plugin\Block\DomainMenuAccessMenuMenuBlock::build()             
         calls parent::build() but                                                                            
         Drupal\domain_menu_access_menu_block\Plugin\Block\DomainMenuAccessMenuMenuBlock                      
         does not extend any class.                                                                           
  32     Access to an undefined property                                                                      
         Drupal\domain_menu_access_menu_block\Plugin\Block\DomainMenuAccessMenuMenuBlock::$menuTree.          
         💡 Learn more:                                                                                       
            https://phpstan.org/blog/solving-phpstan-access-to-undefined-property                             
  35     Access to an undefined property                                                                      
         Drupal\domain_menu_access_menu_block\Plugin\Block\DomainMenuAccessMenuMenuBlock::$configuration.     
         💡 Learn more:                                                                                       
            https://phpstan.org/blog/solving-phpstan-access-to-undefined-property                             
  36     Access to an undefined property                                                                      
         Drupal\domain_menu_access_menu_block\Plugin\Block\DomainMenuAccessMenuMenuBlock::$configuration.     
         💡 Learn more:                                                                                       
            https://phpstan.org/blog/solving-phpstan-access-to-undefined-property                             
  40     Access to an undefined property                                                                      
         Drupal\domain_menu_access_menu_block\Plugin\Block\DomainMenuAccessMenuMenuBlock::$configuration.     
         💡 Learn more:                                                                                       
            https://phpstan.org/blog/solving-phpstan-access-to-undefined-property                             
  41     Access to an undefined property                                                                      
         Drupal\domain_menu_access_menu_block\Plugin\Block\DomainMenuAccessMenuMenuBlock::$configuration.     
         💡 Learn more:                                                                                       
            https://phpstan.org/blog/solving-phpstan-access-to-undefined-property                             
  42     Access to an undefined property                                                                      
         Drupal\domain_menu_access_menu_block\Plugin\Block\DomainMenuAccessMenuMenuBlock::$configuration.     
         💡 Learn more:                                                                                       
            https://phpstan.org/blog/solving-phpstan-access-to-undefined-property                             
  43     Access to an undefined property                                                                      
         Drupal\domain_menu_access_menu_block\Plugin\Block\DomainMenuAccessMenuMenuBlock::$configuration.     
         💡 Learn more:                                                                                       
            https://phpstan.org/blog/solving-phpstan-access-to-undefined-property                             
  61     Access to an undefined property                                                                      
         Drupal\domain_menu_access_menu_block\Plugin\Block\DomainMenuAccessMenuMenuBlock::$menuTree.          
         💡 Learn more:                                                                                       
            https://phpstan.org/blog/solving-phpstan-access-to-undefined-property                             
  79     Access to an undefined property                                                                      
         Drupal\domain_menu_access_menu_block\Plugin\Block\DomainMenuAccessMenuMenuBlock::$menuTree.          
         💡 Learn more:                                                                                       
            https://phpstan.org/blog/solving-phpstan-access-to-undefined-property                             
  99     Access to an undefined property                                                                      
         Drupal\domain_menu_access_menu_block\Plugin\Block\DomainMenuAccessMenuMenuBlock::$menuTree.          
         💡 Learn more:                                                                                       
            https://phpstan.org/blog/solving-phpstan-access-to-undefined-property                             
  107    Access to an undefined property                                                                      
         Drupal\domain_menu_access_menu_block\Plugin\Block\DomainMenuAccessMenuMenuBlock::$configuration.     
         💡 Learn more:                                                                                       
            https://phpstan.org/blog/solving-phpstan-access-to-undefined-property                             
  107    Access to an undefined property                                                                      
         Drupal\domain_menu_access_menu_block\Plugin\Block\DomainMenuAccessMenuMenuBlock::$configuration.     
         💡 Learn more:                                                                                       
            https://phpstan.org/blog/solving-phpstan-access-to-undefined-property                             
  114    Access to an undefined property                                                                      
         Drupal\domain_menu_access_menu_block\Plugin\Block\DomainMenuAccessMenuMenuBlock::$menuTree.          
         💡 Learn more:                                                                                       
            https://phpstan.org/blog/solving-phpstan-access-to-undefined-property                             
  128    Access to an undefined property                                                                      
         Drupal\domain_menu_access_menu_block\Plugin\Block\DomainMenuAccessMenuMenuBlock::$menuTree.          
         💡 Learn more:                                                                                       
            https://phpstan.org/blog/solving-phpstan-access-to-undefined-property                             
  135    Access to an undefined property                                                                      
         Drupal\domain_menu_access_menu_block\Plugin\Block\DomainMenuAccessMenuMenuBlock::$menuTree.          
         💡 Learn more:                                                                                       
            https://phpstan.org/blog/solving-phpstan-access-to-undefined-property                             
  136    Access to an undefined property                                                                      
         Drupal\domain_menu_access_menu_block\Plugin\Block\DomainMenuAccessMenuMenuBlock::$menuTree.          
         💡 Learn more:                                                                                       
            https://phpstan.org/blog/solving-phpstan-access-to-undefined-property                             
  144    Call to an undefined method                                                                          
         Drupal\domain_menu_access_menu_block\Plugin\Block\DomainMenuAccessMenuMenuBlock::getBlockLabel().    
  144    Call to an undefined method                                                                          
         Drupal\domain_menu_access_menu_block\Plugin\Block\DomainMenuAccessMenuMenuBlock::label().            
  152    Access to an undefined property                                                                      
         Drupal\domain_menu_access_menu_block\Plugin\Block\DomainMenuAccessMenuMenuBlock::$configuration.     
         💡 Learn more:                                                                                       
            https://phpstan.org/blog/solving-phpstan-access-to-undefined-property                             
  180    \Drupal calls should be avoided in classes, use dependency injection                                 
         instead                                                                                              
  190    Drupal\domain_menu_access_menu_block\Plugin\Block\DomainMenuAccessMenuMenuBlock::getCacheContexts()  
         calls parent::getCacheContexts() but                                                                 
         Drupal\domain_menu_access_menu_block\Plugin\Block\DomainMenuAccessMenuMenuBlock                      
         does not extend any class.                                                                           
 ------ ----------------------------------------------------------------------------------------------------- 
 ------ ---------------------------------------------------------------------- 
  Line   src/Plugin/Block/DomainMenuAccessMenuBlock.php                        
 ------ ---------------------------------------------------------------------- 
  68     \Drupal calls should be avoided in classes, use dependency injection  
         instead                                                               
 ------ ---------------------------------------------------------------------- 
 [ERROR] Found 26 errors    

Steps to reproduce

Run PHPStan

Proposed resolution

Fix issues

Remaining tasks

Create MR

User interface changes

-

API changes

-

Data model changes

-

📌 Task
Status

Fixed

Version

2.0

Component

Code

Created by

🇧🇪Belgium tim-diels Belgium 🇧🇪

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

Production build 0.71.5 2024