Remove extraneous elses

[eileenmcnaughton]

Overview

Further removal of elses directly after a return

Same as #19168 but a different file

This looks like a huge code change but it's just removing meaningless elses and reformatting view using
https://github.com/civicrm/civicrm-core/pull/19171/files?w=1 for better readability

Before

Constructs like

if ($a = 1) {
  return;
}
else {
  // do the thing
}

After

if ($a = 1) {
  return;
}
 // do the thing

Technical Details

[civibot]

(Standard links)

• If this is your first pull-request for CiviCRM, please browse CONTRIBUTING.md for information about the development and testing processes.
• If you are reviewing this pull-request, you may wish to consult the test sites and the Review Standards (long template, short template).

[eileenmcnaughton]

This looks wonky in web view but

[eileenmcnaughton]

double return

[eileenmcnaughton]

sigh @seamuslee001 all these files are hard to make 'the change' in

[seamuslee001]

If Jenkins likes it then I do too