karaolidis/fbchat
Archived
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix IndexError: list index out of range bug

Browse Source 
When the returned `short_name` is null, `fbchat` throws an exception:

```python
  File "/usr/local/lib/python2.7/site-packages/fbchat/client.py", line 792, in fetchThreadList
    return [graphql_to_thread(node) for node in j['viewer']['message_threads']['nodes']]
  File "/usr/local/lib/python2.7/site-packages/fbchat/graphql.py", line 193, in graphql_to_thread
    last_name=user.get('name').split(user.get('short_name'),1)[1].strip(),
IndexError: list index out of range
```

This commit fixes that scenario by accessing the last item in the list via `.pop()` instead of via `[1]`
This commit is contained in:
Oren
2018-05-07 19:50:43 +03:00
committed by GitHub
parent 884af48270
commit edc655bae7
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
fbchat/graphql.py
View File

@@ -190,7 +190,7 @@ def graphql_to_thread(thread):
url=user.get('url'), url=user.get('url'),
name=user.get('name'), name=user.get('name'),
first_name=user.get('short_name'), first_name=user.get('short_name'),
last_name=user.get('name').split(user.get('short_name'),1)[1].strip(), last_name=user.get('name').split(user.get('short_name'),1).pop().strip(),
is_friend=user.get('is_viewer_friend'), is_friend=user.get('is_viewer_friend'),
gender=GENDERS.get(user.get('gender')), gender=GENDERS.get(user.get('gender')),
affinity=user.get('affinity'), affinity=user.get('affinity'),