tmp: add default roles on the fly

... just for testing at the moment. System roles should really be
assigned to the user beforehand as part of the onboarding process...

src/main/java/org/ohdsi/webapi/shiro/PermissionManager.java

@@ -213,10 +213,12 @@ public UserEntity registerUser(final String login, final String name, final User
         removeAllUserRolesFromUser(login, user);
         // add back just the given newUserRoles:
         addRolesForUser(login, userOrigin, user, newUserRoles, false);
+        // make sure the default roles are there: TODO - discuss if really necessary....
+        addDefaultRolesForUser(login, userOrigin, user, defaultRoles);
       }
       // get user again, fresh from db with all new roles:
       user = userRepository.findOne(user.getId());
-      return user;
+      return user;  // >>>>>>>>>> RETURN! Add else for readability???
     }
 
     checkRoleIsAbsent(login, false, "User with such login has been improperly removed from the database. " +
@@ -232,7 +234,7 @@ public UserEntity registerUser(final String login, final String name, final User
     this.addUser(user, personalRole, userOrigin, null);
     addRolesForUser(login, userOrigin, user, newUserRoles, false);
     addDefaultRolesForUser(login, userOrigin, user, defaultRoles);
-    // // get user again, fresh from db with all new roles:
+    // get user again, fresh from db with all new roles:
     user = userRepository.findOne(user.getId());
     return user;
   }
@@ -500,15 +502,15 @@ private UserRoleEntity addUser(final UserEntity user, final RoleEntity role,
                                  final UserOrigin userOrigin, final String status) {
     UserRoleEntity relation = this.userRoleRepository.findByUserAndRole(user, role);
     if (relation == null) {
-      logger.debug("The system role={} is new for this user. Adding...", role.getName());
+      logger.debug("The role={} is new for this user. Adding...", role.getName());
       relation = new UserRoleEntity();
       relation.setUser(user);
       relation.setRole(role);
       relation.setStatus(status);
       relation.setOrigin(userOrigin);
       relation = this.userRoleRepository.save(relation);
     } else {
-        logger.debug("The user already had the system role={}", role.getName());
+        logger.debug("The user already had the role={}", role.getName());
     }
 
     return relation;