These warnings show up three (3) times on verifying a Remote Import server:

  • Declaration of Provision_Service_remote_import_hostmaster::fetch_site($site) should be compatible with Provision_Service_remote_import::fetch_site() hostmaster.php:222
  • Declaration of Provision_Service_remote_import_hostmaster::fetch($path, $dest) should be compatible with Provision_Service::fetch($path = NULL) hostmaster.php:222

Not sure if it affects anything yet.

CommentFileSizeAuthor
#5 overriden-2826739-5.patch1.42 KBhelmo
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

colan created an issue. See original summary.

colan’s picture

colan
he/him
Primary language English
Location Toronto 🇨🇦
CreditAttribution: colan at Colan Schwartz Consulting commented

This is currently preventing the process from working. For a workaround, see #2827791: List Remote Sites task produces no results with warnings.

colan’s picture

colan
he/him
Primary language English
Location Toronto 🇨🇦
CreditAttribution: colan at Colan Schwartz Consulting commented
Priority: Normal » Critical
colan’s picture

colan
he/him
Primary language English
Location Toronto 🇨🇦
CreditAttribution: colan at Colan Schwartz Consulting commented
Priority: Critical » Normal
helmo’s picture

helmo CreditAttribution: helmo as a volunteer and at Initfour websolutions commented
Status: Active » Needs review
FileSize
overriden-2826739-5.patch1.42 KB

This totally untested patch might be what we need here.

  • colan committed 7abfd46 on 7.x-3.x authored by helmo 
    Issue #2826739 by helmo: Updated method signatures to match class...
colan’s picture

colan
he/him
Primary language English
Location Toronto 🇨🇦
CreditAttribution: colan at Colan Schwartz Consulting commented
Status: Needs review » Fixed

Looks good in my testing. Warnings go away & site appears to import correctly.

Status: Fixed » Closed (fixed)

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