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

Не заполняется имя контакта #41

Open
ptz-nerf opened this issue Jul 11, 2018 · 5 comments
Open

Не заполняется имя контакта #41

ptz-nerf opened this issue Jul 11, 2018 · 5 comments
Labels

Comments

@ptz-nerf
Copy link

При получении данных контакта, заполняется только номер телефона.

/**
* Set the value of Phone number of the contact from contact array.
*
* @param array contact
*
* @return self
*/
public function setConcat($contact)
{
$this->phone_number = $contact['phone_number'];
return $this;
}
}

@Bogdaan
Copy link
Owner

Bogdaan commented Jul 11, 2018

Да, похоже это баг - можете сделать PR ?

@ptz-nerf
Copy link
Author

Да, но не в ближайшее время.

@nickender
Copy link
Contributor

nickender commented Jul 22, 2018

Проблема в конструкторе для класса Etinity. Если у дочернего класса есть свойство propertiesMap, то будут обрабатываться только те свойства для которых задан и имеет сеттер, прочие свойства будет игнорироваться.

Исправленный код ниже.

#44

@ptz-nerf
Copy link
Author

Да, там есть обозначенная проблема в конструкторе, но по-хорошему нужно заменить полностью весь if на приведенный вариант.
Но это не решит проблему (там нет свойства contact). Там уже есть сеттер setConcat c опечаткой в имени, в который попадают данные, нужно просто их разобрать $this->name = $contact['name'];.

У меня сейчас нет возможности, плевое дело, постараюсь сделать позже...

@nickender
Copy link
Contributor

Тут дело не только в отсутствии name. Cообщение типа Contact может себе содержать поле text. Туда, например, помещается содержимое ActionBody кнопки с типом share-phone. Поскольку ответ 'share-phone' не перехватывается OnText как в случае с 'reply' ибо типизируется как Contact, пришлось в наглую добавить для класса Contact свойства text, метод getText и изменить toArray.

Зачем? А что бы юзер в ответ на запрос собственного телефона не скинул произвольный контакт из телефонной книги в ActionBody кладу рандомный токен, а потом проверяю.

@Bogdaan Bogdaan added the bug label Jul 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants