PHP 8.2 Deprecated function: Creation of dynamic property Drupal\multiple_registration\Plugin\Derivative\MultipleRegistrationLocalTasks::$base_plugin_id is deprecated

Created on 31 August 2023, 10 months ago
Updated 7 October 2023, 9 months ago

Looks like it's caused by a typo in class MultipleRegistrationLocalTasks

$this->base_plugin_id = $base_plugin_id;
should be
$this->basePluginId = $base_plugin_id;

Patch attached

🐛 Bug report
Status

Fixed

Version

3.0

Component

Code

Created by

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

Comments & Activities

  • Issue created by @mikcat
  • 🇮🇳India Mohd Sahzad

    i have fixed this issue PHP 8.2 Deprecated function: Creation of dynamic property Drupal\multiple_registration\Plugin\Derivative\MultipleRegistrationLocalTasks::$base_plugin_id is deprecated

  • Assigned to gun2dru
  • @gun2dru opened merge request.
  • Status changed to Needs review 9 months ago
  • 🇧🇾Belarus gun2dru

    As a part of Merge request !14 fixed deprecation issues:

    vendor/bin/drupal-check -d modules/contrib/multiple_registration/
     10/10 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%
    
     ------ -------------------------------------------------- 
      Line   multiple_registration.module                      
     ------ -------------------------------------------------- 
      87     Call to an undefined method                       
             Drupal\Core\Form\FormInterface::getEntity().      
      176    Call to an undefined method                       
             Drupal\Core\Form\FormInterface::getEntity().      
      395    Variable $custom_form_mode might not be defined.  
     ------ -------------------------------------------------- 
    
     ------ ------------------------------------------------- 
      Line   src/AvailableUserRolesService.php                
     ------ ------------------------------------------------- 
      54     Call to an undefined method                      
             Drupal\user\RoleStorageInterface::getStorage().  
     ------ ------------------------------------------------- 
    
     ------ --------------------------------------------------- 
      Line   src/Controller/MultipleRegistrationController.php  
     ------ --------------------------------------------------- 
      248    Variable $rows might not be defined.               
     ------ --------------------------------------------------- 
    
     ------ ----------------------------------------------------------------------- 
      Line   src/Plugin/Derivative/MultipleRegistrationLocalTasks.php               
     ------ ----------------------------------------------------------------------- 
      42     Access to an undefined property                                        
             Drupal\multiple_registration\Plugin\Derivative\MultipleRegistrationLo  
             calTasks::$base_plugin_id.                                             
     ------ ----------------------------------------------------------------------- 
    
     [ERROR] Found 6 errors
    
    10/10 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%
    
    [OK] No errors                                                                 
    
  • First commit to issue fork.
  • 🇺🇦Ukraine ysamoylenko

    @gun2dru Thank you for your MR and for fixing the deprecation issues. The changes will be merged soon.

  • Status changed to RTBC 9 months ago
  • Status changed to Fixed 9 months ago
  • 🇺🇦Ukraine ysamoylenko

    The changes were applied to the latest module dev version. Thanks for your contribution.

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

Production build 0.69.0 2024