Merge pull request #5082 from garychapman/bot-helpers-changes

Fixed code that checks for empty object

packages/rocketchat-bot-helpers/server/index.js

@@ -76,7 +76,7 @@ class BotHelpers {
 	// "public" properties accessed by getters
 	// allUsers / onlineUsers return whichever properties are enabled by settings
 	get allUsers() {
-		if (!this.userFields.length) {
+		if (!Object.keys(this.userFields).length) {
 			this.requestError();
 			return false;
 		} else {
@@ -84,7 +84,7 @@ class BotHelpers {
 		}
 	}
 	get onlineUsers() {
-		if (!this.userFields.length) {
+		if (!Object.keys(this.userFields).length) {
 			this.requestError();
 			return false;
 		} else {