Problem/Motivation

drupal-check results on commit hash:
source : [git] https://git.drupal.org/project/quick_node_clone f06e4252ebfe7cd91b4f02b89f9197bdae9bc4a6
source : https://cgit.drupalcode.org/quick_node_clone 


 ------ --------------------------------------------------- 
  Line   src/Form/QuickNodeCloneNodeForm.php                
 ------ --------------------------------------------------- 
  42     Call to deprecated method link() of class          
         Drupal\Core\Entity\EntityInterface.                
  56     Call to deprecated function drupal_set_message().  
  76     Call to deprecated function drupal_set_message().  
 ------ --------------------------------------------------- 

 [ERROR] Found 3 errors

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

CommentFileSizeAuthor
#4 quick_node_clone-fixed_deprecated_code-3042819-4.patch1.73 KBi-trokhanenko
#2 quick_node_clone-fixed_deprecated_code-3042819-2.patch1.33 KBi-trokhanenko

Comments

mcdwayne created an issue. See original summary.

i-trokhanenko’s picture

i-trokhanenko
Location Lutsk 🇺🇦
commented
Status: Active » Needs review
StatusFileSize
new quick_node_clone-fixed_deprecated_code-3042819-2.patch1.33 KB

Please review!

Status: Needs review » Needs work

The last submitted patch, 2: quick_node_clone-fixed_deprecated_code-3042819-2.patch, failed testing. View results

i-trokhanenko’s picture

i-trokhanenko
Location Lutsk 🇺🇦
commented
Status: Needs work » Needs review
StatusFileSize
new quick_node_clone-fixed_deprecated_code-3042819-4.patch1.73 KB

Used toString() for Link object. Please review #4 patch!

Roman Dyn’s picture

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

Good job! Change looks good. Please commit.

Anonymous’s picture

Anonymous (not verified) commented
Status: Reviewed & tested by the community » Fixed

Committed, thanks!

Status: Fixed » Closed (fixed)

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