Code cleanup: installer/installer

Cleans up useless uses of ```else``` and unnecessary indentation due to 'reverse-logic' in ```if```s.

So this improves readability of code by trying to eliminate the use of ```else``` statements and keeping the indentation to the bare minimum.

For instance, instead of wrapping stuff in multiple 'positive' ```if``` checks, simply do 'negative' ```if```checks and return early.
This means you don't need to nest if statements.

The same can be done with for/foreach loops, but then do ```continue``` in 'negative' ```if``` checks.

Also removed ```else``` statements if the previous ```if```` block ends with a return, like:
```
if($something)
{
   doStuff();;

   return $this;
}
else
{
   doOtherStuff();

   return $that;
}
```
Can be replaced with:
```
if($something)
{
   doStuff();;

   return $this;
}

doOtherStuff();

return $that;
```

### Testing Instructions
This affects the JInstaller, so test the installer in Joomla.

### Just one file?
This is the first file I went through to improve a bit (can of course be improved even more with more effort). If this fares well, I'll do other files too.

### Credits
The improvements are based on a few rules from PHP Object Callisthenics. Kudos to @rdohms for inspiring the PHP and Joomla developers with that!
Hmm.  Maybe not apparently.

```sh
phpcs --report=full --extensions=php --standard=build/phpcs/Joomla/ruleset.xml libraries/cms/installer/installer.php

FILE: libraries/cms/installer/installer.php
--------------------------------------------------------------------------------
FOUND 16 ERROR(S) AFFECTING 16 LINE(S)
--------------------------------------------------------------------------------
  495 | ERROR | Each line in a multi-line IF statement must begin with a
      |       | boolean operator
  497 | ERROR | Closing parenthesis of a multi-line IF statement must be on the
      |       | same line
  658 | ERROR | Each line in a multi-line IF statement must begin with a
      |       | boolean operator
  660 | ERROR | Closing parenthesis of a multi-line IF statement must be on the
      |       | same line
  728 | ERROR | Each line in a multi-line IF statement must begin with a
      |       | boolean operator
  730 | ERROR | Closing parenthesis of a multi-line IF statement must be on the
      |       | same line
  780 | ERROR | Each line in a multi-line IF statement must begin with a
      |       | boolean operator
  782 | ERROR | Closing parenthesis of a multi-line IF statement must be on the
      |       | same line
 1602 | ERROR | Expected "if (...)\n...{...}\n...else\n"; found "if
      |       | (...)\n...{...}\n...else "
 1607 | ERROR | Expected "if (...)\n...{...}\n...else\n"; found "if
      |       | (...)\n...{...}\n...else "
 1622 | ERROR | Expected "if (...)\n...{...}\n...else\n"; found "if
      |       | (...)\n...{...}\n...else "
 1644 | ERROR | Expected "if (...)\n...{...}\n...else\n"; found "if
      |       | (...)\n...{...}\n...else "
 1658 | ERROR | Expected "if (...)\n...{...}\n...else\n"; found "if
      |       | (...)\n...{...}\n...else "
 1674 | ERROR | Expected "if (...)\n...{...}\n...else\n"; found "if
      |       | (...)\n...{...}\n...else "
 1687 | ERROR | Expected "if (...)\n...{...}\n...else\n"; found "if
      |       | (...)\n...{...}\n...else "
 1692 | ERROR | Usage of ELSE IF is not allowed; use ELSEIF instead
--------------------------------------------------------------------------------
```