Unnecessary else in metatag_post_update_convert_author_data()

Problem/Motivation

Inside of metatag.post_update.php there is a function metatag_post_update_convert_author_data there is an unused else at this point:

if ($sandbox['total_records'] > 0) {
    return (string) t('Processed @processed of @total overridden Metatag records.', [
      '@processed' => $sandbox['records_processed'],
      '@total' => $sandbox['total_records'],
    ]);
  }
  else {
    return (string) t("There were no overridden Metatag records.");
  }

And there is a best practice recommendation to avoid using else if isn't necessary. https://freek.dev/2212-avoid-using-else

Proposed resolution

We can just remove this unnecessary else

Remaining tasks

User interface changes

API changes

Data model changes