[NFC] throw exceptions, single quotes, declare exceptions, remove unused param

[eileenmcnaughton]

Overview

Minor cleanup to known conventions

Before

• exceptions not declared
• non-strict checking used when one side is clearly a string
• a mis-cased function call
• double quotes when a single will do
• a function takes a parameter & entirely ignores it. This class is not extending anything

After

Above fixed

Technical Details

All minor stuff - fixing to reduce distractions

Comments

[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).