Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Усовершенствовать поиск: сделать строгий, по полю с определенным id, возврат false если строка поиска пуста #15

Open
s-belichenko opened this issue Dec 26, 2016 · 2 comments

Comments

@s-belichenko
Copy link
Owner

Вопрос скорее ко всем, чем только к автору.

  1. Про строгий поиск. В данный момент поиск реализован не строгий, то есть если я, допустим, буду искать по контактам почту, то по строке [email protected] найдется и контакт, где упоминается почта [email protected].

  2. Про поиск по определенному полю. Возможны ситуации, когда например некая информация будет использоваться сразу в нескольких местах сущности. Например, есть имя Stanislav, и мы хотим найти по нему всех людей с таким имененем. Но одновременно, памятуя про пункт 1, мы найдем и всех, у кого есть почта Stanislav@blah_blah.blah.

  3. Иногда бывает, что есть разные формы ввода на сайте, но в итоге обрабатываются они одним и тем же кодом, и так как в этом случае какое-то поле у формы может отсутствовать, мы при поиску по нему (ради проверки на уникальность) получим полный массив контактов, или сделок, в общем каких-то сущностей.

Реализовать довольно просто, мне кажется, в конструкторе объекта Request в месте "case Request::GET:" добавить проверку $params на пустоту. Кто что скажет?

@webdenis77
Copy link

  1. Вы ведь понимаете, что эта библиотека - лишь обертка над методами API? И по get-запросу возвращает лишь то, что получила от amocrm. Вопрос строгого поиска - вопрос к разработчикам amocrm.
  2. То же самое.
  3. Тут не уловил суть.

Фильтрацию результатов поиска можно реализовать на стороне библиотеки, да, но у меня планов таких нет. Вообще понимаю, что либа стремительно теряет актуальность. Не умеет работать с неразобранным и воронками, например. Также после релиза октябрьской беты amocrm в API появились покупатели, периоды покупателей, каталоги, еще что-то. Что с этим делать непонятно.

Пробежался по форкам - есть один, который ушел дальше оригинала - https://github.com/gitkv/amocrm, там работа с неразобранным вроде реализована, работу не проверял, но судя по коммитам - есть.

Есть еще https://github.com/dotzero/amocrm-php и https://github.com/mb24dev/amocrm. Судя по описаниям, тоже умеют больше.

Я использую только тот функционал, который описан в README (в основном добавление базовых сущностей "контакт/сделка" и получение инфы об этих же сущностях, плюс вебхуки), и пока разработчики amocrm не обрушат обратную совместимость, меня все будет устраивать в нынешнем виде.

Видится три варианта развития (или не развития):

  1. Библиотеку я замораживаю в текущем состоянии, новый функционал добавляться не будет, жирно об этом пишу первой строчкой в ридми. Хотите изменений и доработок - форкайте на здоровье.
  2. Добавляю в коллабораторы заинтересованных в активной разработке. Создаем ветку 2.0, там рефакторинг + новые фичи. По завершении доработки, 2.0 переезжает в master. Пострадают только те, кто в зависимостях композера указывает бранчи вместо номера версии. Надеюсь, таких нет. Из плюсов - вы получаете более-менее раскрученную базу и участвуете в разработке самой популярной библиотеки для работы с amocrm на гитхабе. И за счет добавления новых фич, делаете ее еще популярней. Строгий поиск и поиск по конкретным полям - вообще будет "киллер-фичей" до тех пор, конечно, пока amocrm не реализуют это на своей стороне. Из минусов - авторство не за вами. Участвовать буду, но разработке могу уделять не больше одного рабочего дня в месяц.
  3. Amocrm запиливает нормальную PHP-библиотеку своими силами.

Жду комментариев и предложений, вопрос планирую закрыть до конца января.

@s-belichenko
Copy link
Owner Author

Я бы за второй вариант выступил. Авторство дело хорошее, но все равно в идеале хотелось бы все свести к одному репозиторию, так будет всем удобнее, и новеньким, и стареньким. Да и авторство, по сути, все равно за вами, так как те форки это просто форки, не более.
Предложил бы позвать тех ребят сюда и узнать их мнение. Сделать самому или подождать?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants