diff --git a/extensions/guacamole-auth-jdbc/modules/guacamole-auth-jdbc-base/src/main/java/org/apache/guacamole/auth/jdbc/usergroup/UserGroupService.java b/extensions/guacamole-auth-jdbc/modules/guacamole-auth-jdbc-base/src/main/java/org/apache/guacamole/auth/jdbc/usergroup/UserGroupService.java index 0f4a21683..3c42572ce 100644 --- a/extensions/guacamole-auth-jdbc/modules/guacamole-auth-jdbc-base/src/main/java/org/apache/guacamole/auth/jdbc/usergroup/UserGroupService.java +++ b/extensions/guacamole-auth-jdbc/modules/guacamole-auth-jdbc-base/src/main/java/org/apache/guacamole/auth/jdbc/usergroup/UserGroupService.java @@ -21,8 +21,6 @@ package org.apache.guacamole.auth.jdbc.usergroup; import com.google.inject.Inject; import com.google.inject.Provider; -import java.util.Collection; -import java.util.Collections; import org.apache.guacamole.auth.jdbc.base.ModeledDirectoryObjectMapper; import org.apache.guacamole.auth.jdbc.base.ModeledDirectoryObjectService; import org.apache.guacamole.GuacamoleClientException; @@ -147,8 +145,8 @@ public class UserGroupService extends ModeledDirectoryObjectService existing = userGroupMapper.select(Collections.singleton(model.getIdentifier())); - if (!existing.isEmpty()) + UserGroupModel existing = userGroupMapper.selectOne(model.getIdentifier()); + if (existing != null) throw new GuacamoleClientException("Group \"" + model.getIdentifier() + "\" already exists."); // Create base entity object, implicitly populating underlying entity ID