raelgc / scudcloud

ScudCloud - Slack for Linux
https://launchpad.net/~rael-gc/+archive/ubuntu/scudcloud
MIT License
1.22k stars 99 forks source link

Fail to another another team on python3.6 #527

Closed alfredopalhares closed 7 years ago

alfredopalhares commented 7 years ago

ScudCloud Version

ScudCloud 1.40
Python 3.6.0
Qt 5.8.0
PyQt 5.8
SIP 4.19.1

Distro and Desktop info

Expected behavior

A new team tab should appear

Actual behavior

Scudcloud crashes when addding another team.

-> % scudcloud
QSocketNotifier: Can only be used with threads started with QThread
QSocketNotifier: Can only be used with threads started with QThread
Traceback (most recent call last):
  File "/usr/lib/python3.6/site-packages/scudcloud/leftpane.py", line 63, in switchTo
    self.window.switchTo(url)
  File "/usr/lib/python3.6/site-packages/scudcloud/scudcloud.py", line 358, in switchTo
    self.quicklist(self.current().listChannels())
  File "/usr/lib/python3.6/site-packages/scudcloud/scudcloud.py", line 433, in quicklist
    if hasattr(c, '__getitem__') and c['is_member']:
TypeError: string indices must be integers
[1]    14568 abort (core dumped)  scudcloud

Steps to reproduce

  1. Add another team
  2. Insert Url / email / password
alfredopalhares commented 7 years ago

ping

raelgc commented 7 years ago

Hi @masterkorp. Unfortunatelly I cannot reproduce this under Ubuntu, because it's using python3.5.

I renamed the issue title to be clear about the python version.

I'll need to install python3.6 to try to fix this under it.

raelgc commented 7 years ago

I believe this is no more an issue with several fixes in Qt5 version. Let me know if this is still happens.