Problem/Motivation

Discover if there are any problems when migrating to Drupal 9.

Proposed resolution

Ran the checking tool against commit 3eb2fca6:

./vendor/bin/drupal-check modules/contrib/term_csv_export_import/
 5/5 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%

 ------ --------------------------------------------------- 
  Line   src/Controller/ImportController.php                
 ------ --------------------------------------------------- 
  47     Call to deprecated function drupal_set_message().  
  80     Call to deprecated function drupal_set_message().  
  121    Call to deprecated function drupal_set_message().  
  129    Call to deprecated function drupal_set_message().  
  148    Call to deprecated function drupal_set_message().  
  244    Call to deprecated function drupal_set_message().  
  267    Call to deprecated function drupal_set_message().  
  276    Call to deprecated function drupal_set_message().  
  285    Call to deprecated function drupal_set_message().  
 ------ --------------------------------------------------- 


 [ERROR] Found 9 errors

Remaining tasks

Replace calls to deprecated code.

CommentFileSizeAuthor
#2 3109001-2.patch5.18 KBJohn Cook
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

John Cook created an issue. See original summary.

John Cook’s picture

John Cook CreditAttribution: John Cook at Creode commented
Status: Active » Needs review
Issue tags: +Drupal 9 compatibility, +ContributionWeekend2020
FileSize
3109001-2.patch5.18 KB

I've created a patch to address the deprecated code.

Ankush_03’s picture

Ankush_03
Location Gurgaon, India 🇮🇳
CreditAttribution: Ankush_03 at Srijan | A Material+ Company for Srijan | A Material+ Company commented
Status: Needs review » Reviewed & tested by the community

Patch Working fine !

el1_1el’s picture

el1_1el CreditAttribution: el1_1el commented
Status: Reviewed & tested by the community » Fixed

Thanks!

Status: Fixed » Closed (fixed)

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