php drupal-check.phar modules/contrib/fz152/
 16/16 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%

 ------ --------------------------------------------------- 
  Line   src/Form/Fz152Settings.php                         
 ------ --------------------------------------------------- 
  66     Call to deprecated function drupal_set_message().  
 ------ --------------------------------------------------- 

 ------ --------------------------------------------------- 
  Line   src/Form/Fz152SettingsPage.php                     
 ------ --------------------------------------------------- 
  76     Call to deprecated function drupal_set_message().  
 ------ --------------------------------------------------- 

                                                                                                                        
 [ERROR] Found 2 errors
CommentFileSizeAuthor
#5 3085382-5.patch238 bytesmo_farhaz
#2 deprecated-3085382-2.patch1.13 KBsergiu stici

Comments

Sergiu Stici created an issue. See original summary.

sergiu stici’s picture

sergiu stici commented
Status: Active » Needs review
StatusFileSize
new deprecated-3085382-2.patch1.13 KB

Here is the patch, please review.

mo_farhaz’s picture

mo_farhaz commented
Assigned: Unassigned » mo_farhaz
Status: Needs review » Reviewed & tested by the community

Thanks for the patch @sergi stici

mo_farhaz’s picture

mo_farhaz commented

dd core_version_requirement: ^8 || ^9 to fz152.info.yml to designate that the module is compatible with Drupal 9. See https://www.drupal.org/node/3070687.

mo_farhaz’s picture

mo_farhaz commented
Status: Reviewed & tested by the community » Needs review
StatusFileSize
new 3085382-5.patch238 bytes

Core version requirement added.
please review it.

mo_farhaz’s picture

mo_farhaz commented
Assigned: mo_farhaz » Unassigned
dillix’s picture

dillix commented
Status: Needs review » Reviewed & tested by the community

I tested latest version and patch #5 with drupal rector and there is no errors.

adubovskoy’s picture

adubovskoy commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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