php drupal-check.phar modules/contrib/high_contrast/
 9/9 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%

 ------ ---------------------------------------------------- 
  Line   src/Form/HighContrastConfigurationForm.php          
 ------ ---------------------------------------------------- 
  195    Call to deprecated function file_unmanaged_copy().  
 ------ ---------------------------------------------------- 

                                                                                                                        
 [ERROR] Found 1 error
CommentFileSizeAuthor
#3 deprecated-3087000-3.patch2.91 KBneograph734
#2 deprecated-3087000-2.patch2.15 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-3087000-2.patch2.15 KB

Here is the patch, please review.

neograph734’s picture

neograph734
he/him
Location Netherlands
commented
StatusFileSize
new deprecated-3087000-3.patch2.91 KB

Thanks for the effort. In file_unmanaged_copy() the second parameter could be omitted, but this is no longer the case with fileSystem->copy(). I had to alter the patch a bit, please find it attached.

pifagor’s picture

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

  • pifagor committed 8591eb9 on 8.x-1.x authored by Neograph734 
    Issue #3087000 by Sergiu Stici, pifagor, Neograph734: Deprecated Code...
pifagor’s picture

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

Status: Fixed » Closed (fixed)

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