Using the same profile with multiple destinations fails when scheduling

When I attempted to create multiple schedules (daily, weekly, monthly) with multiple destinations (/daily, /weekly, /monthly) using the same profile, all the backups were saved into the /daily folder.

After doing a little digging I found the suspect code:

FILE: includes/profiles.inc

/**
   * Get the destination of the profile.
   */
  function get_destination() {
    backup_migrate_include('destinations');
    if (empty($this->destination) && !empty($this->destination_id)) {
      $this->destination = backup_migrate_get_destination($this->destination_id);
    }
    return empty($this->destination) ? NULL : $this->destination;
  }

This code ignores the 'destination' re-assignment' if '$this->destination' is NOT empty, which is what happens when this function gets called on the subsequent destinations.

The following code fixes this:

/**
   * Get the destination of the profile.
   */
  function get_destination() {
    backup_migrate_include('destinations');
    if ((empty($this->destination) && !empty($this->destination_id)) || 
        (!empty($this->destination) && !empty($this->destination_id) && 
        ($this->destination->destination_id != $this->destination_id))) {
      $this->destination = backup_migrate_get_destination($this->destination_id);
    }
    return empty($this->destination) ? NULL : $this->destination;
  }

I have included a patch as well.