Changed the REQUIRE line to comply with the FreeBSD handbook

The REQUIRE should be LOGIN, not DAEMON, per the FreeBSD
Porter's handbook, in the FreeBSD init script. This is because sickbeard runs as a non-root user and
isn't part of the base system. Removed sabnzbd from the REQUIRE as well
since sabnzbd is not required for sickbeard to start and so should not
be listed in the REQUIRE line
Added additional rc variables to the FreeBSD init script

Added rc variables to allow specification of the Sick Beard host, port,
web interface username, and web interface password in /etc/rc.conf so
that users don't have to modify the init script to change these values.
This was done because user's shouldn't be modifying the init script
since init scripts are typically installed by the package and are
overwritten with each update. This changes allows Sick Beard to be
safely packaged into a port when the day comes that it's ready for that.
Removed sickbeard_flags reference from command_args variable

Modified the FreeBSD init script by removing the "${sickbeard_flags}"
reference from command_args because, per the FreeBSD Porter's handbook,
the flags variable should never be part of command_args. This is because
if the flags variable is defined it is added to the command
automatically by rc(8) when the init script is run.

Unfortunately we currently can't define the flags variable because it
gets added as a flag to /usr/sbin/daemon, not SickBeard.py
Cleaned up the output of the status function of the FreeBSD init script

The "status" command showed the ps usage information in the event it was
run when Sick Beard wasn't running. This is because no PID is provided
to "ps -p" in that instance. I redirected STDERR of the "ps -p" command
to /dev/null to resovle the bug
Metadata: ALL -- Minor cleanup, standardizations, remove unused variables. Removed namespace (was incorrect anyways)
Metadata: XBMC -- Strip leading/trailing spaces off Actor's names.
Metadata: TIVO -- Override 'non supported' functions to prevent erroneous metadata from being created.
Per Tivo docs, the value for rating can only be a whole number or a half..  example: 1, 1.5, 2, 2.5, 3, 3.5, 4. We use 0 to 4 and also return a float with the precision of 1 place.. (1.0).. waiting to hear back if this is really not allowed.
Metadata: MB -- Re-order and cleaned up the .xml generation to match what MB3 is generating. Note that for mult-ep it looks like they aggregate the episodes into the first ep.
Merge branch 'change_metadata_updates_PR786-thezoggy' into development

* change_metadata_updates_PR786-thezoggy:

  Metadata: ALL
  Minor cleanup, standardizations, remove unused variables. Removed namespace (was incorrect anyways)

  Metadata: XBMC
  Strip leading/trailing spaces off Actor's names.

  Metadata: TIVO
  Override 'non supported' functions to prevent erroneous metadata from being created.
  Per Tivo docs, the value for rating can only be a whole number or a half..  example: 1, 1.5, 2, 2.5, 3, 3.5, 4.
  We use 0 to 4 and also return a float with the precision of 1 place.. (1.0).. waiting to hear back if this is really not allowed.

  Metadata: MB
  Re-order and cleaned up the .xml generation to match what MB3 is generating.
  Note that for mult-ep it looks like they aggregate the episodes into the first ep.

  closes gh-786
Redirect fixes and pep8 related corrections.
Merge pull request #787 from thezoggy/dev--redirect_fixes

Redirect fixes and pep8 related corrections.
Merge pull request #772 from nixalot/freebsd_init_update

Update the FreeBSD init script to comply with the standards in the FreeBSD Porter's Handbook
Merge remote-tracking branch 'main/development' into betaseries