Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
При использовании методов findByPhone\findByEmail - нужно обязательно передавать EntityType или null если мы явно планируем искать по всем видам сущностей.
В прошлой версии не требовало явно указывать. имхо так правильнее - чтобы явно было понятно какой результат будет в итоге.
Оставил существующие методы "под контакты" для обратной совместимости
hasDuplicateContacts
hasOneContact
getContactsId
Но в принципе со всеми сущностями можно работать через
hasDuplicates
hasOne
getEntityIds
Для единообразия - можно конечно для каждой сущности по 3 метода дописать.. но не знаю нужно ли
Как следствие - нужны способы проверить есть ли сущность, быстро получить ID сущности, а также задать приоритетность поиска. В разных сценариях бывает нужна первая\последняя сущность.
Например стандартная логика для телефонии - поиск по приоритету лид-контакт-компания с возвратом последнего найденного совпадения. В текущем классе это возможно через getMatchIdByPriority с сортировкой desc