ettoreleandrotognoli / python-ami

Python AMI Client
BSD 3-Clause "New" or "Revised" License
107 stars 66 forks source link

Fixed processing unicoded variables and events #6

Closed kerozin closed 7 years ago

kerozin commented 7 years ago

Ola!

I used CoreShowChannels and encountered issue due to some fields (CallerIDname in my case) was in unicode.

It was easy to find the cause and offer a patch. :) Please consider. Obrigado!

ettoreleandrotognoli commented 7 years ago

Thank you for your contribution. I will check the problem, the use of unicode can break the compatibility with the python3.

kerozin commented 7 years ago

It'll break. you can simple remove str() and it'll it both for python 2&3 because objects str() called for is strings itself.