Issue 217: Swap and adjust the boolean logic to be more JS-safe.

Reverse the bitwise not operations through a double negative.
Add additional checks extra checks.

There is also a ternary being used in the original code that is pointless.
The logic is effectively:
```
  true ?? true : false
```
Get rid of this pointless ternary.

app/services/userService.js

@@ -12,16 +12,16 @@ core.service("UserService", function ($q, StorageService, User, WsApi) {
         userEvents.notify('FETCH');
 
         return WsApi.fetch(currentUser.getMapping().instantiate).then(function (res) {
-            var body = !res.body ? {} : angular.fromJson(res.body);
-            var credentials = { role: !currentUser.role ? appConfig.anonymousRole : currentUser.role };
+            var body = !!res && !!res.body ? angular.fromJson(res.body) : {};
+            var credentials = { role: !!currentUser.role ? currentUser.role : appConfig.anonymousRole };
 
             // Only change credentials when packet structure is valid.
-            if (!!body.payload && !!body.payload.Credentials) {
+            if (!!body && !!body.payload && !!body.payload.Credentials) {
               delete sessionStorage.role;
               credentials = angular.fromJson(res.body).payload.Credentials;
             }
 
-            currentUser.anonymous = !credentials.role || credentials.role === appConfig.anonymousRole ? true : false;
+            currentUser.anonymous = !credentials.role || credentials.role === appConfig.anonymousRole;
 
             // Cannot have a token for the anonymous role.
             if (currentUser.anonymous) {