amocrm
amocrm copied to clipboard
s-belichenko
Дополнить варианты типов задач в Task.php
Вот так на самом деле выглядит список вариантов типов задач:
[task_types] => Array ( [0] => stdClass Object ( [id] => 1 [name] => Связаться с клиентом [code] => FOLLOW_UP ) [1] => stdClass Object ( [id] => 1 [name] => Звонок [code] => CALL ) [2] => stdClass Object ( [id] => 2 [name] => Встреча [code] => MEETING ) [3] => stdClass Object ( [id] => 3 [name] => Письмо [code] => LETTER ) )
Соответственно, вот здесь в Task.php можно было бы добавить пункт со значением 0:
const CALL = 1;
const MEETING = 2;
const LETTER = 3;
Если есть желание, сделаете pull-request на одну строчку кода? Ошибок накопилось, а времени проверять нет совсем
И никого не смутил одинаковый id ?
[0] => stdClass Object
(
[id] => 1
[name] => Связаться с клиентом
[code] => FOLLOW_UP
)
[1] => stdClass Object
(
[id] => 1
[name] => Звонок
[code] => CALL
)
Я собственно к тому, что "физически" это объекты разные в AmoCRM, а значит одинаковый ID - это либо глюк программистов AmoCRM, а заодно и указание на то, что у них нет проверки на уникальной ID в рамках набора таких вот сущностей.
Но для нас это проблема, ведь выбор элементов происходит по этому ID как раз в пакете, который обсуждаем сейчас, верно?
@nabarabane именно что, мне уже ответили что на самом деле эти варианты сущности одинаковые, сейчас жду ответа, зачем так сделано, чтобы в последующем было понятно где и как это можно использовать.
В общем, разработчики ушли от ответа, сказав что в разработке это никак и ни на что не повлияет. Я бы предложил ради дополнительного понимания оставить в коде вот такую конструкцию:
const FOLLOW_UP = 1;
const CALL = 1;
const MEETING = 2;
const LETTER = 3;
Возможно, пометив ее комментарием со ссылкой на это обсуждение или же просто небольшим объяснением, почему так. Как вам всем идея? Я пока что сделал pull request заново с просто комментарием, но если решите, что лучше как-то иначе это пометить в коде - он открыт для правки.