php drupal-check.phar modules/contrib/simple_pass_reset/
 5/5 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%

 ------ ---------------------------------------------------------------------------------------- 
  Line   src/AccessChecks/ResetPassAccessCheck.php                                               
 ------ ---------------------------------------------------------------------------------------- 
  36     Call to deprecated function drupal_set_message().                                       
  38     Call to deprecated method getUsername() of class Drupal\Core\Session\AccountInterface.  
  39     Call to deprecated method getUsername() of class Drupal\Core\Session\AccountInterface.  
  40     Call to deprecated method url() of class Drupal.                                        
  50     Call to deprecated function drupal_set_message().                                       
  50     Call to deprecated method url() of class Drupal.                                        
 ------ ---------------------------------------------------------------------------------------- 

 ------ ------------------------------------------------------------ 
  Line   src/Controller/User.php                                     
 ------ ------------------------------------------------------------ 
  32     Call to deprecated method entityManager() of class Drupal.  
 ------ ------------------------------------------------------------ 

                                                                                                                        
 [ERROR] Found 7 errors 

Comments

Sergiu Stici created an issue. See original summary.

sergiu stici’s picture

Status: Active » Needs review
StatusFileSize
new3.52 KB

Here's the patch, please review.

chris matthews’s picture

jcnventura’s picture

Title: Deprecated Code Report » Drupal 9 readiness

Just needs the following:

diff --git a/simple_pass_reset.info.yml b/simple_pass_reset.info.yml
index 44484c1..8716f5d 100644
--- a/simple_pass_reset.info.yml
+++ b/simple_pass_reset.info.yml
@@ -3,3 +3,4 @@ type: module
 description: 'When resetting a password, skip the unnecessary and confusing one-time login form.'
 package: User Management
 core: 8.x
+core_version_requirement: ^8 || ^9

That @rahul.nahar001 added 9 days ago in #3141683: Automated Drupal 9 compatibility fixes, and that @Mistrae added 25 days ago in #3145690: Drupal 9 readiness. Also, people, please search the issue queue before adding new issues.

jcnventura’s picture

Issue tags: +Drupal 9 compatibility
raunak.singh’s picture

Assigned: Unassigned » raunak.singh
Status: Needs review » Needs work

Need work as per #4 comment

raunak.singh’s picture

Assigned: raunak.singh » Unassigned
Status: Needs work » Needs review
StatusFileSize
new3.88 KB
new294 bytes

Re-roll patch as per #4
Please review this patch.

  • mukila committed 4041f81 on 8.x-1.x
    Issue #3091696: Drupal 9 readiness
    
mukila’s picture

mukila’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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