populateAcquiaSettings() method wrong condition

Problem/Motivation

There is a method populateAcquiaSettings() of a class AcquiaSubsription to which we come from acquia_search_init() function. This method populates the settings from various sources. If the variable table is already populated, it uses those credentials instead of Acquia cloud. If local credentials don't exist, it attempts to set with Acquia cloud:

  public function populateAcquiaSettings() {
    // Attempt to set local credentials first.
    $this->setLocalCredentials();

    // If local credentials don't exist, attempt to set with Acquia cloud.
    if (!$this->settings) {
      $this->setAcquiaCloudCredentials();
    }
  }

The problem is that setLocalCredentials() method, which runs first, always fills $this->settings property. In case local credentials are not set, $this->settings will contain AcquiaSettings object instance with empty properties:

    if (!$identifier || !$key || !$application_uuid) {
      $this->settings = new AcquiaSettings();
      return;
    }

Due to this fact the if (!$this->settings) condition will not work and setAcquiaCloudCredentials() method will never run.