SelectUsersPopupWindow searchPeople should not put % around the name

As we transition to the new Hyland Community, some post links may be temporarily disrupted as we finalize redirects. We apologize for the inconvenience and are working to resolve this as quickly as possible. Details.

olegpavliv

SelectUsersPopupWindow searchPeople() puts percent % around the name. Maybe it was done to do a search in a DB table.

But it is used in the LDAP search in activity-webapp-explorer. After inserting the % symbol the search result is empty in LDAP.

Here is a correct version of searchPeople() method

  protected void searchPeople(String searchText) {
    if (searchText.length() >= 2) {
      matchingUsersTable.removeAllItems();
      
      List<User> results = ProcessEngines.getDefaultProcessEngine()
              .getIdentityService()
              .createUserQuery()
//              .userFullNameLike("%" + searchText + "%")    // this code is incorrect
              .userFullNameLike(searchText)
              .list();
      
      for (User user : results) {
        if (!multiSelect || !selectedUsersTable.containsId(user.getId())) {
          if (ignoredUserIds == null || !ignoredUserIds.contains(user.getId())) {
            addMatchingUser(user.getId(), user.getFirstName() + " " + user.getLastName());
          }
        }
      }
    }
  }
‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍

jbarrez

Is this fixed by https://github.com/Activiti/Activiti/pull/581 ? Or is this something else?