UC-138 SQL to update the user_fbconnect table
UC-138 Add numbering to steps
UC-138 Enforce having the right fbAppId or zero.

* Also assume there is only one mapping returned by all load functions
* Simplfy logic based on above
* Rearrange methods to consolidate class vs instance methods
* Rearrange methods to be closer together if they are relted

There is on exception to the last point, 'loadFromDB'.  Moving this to the 'right' place would have made the code changes that also happened harder to see.  Will move this later.
UC-138 Augment the TODO
UC-138 Do some error checking
UC-138 Simplify the row selection
UC-138 Simplify a bit more ...
UC-138 Make the alters faster
UC-138 Code review updates

* Added unit test
* Added integration test
* more constants
* less globals
UC-138 Make sure the new tests pass in the wild
UC-138 Code review updates

* Moved logging of DB failures to a different place
* Fixed bad flipped logic on an if/else
* Removed return type from controller that has nothing to return
Merge pull request #5458 from Wikia/UC-138

UC-138 Update the user_fbconnect table and code that accesses it