Commit 1e165540 by Jason Song Committed by GitHub

Merge pull request #742 from lepdou/search_user

return users when search user keyword is empty
parents b9bca870 ec448a1a
...@@ -11,7 +11,9 @@ import java.util.List; ...@@ -11,7 +11,9 @@ import java.util.List;
*/ */
public interface UserRepository extends PagingAndSortingRepository<UserPO, Long> { public interface UserRepository extends PagingAndSortingRepository<UserPO, Long> {
List<UserPO> findByUsernameLike(String username); List<UserPO> findFirst20ByEnabled(int enabled);
List<UserPO> findByUsernameLikeAndEnabled(String username, int enabled);
UserPO findByUsername(String username); UserPO findByUsername(String username);
} }
...@@ -18,7 +18,6 @@ import org.springframework.security.provisioning.JdbcUserDetailsManager; ...@@ -18,7 +18,6 @@ import org.springframework.security.provisioning.JdbcUserDetailsManager;
import org.springframework.util.CollectionUtils; import org.springframework.util.CollectionUtils;
import java.util.ArrayList; import java.util.ArrayList;
import java.util.Collections;
import java.util.List; import java.util.List;
import java.util.stream.Collectors; import java.util.stream.Collectors;
...@@ -58,18 +57,19 @@ public class SpringSecurityUserService implements UserService { ...@@ -58,18 +57,19 @@ public class SpringSecurityUserService implements UserService {
@Override @Override
public List<UserInfo> searchUsers(String keyword, int offset, int limit) { public List<UserInfo> searchUsers(String keyword, int offset, int limit) {
List<UserPO> users;
if (StringUtils.isEmpty(keyword)) { if (StringUtils.isEmpty(keyword)) {
return Collections.emptyList(); users = userRepository.findFirst20ByEnabled(1);
} else {
users = userRepository.findByUsernameLikeAndEnabled("%" + keyword + "%", 1);
} }
List<UserPO> userPOs = userRepository.findByUsernameLike("%" + keyword + "%");
List<UserInfo> result = Lists.newArrayList(); List<UserInfo> result = Lists.newArrayList();
if (CollectionUtils.isEmpty(userPOs)) { if (CollectionUtils.isEmpty(users)) {
return result; return result;
} }
result.addAll(userPOs.stream().map(UserPO::toUserInfo).collect(Collectors.toList())); result.addAll(users.stream().map(UserPO::toUserInfo).collect(Collectors.toList()));
return result; return result;
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment