Merge pull request #151 from jhedstrom/150-illegal-string-offset

Fix illegal string offset warning.
jhedstrom · Nov 28, 2017 · 4d11b5d · 4d11b5d
2 parents f900a3a + 55d1ac7
commit 4d11b5d
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/src/Drupal/Driver/DrushDriver.php b/src/Drupal/Driver/DrushDriver.php
@@ -135,10 +135,12 @@ public function bootstrap() {
   protected function isLegacyDrush() {
     try {
       // Try for a drush 9 version.
-      $version = unserialize($this->drush('version', [], ['format' => 'php']));
-      return version_compare($version['drush-version'], '9', '<=');
+      $version = trim($this->drush('version', [], ['format' => 'string']));
+      return version_compare($version, '9', '<=');
     }
     catch (\RuntimeException $e) {
+      // The version of drush is old enough that only `--version` was available,
+      // so this is a legacy version.
       return TRUE;
     }
   }