8/8 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%

 ------ ---------------------------------------------------------------------- 
  Line   src/Repository/CovidRepository.php                                    
 ------ ---------------------------------------------------------------------- 
  25     Call to deprecated function drupal_set_message():                     
         in drupal:8.5.0 and is removed from drupal:9.0.0.                     
         Use \Drupal\Core\Messenger\MessengerInterface::addMessage() instead.  
 ------ ---------------------------------------------------------------------- 

                                                                                                                        
 [ERROR] Found 1 error

------ ----------------------------------------------------------------------
Line src/Repository/CovidRepository.php
------ ----------------------------------------------------------------------
41 Call to deprecated function drupal_set_message():
in drupal:8.5.0 and is removed from drupal:9.0.0.
Use \Drupal\Core\Messenger\MessengerInterface::addMessage() instead.
65 Call to deprecated function drupal_set_message():
in drupal:8.5.0 and is removed from drupal:9.0.0.
Use \Drupal\Core\Messenger\MessengerInterface::addMessage() instead.
81 Call to deprecated function drupal_set_message():
in drupal:8.5.0 and is removed from drupal:9.0.0.
Use \Drupal\Core\Messenger\MessengerInterface::addMessage() instead.
------ ----------------------------------------------------------------------

CommentFileSizeAuthor
#3 D9_Readiness_3124037-3.patch5.44 KBabhijeet.kumar2107
#2 removed-deprecated-code.patch1.15 KBsahana _n

Comments

Sahana _N created an issue. See original summary.

sahana _n’s picture

sahana _n commented
Status: Active » Needs review
StatusFileSize
new removed-deprecated-code.patch1.15 KB

Please review the patch.

abhijeet.kumar2107’s picture

abhijeet.kumar2107 commented
Version: » 8.x-1.x-dev
Issue summary: View changes
Issue tags: +Drupal 9 porting dayDrupal 9 readinessDrupal 9 compatibility
StatusFileSize
new D9_Readiness_3124037-3.patch5.44 KB

Fixed latest deprecated code in latest patch. Please review

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Issue tags: -Drupal 9 porting dayDrupal 9 readinessDrupal 9 compatibility +Drupal 9 porting day, +Drupal 9 compatibility

Issue tags must be separated by commas, not spaces.

abhijeet.kumar2107’s picture

abhijeet.kumar2107 commented

Thanks @kiamlaluno for valuable feedback and will keep in mind in the future.

nitesh624’s picture

nitesh624
he/him
Primary language English
Location Ranchi, India
commented
Assigned: Unassigned » nitesh624
nitesh624’s picture

nitesh624
he/him
Primary language English
Location Ranchi, India
commented
Assigned: nitesh624 » Unassigned
kapil17’s picture

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

Hello @abhijeet.kumar2107

Just applied the patch and its throwing an error. when you are applying dependency injection please make sure the order of using service arguments in services.yml and constructor of the class should be same. I fixed that issue.

Thanks for the patch.

kapil17’s picture

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

Status: Fixed » Closed (fixed)

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