I'm trying to use provision-sync to sync from a normal drush alias, and it always fails.

Figured out it's because it uses d($source)->uri to detemine the domain name for the sites/domain/ path.

Guess what? d('@normal.drush.alias')->uri returns NULL:

To reproduce, create a simple drush alias and print d('alias')->uri

aegir@devshop:~$ drush php
Psy Shell v0.8.14 (PHP 7.0.32-0ubuntu0.16.04.1 — cli) by Justin Hileman
>>> d('@hostmaster.or.other.site')->uri
=> hostmaster.domain.com
>>> d('@custom.drush.alias')->uri
=> null

Patch and branch coming soon.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Jon Pugh created an issue. See original summary.

Jon Pugh’s picture

Jon Pugh’s picture

FileSize
1.21 KB

Fixed comment, check for prop before returning.

  • Jon Pugh committed 02b0f79 on 3012687-d-get-alias-props
    Issue #3012687 by Jon Pugh: Allow d() to pass through normal drush alias...
  • Jon Pugh committed 65c3440 on 3012687-d-get-alias-props
    Issue #3012687: Allow d() to pass through normal drush alias properties
    

  • Jon Pugh committed 02b0f79 on 7.x-3.x
    Issue #3012687 by Jon Pugh: Allow d() to pass through normal drush alias...
  • Jon Pugh committed 65c3440 on 7.x-3.x
    Issue #3012687: Allow d() to pass through normal drush alias properties
    
Jon Pugh’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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