build.sh and docs update

This reverts commit 47c115e699.

.github/workflows/ci.yml

@@ -1,43 +0,0 @@
-name: CI
-
-on:
-  - push
-  - pull_request
-
-jobs:
-
-  build:
-
-    strategy:
-      matrix:
-        python-version:
-          - "3.7"
-          - "3.8"
-          - "3.9"
-          - "3.10"
-
-    name: Python ${{ matrix.python-version }}
-    runs-on: ubuntu-20.04
-
-    steps:
-
-      - uses: actions/checkout@v2
-
-      - name: Set up Python ${{ matrix.python-version }}
-        uses: actions/setup-python@v2
-        with:
-          python-version: ${{ matrix.python-version }}
-
-      - name: Install dependencies
-        run: |
-          pip install -r requirements.txt
-          pip install bandit flake8 pylint
-
-      - name: Lint with flake8
-        run: make flake8
-
-      # - name: Lint with pylint
-      #   run: make pylint
-
-      - name: Lint with bandit
-        run: make bandit

.gitignore

@@ -1,8 +1,5 @@
-.pylint.err
-.pylint.out
 *.pyc
 *.pyo
-
 toxygen/toxcore
 tests/tests
 tests/libs
@@ -12,10 +9,7 @@ tests/avatars
 toxygen/libs
 .idea
 *~
-#*
 *.iml
-*.junk
-
 *.so
 *.log
 toxygen/build
@@ -31,5 +25,4 @@ Toxygen.egg-info
 *.tox
 .cache
 *.db
-*~
+
-Makefile

.rsync.sh

@@ -1,8 +0,0 @@
-#!/bin/sh
-
-find * -name \*.py | xargs grep -l '[ 	]*$' | xargs sed -i -e 's/[	 ]*$//'
-rsync "$@" -vax  --include \*.py \
-	--exclude Toxygen.egg-info --exclude build \
-	--exclude \*.pyc --exclude .pyl\* --exclude \*~ \
-	--exclude __pycache__ --exclude \*.egg-info --exclude \*.new \
-	./ ../toxygen.git/|grep -v /$

README.md

@@ -1,11 +1,16 @@
 # Toxygen
 
-Toxygen is powerful cross-platform [Tox](https://tox.chat/) client
+Toxygen is powerful cross-platform [Tox](https://tox.chat/) client written in pure Python3.
-written in pure Python3.
 
-### [Install](/docs/install.md) - [Contribute](/docs/contributing.md) - [Plugins](/docs/plugins.md) - [Compile](/docs/compile.md) - [Contact](/docs/contact.md)
+[![Release](https://img.shields.io/github/release/toxygen-project/toxygen.svg?style=flat)](https://github.com/toxygen-project/toxygen/releases/latest)
+[![Stars](https://img.shields.io/github/stars/toxygen-project/toxygen.svg?style=flat)](https://github.com/toxygen-project/toxygen/stargazers)
+[![Open issues](https://img.shields.io/github/issues/toxygen-project/toxygen.svg?style=flat)](https://github.com/toxygen-project/toxygen/issues)
+[![License](https://img.shields.io/badge/license-GPLv3-blue.svg?style=flat)](https://raw.githubusercontent.com/toxygen-project/toxygen/master/LICENSE.md)
+[![Build Status](https://travis-ci.org/toxygen-project/toxygen.svg?branch=master)](https://travis-ci.org/toxygen-project/toxygen)
 
-### Supported OS: Linux and Windows (only Linux is tested at the moment)
+### [Install](/docs/install.md) - [Contribute](/docs/contributing.md) - [Plugins](/docs/plugins.md) - [Compile](/docs/compile.md) - [Contact](/docs/contact.md) - [Updater](https://github.com/toxygen-project/toxygen_updater)
+
+### Supported OS: Linux and Windows
 
 ### Features:
 
@@ -27,7 +32,7 @@ written in pure Python3.
 - Faux offline file transfers
 - Inline images
 - Message splitting
-- Proxy support - runs over tor
+- Proxy support
 - Avatars
 - Multiprofile
 - Multilingual
@@ -38,85 +43,22 @@ written in pure Python3.
 - Changing nospam
 - File resuming
 - Read receipts
-- uses gevent
+
-- NGC groups
+### Downloads
-- PyQt5, PyQt6, PySide2, PySide6 via qtpy
+[Releases](https://github.com/toxygen-project/toxygen/releases)
+
+[Download last stable version](https://github.com/toxygen-project/toxygen/archive/master.zip)
+
+[Download develop version](https://github.com/toxygen-project/toxygen/archive/develop.zip)
 
 ### Screenshots
 *Toxygen on Ubuntu and Windows*
 ![Ubuntu](/docs/ubuntu.png)
 ![Windows](/docs/windows.png)
 
-Windows was working but is not currently being tested. AV was working
+### Docs
-but is not currently being tested: we're unsure of handling the AV devices
+[Check /docs/ for more info](/docs/)
-from the commandline. We need to get a working echobot that supports SOCKS5;
-we were working on one in https://git.plastiras.org/emdee/toxygen_wrapper
 
-## Forked
+Also visit [pythonhosted.org/Toxygen/](http://pythonhosted.org/Toxygen/)
 
-This hard-forked from the dead https://github.com/toxygen-project/toxygen
+[Wiki](https://wiki.tox.chat/clients/toxygen)
-```next_gen``` branch.
- 
-See ToDo.md to the current ToDo list.
-
-## Wechat
-
-You can have a [weechat](https://github.com/weechat/qweechat)
-console so that you can have IRC and jabber in a window as well as Tox.
-There's a copy of qweechat in ```thirdparty/qweechat``` backported to
-PyQt5 and integrated into toxygen. Follow the normal instructions for
-adding a ```relay``` to [weechat](https://github.com/weechat/weechat)
-```
-/relay add ipv4.ssl.weechat 9000
-/relay start ipv4.ssl.weechat
-```
-or
-```
-/relay add weechat 9000
-/relay start weechat
-```
-and use the Plugins/Weechat Console to start weechat under Toxygen.
-Then use the File/Connect menu item of the Console to connect to weechat.
-
-Weechat has a Jabber plugin to enable XMPP:
-```
-/python load jabber.el
-/help jabber
-```
-so you can have Tox, IRC and XMPP in the same application!
-See docs/ToxygenWeechat.md
-
-## Install
-
-To install read the requirements.txt and look at the comments; there
-may be things that need installing by hand or decisions to be made
-on supported alternatives.
-
-https://git.plastiras.org/emdee/toxygen_wrapper needs packaging
-on pypi as it is a dependency. Just download and install it from
-https://git.plastiras.org/emdee/toxygen_wrapper
-
-This is being ported to Qt6 using qtpy https://github.com/spyder-ide/qtpy
-It now runs on PyQt5 and should run on PyQt6, and may run on PySide2 and
-PySide6 - YMMV. You will be able to choose between them by setting the
-environment variable QT_API to one of: pyqt5 pyqt6 pyside2 pyside6.
-
-To install it,  look in the Makefile for the install target and type
-```
-make install
-```
-You should set the PIP_EXE_MSYS and PYTHON_EXE_MSYS variables and it does
-```
-	${PIP_EXE_MSYS} --python ${PYTHON_EXE_MSYS} install \
-		--no-deps \
-		--target ${PREFIX}/lib/python${PYTHON_MINOR}/site-packages/ \
-		--upgrade .
-```
-and installs into PREFIX which is usually /usr/local
-
-Up-to-date code is on https://git.plastiras.org/emdee/toxygen
-
-## MultiDevice
-
-Work on this project is suspended until the
-[MultiDevice](https://git.plastiras.org/emdee/tox_profile/wiki/MultiDevice-Announcements-POC) problem is solved. Fork me!

ToDo.md

@@ -1,70 +0,0 @@
-# Toxygen ToDo List
-
-## Bugs
-
-1. There is an agravating bug  where new messages are not put in the
-   current window, and a messages waiting indicator appears. You have
-   to focus out of the window and then back in the window. this may be
-   fixed already
-
-2. The tray icon is flaky and has been disabled - look in app.py
-   for bSHOW_TRAY
-
-## Fix history
-
-## Fix Audio
-
-The code is in there but it's not working. It looks like audio input
-is working but not output. The code is all in there; I may have broken
-it trying to wire up the ability to set the audio device from the
-command line.
-
-## Fix Video
-
-The code is in there but it's not working.  I may have broken it
-trying to wire up the ability to set the video device from the command
-line.
-
-## NGC Groups
-
-1. peer_id There has been a change of API on a field named
-   ```group.peer_id``` The code is broken in places because I have not
-   seen the path to change from the old API ro the new one.
-
-
-## Plugin system
-
-1. Needs better documentation and checking.
-
-2. There's something broken in the way some of them plug into Qt menus.
-
-3. Should the plugins be in toxygen or a separate repo?
-
-4. There needs to be a uniform way for plugins to wire into callbacks.
-
-## check toxygen_wrapper
-
-1. I've broken out toxygen_wrapper to be standalone,
-   https://git.plastiras.org/emdee/toxygen_wrapper but the tox.py
-   needs each call double checking.
-
-2. https://git.plastiras.org/emdee/toxygen_wrapper needs packaging
-   and making a dependency.
-
-## Migration
-
-Migrate PyQt5 to qtpy - done, but I'm not sure qtpy supports PyQt6.
-https://github.com/spyder-ide/qtpy/
-
-Maybe migrate gevent to asyncio, and migrate to
-[qasync](https://github.com/CabbageDevelopment/qasync)
-(see https://git.plastiras.org/emdee/phantompy ).
-
-(Also look at https://pypi.org/project/asyncio-gevent/ but it's dead).
-
-## Standards
-
-There's a standard for Tox clients that this has not been tested against:
-https://tox.gitbooks.io/tox-client-standard/content/general_requirements/general_requirements.html
-https://github.com/Tox/Tox-Client-Standard
-

_Bugs/segv.err

@@ -1,130 +0,0 @@
-0
-TRAC> network.c#1748:net_connect connecting socket 58 to 127.0.0.1:9050
-TRAC> Messenger.c#2709:do_messenger Friend num in DHT 2 != friend num in msger 14
-TRAC> Messenger.c#2723:do_messenger F[--: 0] D3385007C28852C5398393E3338E6AABE5F86EF249BF724E7404233207D4D927
-TRAC> Messenger.c#2723:do_messenger F[--: 1] 98984E104B8A97CC43AF03A27BE159AC1F4CF35FADCC03D6CD5F8D67B5942A56
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 10 bytes
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] =>T   2= 185.87.49.189:3389 (0: OK) | 0000000000000000...00
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] T=>  10= 185.87.49.189:3389 (0: OK) | 010001b95731bd0d...3d
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 10 bytes
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] =>T   2= 37.221.66.161:443 (0: OK) | 0000000000000000...00
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] T=>  10= 37.221.66.161:443 (0: OK) | 01000125dd42a101...bb
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 10 bytes
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] T=>   3= 172.93.52.70:33445 (0: OK) | 0000000000000000...00
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 2 bytes
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] T=>   3= 139.162.110.188:33445 (0: OK) | 0000000000000000...00
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 2 bytes
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] T=>   3= 37.59.63.150:33445 (0: OK) | 0000000000000000...00
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 2 bytes
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] T=>   3= 130.133.110.14:33445 (0: OK) | 0000000000000000...00
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 2 bytes
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] T=>   3= 37.97.185.116:33445 (0: OK) | 0000000000000000...00
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 2 bytes
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] T=>   3= 85.143.221.42:33445 (0: OK) | 0000000000000000...00
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 2 bytes
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] T=>   3= 104.244.74.69:38445 (0: OK) | 0000000000000000...00
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 2 bytes
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] T=>   3= 49.12.229.145:3389 (0: OK) | 0000000000000000...00
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 2 bytes
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] T=>   3= 168.119.209.10:33445 (0: OK) | 0000000000000000...00
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 2 bytes
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] T=>   3= 81.169.136.229:33445 (0: OK) | 0000000000000000...00
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 2 bytes
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] T=>   3= 91.219.59.156:33445 (0: OK) | 0000000000000000...00
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 2 bytes
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] T=>   3= 46.101.197.175:3389 (0: OK) | 0000000000000000...00
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 2 bytes
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] T=>   3= 198.199.98.108:33445 (0: OK) | 0000000000000000...00
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 2 bytes
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] T=>   3= 130.133.110.14:33445 (0: OK) | 0000000000000000...00
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 2 bytes
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] T=>   3= 49.12.229.145:3389 (0: OK) | 0000000000000000...00
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 2 bytes
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] T=>   3= 188.225.9.167:33445 (0: OK) | 0000000000000000...00
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 2 bytes
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] T=>   3= 5.19.249.240:38296 (0: OK) | 0000000000000000...00
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 2 bytes
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] T=>   3= 94.156.35.247:3389 (0: OK) | 0000000000000000...00
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 2 bytes
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 10 bytes
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 10 bytes
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 10 bytes
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] =>T   2= 172.93.52.70:33445 (0: OK) | 0000000000000000...00
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] T=>  10= 172.93.52.70:33445 (0: OK) | 010001ac5d344682...a5
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 10 bytes
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] =>T   2= 139.162.110.188:33445 (0: OK) | 0000000000000000...00
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] T=>  10= 139.162.110.188:33445 (0: OK) | 0100018ba26ebc82...a5
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 10 bytes
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] =>T   2= 37.59.63.150:33445 (0: OK) | 0000000000000000...00
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] T=>  10= 37.59.63.150:33445 (0: OK) | 010001253b3f9682...a5
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 10 bytes
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] =>T   2= 130.133.110.14:33445 (0: OK) | 0000000000000000...00
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] T=>  10= 130.133.110.14:33445 (0: OK) | 01000182856e0e82...a5
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 10 bytes
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] =>T   2= 37.97.185.116:33445 (0: OK) | 0000000000000000...00
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] T=>  10= 37.97.185.116:33445 (0: OK) | 0100012561b97482...a5
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 10 bytes
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] =>T   2= 85.143.221.42:33445 (0: OK) | 0000000000000000...00
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] T=>  10= 85.143.221.42:33445 (0: OK) | 010001558fdd2a82...a5
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 10 bytes
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] =>T   2= 104.244.74.69:38445 (0: OK) | 0000000000000000...00
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] T=>  10= 104.244.74.69:38445 (0: OK) | 01000168f44a4596...2d
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 10 bytes
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] =>T   2= 49.12.229.145:3389 (0: OK) | 0000000000000000...00
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] T=>  10= 49.12.229.145:3389 (0: OK) | 010001310ce5910d...3d
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 10 bytes
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] =>T   2= 168.119.209.10:33445 (0: OK) | 0000000000000000...00
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] T=>  10= 168.119.209.10:33445 (0: OK) | 010001a877d10a82...a5
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 10 bytes
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] =>T   2= 81.169.136.229:33445 (0: OK) | 0000000000000000...00
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] T=>  10= 81.169.136.229:33445 (0: OK) | 01000151a988e582...a5
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 10 bytes
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] =>T   2= 91.219.59.156:33445 (0: OK) | 0000000000000000...00
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] T=>  10= 91.219.59.156:33445 (0: OK) | 0100015bdb3b9c82...a5
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 10 bytes
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] =>T   2= 46.101.197.175:3389 (0: OK) | 0000000000000000...00
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] T=>  10= 46.101.197.175:3389 (0: OK) | 0100012e65c5af0d...3d
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 10 bytes
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] =>T   2= 198.199.98.108:33445 (0: OK) | 0000000000000000...00
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] T=>  10= 198.199.98.108:33445 (0: OK) | 010001c6c7626c82...a5
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 10 bytes
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] =>T   2= 130.133.110.14:33445 (0: OK) | 0000000000000000...00
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] T=>  10= 130.133.110.14:33445 (0: OK) | 01000182856e0e82...a5
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 10 bytes
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] =>T   2= 49.12.229.145:3389 (0: OK) | 0000000000000000...00
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] T=>  10= 49.12.229.145:3389 (0: OK) | 010001310ce5910d...3d
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 10 bytes
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] =>T   2= 188.225.9.167:33445 (0: OK) | 0000000000000000...00
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] T=>  10= 188.225.9.167:33445 (0: OK) | 010001bce109a782...a5
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 10 bytes
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] =>T   2= 5.19.249.240:38296 (0: OK) | 0000000000000000...00
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] T=>  10= 5.19.249.240:38296 (0: OK) | 0100010513f9f095...98
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 10 bytes
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] =>T   2= 94.156.35.247:3389 (0: OK) | 0000000000000000...00
-TRAC> network.c#789:loglogdata [05 = <unknown>           ] T=>  10= 94.156.35.247:3389 (0: OK) | 0100015e9c23f70d...3d
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 10 bytes
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 10 bytes
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 10 bytes
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 10 bytes
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 10 bytes
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 10 bytes
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 10 bytes
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 10 bytes
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 10 bytes
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 10 bytes
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 10 bytes
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 10 bytes
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 10 bytes
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 10 bytes
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 10 bytes
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 10 bytes
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 10 bytes
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 10 bytes
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 10 bytes
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 10 bytes
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 10 bytes
-TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 10 bytes
-app.contacts.contacts_manager INFO update_groups_numbers len(groups)={len(groups)}
-
-Thread 76 "ToxIterateThrea" received signal SIGSEGV, Segmentation fault.
-[Switching to Thread 0x7ffedcb6b640 (LWP 2950427)]

_Bugs/tox.abilinski.com.ping

@@ -1,11 +0,0 @@
- ping tox.abilinski.com
-ping: socket: Address family not supported by protocol
-PING tox.abilinski.com (172.103.226.229) 56(84) bytes of data.
-64 bytes from 172.103.226.229.cable.tpia.cipherkey.com (172.103.226.229): icmp_seq=1 ttl=48 time=86.6 ms
-64 bytes from 172.103.226.229.cable.tpia.cipherkey.com (172.103.226.229): icmp_seq=2 ttl=48 time=83.1 ms
-64 bytes from 172.103.226.229.cable.tpia.cipherkey.com (172.103.226.229): icmp_seq=3 ttl=48 time=82.9 ms
-64 bytes from 172.103.226.229.cable.tpia.cipherkey.com (172.103.226.229): icmp_seq=4 ttl=48 time=83.4 ms
-64 bytes from 172.103.226.229.cable.tpia.cipherkey.com (172.103.226.229): icmp_seq=5 ttl=48 time=102 ms
-64 bytes from 172.103.226.229.cable.tpia.cipherkey.com (172.103.226.229): icmp_seq=6 ttl=48 time=87.4 ms
-64 bytes from 172.103.226.229.cable.tpia.cipherkey.com (172.103.226.229): icmp_seq=7 ttl=48 time=84.9 ms
-^C

docs/ToxygenWeechat.md

@@ -1,170 +0,0 @@
-## Toxygen Weechat
-
-You can have a [weechat](https://github.com/weechat/qweechat)
-console so that you can have IRC and jabber in a window as well as Tox.
-There's a copy of qweechat in ```thirdparty/qweechat``` backported to
-PyQt5 and integrated into toxygen. Follow the normal instructions for
-adding a ```relay``` to [weechat](https://github.com/weechat/weechat)
-```
-/relay add ipv4.ssl.weechat 9000
-/relay start ipv4.ssl.weechat
-```
-or
-```
-/set relay.network.ipv6 off
-/set relay.network.password password
-/relay add weechat 9000
-/relay start weechat
-```
-and use the Plugins/Weechat Console to start weechat under Toxygen.
-Then use the File/Connect menu item of the Console to connect to weechat.
-
-Weechat has a Jabber plugin to enable XMPP:
-```
-/python load jabber.el
-/help jabber
-```
-so you can have Tox, IRC and XMPP in the same application!
-
-### Creating servers for IRC over Tor
-
-Create a proxy called tor
-```
-/proxy add tor socks5 127.0.0.1 9050
-```
-
-It should now show up in the list of proxies.
-```
-/proxy list
-```
-
-```
-/nick NickName
-```
-
-## TLS certificates
-
-[Create a Self-signed Certificate](https://www.oftc.net/NickServ/CertFP/)
-
-Choose a NickName you will identify as.
-
-Create a directory for your certificates ~/.config/weechat/ssl/ 
-and make a subdirectory for each server ~/.config/weechat/ssl/irc.oftc.net/ 
-
-Change to the server directory and use openssl to make a keypair and answer the questions:
-```
-openssl req -nodes -newkey rsa:2048 -keyout NickName.key -x509 -days 3650 -out NickName.cer
-chmod 400 NickName.key
-```
-We now combine certificate and key to a single file NickName.pem 
-```
-cat NickName.cer NickName.key > NickName.pem
-chmod 400 NickName.pem
-```
-
-Do this for each server you want to connect to, or just use one for all of them.
-
-### Libera TokTok channel
-
-The main discussion forum for Tox is the #TokTok channel on libera.
-
-https://mox.sh/sysadmin/secure-irc-connection-to-freenode-with-tor-and-weechat/
-We have to create an account without Tor, this is a requirement to use TOR:
-Connect to irc.libera.chat without Tor and register 
-```
-/msg NickServ identify NickName password
-/msg NickServ REGISTER mypassword mycoolemail@example.com
-/msg NickServ SET PRIVATE ON
-```
-Confirm registration after getting the mail with the code:
-```
-/msg NickServ VERIFY REGISTER NickName code1235678
-```
-
-Libera has an onion server so we can map an address in tor. Add this
-to your /etc/tor/torrc
-```
-MapAddress palladium.libera.chat libera75jm6of4wxpxt4aynol3xjmbtxgfyjpu34ss4d7r7q2v5zrpyd.onion
-```
-Or without the MapAddress just use
-libera75jm6of4wxpxt4aynol3xjmbtxgfyjpu34ss4d7r7q2v5zrpyd.onion
-as the server address below, but set tls_verify to off.
-
-Define the server in weechat
-https://www.weechat.org/files/doc/stable/weechat_user.en.html#irc_sasl_authentication
-```
-/server remove libera
-/server add libera palladium.libera.chat/6697 -tls -tls_verify
-/set irc.server.libera.ipv6 off
-/set irc.server.libera.proxy tor
-/set irc.server.libera.username NickName
-/set irc.server.libera.password password
-/set irc.server.libera.nicks NickName
-/set irc.server.libera.tls on
-/set irc.server.libera.tls_cert "${weechat_config_dir}/ssl/libera.chat/NickName.pem"
-```
-
-```
-/set irc.server.libera.sasl_mechanism ecdsa-nist256p-challenge
-/set irc.server.libera.sasl_username "NickName"
-/set irc.server.libera.sasl_key "${weechat_config_dir}/ssl/libera.chat/NickName.pem"
-```
-
-Disconnect and connect back to the server. 
-```
-/disconnect libera
-/connect libera
-```
-
-/msg nickserv identify password NickName
-
-
-### oftc.net
-
-To use oftc.net over tor, you need to authenticate by SSL certificates.
-
-
-Define the server in weechat
-```
-/server remove irc.oftc.net
-/server add OFTC irc.oftc.net/6697  -tls -tls_verify
-/set irc.server.OFTC.ipv6 off
-/set irc.server.OFTC.proxy tor
-/set irc.server.OFTC.username NickName
-/set irc.server.OFTC.nicks NickName
-/set irc.server.OFTC.tls on
-/set irc.server.OFTC.tls_cert "${weechat_config_dir}/ssl/irc.oftc.chat/NickName.pem"
-
-# Disconnect and connect back to the server. 
-/disconnect OFTC
-/connect OFTC
-``` 
-You must be identified in order to validate using certs
-```
-/msg nickserv identify password NickName
-```
-To allow NickServ to identify you based on this certificate you need
-to associate the certificate fingerprint with your nick. To do this
-issue the command cert add to Nickserv (try /msg nickserv helpcert).
-```
-/msg nickserv cert add
-```
-
-### Privacy
-
-[Add somes settings bellow to weechat](https://szorfein.github.io/weechat/tor/configure-weechat/).
-Detail from [faq](https://weechat.org/files/doc/weechat_faq.en.html#security).
-
-```
-/set irc.server_default.msg_part ""
-/set irc.server_default.msg_quit ""
-/set irc.ctcp.clientinfo ""
-/set irc.ctcp.finger ""
-/set irc.ctcp.source ""
-/set irc.ctcp.time ""
-/set irc.ctcp.userinfo ""
-/set irc.ctcp.version ""
-/set irc.ctcp.ping ""
-/plugin unload xfer
-/set weechat.plugin.autoload "*,!xfer"
-```

docs/contact.md

@@ -1,6 +1,5 @@
 # Contact us:
 
-1) https://git.plastiras.org/emdee/toxygen/issues
+1) Using GitHub - open issue
 
-2) Use Toxygen Tox Group (NGC) - 
+2) Use Toxygen Tox Group (NGC) - ID: 59D68B2709E81A679CF91416CB0E3692851C6CFCABEFF98B7131E3805A6D75FA
-ID: 59D68B2709E81A679CF91416CB0E3692851C6CFCABEFF98B7131E3805A6D75FA

docs/contributing.md

@@ -7,15 +7,12 @@ Help us find all bugs in Toxygen! Please provide following info:
 - Toxygen executable info - python executable (.py), precompiled binary, from package etc.
 - Steps to reproduce the bug
 
-Want to see new feature in Toxygen? 
+Want to see new feature in Toxygen? [Ask for it!](https://github.com/toxygen-project/toxygen/issues)
-[Ask for it!](https://git.plastiras.org/emdee/toxygen/issues)
 
 # Pull requests
 
 Developer? Feel free to open pull request. Our dev team is small so we glad to get help.
-Don't know what to do? Improve UI, fix 
+Don't know what to do? Improve UI, fix [issues](https://github.com/toxygen-project/toxygen/issues) or implement features from our TODO list.
-[issues](https://git.plastiras.org/emdee/toxygen/issues) 
-or implement features from our TODO list.
 You can find our TODO's in code, issues list and [here](/README.md). Also you can implement [plugins](/docs/plugins.md) for Toxygen.
 
 Note that we have a lot of branches for different purposes. Master branch is for stable versions (releases) only, so I recommend to open PR's to develop branch. Development of next Toxygen version usually goes there. Other branches used for implementing different tasks such as file transfers improvements or audio calls implementation etc.

docs/install.md

@@ -1,15 +1,33 @@
 # How to install Toxygen
 
+## Use precompiled binary (recommended for users):
+[Check our releases page](https://github.com/toxygen-project/toxygen/releases)
+
+## Using pip3
+
+### Windows
+
+``pip install toxygen``
+
+Run app using ``toxygen`` command.
+
 ### Linux
 
-1. Install [c-toxcore](https://github.com/TokTok/c-toxcore/)
+1. Install [toxcore](https://github.com/irungentoo/toxcore/blob/master/INSTALL.md) with toxav support in your system (install in /usr/lib/)
 2. Install PortAudio: 
 ``sudo apt-get install portaudio19-dev``
 3. For 32-bit Linux install PyQt5: ``sudo apt-get install python3-pyqt5``
 4. Install [OpenCV](http://docs.opencv.org/trunk/d7/d9f/tutorial_linux_install.html) or via ``sudo pip3 install opencv-python``
-5. Install [toxygen](https://git.plastiras.org/emdee/toxygen/)
+5. Install toxygen: 
+``sudo pip3 install toxygen``
 6. Run toxygen using ``toxygen`` command.
 
+## Packages
+
+Arch Linux: [AUR](https://aur.archlinux.org/packages/toxygen-git/)
+
+Debian/Ubuntu: [tox.chat](https://tox.chat/download.html#gnulinux)
+
 ## From source code (recommended for developers)
 
 ### Windows
@@ -21,31 +39,34 @@ Note: 32-bit Python isn't supported due to bug with videocalls. It is strictly r
 3. Install PyAudio: ``pip install pyaudio``
 4. Install numpy: ``pip install numpy``
 5. Install OpenCV: ``pip install opencv-python``
-6. git clone --depth=1 https://git.plastiras.org/emdee/toxygen/
+6. [Download toxygen](https://github.com/toxygen-project/toxygen/archive/master.zip)
-7. I don't know
+7. Unpack archive
 8. Download latest libtox.dll build, download latest libsodium.a build, put it into \toxygen\libs\
 9. Run \toxygen\main.py.
 
+Optional: install toxygen using setup.py: ``python setup.py install``
+
+[libtox.dll for 32-bit Python](https://build.tox.chat/view/libtoxcore/job/libtoxcore_build_windows_x86_shared_release/lastSuccessfulBuild/artifact/libtoxcore_build_windows_x86_shared_release.zip)
+
+[libtox.dll for 64-bit Python](https://build.tox.chat/view/libtoxcore/job/libtoxcore_build_windows_x86-64_shared_release/lastSuccessfulBuild/artifact/libtoxcore_build_windows_x86-64_shared_release.zip)
+
+[libsodium.a for 32-bit Python](https://build.tox.chat/view/libsodium/job/libsodium_build_windows_x86_static_release/lastSuccessfulBuild/artifact/libsodium_build_windows_x86_static_release.zip)
+
+[libsodium.a for 64-bit Python](https://build.tox.chat/view/libsodium/job/libsodium_build_windows_x86-64_static_release/lastSuccessfulBuild/artifact/libsodium_build_windows_x86-64_static_release.zip)
+
 ### Linux
 
 1. Install latest Python3: 
 ``sudo apt-get install python3``
-2. Install PyQt5: ``sudo apt-get install python3-pyqt5``
+2. Install PyQt5: ``sudo apt-get install python3-pyqt5`` or ``sudo pip3 install pyqt5``
-3. Install [toxcore](https://github.com/TokTok/c-toxcore) with toxav support)
+3. Install [toxcore](https://github.com/irungentoo/toxcore/blob/master/INSTALL.md) with toxav support in your system (install in /usr/lib/)
-4. Install PyAudio: ``sudo apt-get install portaudio19-dev python3-pyaudio`` (or ``sudo pip3 install pyaudio``)
+4. Install PyAudio: 
-5. Install toxygen_wrapper https://git.plastiras.org/emdee/toxygen_wrapper
+``sudo apt-get install portaudio19-dev`` and ``sudo apt-get install python3-pyaudio`` (or ``sudo pip3 install pyaudio``)
-6. Install the rest of the requirements: ``sudo pip3 install -m requirements.txt``
+5. Install NumPy: ``sudo pip3 install numpy``
-7. git clone --depth=1 [toxygen](https://git.plastiras.org/emdee/toxygen/)
+6. Install [OpenCV](http://docs.opencv.org/trunk/d7/d9f/tutorial_linux_install.html) or via ``sudo pip3 install opencv-python``
-8. Look in the Makefile for the install target and type
+7. [Download toxygen](https://github.com/toxygen-project/toxygen/archive/master.zip)
-``
+8. Unpack archive
-make install
-``
-You should set the PIP_EXE_MSYS and PYTHON_EXE_MSYS variables and it does
-``
-	${PIP_EXE_MSYS} --python ${PYTHON_EXE_MSYS} install \
-		--target ${PREFIX}/lib/python${PYTHON_MINOR}/site-packages/ \
-		--upgrade .
-``
 9. Run app:
-``python3 ${PREFIX}/lib/python${PYTHON_MINOR}/site-packages/bin/toxygen``
+``python3 main.py``
 
+Optional: install toxygen using setup.py: ``python3 setup.py install``

docs/plugin_api.md

@@ -1,6 +1,6 @@
 # Plugins API
 
-In Toxygen plugin is single python module (.py file) and directory with data associated with it. 
+In Toxygen plugin is single python (supported Python 3.4 - 3.6) module (.py file) and directory with data associated with it. 
 Every module must contain one class derived from PluginSuperClass defined in [plugin_super_class.py](/src/plugins/plugin_super_class.py). Instance of this class will be created by PluginLoader class (defined in [plugin_support.py](/src/plugin_support.py) ). This class can enable/disable plugins and send data to it. 
 
 Every plugin has its own full name and unique short name (1-5 symbols). Main app can get it using special methods. 
@@ -53,5 +53,5 @@ Plugin's methods MUST NOT raise exceptions.
 
 # Examples
 
-You can find examples in [official repo](https://git.plastiras.org/emdee/toxygen_plugins)
+You can find examples in [official repo](https://github.com/toxygen-project/toxygen_plugins)
 

docs/todo.md

@@ -1,70 +0,0 @@
-# Toxygen ToDo List
-
-## Bugs
-
-1. There is an agravating bug  where new messages are not put in the
-   current window, and a messages waiting indicator appears. You have
-   to focus out of the window and then back in the window. this may be
-   fixed already
-
-2. The tray icon is flaky and has been disabled - look in app.py
-   for bSHOW_TRAY
-
-## Fix history
-
-## Fix Audio
-
-The code is in there but it's not working. It looks like audio input
-is working but not output. The code is all in there; I may have broken
-it trying to wire up the ability to set the audio device from the
-command line.
-
-## Fix Video
-
-The code is in there but it's not working.  I may have broken it
-trying to wire up the ability to set the video device from the command
-line.
-
-## NGC Groups
-
-1. peer_id There has been a change of API on a field named
-   ```group.peer_id``` The code is broken in places because I have not
-   seen the path to change from the old API ro the new one.
-
-
-## Plugin system
-
-1. Needs better documentation and checking.
-
-2. There's something broken in the way some of them plug into Qt menus.
-
-3. Should the plugins be in toxygen or a separate repo?
-
-4. There needs to be a uniform way for plugins to wire into callbacks.
-
-## check toxygen_wrapper
-
-1. I've broken out toxygen_wrapper to be standalone,
-   https://git.plastiras.org/emdee/toxygen_wrapper but the tox.py
-   needs each call double checking.
-
-2. https://git.plastiras.org/emdee/toxygen_wrapper needs packaging
-   and making a dependency.
-
-## Migration
-
-Migrate PyQt5 to qtpy - done, but I'm not sure qtpy supports PyQt6.
-https://github.com/spyder-ide/qtpy/
-
-Maybe migrate gevent to asyncio, and migrate to
-[qasync](https://github.com/CabbageDevelopment/qasync)
-(see https://git.plastiras.org/emdee/phantompy ).
-
-(Also look at https://pypi.org/project/asyncio-gevent/ but it's dead).
-
-## Standards
-
-There's a standard for Tox clients that this has not been tested against:
-https://tox.gitbooks.io/tox-client-standard/content/general_requirements/general_requirements.html
-https://github.com/Tox/Tox-Client-Standard
-

pyproject.tom

@@ -1,51 +0,0 @@
-[project]
-name = "stem_examples"
-description = "examples of using stem"
-authors = [{ name = "emdee", email = "emdee@spm.plastiras.org" } ]
-requires-python = ">=3.6"
-dependencies = [
-    'stem',
-]
-keywords = ["stem", "python3", "tor"]
-classifiers = [
-    "License :: OSI Approved",
-    "Operating System :: POSIX :: BSD :: FreeBSD",
-    "Operating System :: POSIX :: Linux",
-    "Programming Language :: Python :: 3 :: Only",
-    "Programming Language :: Python :: 3.6",
-    "Programming Language :: Python :: 3.7",
-    "Programming Language :: Python :: 3.8",
-    "Programming Language :: Python :: 3.9",
-    "Programming Language :: Python :: 3.10",
-    "Programming Language :: Python :: 3.11",
-    "Programming Language :: Python :: Implementation :: CPython",
-    ]
-#
-dynamic = ["version", "readme", ] # cannot be dynamic ['license']
-
-[project.scripts]
-toxygen = "toxygen.toxygen.main:main"
-
-#[project.license]
-#file = "LICENSE.md"
-
-[project.urls]
-repository = "https://git.plastiras.org/emdee/toxygen"
-
-[build-system]
-requires = ["setuptools >= 61.0"]
-build-backend = "setuptools.build_meta"
-
-[tool.setuptools.dynamic]
-version = {attr = "stem_examples.__version__"} 
-readme = {file = ["README.md", "ToDo.txt"]}
-
-[tool.setuptools]
-packages = ["toxygen"]
-
-[tool.setuptools.packages.find]
-where = "toxygen"
-
-[tool.setuptools.packages.package-data]
-"*" = ["*.ui"]
-

pyproject.toml

@@ -1,56 +0,0 @@
-[project]
-name = "toxygen"
-description = "examples of using stem"
-authors = [{ name = "emdee", email = "emdee@spm.plastiras.org" } ]
-requires-python = ">=3.7"
-keywords = ["stem", "python3", "tox"]
-classifiers = [
-  # How mature is this project? Common values are
-  #   3 - Alpha
-  #   4 - Beta
-  #   5 - Production/Stable
-  "Development Status :: 4 - Beta",
-
-  # Indicate who your project is intended for
-  "Intended Audience :: Developers",
-
-  # Specify the Python versions you support here.
-  "Programming Language :: Python :: 3",
-  "License :: OSI Approved",
-  "Operating System :: POSIX :: BSD :: FreeBSD",
-  "Operating System :: POSIX :: Linux",
-  "Programming Language :: Python :: 3 :: Only",
-  "Programming Language :: Python :: 3.6",
-  "Programming Language :: Python :: 3.7",
-  "Programming Language :: Python :: 3.8",
-  "Programming Language :: Python :: 3.9",
-  "Programming Language :: Python :: 3.10",
-  "Programming Language :: Python :: 3.11",
-  "Programming Language :: Python :: Implementation :: CPython",
-]
-#
-dynamic = ["version", "readme", "dependencies"] # cannot be dynamic ['license']
-
-[project.gui-scripts]
-toxygen = "toxygen.__main__:main"
-
-[project.optional-dependencies]
-weechat = ["weechat"]
-
-#[project.license]
-#file = "LICENSE.md"
-
-[project.urls]
-repository = "https://git.plastiras.org/emdee/toxygen"
-
-[build-system]
-requires = ["setuptools >= 61.0"]
-build-backend = "setuptools.build_meta"
-
-[tool.setuptools.dynamic]
-version = {attr = "toxygen.app.__version__"} 
-readme = {file = ["README.md", "ToDo.txt"]}
-dependencies = {file = ["requirements.txt"]}
-
-[tool.setuptools]
-packages = ["toxygen"]

requirements.txt

@@ -1,21 +0,0 @@
-# the versions are the current ones tested - may work with earlier versions
-# choose one of PyQt5 PyQt6 PySide2 PySide6
-# for now PyQt5 and PyQt6 is working, and most of the testing is PyQt5
-# usually this is installed by your OS package manager and pip may not
-# detect the right version, so we leave these commented
-# PyQt5 >= 5.15.10
-QtPy >= 2.4.1
-PyAudio >= 0.2.13
-numpy >= 1.26.1
-opencv_python >= 4.8.0
-pydenticon >= 0.3.1
-pillow >= 10.2.0
-gevent >= 23.9.1
-greenlet >= 2.0.2
-# this is optional
-coloredlogs >= 15.0.1
-# this is optional
-# qtconsole >= 5.4.3
-# this is not on pypi yet - get it from
-# https://git.plastiras.org/emdee/toxygen_wrapper
-# toxygen_wrapper == 1.0.0

setup.cfg

@@ -1,54 +0,0 @@
-[metadata]
-classifiers =
-    License :: OSI Approved
-    License :: OSI Approved :: BSD 1-clause
-    Intended Audience :: Web Developers
-    Operating System :: Microsoft :: Windows
-    Operating System :: POSIX :: BSD :: FreeBSD
-    Operating System :: POSIX :: Linux
-    Programming Language :: Python :: 3 :: Only
-    Programming Language :: Python :: 3.6
-    Programming Language :: Python :: 3.7
-    Programming Language :: Python :: 3.8
-    Programming Language :: Python :: 3.9
-    Programming Language :: Python :: 3.10
-    Programming Language :: Python :: 3.11
-    Programming Language :: Python :: Implementation :: CPython
-
-[options]
-zip_safe = false
-python_requires = ~=3.7
-include_package_data =
-    "*" = ["*.ui", "*.txt", "*.png", "*.ico", "*.gif", "*.wav"]
-
-[options.entry_points]
-console_scripts =
-    toxygen = toxygen.__main__:iMain
-
-[easy_install]
-zip_ok = false
-
-[flake8]
-jobs = 1
-max-line-length = 88
-ignore =
-    E111
-    E114
-    E128
-    E225
-    E261
-    E302
-    E305
-    E402
-    E501
-    E502
-    E541
-    E701
-    E702
-    E704
-    E722
-    E741
-    F508
-    F541
-    W503
-    W601

setup.py

@@ -1,53 +1,93 @@
-import sys
-import os
 from setuptools import setup
 from setuptools.command.install import install
+from platform import system
+from subprocess import call
+import main
+import sys
+import os
+from utils.util import curr_directory, join_path
 
-version = '1.0.0'
 
-MODULES = open('requirements.txt', 'rt').readlines()
+version = main.__version__ + '.0'
+
+
+if system() == 'Windows':
+    MODULES = ['PyQt5', 'PyAudio', 'numpy', 'opencv-python', 'pydenticon']
+else:
+    MODULES = []
+    try:
+        import pyaudio
+    except ImportError:
+        MODULES.append('PyAudio')
+    try:
+        import PyQt5
+    except ImportError:
+        MODULES.append('PyQt5')
+    try:
+        import numpy
+    except ImportError:
+        MODULES.append('numpy')
+    try:
+        import cv2
+    except ImportError:
+        MODULES.append('opencv-python')
+    try:
+        import pydenticon
+    except ImportError:
+        MODULES.append('pydenticon')
+
 
 def get_packages():
-    directory = os.path.join(os.path.dirname(__file__), 'tox_wrapper')
+    directory = join_path(curr_directory(__file__), 'toxygen')
     for root, dirs, files in os.walk(directory):
         packages = map(lambda d: 'toxygen.' + d, dirs)
         packages = ['toxygen'] + list(packages)
+
         return packages
 
+
 class InstallScript(install):
     """This class configures Toxygen after installation"""
 
     def run(self):
         install.run(self)
+        try:
+            if system() != 'Windows':
+                call(["toxygen", "--clean"])
+        except:
+            try:
+                params = list(filter(lambda x: x.startswith('--prefix='), sys.argv))
+                if params:
+                    path = params[0][len('--prefix='):]
+                    if path[-1] not in ('/', '\\'):
+                        path += '/'
+                    path += 'bin/toxygen'
+                    if system() != 'Windows':
+                        call([path, "--clean"])
+            except:
+                pass
+
 
 setup(name='Toxygen',
       version=version,
       description='Toxygen - Tox client',
       long_description='Toxygen is powerful Tox client written in Python3',
-      url='https://git.plastiras.org/emdee/toxygen/',
+      url='https://github.com/toxygen-project/toxygen/',
-      keywords='toxygen Tox messenger',
+      keywords='toxygen tox messenger',
       author='Ingvar',
-      maintainer='',
+      maintainer='Ingvar',
       license='GPL3',
       packages=get_packages(),
       install_requires=MODULES,
       include_package_data=True,
       classifiers=[
           'Programming Language :: Python :: 3 :: Only',
-    "Programming Language :: Python :: 3.6",
+          'Programming Language :: Python :: 3.5',
-    "Programming Language :: Python :: 3.7",
+          'Programming Language :: Python :: 3.6',
-    "Programming Language :: Python :: 3.8",
-    "Programming Language :: Python :: 3.9",
-    "Programming Language :: Python :: 3.10",
-    "Programming Language :: Python :: 3.11",
-          'Programming Language :: Python :: 3.11',
       ],
       entry_points={
           'console_scripts': ['toxygen=toxygen.main:main']
       },
-      package_data={"": ["*.ui"],},
       cmdclass={
-          'install': InstallScript,
+          'install': InstallScript
-      },
+      })
-      zip_safe=False
-      )

setup.py.dst

@@ -1,53 +0,0 @@
-import sys
-import os
-from setuptools import setup
-from setuptools.command.install import install
-
-version = '1.0.0'
-
-MODULES = open('requirements.txt', 'rt').readlines()
-
-def get_packages():
-    directory = os.path.join(os.path.dirname(__file__), 'tox_wrapper')
-    for root, dirs, files in os.walk(directory):
-        packages = map(lambda d: 'toxygen.' + d, dirs)
-        packages = ['toxygen'] + list(packages)
-        return packages
-
-class InstallScript(install):
-    """This class configures Toxygen after installation"""
-
-    def run(self):
-        install.run(self)
-
-setup(name='Toxygen',
-      version=version,
-      description='Toxygen - Tox client',
-      long_description='Toxygen is powerful Tox client written in Python3',
-      url='https://git.plastiras.org/emdee/toxygen/',
-      keywords='toxygen Tox messenger',
-      author='Ingvar',
-      maintainer='',
-      license='GPL3',
-      packages=get_packages(),
-      install_requires=MODULES,
-      include_package_data=True,
-      classifiers=[
-          'Programming Language :: Python :: 3 :: Only',
-    "Programming Language :: Python :: 3.6",
-    "Programming Language :: Python :: 3.7",
-    "Programming Language :: Python :: 3.8",
-    "Programming Language :: Python :: 3.9",
-    "Programming Language :: Python :: 3.10",
-    "Programming Language :: Python :: 3.11",
-          'Programming Language :: Python :: 3.11',
-      ],
-      entry_points={
-          'console_scripts': ['toxygen=toxygen.main:main']
-      },
-      package_data={"": ["*.ui"],},
-      cmdclass={
-          'install': InstallScript,
-      },
-      zip_safe=False
-      )

toxygen/.pylint.sh

@@ -1,14 +0,0 @@
-#!/bin/sh
-
-ROLE=logging
-/var/local/bin/pydev_pylint.bash -E -f text *py [a-nr-z]*/*py  >.pylint.err
-/var/local/bin/pydev_pylint.bash *py [a-nr-z]*/*py  >.pylint.out
-
-sed -e "/Module 'os' has no/d" \
-    -e "/Undefined variable 'app'/d" \
-    -e '/tests\//d' \
-    -e "/Instance of 'Curl' has no /d" \
-    -e "/No name 'path' in module 'os' /d" \
-    -e "/ in module 'os'/d" \
-    -e "/.bak\//d" \
-	-i .pylint.err .pylint.out

toxygen/__init__.py

@@ -5,5 +5,4 @@ path = os.path.dirname(os.path.realpath(__file__))  # curr dir
 
 sys.path.insert(0, os.path.join(path, 'styles'))
 sys.path.insert(0, os.path.join(path, 'plugins'))
-sys.path.insert(0, os.path.join(path, 'third_party'))
 sys.path.insert(0, path)

toxygen/__main__.py

@@ -1,370 +0,0 @@
-# -*- mode: python; indent-tabs-mode: nil; py-indent-offset: 4; coding: utf-8 -*-
-import sys
-import os
-import app
-import logging
-import signal
-import time
-
-from gevent import monkey; monkey.patch_all(); del monkey   # noqa
-
-import faulthandler
-faulthandler.enable()
-
-import warnings
-warnings.filterwarnings('ignore')
-
-import tox_wrapper.tests.support_testing as ts
-try:
-    from trepan.interfaces import server as Mserver
-    from trepan.api import debug
-except:
-    print('trepan3 TCP server NOT enabled.')
-else:
-    import signal
-    try:
-        signal.signal(signal.SIGUSR1, ts.trepan_handler)
-        print('trepan3 TCP server enabled on port 6666.')
-    except: pass
-
-from user_data.settings import *
-from user_data.settings import Settings
-from user_data import settings
-import utils.util as util
-with ts.ignoreStderr():
-    import pyaudio
-
-__maintainer__ = 'Ingvar'
-__version__ = '1.0.0' # was 0.5.0+
-
-sleep = time.sleep
-
-os.environ['QT_API'] = os.environ.get('QT_API', 'pyqt5')
-
-def reset() -> None:
-    Settings.reset_auto_profile()
-
-def clean() -> None:
-    """Removes libs folder"""
-    directory = util.get_libs_directory()
-    util.remove(directory)
-
-def print_toxygen_version() -> None:
-    print('toxygen ' + __version__)
-
-def setup_default_audio():
-    # need:
-    audio = ts.get_audio()
-    # unfinished
-    global oPYA
-    oPYA = pyaudio.PyAudio()
-    audio['output_devices'] = dict()
-    i = oPYA.get_device_count()
-    while i > 0:
-        i -= 1
-        if oPYA.get_device_info_by_index(i)['maxOutputChannels'] == 0:
-            continue
-        audio['output_devices'][i] = oPYA.get_device_info_by_index(i)['name']
-    i = oPYA.get_device_count()
-    audio['input_devices'] = dict()
-    while i > 0:
-        i -= 1
-        if oPYA.get_device_info_by_index(i)['maxInputChannels'] == 0:
-            continue
-        audio['input_devices'][i] = oPYA.get_device_info_by_index(i)['name']
-    return audio
-
-def setup_video(oArgs):
-    video = setup_default_video()
-    # this is messed up - no video_input in oArgs
-    #     parser.add_argument('--video_input', type=str,)
-    if not hasattr(oArgs, 'video_input'):
-        video['device'] = video['output_devices'][0]
-    elif oArgs.video_input == '-1':
-        video['device'] = video['output_devices'][1]
-    else:
-        video['device'] = oArgs.video_input
-    return video
-
-def setup_audio(oArgs):
-    global oPYA
-    audio = setup_default_audio()
-    for k,v in audio['input_devices'].items():
-        if v == 'default' and 'input' not in audio:
-            audio['input'] = k
-        if v == getattr(oArgs, 'audio_input'):
-            audio['input'] = k
-            LOG.debug(f"Setting audio['input'] {k} = {v} {k}")
-            break
-    for k,v in audio['output_devices'].items():
-        if v == 'default' and 'output' not in audio:
-            audio['output'] = k
-        if v == getattr(oArgs, 'audio_output'):
-            audio['output'] = k
-            LOG.debug(f"Setting audio['output'] {k} = {v} " +str(k))
-            break
-
-    if hasattr(oArgs, 'mode') and getattr(oArgs, 'mode') > 1:
-        audio['enabled'] = True
-        audio['audio_enabled'] = True
-        audio['video_enabled'] = True
-    elif hasattr(oArgs, 'mode') and getattr(oArgs, 'mode') > 0:
-        audio['enabled'] = True
-        audio['audio_enabled'] = False
-        audio['video_enabled'] = True
-    else:
-        audio['enabled'] = False
-        audio['audio_enabled'] = False
-        audio['video_enabled'] = False
-
-    return audio
-
-    i = getattr(oArgs, 'audio_output')
-    if i >= 0:
-        try:
-            elt = oPYA.get_device_info_by_index(i)
-            if i >= 0 and ( 'maxOutputChannels' not in elt or \
-                            elt['maxOutputChannels'] == 0):
-                LOG.warn(f"Audio output device has no output channels:  {i}")
-                oArgs.audio_output = -1
-        except OSError as e:
-            LOG.warn("Audio output device error looking for maxOutputChannels: " \
-                      +str(i) +' ' +str(e))
-            oArgs.audio_output = -1
-
-    if getattr(oArgs, 'audio_output') < 0:
-        LOG.info("Choose an output device:")
-        i = oPYA.get_device_count()
-        while i > 0:
-            i -= 1
-            if oPYA.get_device_info_by_index(i)['maxOutputChannels'] == 0:
-                continue
-            LOG.info(str(i) \
-                  +' ' +oPYA.get_device_info_by_index(i)['name'] \
-                  +' ' +str(oPYA.get_device_info_by_index(i)['defaultSampleRate'])
-                  )
-        return 0
-
-    i = getattr(oArgs, 'audio_input')
-    if i >= 0:
-        try:
-            elt = oPYA.get_device_info_by_index(i)
-            if i >= 0 and ( 'maxInputChannels' not in elt or \
-                            elt['maxInputChannels'] == 0):
-                LOG.warn(f"Audio input device has no input channels:  {i}")
-                setattr(oArgs, 'audio_input', -1)
-        except OSError as e:
-            LOG.warn("Audio input device error looking for maxInputChannels: " \
-                      +str(i) +' ' +str(e))
-            setattr(oArgs, 'audio_input', -1)
-    if getattr(oArgs, 'audio_input') < 0:
-        LOG.info("Choose an input device:")
-        i = oPYA.get_device_count()
-        while i > 0:
-            i -= 1
-            if oPYA.get_device_info_by_index(i)['maxInputChannels'] == 0:
-                continue
-            LOG.info(str(i) \
-                     +' ' +oPYA.get_device_info_by_index(i)['name']
-                     +' ' +str(oPYA.get_device_info_by_index(i)['defaultSampleRate'])
-                  )
-        return 0
-
-def setup_default_video():
-    default_video = ["-1"]
-    default_video.extend(ts.get_video_indexes())
-    LOG.info(f"Video input choices: {default_video}")
-    video = {'device': -1, 'width': 320, 'height': 240, 'x': 0, 'y': 0}
-    video['output_devices'] = default_video
-    return video
-
-def main_parser(_=None, iMode=2):
-    if not os.path.exists('/proc/sys/net/ipv6'):
-        bIpV6 = 'False'
-    else:
-        bIpV6 = 'True'
-    lIpV6Choices=[bIpV6, 'False']
-
-    audio = setup_default_audio()
-    default_video = setup_default_video()['output_devices']
-
-    parser = ts.oMainArgparser()
-    parser.add_argument('--version', action='store_true', help='Prints Toxygen version')
-    parser.add_argument('--clean', action='store_true', help='Delete toxcore libs from libs folder')
-    parser.add_argument('--reset', action='store_true', help='Reset default profile')
-    parser.add_argument('--uri', type=str, default='',
-                        help='Add specified Tox ID to friends')
-    parser.add_argument('--auto_accept_path', '--auto-accept-path', type=str,
-                        default=os.path.join(os.environ['HOME'], 'Downloads'),
-                        help="auto_accept_path")
-#    parser.add_argument('--mode', type=int, default=iMode,
-#                        help='Mode: 0=chat 1=chat+audio 2=chat+audio+video default: 0')
-    parser.add_argument('--font', type=str, default="Courier",
-                        help='Message font')
-    parser.add_argument('--message_font_size', type=int, default=15,
-                        help='Font size in pixels')
-    parser.add_argument('--local_discovery_enabled',type=str,
-                        default='False', choices=['True','False'],
-                        help='Look on the local lan')
-    parser.add_argument('--compact_mode',type=str,
-                        default='True', choices=['True','False'],
-                        help='Compact mode')
-    parser.add_argument('--allow_inline',type=str,
-                        default='False', choices=['True','False'],
-                        help='Dis/Enable allow_inline')
-    parser.add_argument('--notifications',type=str,
-                        default='True', choices=['True','False'],
-                        help='Dis/Enable notifications')
-    parser.add_argument('--sound_notifications',type=str,
-                        default='True', choices=['True','False'],
-                        help='Enable sound notifications')
-    parser.add_argument('--calls_sound',type=str,
-                        default='True', choices=['True','False'],
-                        help='Enable calls_sound')
-    parser.add_argument('--core_logging',type=str,
-                        default='False', choices=['True','False'],
-                        help='Dis/Enable Toxcore notifications')
-    parser.add_argument('--save_history',type=str,
-                        default='True', choices=['True','False'],
-                        help='En/Disable save history')
-    parser.add_argument('--update', type=int, default=0,
-                        choices=[0,0],
-                        help='Update program (broken)')
-    parser.add_argument('--video_input', type=str,
-                        default=-1,
-                        choices=default_video,
-                        help="Video input device number - /dev/video?")
-    parser.add_argument('--audio_input', type=str,
-                        default=oPYA.get_default_input_device_info()['name'],
-                        choices=audio['input_devices'].values(),
-                        help="Audio input device name - aplay -L for help")
-    parser.add_argument('--audio_output', type=str,
-                        default=oPYA.get_default_output_device_info()['index'],
-                        choices=audio['output_devices'].values(),
-                        help="Audio output device number - -1 for help")
-    parser.add_argument('--theme', type=str, default='default',
-                        choices=['dark', 'default'],
-                        help='Theme - style of UI')
-#    parser.add_argument('--sleep', type=str, default='time',
-#                        # could expand this to tk, gtk, gevent...
-#                        choices=['qt','gevent','time'],
-#                        help='Sleep method - one of qt, gevent , time')
-    supported_languages = settings.supported_languages()
-    parser.add_argument('--language', type=str, default='English',
-                        choices=supported_languages,
-                        help='Languages')
-    parser.add_argument('profile', type=str, nargs='?', default=None,
-                        help='Path to Tox profile')
-    return parser
-
-# clean out the unchanged settings so these can override the profile
-lKEEP_SETTINGS = ['uri',
-                  'profile',
-                  'loglevel',
-                  'logfile',
-                  'mode',
-
-                  # dunno
-                  'audio_input',
-                  'audio_output',
-                  'audio',
-                  'video',
-
-                  'ipv6_enabled',
-                  'udp_enabled',
-                  'local_discovery_enabled',
-                  'trace_enabled',
-
-                  'theme',
-                  'network',
-                  'message_font_size',
-                  'font',
-                  'save_history',
-                  'language',
-                  'update',
-                  'proxy_host',
-                  'proxy_type',
-                  'proxy_port',
-                  'core_logging',
-                  'audio',
-                  'video'
-                  ] # , 'nodes_json'
-
-class A(): pass
-
-def main(lArgs=None) -> int:
-    global oPYA
-    from argparse import Namespace
-    if lArgs is None:
-        lArgs = sys.argv[1:]
-    parser = main_parser()
-    default_ns = parser.parse_args([])
-    oArgs = parser.parse_args(lArgs)
-
-    if oArgs.version:
-        print_toxygen_version()
-        return 0
-
-    if oArgs.clean:
-        clean()
-        return 0
-
-    if oArgs.reset:
-        reset()
-        return 0
-
-    # if getattr(oArgs, 'network') in ['newlocal', 'localnew']: oArgs.network = 'new'
-
-    # clean out the unchanged settings so these can override the profile
-    for key in default_ns.__dict__.keys():
-        if key in lKEEP_SETTINGS: continue
-        if not hasattr(oArgs, key): continue
-        if getattr(default_ns, key) == getattr(oArgs, key):
-            delattr(oArgs, key)
-
-    ts.clean_booleans(oArgs)
-
-    aArgs = A()
-    for key in oArgs.__dict__.keys():
-        setattr(aArgs, key, getattr(oArgs, key))
-
-    #setattr(aArgs, 'video', setup_video(oArgs))
-    aArgs.video = setup_video(oArgs)
-    assert 'video' in aArgs.__dict__
-
-    #setattr(aArgs, 'audio', setup_audio(oArgs))
-    aArgs.audio = setup_audio(oArgs)
-    assert 'audio' in aArgs.__dict__
-    oArgs = aArgs
-
-    oApp = app.App(__version__, oArgs)
-    # for pyqtconsole
-    try:
-        setattr(__builtins__, 'app', oApp)
-    except Exception as e:
-        pass
-    i = oApp.iMain()
-    return i
-
-if __name__ == '__main__':
-    iRet = 0
-    try:
-        iRet = main(sys.argv[1:])
-    except KeyboardInterrupt:
-        iRet = 0
-    except SystemExit as e:
-        iRet = e
-    except Exception as e:
-        import traceback
-        sys.stderr.write(f"Exception from main  {e}" \
-                         +'\n' + traceback.format_exc() +'\n' )
-        iRet = 1
-
-    # Exception ignored in: <module 'threading' from '/usr/lib/python3.9/threading.py'>
-    # File "/usr/lib/python3.9/threading.py", line 1428, in _shutdown
-    # lock.acquire()
-    # gevent.exceptions.LoopExit as e:
-    # This operation would block forever
-    sys.stderr.write('Calling sys.exit' +'\n')
-    with ts.ignoreStdout():
-        sys.exit(iRet)

toxygen/av/call.py

@@ -17,7 +17,9 @@ class Call:
 
     is_active = property(get_is_active, set_is_active)
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Audio
+    # -----------------------------------------------------------------------------------------------------------------
 
     def get_in_audio(self):
         return self._in_audio
@@ -35,7 +37,9 @@ class Call:
 
     out_audio = property(get_out_audio, set_out_audio)
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Video
+    # -----------------------------------------------------------------------------------------------------------------
 
     def get_in_video(self):
         return self._in_video

toxygen/av/calls.py

@@ -1,47 +1,21 @@
-# -*- mode: python; indent-tabs-mode: nil; py-indent-offset: 4; coding: utf-8 -*-
+import pyaudio
 import time
 import threading
+from wrapper.toxav_enums import *
+import cv2
 import itertools
-
+import numpy as np
-from tox_wrapper.toxav_enums import *
-from tox_wrapper.tests import support_testing as ts
-from tox_wrapper.tests.support_testing import LOG_ERROR, LOG_WARN, LOG_INFO, LOG_DEBUG, LOG_TRACE
-
-with ts.ignoreStderr():
-    import pyaudio
 from av import screen_sharing
 from av.call import Call
 import common.tox_save
 
-from utils import ui as util_ui
-import tox_wrapper.tests.support_testing as ts
-from middleware.threads import invoke_in_main_thread
-from middleware.threads import BaseThread
-
-sleep = time.sleep
-
-global LOG
-import logging
-LOG = logging.getLogger('app.'+__name__)
-
-TIMER_TIMEOUT = 30.0
-bSTREAM_CALLBACK = False
-iFPS = 25
 
 class AV(common.tox_save.ToxAvSave):
 
     def __init__(self, toxav, settings):
         super().__init__(toxav)
-        self._toxav = toxav
         self._settings = settings
         self._running = True
-        s = settings
-        if 'video' not in s:
-            LOG.warn("AV.__init__ 'video' not in s" )
-            LOG.debug(f"AV.__init__ {s!r}" )
-        elif 'device' not in s['video']:
-            LOG.warn("AV.__init__ 'device' not in s.video" )
-            LOG.debug(f"AV.__init__ {s['video']!r}" )
 
         self._calls = {}  # dict: key - friend number, value - Call instance
 
@@ -51,30 +25,17 @@ class AV(common.tox_save.ToxAvSave):
         self._audio_running = False
         self._out_stream = None
 
+        self._audio_rate = 8000
         self._audio_channels = 1
         self._audio_duration = 60
-        self._audio_rate_pa = 48000
+        self._audio_sample_count = self._audio_rate * self._audio_channels * self._audio_duration // 1000
-        self._audio_rate_tox = 48000
-        self._audio_rate_pa = 48000
-        self._audio_krate_tox_audio = self._audio_rate_tox // 1000
-        self._audio_krate_tox_video = 5000
-        self._audio_sample_count_pa = self._audio_rate_pa * self._audio_channels * self._audio_duration // 1000
-        self._audio_sample_count_tox = self._audio_rate_tox * self._audio_channels * self._audio_duration // 1000
 
         self._video = None
         self._video_thread = None
-        self._video_running = None
+        self._video_running = False
 
-        self._video_width = 320
+        self._video_width = 640
-        self._video_height = 240
+        self._video_height = 480
-
-        # was iOutput = self._settings._args.audio['output']
-        iInput = self._settings['audio']['input']
-        self.lPaSampleratesI = ts.lSdSamplerates(iInput)
-        iOutput = self._settings['audio']['output']
-        self.lPaSampleratesO = ts.lSdSamplerates(iOutput)
-        global oPYA
-        oPYA = self._audio = pyaudio.PyAudio()
 
     def stop(self):
         self._running = False
@@ -84,75 +45,34 @@ class AV(common.tox_save.ToxAvSave):
     def __contains__(self, friend_number):
         return friend_number in self._calls
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Calls
+    # -----------------------------------------------------------------------------------------------------------------
 
     def __call__(self, friend_number, audio, video):
         """Call friend with specified number"""
-        if friend_number in self._calls:
+        self._toxav.call(friend_number, 32 if audio else 0, 5000 if video else 0)
-            LOG.warn(f"__call__ already has {friend_number}")
-            return
-        if self._audio_krate_tox_audio not in ts.lToxSampleratesK:
-            LOG.warn(f"__call__ {self._audio_krate_tox_audio} not in {ts.lToxSampleratesK}")
-
-        try:
-            self._toxav.call(friend_number,
-                             self._audio_krate_tox_audio if audio else 0,
-                             self._audio_krate_tox_video if video else 0)
-        except Exception as e:
-            LOG.warn(f"_toxav.call already has {friend_number}")
-            return
         self._calls[friend_number] = Call(audio, video)
-        threading.Timer(TIMER_TIMEOUT,
+        threading.Timer(30.0, lambda: self.finish_not_started_call(friend_number)).start()
-                        lambda: self.finish_not_started_call(friend_number)).start()
 
     def accept_call(self, friend_number, audio_enabled, video_enabled):
-        # obsolete
-        return self.call_accept_call(friend_number, audio_enabled, video_enabled)
-
-    def call_accept_call(self, friend_number, audio_enabled, video_enabled):
-        LOG.debug(f"call_accept_call from {friend_number} {self._running}" +
-                  f"{audio_enabled} {video_enabled}")
-        # import pdb; pdb.set_trace() - gets into q Qt exec_ problem
-        # ts.trepan_handler()
-
-        if self._audio_krate_tox_audio not in ts.lToxSampleratesK:
-            LOG.warn(f"__call__ {self._audio_krate_tox_audio} not in {ts.lToxSampleratesK}")
         if self._running:
             self._calls[friend_number] = Call(audio_enabled, video_enabled)
-            # audio_bit_rate: Audio bit rate in Kb/sec. Set this to 0 to disable audio sending.
+            self._toxav.answer(friend_number, 32 if audio_enabled else 0, 5000 if video_enabled else 0)
-            # video_bit_rate: Video bit rate in Kb/sec. Set this to 0 to disable video sending.
-            try:
-                self._toxav.answer(friend_number,
-                                   self._audio_krate_tox_audio if audio_enabled else 0,
-                                   self._audio_krate_tox_video if video_enabled else 0)
-            except Exception as e:
-                LOG.debug(f"AV accept_call error from {friend_number} {self._running}" +
-                          f"{e}")
-                raise
             if audio_enabled:
-                # may raise
                 self.start_audio_thread()
             if video_enabled:
-                # may raise
                 self.start_video_thread()
 
     def finish_call(self, friend_number, by_friend=False):
-        LOG.debug(f"finish_call  {friend_number}")
         if not by_friend:
             self._toxav.call_control(friend_number, TOXAV_CALL_CONTROL['CANCEL'])
         if friend_number in self._calls:
             del self._calls[friend_number]
-        try:
-            # AttributeError: 'int' object has no attribute 'out_audio'
         if not len(list(filter(lambda c: c.out_audio, self._calls))):
             self.stop_audio_thread()
         if not len(list(filter(lambda c: c.out_video, self._calls))):
             self.stop_video_thread()
-        except Exception as e:
-            LOG.error(f"finish_call FixMe:   {e}")
-            # dunno
-            self.stop_audio_thread()
-            self.stop_video_thread()
 
     def finish_not_started_call(self, friend_number):
         if friend_number in self:
@@ -164,7 +84,6 @@ class AV(common.tox_save.ToxAvSave):
         """
         New call state
         """
-        LOG.debug(f"toxav_call_state_cb {friend_number}")
         call = self._calls[friend_number]
         call.is_active = True
 
@@ -180,90 +99,39 @@ class AV(common.tox_save.ToxAvSave):
     def is_video_call(self, number):
         return number in self and self._calls[number].in_video
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Threads
+    # -----------------------------------------------------------------------------------------------------------------
 
     def start_audio_thread(self):
         """
         Start audio sending
-        from a callback
         """
-        global oPYA
-        # was iInput = self._settings._args.audio['input']
-        iInput = self._settings['audio']['input']
         if self._audio_thread is not None:
-            LOG_WARN(f"start_audio_thread device={iInput}")
             return
-        LOG_DEBUG(f"start_audio_thread device={iInput}")
-        lPaSamplerates = ts.lSdSamplerates(iInput)
-        if not(len(lPaSamplerates)):
-            e = f"No supported sample rates for device: audio[input]={iInput!r}"
-            LOG_ERROR(f"start_audio_thread {e}")
-            #?? dunno - cancel call?
-            return
-        if not self._audio_rate_pa in lPaSamplerates:
-            LOG_WARN(f"{self._audio_rate_pa} not in {lPaSamplerates!r}")
-            if False:
-                self._audio_rate_pa = oPYA.get_device_info_by_index(iInput)['defaultSampleRate']
-            else:
-                LOG_WARN(f"Setting audio_rate to: {lPaSamplerates[0]}")
-                self._audio_rate_pa = lPaSamplerates[0]
 
-        try:
+        self._audio_running = True
-            LOG_DEBUG( f"start_audio_thread framerate: {self._audio_rate_pa}" \
-                     +f" device: {iInput}"
-                     +f" supported: {lPaSamplerates!r}")
-            if self._audio_rate_pa not in lPaSamplerates:
-                LOG_WARN(f"PAudio sampling rate was {self._audio_rate_pa} changed to {lPaSamplerates[0]}")
-                self._audio_rate_pa = lPaSamplerates[0]
 
-            if bSTREAM_CALLBACK:
+        self._audio = pyaudio.PyAudio()
-                self._audio_stream = oPYA.open(format=pyaudio.paInt16,
+        self._audio_stream = self._audio.open(format=pyaudio.paInt16,
-                                               rate=self._audio_rate_pa,
+                                              rate=self._audio_rate,
                                               channels=self._audio_channels,
                                               input=True,
-                                               input_device_index=iInput,
+                                              input_device_index=self._settings.audio['input'],
-                                               frames_per_buffer=self._audio_sample_count_pa * 10,
+                                              frames_per_buffer=self._audio_sample_count * 10)
-                                               stream_callback=self.send_audio_data)
-                self._audio_running = True
-                self._audio_stream.start_stream()
-                while self._audio_stream.is_active():
-                    sleep(0.1)
-                self._audio_stream.stop_stream()
-                self._audio_stream.close()
 
-            else:
+        self._audio_thread = threading.Thread(target=self.send_audio)
-                self._audio_stream = oPYA.open(format=pyaudio.paInt16,
-                                               rate=self._audio_rate_pa,
-                                               channels=self._audio_channels,
-                                               input=True,
-                                               input_device_index=iInput,
-                                               frames_per_buffer=self._audio_sample_count_pa * 10)
-                self._audio_running = True
-                self._audio_thread = BaseThread(target=self.send_audio,
-                                                      name='_audio_thread')
         self._audio_thread.start()
 
-        except Exception as e:
-            LOG.error(f"Starting self._audio.open {e}")
-            LOG.debug(repr(dict(format=pyaudio.paInt16,
-                                rate=self._audio_rate_pa,
-                                channels=self._audio_channels,
-                                input=True,
-                                input_device_index=iInput,
-                                frames_per_buffer=self._audio_sample_count_pa * 10)))
-            # catcher in place in calls_manager? not if from a callback
-            # calls_manager._call.toxav_call_state_cb(friend_number, mask)
-            # raise RuntimeError(e)
-            return
-        else:
-            LOG_DEBUG(f"start_audio_thread {self._audio_stream!r}")
-
     def stop_audio_thread(self):
 
         if self._audio_thread is None:
             return
+
         self._audio_running = False
 
+        self._audio_thread.join()
+
         self._audio_thread = None
         self._audio_stream = None
         self._audio = None
@@ -276,197 +144,96 @@ class AV(common.tox_save.ToxAvSave):
     def start_video_thread(self):
         if self._video_thread is not None:
             return
-        s = self._settings
-        if 'video' not in s:
-            LOG.warn("AV.__init__ 'video' not in s" )
-            LOG.debug(f"start_video_thread {s!r}" )
-            raise RuntimeError("start_video_thread not 'video' in s)" )
-        elif 'device' not in s['video']:
-            LOG.error("start_video_thread not 'device' in s['video']" )
-            LOG.debug(f"start_video_thread {s['video']!r}" )
-            raise RuntimeError("start_video_thread not 'device' ins s['video']" )
-        self._video_width = s['video']['width']
-        self._video_height = s['video']['height']
-
-        # dunno
-        if True or s['video']['device'] == -1:
-            self._video = screen_sharing.DesktopGrabber(s['video']['x'],
-                                                        s['video']['y'],
-                                                        s['video']['width'],
-                                                        s['video']['height'])
-        else:
-            with ts.ignoreStdout():
-                import cv2
-            if s['video']['device'] == 0:
-                # webcam
-                self._video = cv2.VideoCapture(s['video']['device'], cv2.DSHOW)
-            else:
-                self._video = cv2.VideoCapture(s['video']['device'])
-            self._video.set(cv2.CAP_PROP_FPS, iFPS)
-            self._video.set(cv2.CAP_PROP_FRAME_WIDTH, self._video_width)
-            self._video.set(cv2.CAP_PROP_FRAME_HEIGHT, self._video_height)
-#            self._video.set(cv2.CAP_PROP_FOURCC, cv2.VideoWriter_fourcc('M', 'J', 'P', 'G'))
-        if self._video is None:
-            LOG.error("start_video_thread " \
-                     +f" device: {s['video']['device']}" \
-                     +f" supported: {s['video']['width']} {s['video']['height']}")
-            return
-        LOG.info("start_video_thread " \
-                 +f" device: {s['video']['device']}" \
-                 +f" supported: {s['video']['width']} {s['video']['height']}")
 
         self._video_running = True
-        self._video_thread = BaseThread(target=self.send_video,
+        self._video_width = s.video['width']
-                                        name='_video_thread')
+        self._video_height = s.video['height']
+
+        if s.video['device'] == -1:
+            self._video = screen_sharing.DesktopGrabber(self._settings.video['x'], self._settings.video['y'],
+                                                        self._settings.video['width'], self._settings.video['height'])
+        else:
+            self._video = cv2.VideoCapture(self._settings.video['device'])
+            self._video.set(cv2.CAP_PROP_FPS, 25)
+            self._video.set(cv2.CAP_PROP_FRAME_WIDTH, self._video_width)
+            self._video.set(cv2.CAP_PROP_FRAME_HEIGHT, self._video_height)
+
+        self._video_thread = threading.Thread(target=self.send_video)
         self._video_thread.start()
 
-    def stop_video_thread(self) -> None:
+    def stop_video_thread(self):
         if self._video_thread is None:
             return
 
         self._video_running = False
-        i = 0
+        self._video_thread.join()
-        while i < ts.iTHREAD_JOINS:
-            self._video_thread.join(ts.iTHREAD_TIMEOUT)
-            try:
-                if not self._video_thread.is_alive(): break
-            except:
-                # AttributeError: 'NoneType' object has no attribute 'join'
-                break
-            i = i + 1
-        else:
-            LOG.warn("self._video_thread.is_alive BLOCKED")
         self._video_thread = None
         self._video = None
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Incoming chunks
+    # -----------------------------------------------------------------------------------------------------------------
 
-    def audio_chunk(self, samples, channels_count, rate) -> None:
+    def audio_chunk(self, samples, channels_count, rate):
         """
         Incoming chunk
         """
 
         if self._out_stream is None:
-            # was iOutput = self._settings._args.audio['output']
+            self._out_stream = self._audio.open(format=pyaudio.paInt16,
-            iOutput = self._settings['audio']['output']
-            if not rate in self.lPaSampleratesO:
-                LOG.warn(f"{rate} not in {self.lPaSampleratesO!r}")
-                if False:
-                    rate = oPYA.get_device_info_by_index(iOutput)['defaultSampleRate']
-                LOG.warn(f"Setting audio_rate to: {self.lPaSampleratesO[0]}")
-                rate = self.lPaSampleratesO[0]
-            try:
-                with ts.ignoreStderr():
-                    self._out_stream = oPYA.open(format=pyaudio.paInt16,
                                                 channels=channels_count,
                                                 rate=rate,
-                                                 output_device_index=iOutput,
+                                                output_device_index=self._settings.audio['output'],
                                                 output=True)
-            except Exception as e:
-                LOG.error(f"Error playing audio_chunk creating self._out_stream   {e}")
-                invoke_in_main_thread(util_ui.message_box,
-                                    str(e),
-                                    util_ui.tr("Error Chunking audio"))
-                # dunno
-                self.stop()
-                return
-
-        iOutput = self._settings['audio']['output']
-        LOG.debug(f"audio_chunk output_device_index={iOutput} rate={rate} channels={channels_count}")
         self._out_stream.write(samples)
 
+    # -----------------------------------------------------------------------------------------------------------------
     # AV sending
+    # -----------------------------------------------------------------------------------------------------------------
 
-    def send_audio_data(self, data, count, *largs, **kwargs) -> None:
+    def send_audio(self):
-        pcm = data
-        # :param sampling_rate: Audio sampling rate used in this frame.
-        if self._toxav is None:
-            raise RuntimeError("_toxav not initialized")
-        if self._audio_rate_tox not in ts.lToxSamplerates:
-            LOG.warn(f"ToxAudio sampling rate was {self._audio_rate_tox} changed to {ts.lToxSamplerates[0]}")
-            self._audio_rate_tox = ts.lToxSamplerates[0]
-
-        for friend_num in self._calls:
-            if self._calls[friend_num].out_audio:
-                try:
-                    # app.av.calls ERROR Error send_audio:   One of the frame parameters was invalid. E.g. the resolution may be too small or too large, or the audio sampling rate may be unsupported
-                    # app.av.calls ERROR Error send_audio audio_send_frame: This client is currently not in a call with the friend.
-                    self._toxav.audio_send_frame(friend_num,
-                                                 pcm,
-                                                 count,
-                                                 self._audio_channels,
-                                                 self._audio_rate_tox)
-                except Exception as e:
-                   LOG.error(f"Error send_audio audio_send_frame: {e}")
-                   LOG.debug(f"send_audio self._audio_rate_tox={self._audio_rate_tox} self._audio_channels={self._audio_channels}")
-#                   invoke_in_main_thread(util_ui.message_box,
-#                                    str(e),
-#                                    util_ui.tr("Error send_audio audio_send_frame"))
-                   pass
-
-    def send_audio(self) -> None:
         """
         This method sends audio to friends
         """
-        i=0
+
-        count = self._audio_sample_count_tox
-        LOG.debug(f"send_audio stream={self._audio_stream}")
         while self._audio_running:
             try:
-                pcm = self._audio_stream.read(count, exception_on_overflow=False)
+                pcm = self._audio_stream.read(self._audio_sample_count)
-                if not pcm:
+                if pcm:
-                    sleep(0.1)
+                    for friend_num in self._calls:
-                else:
+                        if self._calls[friend_num].out_audio:
-                    self.send_audio_data(pcm, count)
+                            try:
+                                self._toxav.audio_send_frame(friend_num, pcm, self._audio_sample_count,
+                                                             self._audio_channels, self._audio_rate)
                             except:
-                LOG_DEBUG(f"error send_audio {i}")
+                                pass
-            else:
+            except:
-                LOG_TRACE(f"send_audio {i}")
+                pass
-            i += 1
-            sleep(0.01)
 
-    def send_video(self) -> None:
+            time.sleep(0.01)
+
+    def send_video(self):
         """
         This method sends video to friends
         """
-        LOG.debug(f"send_video thread={threading.current_thread().name}"
-                  +f" self._video_running={self._video_running}"
-                  +f" device: {self._settings['video']['device']}" )
         while self._video_running:
             try:
                 result, frame = self._video.read()
-                if not result:
+                if result:
-                    LOG.warn(f"send_video video_send_frame _video.read result={result}")
-                    break
-                if frame is None:
-                    LOG.warn(f"send_video video_send_frame _video.read result={result} frame={frame}")
-                    continue
-                else:
-                    LOG_TRACE(f"send_video video_send_frame _video.read result={result}")
                     height, width, channels = frame.shape
-                    friends = []
                     for friend_num in self._calls:
                         if self._calls[friend_num].out_video:
-                            friends.append(friend_num)
-                    if len(friends) == 0:
-                        LOG.warn(f"send_video video_send_frame no friends")
-                    else:
-                        LOG_TRACE(f"send_video video_send_frame {friends}")
-                        friend_num = friends[0]
                             try:
                                 y, u, v = self.convert_bgr_to_yuv(frame)
                                 self._toxav.video_send_frame(friend_num, width, height, y, u, v)
-                        except Exception as e:
+                            except:
-                            LOG.debug(f"send_video video_send_frame ERROR {e}")
+                                pass
+            except:
                 pass
 
-            except Exception as e:
+            time.sleep(0.01)
-                LOG.error(f"send_video video_send_frame {e}")
-                pass
 
-            sleep( 1.0/iFPS)
+    def convert_bgr_to_yuv(self, frame):
-
-    def convert_bgr_to_yuv(self, frame) -> tuple:
         """
         :param frame: input bgr frame
         :return y, u, v: y, u, v values of frame
@@ -497,14 +264,11 @@ class AV(common.tox_save.ToxAvSave):
         Y, U, V can be extracted using slices and joined in one list using itertools.chain.from_iterable()
         Function returns bytes(y), bytes(u), bytes(v), because it is required for ctypes
         """
-        with ts.ignoreStdout():
-            import cv2
         frame = cv2.cvtColor(frame, cv2.COLOR_BGR2YUV_I420)
 
         y = frame[:self._video_height, :]
         y = list(itertools.chain.from_iterable(y))
 
-        import numpy as np
         u = np.zeros((self._video_height // 2, self._video_width // 2), dtype=np.int)
         u[::2, :] = frame[self._video_height:self._video_height * 5 // 4, :self._video_width // 2]
         u[1::2, :] = frame[self._video_height:self._video_height * 5 // 4, self._video_width // 2:]

toxygen/av/calls_manager.py

@@ -1,36 +1,29 @@
-# -*- mode: python; indent-tabs-mode: nil; py-indent-offset: 4; coding: utf-8 -*-
-
-import sys
 import threading
-
+import cv2
 import av.calls
 from messenger.messages import *
 from ui import av_widgets
 import common.event as event
-import utils.ui as util_ui
 
-global LOG
-import logging
-LOG = logging.getLogger('app.'+__name__)
 
 class CallsManager:
 
-    def __init__(self, toxav, settings, main_screen, contacts_manager, app=None):
+    def __init__(self, toxav, settings, screen, contacts_manager):
-        self._callav = av.calls.AV(toxav, settings)  # object with data about calls
+        self._call = av.calls.AV(toxav, settings)  # object with data about calls
-        self._call = self._callav
         self._call_widgets = {}  # dict of incoming call widgets
         self._incoming_calls = set()
         self._settings = settings
-        self._main_screen = main_screen
+        self._screen = screen
         self._contacts_manager = contacts_manager
         self._call_started_event = event.Event()  # friend_number, audio, video, is_outgoing
         self._call_finished_event = event.Event()  # friend_number, is_declined
-        self._app = app
 
     def set_toxav(self, toxav):
-        self._callav.set_toxav(toxav)
+        self._call.set_toxav(toxav)
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Events
+    # -----------------------------------------------------------------------------------------------------------------
 
     def get_call_started_event(self):
         return self._call_started_event
@@ -42,33 +35,35 @@ class CallsManager:
 
     call_finished_event = property(get_call_finished_event)
 
+    # -----------------------------------------------------------------------------------------------------------------
     # AV support
+    # -----------------------------------------------------------------------------------------------------------------
 
     def call_click(self, audio=True, video=False):
         """User clicked audio button in main window"""
         num = self._contacts_manager.get_active_number()
         if not self._contacts_manager.is_active_a_friend():
             return
-        if num not in self._callav and self._contacts_manager.is_active_online():  # start call
+        if num not in self._call and self._contacts_manager.is_active_online():  # start call
-            if not self._settings['audio']['enabled']:
+            if not self._settings.audio['enabled']:
                 return
-            self._callav(num, audio, video)
+            self._call(num, audio, video)
-            self._main_screen.active_call()
+            self._screen.active_call()
             self._call_started_event(num, audio, video, True)
-        elif num in self._callav:  # finish or cancel call if you call with active friend
+        elif num in self._call:  # finish or cancel call if you call with active friend
             self.stop_call(num, False)
 
     def incoming_call(self, audio, video, friend_number):
         """
         Incoming call from friend.
         """
-        LOG.debug(__name__ +f" incoming_call  {friend_number}")
+        if not self._settings.audio['enabled']:
-        # if not self._settings['audio']['enabled']: return
+            return
         friend = self._contacts_manager.get_friend_by_number(friend_number)
         self._call_started_event(friend_number, audio, video, False)
         self._incoming_calls.add(friend_number)
         if friend_number == self._contacts_manager.get_active_number():
-            self._main_screen.incoming_call()
+            self._screen.incoming_call()
         else:
             friend.actions = True
         text = util_ui.tr("Incoming video call") if video else util_ui.tr("Incoming audio call")
@@ -79,83 +74,43 @@ class CallsManager:
     def accept_call(self, friend_number, audio, video):
         """
         Accept incoming call with audio or video
-        Called from a thread
         """
-
+        self._call.accept_call(friend_number, audio, video)
-        LOG.debug(f"CM accept_call from {friend_number} {audio} {video}")
+        self._screen.active_call()
-        sys.stdout.flush()
-
-        try:
-            self._callav.call_accept_call(friend_number, audio, video)
-        except Exception as e:
-            LOG.error(f"accept_call _call.accept_call ERROR for {friend_number} {e}")
-            self._main_screen.call_finished()
-            if hasattr(self._main_screen, '_settings') and \
-              'audio' in self._main_screen._settings and \
-              'input' in self._main_screen._settings['audio']:
-                iInput = self._settings['audio']['input']
-                iOutput = self._settings['audio']['output']
-                iVideo = self._settings['video']['device']
-                LOG.debug(f"iInput={iInput} iOutput={iOutput} iVideo={iVideo}")
-            elif hasattr(self._main_screen, '_settings') and \
-              hasattr(self._main_screen._settings, 'audio') and \
-              'input' not in self._main_screen._settings['audio']:
-                LOG.warn(f"'audio' not in {self._main_screen._settings!r}")
-            elif hasattr(self._main_screen, '_settings') and \
-              hasattr(self._main_screen._settings, 'audio') and \
-              'input' not in self._main_screen._settings['audio']:
-                LOG.warn(f"'audio' not in {self._main_screen._settings!r}")
-            else:
-                LOG.warn(f"_settings not in self._main_screen")
-            util_ui.message_box(str(e),
-                            util_ui.tr('ERROR Accepting call from {friend_number}'))
-        else:
-            self._main_screen.active_call()
-
-        finally:
-            # does not terminate call - just the av_widget
         if friend_number in self._incoming_calls:
             self._incoming_calls.remove(friend_number)
-            try:
-                self._call_widgets[friend_number].close()
         del self._call_widgets[friend_number]
-            except:
-                # RuntimeError: wrapped C/C++ object of type IncomingCallWidget has been deleted
-
-                pass
-            LOG.debug(f" closed self._call_widgets[{friend_number}]")
 
     def stop_call(self, friend_number, by_friend):
         """
         Stop call with friend
         """
-        LOG.debug(__name__+f" stop_call {friend_number}")
         if friend_number in self._incoming_calls:
             self._incoming_calls.remove(friend_number)
             is_declined = True
         else:
             is_declined = False
-        self._main_screen.call_finished()
+        self._screen.call_finished()
-        self._callav.finish_call(friend_number, by_friend)  # finish or decline call
+        is_video = self._call.is_video_call(friend_number)
+        self._call.finish_call(friend_number, by_friend)  # finish or decline call
         if friend_number in self._call_widgets:
             self._call_widgets[friend_number].close()
             del self._call_widgets[friend_number]
 
         def destroy_window():
-            #??? FixMed
-            is_video = self._callav.is_video_call(friend_number)
             if is_video:
-                import cv2
                 cv2.destroyWindow(str(friend_number))
 
         threading.Timer(2.0, destroy_window).start()
         self._call_finished_event(friend_number, is_declined)
 
     def friend_exit(self, friend_number):
-        if friend_number in self._callav:
+        if friend_number in self._call:
-            self._callav.finish_call(friend_number, True)
+            self._call.finish_call(friend_number, True)
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Private methods
+    # -----------------------------------------------------------------------------------------------------------------
 
     def _get_incoming_call_widget(self, friend_number, text, friend_name):
         return av_widgets.IncomingCallWidget(self._settings, self, friend_number, text, friend_name)

toxygen/av/screen_sharing.py

@@ -1,4 +1,5 @@
-from  qtpy import QtWidgets
+import numpy as np
+from PyQt5 import QtWidgets
 
 
 class DesktopGrabber:
@@ -16,7 +17,6 @@ class DesktopGrabber:
         pixmap = self._screen.grabWindow(0, self._x, self._y, self._width, self._height)
         image = pixmap.toImage()
         s = image.bits().asstring(self._width * self._height * 4)
-        import numpy as np
         arr = np.fromstring(s, dtype=np.uint8).reshape((self._height, self._width, 4))
 
         return True, arr

toxygen/bootstrap/bootstrap.py

@@ -1,48 +1,83 @@
-# -*- mode: python; indent-tabs-mode: nil; py-indent-offset: 4; coding: utf-8 -*-
 import random
 import urllib.request
 from utils.util import *
-from  qtpy import QtNetwork
+from PyQt5 import QtNetwork, QtCore
-from  qtpy import QtCore
+import json
-try:
-    import certifi
-    from io import BytesIO
-except ImportError:
-    certifi = None
 
-from user_data.settings import get_user_config_path
-from tox_wrapper.tests.support_testing import _get_nodes_path
-from tox_wrapper.tests.support_http import download_url
-import tox_wrapper.tests.support_testing as ts
 
-global LOG
+DEFAULT_NODES_COUNT = 4
-import logging
-LOG = logging.getLogger('app.'+'bootstrap')
 
-def download_nodes_list(settings, oArgs):
+
+class Node:
+
+    def __init__(self, node):
+        self._ip, self._port, self._tox_key = node['ipv4'], node['port'], node['public_key']
+        self._priority = random.randint(1, 1000000) if node['status_tcp'] and node['status_udp'] else 0
+
+    def get_priority(self):
+        return self._priority
+
+    priority = property(get_priority)
+
+    def get_data(self):
+        return self._ip, self._port, self._tox_key
+
+
+def generate_nodes(nodes_count=DEFAULT_NODES_COUNT):
+    with open(_get_nodes_path(), 'rt') as fl:
+        json_nodes = json.loads(fl.read())['nodes']
+    nodes = map(lambda json_node: Node(json_node), json_nodes)
+    nodes = filter(lambda n: n.priority > 0, nodes)
+    sorted_nodes = sorted(nodes, key=lambda x: x.priority)
+    if nodes_count is not None:
+        sorted_nodes = sorted_nodes[-DEFAULT_NODES_COUNT:]
+    for node in sorted_nodes:
+        yield node.get_data()
+
+
+def download_nodes_list(settings):
+    url = 'https://nodes.tox.chat/json'
     if not settings['download_nodes_list']:
-        return ''
+        return
-    if not ts.bAreWeConnected():
-        return ''
-    url = settings['download_nodes_url']
-    path = _get_nodes_path(oArgs=oArgs)
-    # dont download blindly so we can edit the file and not block on startup
-    if os.path.isfile(path):
-        with open(path, 'rt') as fl:
-            result = fl.read()
-            return result
-    LOG.debug("downloading list of nodes")
-    result = download_url(url, settings._app._settings)
-    if not result:
-        LOG.warn("failed downloading list of nodes")
-        return ''
-    LOG.info("downloaded list of nodes")
-    _save_nodes(result, settings._app)
-    return result
 
-def _save_nodes(nodes, app):
+    if not settings['proxy_type']:  # no proxy
+        try:
+            req = urllib.request.Request(url)
+            req.add_header('Content-Type', 'application/json')
+            response = urllib.request.urlopen(req)
+            result = response.read()
+            _save_nodes(result)
+        except Exception as ex:
+            log('TOX nodes loading error: ' + str(ex))
+    else:  # proxy
+        netman = QtNetwork.QNetworkAccessManager()
+        proxy = QtNetwork.QNetworkProxy()
+        proxy.setType(
+            QtNetwork.QNetworkProxy.Socks5Proxy if settings['proxy_type'] == 2 else QtNetwork.QNetworkProxy.HttpProxy)
+        proxy.setHostName(settings['proxy_host'])
+        proxy.setPort(settings['proxy_port'])
+        netman.setProxy(proxy)
+        try:
+            request = QtNetwork.QNetworkRequest()
+            request.setUrl(QtCore.QUrl(url))
+            reply = netman.get(request)
+
+            while not reply.isFinished():
+                QtCore.QThread.msleep(1)
+                QtCore.QCoreApplication.processEvents()
+            data = bytes(reply.readAll().data())
+            _save_nodes(data)
+        except Exception as ex:
+            log('TOX nodes loading error: ' + str(ex))
+
+
+def _get_nodes_path():
+    return join_path(curr_directory(__file__), 'nodes.json')
+
+
+def _save_nodes(nodes):
     if not nodes:
         return
-    with open(_get_nodes_path(app._args), 'wb') as fl:
+    print('Saving nodes...')
-        LOG.info("Saving nodes to " +_get_nodes_path(app._args))
+    with open(_get_nodes_path(), 'wb') as fl:
         fl.write(nodes)

toxygen/contacts/basecontact.py

@@ -1,7 +1,6 @@
-# -*- mode: python; indent-tabs-mode: nil; py-indent-offset: 4; coding: utf-8 -*-
 from user_data.settings import *
-from  qtpy import QtCore, QtGui
+from PyQt5 import QtCore, QtGui
-from tox_wrapper.toxcore_enums_and_consts import TOX_PUBLIC_KEY_SIZE
+from wrapper.toxcore_enums_and_consts import TOX_PUBLIC_KEY_SIZE
 import utils.util as util
 import common.event as event
 import contacts.common as common
@@ -15,27 +14,26 @@ class BaseContact:
     Base class for all contacts.
     """
 
-    def __init__(self, profile_manager, name, status_message, widget, tox_id, kind=''):
+    def __init__(self, profile_manager, name, status_message, widget, tox_id):
         """
         :param name: name, example: 'Toxygen user'
         :param status_message: status message, example: 'Toxing on Toxygen'
         :param widget: ContactItem instance
         :param tox_id: tox id of contact
-        :param kind: one of ['bot', 'friend', 'group', 'invite', 'grouppeer', '']
         """
         self._profile_manager = profile_manager
         self._name, self._status_message = name, status_message
-        self._kind = kind
         self._status, self._widget = None, widget
         self._tox_id = tox_id
-
         self._name_changed_event = event.Event()
         self._status_message_changed_event = event.Event()
         self._status_changed_event = event.Event()
         self._avatar_changed_event = event.Event()
         self.init_widget()
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Name - current name or alias of user
+    # -----------------------------------------------------------------------------------------------------------------
 
     def get_name(self):
         return self._name
@@ -55,7 +53,9 @@ class BaseContact:
 
     name_changed_event = property(get_name_changed_event)
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Status message
+    # -----------------------------------------------------------------------------------------------------------------
 
     def get_status_message(self):
         return self._status_message
@@ -75,7 +75,9 @@ class BaseContact:
 
     status_message_changed_event = property(get_status_message_changed_event)
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Status
+    # -----------------------------------------------------------------------------------------------------------------
 
     def get_status(self):
         return self._status
@@ -94,20 +96,23 @@ class BaseContact:
 
     status_changed_event = property(get_status_changed_event)
 
+    # -----------------------------------------------------------------------------------------------------------------
     # TOX ID. WARNING: for friend it will return public key, for profile - full address
+    # -----------------------------------------------------------------------------------------------------------------
 
     def get_tox_id(self):
         return self._tox_id
 
     tox_id = property(get_tox_id)
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Avatars
+    # -----------------------------------------------------------------------------------------------------------------
 
     def load_avatar(self):
         """
         Tries to load avatar of contact or uses default avatar
         """
-        try:
         avatar_path = self.get_avatar_path()
         width = self._widget.avatar_label.width()
         pixmap = QtGui.QPixmap(avatar_path)
@@ -115,8 +120,6 @@ class BaseContact:
                                                           QtCore.Qt.SmoothTransformation))
         self._widget.avatar_label.repaint()
         self._avatar_changed_event(avatar_path)
-        except Exception as e:
-            pass
 
     def reset_avatar(self, generate_new):
         avatar_path = self.get_avatar_path()
@@ -158,23 +161,19 @@ class BaseContact:
 
     avatar_changed_event = property(get_avatar_changed_event)
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Widgets
+    # -----------------------------------------------------------------------------------------------------------------
 
     def init_widget(self):
-        # File "/mnt/o/var/local/src/toxygen/toxygen/contacts/contacts_manager.py", line 252, in filtration_and_sorting
-        # contact.set_widget(item_widget)
-        # File "/mnt/o/var/local/src/toxygen/toxygen/contacts/contact.py", line 320, in set_widget
-        if not self._widget:
-            LOG.warn("BC.init_widget self._widget is NULL")
-            return
         self._widget.name.setText(self._name)
         self._widget.status_message.setText(self._status_message)
-        if hasattr(self._widget, 'kind'):
-            self._widget.kind.setText(self._kind)
         self._widget.connection_status.update(self._status)
         self.load_avatar()
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Private methods
+    # -----------------------------------------------------------------------------------------------------------------
 
     @staticmethod
     def _get_default_avatar_path():

toxygen/contacts/common.py

@@ -2,7 +2,9 @@ from pydenticon import Generator
 import hashlib
 
 
+# -----------------------------------------------------------------------------------------------------------------
 # Typing notifications
+# -----------------------------------------------------------------------------------------------------------------
 
 class BaseTypingNotificationHandler:
 
@@ -28,7 +30,9 @@ class FriendTypingNotificationHandler(BaseTypingNotificationHandler):
 BaseTypingNotificationHandler.DEFAULT_HANDLER = BaseTypingNotificationHandler()
 
 
+# -----------------------------------------------------------------------------------------------------------------
 # Identicons support
+# -----------------------------------------------------------------------------------------------------------------
 
 
 def generate_avatar(public_key):

toxygen/contacts/contact.py

@@ -1,17 +1,10 @@
-# -*- mode: python; indent-tabs-mode: nil; py-indent-offset: 4; coding: utf-8 -*-
+from history.database import *
-from history.database import TIMEOUT, \
-    SAVE_MESSAGES, MESSAGE_AUTHOR
-
 from contacts import basecontact, common
 from messenger.messages import *
 from contacts.contact_menu import *
 from file_transfers import file_transfers as ft
 import re
 
-# LOG=util.log
-global LOG
-import logging
-LOG = logging.getLogger('app.'+__name__)
 
 class Contact(basecontact.BaseContact):
     """
@@ -42,7 +35,9 @@ class Contact(basecontact.BaseContact):
         if hasattr(self, '_message_getter'):
             del self._message_getter
 
+    # -----------------------------------------------------------------------------------------------------------------
     # History support
+    # -----------------------------------------------------------------------------------------------------------------
 
     def load_corr(self, first_time=True):
         """
@@ -119,7 +114,9 @@ class Contact(basecontact.BaseContact):
 
         return TextMessage(message, author, unix_time, message_type, unique_id)
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Unsent messages
+    # -----------------------------------------------------------------------------------------------------------------
 
     def get_unsent_messages(self):
         """
@@ -132,11 +129,8 @@ class Contact(basecontact.BaseContact):
         """
         :return list of unsent messages for saving
         """
-#                               and m.tox_message_id == tox_message_id,
+        messages = filter(lambda m: m.type in (MESSAGE_TYPE['TEXT'], MESSAGE_TYPE['ACTION'])
-        messages = filter(lambda m: m.author is not None
+                                    and m.author.type == MESSAGE_AUTHOR['NOT_SENT'], self._corr)
-                              and m.author.type == MESSAGE_AUTHOR['NOT_SENT'],
-                              self._corr)
-        # was message = list(...)[0]
         return list(messages)
 
     def mark_as_sent(self, tox_message_id):
@@ -145,10 +139,11 @@ class Contact(basecontact.BaseContact):
                                             and m.tox_message_id == tox_message_id, self._corr))[0]
             message.mark_as_sent()
         except Exception as ex:
-            #   wrapped C/C++ object of type QLabel has been deleted
+            util.log('Mark as sent ex: ' + str(ex))
-            LOG.error(f"Mark as sent:  {ex!s}")
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Message deletion
+    # -----------------------------------------------------------------------------------------------------------------
 
     def delete_message(self, message_id):
         elem = list(filter(lambda m: m.message_id == message_id, self._corr))[0]
@@ -194,7 +189,9 @@ class Contact(basecontact.BaseContact):
                                      self._corr))
             self._unsaved_messages = len(self.get_unsent_messages())
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Chat history search
+    # -----------------------------------------------------------------------------------------------------------------
 
     def search_string(self, search_string):
         self._search_string, self._search_index = search_string, 0
@@ -227,7 +224,9 @@ class Contact(basecontact.BaseContact):
                 return i
         return None  # not found
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Current text - text from message area
+    # -----------------------------------------------------------------------------------------------------------------
 
     def get_curr_text(self):
         return self._curr_text
@@ -237,7 +236,9 @@ class Contact(basecontact.BaseContact):
 
     curr_text = property(get_curr_text, set_curr_text)
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Alias support
+    # -----------------------------------------------------------------------------------------------------------------
 
     def set_name(self, value):
         """
@@ -253,7 +254,9 @@ class Contact(basecontact.BaseContact):
     def has_alias(self):
         return self._alias
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Visibility in friends' list
+    # -----------------------------------------------------------------------------------------------------------------
 
     def get_visibility(self):
         return self._visible
@@ -263,7 +266,9 @@ class Contact(basecontact.BaseContact):
 
     visibility = property(get_visibility, set_visibility)
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Unread messages and other actions from friend
+    # -----------------------------------------------------------------------------------------------------------------
 
     def get_actions(self):
         return self._new_actions
@@ -291,7 +296,9 @@ class Contact(basecontact.BaseContact):
 
     messages = property(get_messages)
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Friend's or group's number (can be used in toxcore)
+    # -----------------------------------------------------------------------------------------------------------------
 
     def get_number(self):
         return self._number
@@ -301,19 +308,25 @@ class Contact(basecontact.BaseContact):
 
     number = property(get_number, set_number)
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Typing notifications
+    # -----------------------------------------------------------------------------------------------------------------
 
     def get_typing_notification_handler(self):
         return common.BaseTypingNotificationHandler.DEFAULT_HANDLER
 
     typing_notification_handler = property(get_typing_notification_handler)
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Context menu support
+    # -----------------------------------------------------------------------------------------------------------------
 
     def get_context_menu_generator(self):
         return BaseContactMenuGenerator(self)
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Filtration support
+    # -----------------------------------------------------------------------------------------------------------------
 
     def set_widget(self, widget):
         self._widget = widget

toxygen/contacts/contact_menu.py

@@ -1,14 +1,10 @@
-# -*- mode: python; indent-tabs-mode: nil; py-indent-offset: 4; coding: utf-8 -*-
+from PyQt5 import QtWidgets
-from  qtpy import QtWidgets
-
 import utils.ui as util_ui
-from tox_wrapper.toxcore_enums_and_consts import *
 
-global LOG
-import logging
-LOG = logging.getLogger('app')
 
+# -----------------------------------------------------------------------------------------------------------------
 # Builder
+# -----------------------------------------------------------------------------------------------------------------
 
 def _create_menu(menu_name, parent):
     menu_name = menu_name or ''
@@ -81,7 +77,9 @@ class ContactMenuBuilder:
         self._actions[self._index] = (text, handler)
         self._index += 1
 
+# -----------------------------------------------------------------------------------------------------------------
 # Generators
+# -----------------------------------------------------------------------------------------------------------------
 
 
 class BaseContactMenuGenerator:
@@ -92,15 +90,17 @@ class BaseContactMenuGenerator:
     def generate(self, plugin_loader, contacts_manager, main_screen, settings, number, groups_service, history_loader):
         return ContactMenuBuilder().build()
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Private methods
+    # -----------------------------------------------------------------------------------------------------------------
 
     def _generate_copy_menu_builder(self, main_screen):
         copy_menu_builder = ContactMenuBuilder()
         (copy_menu_builder
          .with_name(util_ui.tr('Copy'))
          .with_action(util_ui.tr('Name'), lambda: main_screen.copy_text(self._contact.name))
-         .with_action(util_ui.tr("Status message"), lambda: main_screen.copy_text(self._contact.status_message))
+         .with_action(util_ui.tr('Status message'), lambda: main_screen.copy_text(self._contact.status_message))
-         .with_action(util_ui.tr("Public key"), lambda: main_screen.copy_text(self._contact.tox_id))
+         .with_action(util_ui.tr('Public key'), lambda: main_screen.copy_text(self._contact.tox_id))
          )
 
         return copy_menu_builder
@@ -108,11 +108,11 @@ class BaseContactMenuGenerator:
     def _generate_history_menu_builder(self, history_loader, main_screen):
         history_menu_builder = ContactMenuBuilder()
         (history_menu_builder
-         .with_name(util_ui.tr("Chat history"))
+         .with_name(util_ui.tr('Chat history'))
-         .with_action(util_ui.tr("Clear history"), lambda: history_loader.clear_history(self._contact)
+         .with_action(util_ui.tr('Clear history'), lambda: history_loader.clear_history(self._contact)
                                                            or main_screen.messages.clear())
-         .with_action(util_ui.tr("Export as text"), lambda: history_loader.export_history(self._contact))
+         .with_action(util_ui.tr('Export as text'), lambda: history_loader.export_history(self._contact))
-         .with_action(util_ui.tr("Export as HTML"), lambda: history_loader.export_history(self._contact, False))
+         .with_action(util_ui.tr('Export as HTML'), lambda: history_loader.export_history(self._contact, False))
          )
 
         return history_menu_builder
@@ -127,16 +127,16 @@ class FriendMenuGenerator(BaseContactMenuGenerator):
         groups_menu_builder = self._generate_groups_menu(contacts_manager, groups_service)
 
         allowed = self._contact.tox_id in settings['auto_accept_from_friends']
-        auto = util_ui.tr("Disallow auto accept") if allowed else util_ui.tr('Allow auto accept')
+        auto = util_ui.tr('Disallow auto accept') if allowed else util_ui.tr('Allow auto accept')
 
         builder = ContactMenuBuilder()
         menu = (builder
-                .with_action(util_ui.tr("Set alias"), lambda: main_screen.set_alias(number))
+                .with_action(util_ui.tr('Set alias'), lambda: main_screen.set_alias(number))
                 .with_submenu(history_menu_builder)
                 .with_submenu(copy_menu_builder)
                 .with_action(auto, lambda: main_screen.auto_accept(number, not allowed))
-                .with_action(util_ui.tr("Remove friend"), lambda: main_screen.remove_friend(number))
+                .with_action(util_ui.tr('Remove friend'), lambda: main_screen.remove_friend(number))
-                .with_action(util_ui.tr("Block friend"), lambda: main_screen.block_friend(number))
+                .with_action(util_ui.tr('Block friend'), lambda: main_screen.block_friend(number))
                 .with_action(util_ui.tr('Notes'), lambda: main_screen.show_note(self._contact))
                 .with_optional_submenu(plugins_menu_builder)
                 .with_optional_submenu(groups_menu_builder)
@@ -144,7 +144,9 @@ class FriendMenuGenerator(BaseContactMenuGenerator):
 
         return menu
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Private methods
+    # -----------------------------------------------------------------------------------------------------------------
 
     @staticmethod
     def _generate_plugins_menu_builder(plugin_loader, number):
@@ -163,13 +165,11 @@ class FriendMenuGenerator(BaseContactMenuGenerator):
 
     def _generate_groups_menu(self, contacts_manager, groups_service):
         chats = contacts_manager.get_group_chats()
-        LOG.debug(f"_generate_groups_menu len(chats)={len(chats)} or self._contact.status={self._contact.status}")
         if not len(chats) or self._contact.status is None:
-            #? return None
+            return None
-            pass
         groups_menu_builder = ContactMenuBuilder()
         (groups_menu_builder
-         .with_name(util_ui.tr("Invite to group"))
+         .with_name(util_ui.tr('Invite to group'))
          .with_actions([(g.name, lambda: groups_service.invite_friend(self._contact.number, g.number)) for g in chats])
          )
 
@@ -184,26 +184,26 @@ class GroupMenuGenerator(BaseContactMenuGenerator):
 
         builder = ContactMenuBuilder()
         menu = (builder
-                .with_action(util_ui.tr("Set alias"), lambda: main_screen.set_alias(number))
+                .with_action(util_ui.tr('Set alias'), lambda: main_screen.set_alias(number))
                 .with_submenu(copy_menu_builder)
                 .with_submenu(history_menu_builder)
-                .with_optional_action(util_ui.tr("Manage group"),
+                .with_optional_action(util_ui.tr('Manage group'),
                                       lambda: groups_service.show_group_management_screen(self._contact),
                                       self._contact.is_self_founder())
-                .with_optional_action(util_ui.tr("Group settings"),
+                .with_optional_action(util_ui.tr('Group settings'),
                                       lambda: groups_service.show_group_settings_screen(self._contact),
                                       not self._contact.is_self_founder())
-                .with_optional_action(util_ui.tr("Set topic"),
+                .with_optional_action(util_ui.tr('Set topic'),
                                       lambda: groups_service.set_group_topic(self._contact),
                                       self._contact.is_self_moderator_or_founder())
-#                .with_action(util_ui.tr("Bans list"),
+                .with_action(util_ui.tr('Bans list'),
-#                             lambda: groups_service.show_bans_list(self._contact))
+                             lambda: groups_service.show_bans_list(self._contact))
-                .with_action(util_ui.tr("Reconnect to group"),
+                .with_action(util_ui.tr('Reconnect to group'),
                              lambda: groups_service.reconnect_to_group(self._contact.number))
-                .with_optional_action(util_ui.tr("Disconnect from group"),
+                .with_optional_action(util_ui.tr('Disconnect from group'),
                                       lambda: groups_service.disconnect_from_group(self._contact.number),
                                       self._contact.status is not None)
-                .with_action(util_ui.tr("Leave group"), lambda: groups_service.leave_group(self._contact.number))
+                .with_action(util_ui.tr('Leave group'), lambda: groups_service.leave_group(self._contact.number))
                 .with_action(util_ui.tr('Notes'), lambda: main_screen.show_note(self._contact))
                 ).build()
 
@@ -218,10 +218,10 @@ class GroupPeerMenuGenerator(BaseContactMenuGenerator):
 
         builder = ContactMenuBuilder()
         menu = (builder
-                .with_action(util_ui.tr("Set alias"), lambda: main_screen.set_alias(number))
+                .with_action(util_ui.tr('Set alias'), lambda: main_screen.set_alias(number))
                 .with_submenu(copy_menu_builder)
                 .with_submenu(history_menu_builder)
-                .with_action(util_ui.tr("Quit chat"),
+                .with_action(util_ui.tr('Quit chat'),
                              lambda: contacts_manager.remove_group_peer(self._contact))
                 .with_action(util_ui.tr('Notes'), lambda: main_screen.show_note(self._contact))
                 ).build()

toxygen/contacts/contact_provider.py

@@ -1,32 +1,22 @@
-# -*- mode: python; indent-tabs-mode: nil; py-indent-offset: 4; coding: utf-8 -*-
-
 import common.tox_save as tox_save
 
-global LOG
-import logging
-LOG = logging.getLogger(__name__)
-
-# callbacks can be called in any thread so were being careful
-from av.calls import LOG_ERROR, LOG_WARN, LOG_INFO, LOG_DEBUG, LOG_TRACE
 
 class ContactProvider(tox_save.ToxSave):
 
-    def __init__(self, tox, friend_factory, group_factory, group_peer_factory, app=None):
+    def __init__(self, tox, friend_factory, group_factory, group_peer_factory):
         super().__init__(tox)
         self._friend_factory = friend_factory
         self._group_factory = group_factory
         self._group_peer_factory = group_peer_factory
         self._cache = {}  # key - contact's public key, value - contact instance
-        self._app = app
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Friends
+    # -----------------------------------------------------------------------------------------------------------------
 
     def get_friend_by_number(self, friend_number):
-        try:
         public_key = self._tox.friend_get_public_key(friend_number)
-        except Exception as e:
+
-            LOG_WARN(f"CP.get_friend_by_number NO {friend_number} {e} ")
-            return None
         return self.get_friend_by_public_key(public_key)
 
     def get_friend_by_public_key(self, public_key):
@@ -34,118 +24,67 @@ class ContactProvider(tox_save.ToxSave):
         if friend is not None:
             return friend
         friend = self._friend_factory.create_friend_by_public_key(public_key)
-        if friend is None:
-            LOG_WARN(f"CP.get_friend_by_public_key NULL {friend} ")
-        else:
         self._add_to_cache(public_key, friend)
-            LOG_DEBUG(f"CP.get_friend_by_public_key ADDED {friend} ")
+
         return friend
 
     def get_all_friends(self):
-        if self._app and self._app.bAppExiting:
-            return
-        try:
         friend_numbers = self._tox.self_get_friend_list()
-        except Exception as e:
-            LOG_WARN(f"CP.get_all_friends EXCEPTION {e} ")
-            return None
         friends = map(lambda n: self.get_friend_by_number(n), friend_numbers)
+
         return list(friends)
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Groups
+    # -----------------------------------------------------------------------------------------------------------------
 
     def get_all_groups(self):
-        """from callbacks"""
+        group_numbers = range(self._tox.group_get_number_groups())
-        try:
+        groups = map(lambda n: self.get_group_by_number(n), group_numbers)
-            len_groups = self._tox.group_get_number_groups()
+
-            group_numbers = range(len_groups)
+        return list(groups)
-        except Exception as e:
-            return None
-        groups = list(map(lambda n: self.get_group_by_number(n), group_numbers))
-        # failsafe in case there are bogus None groups?
-        fgroups = list(filter(lambda x: x, groups))
-        if len(fgroups) != len_groups:
-            LOG_WARN(f"CP.are there are bogus None groups in libtoxcore? {len(fgroups)} != {len_groups}")
-            for group_num in group_numbers:
-                group = self.get_group_by_number(group_num)
-                if group is None:
-                    LOG_ERROR(f"There are bogus None groups in libtoxcore {group_num}!")
-                    # fixme: do something
-            groups = fgroups
-        return groups
 
     def get_group_by_number(self, group_number):
-        group = None
+        public_key = self._tox.group_get_chat_id(group_number)
-        try:
-#            LOG_DEBUG(f"CP.CP.group_get_number {group_number} ")
-            # original code
-            chat_id = self._tox.group_get_chat_id(group_number)
-            if chat_id is None:
-                LOG_ERROR(f"get_group_by_number NULL chat_id ({group_number})")
-            elif chat_id == '-1':
-                LOG_ERROR(f"get_group_by_number <0 chat_id ({group_number})")
-            else:
-                LOG_INFO(f"CP.group_get_number {group_number} {chat_id}")
-                group = self.get_group_by_chat_id(chat_id)
-                if group is None or group  == '-1':
-                    LOG_WARN(f"CP.get_group_by_number leaving {group} ({group_number})")
-                    #? iRet = self._tox.group_leave(group_number)
-                    # invoke in main thread?
-                    # self._contacts_manager.delete_group(group_number)
-            return group
-        except Exception as e:
-            LOG_WARN(f"CP.group_get_number {group_number} {e}")
-            return None
 
-    def get_group_by_chat_id(self, chat_id):
+        return self.get_group_by_public_key(public_key)
-        group = self._get_contact_from_cache(chat_id)
-        if group is not None:
-            return group
-        group = self._group_factory.create_group_by_chat_id(chat_id)
-        if group is None:
-            LOG_ERROR(f"get_group_by_chat_id NULL chat_id={chat_id}")
-        else:
-            self._add_to_cache(chat_id, group)
-
-        return group
 
     def get_group_by_public_key(self, public_key):
         group = self._get_contact_from_cache(public_key)
         if group is not None:
             return group
         group = self._group_factory.create_group_by_public_key(public_key)
-        if group is None:
-            LOG_WARN(f"get_group_by_public_key NULL group public_key={public_key}")
-        else:
         self._add_to_cache(public_key, group)
 
         return group
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Group peers
+    # -----------------------------------------------------------------------------------------------------------------
 
     def get_all_group_peers(self):
         return list()
 
     def get_group_peer_by_id(self, group, peer_id):
         peer = group.get_peer_by_id(peer_id)
-        if peer is not None:
+
         return self._get_group_peer(group, peer)
-        LOG_WARN(f"get_group_peer_by_id peer_id={peer_id}")
-        return None
 
     def get_group_peer_by_public_key(self, group, public_key):
         peer = group.get_peer_by_public_key(public_key)
-        if peer is not None:
-            return self._get_group_peer(group, peer)
-        LOG_WARN(f"get_group_peer_by_public_key public_key={public_key}")
-        return None
 
+        return self._get_group_peer(group, peer)
+
+    # -----------------------------------------------------------------------------------------------------------------
     # All contacts
+    # -----------------------------------------------------------------------------------------------------------------
 
     def get_all(self):
         return self.get_all_friends() + self.get_all_groups() + self.get_all_group_peers()
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Caching
+    # -----------------------------------------------------------------------------------------------------------------
 
     def clear_cache(self):
         self._cache.clear()
@@ -154,7 +93,9 @@ class ContactProvider(tox_save.ToxSave):
         if contact_public_key in self._cache:
             del self._cache[contact_public_key]
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Private methods
+    # -----------------------------------------------------------------------------------------------------------------
 
     def _get_contact_from_cache(self, public_key):
         return self._cache[public_key] if public_key in self._cache else None

toxygen/contacts/contacts_manager.py

@@ -1,38 +1,9 @@
-# -*- mode: python; indent-tabs-mode: nil; py-indent-offset: 4; coding: utf-8 -*-
-
-import traceback
-
 from contacts.friend import Friend
 from contacts.group_chat import GroupChat
 from messenger.messages import *
 from common.tox_save import ToxSave
 from contacts.group_peer_contact import GroupPeerContact
-from groups.group_peer import GroupChatPeer
-from middleware.callbacks import LOG_ERROR, LOG_WARN, LOG_INFO, LOG_DEBUG, LOG_TRACE
-import tox_wrapper.toxcore_enums_and_consts as enums
 
-# LOG=util.log
-global LOG
-import logging
-LOG = logging.getLogger('app.'+__name__)
-
-
-UINT32_MAX = 2 ** 32 -1
-
-def set_contact_kind(contact):
-    bInvite = len(contact.name) == enums.TOX_PUBLIC_KEY_SIZE * 2 and \
-      contact.status_message == ''
-    bBot = not bInvite and contact.name.lower().endswith(' bot')
-    if type(contact) == Friend and bInvite:
-        contact._kind = 'invite'
-    elif type(contact) == Friend and bBot:
-        contact._kind = 'bot'
-    elif type(contact) == Friend:
-        contact._kind = 'friend'
-    elif type(contact) == GroupChat:
-        contact._kind = 'group'
-    elif type(contact) == GroupChatPeer:
-        contact._kind = 'grouppeer'
 
 class ContactsManager(ToxSave):
     """
@@ -44,14 +15,12 @@ class ContactsManager(ToxSave):
         super().__init__(tox)
         self._settings = settings
         self._screen = screen
-        self._ms = screen
         self._profile_manager = profile_manager
         self._contact_provider = contact_provider
         self._tox_dns = tox_dns
         self._messages_items_factory = messages_items_factory
         self._messages = screen.messages
-        self._contacts = []
+        self._contacts, self._active_contact = [], -1
-        self._active_contact = -1
         self._active_contact_changed = Event()
         self._sorting = settings['sorting']
         self._filter_string = ''
@@ -59,11 +28,6 @@ class ContactsManager(ToxSave):
         self._history = history
         self._load_contacts()
 
-    def _log(self, s):
-        try:
-            self._ms._log(s)
-        except: pass
-
     def get_contact(self, num):
         if num < 0 or num >= len(self._contacts):
             return None
@@ -89,29 +53,19 @@ class ContactsManager(ToxSave):
         return self.get_curr_contact().number == group_number
 
     def is_contact_active(self, contact):
-        if self._active_contact == -1:
-#            LOG.debug("No self._active_contact")
-            return False
-        if self._active_contact >= len(self._contacts):
-            LOG.warn(f"ERROR _active_contact={self._active_contact} >= contacts len={len(self._contacts)}")
-            return False
-        if not self._contacts[self._active_contact]:
-            LOG.warn(f"ERROR NULL {self._contacts[self._active_contact]}  {contact.tox_id}")
-            return False
-
-        if not hasattr(contact, 'tox_id'):
-            LOG.warn(f"ERROR is_contact_active no contact.tox_id {type(contact)}  contact={contact}")
-            return False
-
         return self._contacts[self._active_contact].tox_id == contact.tox_id
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Reconnection support
+    # -----------------------------------------------------------------------------------------------------------------
 
     def reset_contacts_statuses(self):
         for contact in self._contacts:
             contact.status = None
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Work with active friend
+    # -----------------------------------------------------------------------------------------------------------------
 
     def get_active(self):
         return self._active_contact
@@ -141,16 +95,11 @@ class ContactsManager(ToxSave):
                 current_contact.remove_messages_widgets()  # TODO: if required
                 self._unsubscribe_from_events(current_contact)
 
-            if self._active_contact >= 0 and self._active_contact != value:
+            if self._active_contact + 1 and self._active_contact != value:
                 try:
                     current_contact.curr_text = self._screen.messageEdit.toPlainText()
                 except:
                     pass
-
-            # IndexError: list index out of range
-            if value >= len(self._contacts):
-                LOG.warn("CM.set_active value too big: {{self._contacts}}")
-                return
             contact = self._contacts[value]
             self._subscribe_to_events(contact)
             contact.remove_invalid_unsent_files()
@@ -179,9 +128,10 @@ class ContactsManager(ToxSave):
             #     self._screen.call_finished()
             self._set_current_contact_data(contact)
             self._active_contact_changed(contact)
-        except Exception as e:  # no friend found. ignore
+        except Exception as ex:  # no friend found. ignore
-            LOG.warn(f"CM.set_active EXCEPTION  value:{value} len={len(self._contacts)} {e}")
+            util.log('Friend value: ' + str(value))
-            # gulp raise
+            util.log('Error in set active: ' + str(ex))
+            raise
 
     active_contact = property(get_active, set_active)
 
@@ -203,23 +153,21 @@ class ContactsManager(ToxSave):
     def is_active_a_group_chat_peer(self):
         return type(self.get_curr_contact()) is GroupPeerContact
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Filtration
+    # -----------------------------------------------------------------------------------------------------------------
 
     def filtration_and_sorting(self, sorting=0, filter_str=''):
         """
         Filtration of friends list
         :param sorting: 0 - no sorting, 1 - online only, 2 - online first, 3 - by name,
-        4 - online and by name, 5 - online first and by name, 6 kind
+        4 - online and by name, 5 - online first and by name
         :param filter_str: show contacts which name contains this substring
         """
         filter_str = filter_str.lower()
         current_contact = self.get_curr_contact()
 
-        for index, contact in enumerate(self._contacts):
+        if sorting > 5 or sorting < 0:
-            if not contact._kind:
-                set_contact_kind(contact)
-
-        if sorting > 6 or sorting < 0:
             sorting = 0
 
         if sorting in (1, 2, 4, 5):  # online first
@@ -235,30 +183,18 @@ class ContactsManager(ToxSave):
             part2 = sorted(part2, key=key_lambda)
             self._contacts = part1 + part2
         elif sorting == 0:
-            # AttributeError: 'NoneType' object has no attribute 'number'
-            for (i, contact) in enumerate(self._contacts):
-                if contact is None or not hasattr(contact, 'number'):
-                    LOG.error(f"Contact {i} is None or not hasattr 'number'")
-                    del self._contacts[i]
-                    continue
             contacts = sorted(self._contacts, key=lambda c: c.number)
             friends = filter(lambda c: type(c) is Friend, contacts)
             groups = filter(lambda c: type(c) is GroupChat, contacts)
             group_peers = filter(lambda c: type(c) is GroupPeerContact, contacts)
             self._contacts = list(friends) + list(groups) + list(group_peers)
-        elif sorting == 6:
-            self._contacts = sorted(self._contacts, key=lambda x: x._kind)
         else:
             self._contacts = sorted(self._contacts, key=lambda x: x.name.lower())
 
-
         # change item widgets
         for index, contact in enumerate(self._contacts):
             list_item = self._screen.friends_list.item(index)
             item_widget = self._screen.friends_list.itemWidget(list_item)
-            if not item_widget:
-                LOG_WARN("CM.filtration_and_sorting( item_widget is NULL")
-                continue
             contact.set_widget(item_widget)
 
         for index, friend in enumerate(self._contacts):
@@ -286,7 +222,9 @@ class ContactsManager(ToxSave):
         """
         self.filtration_and_sorting(self._sorting, self._filter_string)
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Contact getters
+    # -----------------------------------------------------------------------------------------------------------------
 
     def get_friend_by_number(self, number):
         return list(filter(lambda c: c.number == number and type(c) is Friend, self._contacts))[0]
@@ -297,15 +235,9 @@ class ContactsManager(ToxSave):
     def get_or_create_group_peer_contact(self, group_number, peer_id):
         group = self.get_group_by_number(group_number)
         peer = group.get_peer_by_id(peer_id)
-        if peer is None:
-            LOG.warn(f'get_or_create_group_peer_contact group_number={group_number} peer_id={peer_id} peer={peer}')
-            return None
-        LOG.debug(f'get_or_create_group_peer_contact group_number={group_number} peer_id={peer_id} peer={peer}')
         if not self.check_if_contact_exists(peer.public_key):
-            contact = self.add_group_peer(group, peer)
+            self.add_group_peer(group, peer)
-            # dunno
+
-            return contact
-        # me - later wrong kind of object?
         return self.get_contact_by_tox_id(peer.public_key)
 
     def check_if_contact_exists(self, tox_id):
@@ -323,7 +255,9 @@ class ContactsManager(ToxSave):
     def is_active_online(self):
         return self._active_contact + 1 and self.get_curr_contact().status is not None
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Work with friends (remove, block, set alias, get public key)
+    # -----------------------------------------------------------------------------------------------------------------
 
     def set_alias(self, num):
         """
@@ -365,10 +299,7 @@ class ContactsManager(ToxSave):
         """
         friend = self._contacts[num]
         self._cleanup_contact_data(friend)
-        try:
         self._tox.friend_delete(friend.number)
-        except Exception as e:
-            LOG.warn(f"'There was no friend with the given friend number {e}")
         self._delete_contact(num)
 
     def add_friend(self, tox_id):
@@ -383,8 +314,8 @@ class ContactsManager(ToxSave):
         """
         Block user with specified tox id (or public key) - delete from friends list and ignore friend requests
         """
-        tox_id = tox_id[:enums.TOX_PUBLIC_KEY_SIZE * 2]
+        tox_id = tox_id[:TOX_PUBLIC_KEY_SIZE * 2]
-        if tox_id == self._tox.self_get_address()[:enums.TOX_PUBLIC_KEY_SIZE * 2]:
+        if tox_id == self._tox.self_get_address[:TOX_PUBLIC_KEY_SIZE * 2]:
             return
         if tox_id not in self._settings['blocked']:
             self._settings['blocked'].append(tox_id)
@@ -408,25 +339,19 @@ class ContactsManager(ToxSave):
             self.add_friend(tox_id)
             self.save_profile()
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Groups support
+    # -----------------------------------------------------------------------------------------------------------------
 
     def get_group_chats(self):
         return list(filter(lambda c: type(c) is GroupChat, self._contacts))
 
     def add_group(self, group_number):
-        index = len(self._contacts)
         group = self._contact_provider.get_group_by_number(group_number)
-        if group is None:
+        index = len(self._contacts)
-            LOG.warn(f"CM.add_group: NULL group from group_number={group_number}")
-        elif type(group) == int and group < 0:
-            LOG.warn(f"CM.add_group: NO group from group={group} group_number={group_number}")
-        else:
-            LOG.info(f"CM.add_group: Adding group {group._name}")
         self._contacts.append(group)
-            LOG.info(f"contacts_manager.add_group: saving profile")
-            self._save_profile()
         group.reset_avatar(self._settings['identicons'])
-            LOG.info(f"contacts_manager.add_group: setting active")
+        self._save_profile()
         self.set_active(index)
         self.update_filtration()
 
@@ -436,21 +361,20 @@ class ContactsManager(ToxSave):
         num = self._contacts.index(group)
         self._delete_contact(num)
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Groups private messaging
+    # -----------------------------------------------------------------------------------------------------------------
 
     def add_group_peer(self, group, peer):
         contact = self._contact_provider.get_group_peer_by_id(group, peer.id)
         if self.check_if_contact_exists(contact.tox_id):
             return
-        contact._kind = 'grouppeer'
         self._contacts.append(contact)
         contact.reset_avatar(self._settings['identicons'])
         self._save_profile()
-        return contact
 
     def remove_group_peer_by_id(self, group, peer_id):
         peer = group.get_peer_by_id(peer_id)
-        if peer: # broken
         if not self.check_if_contact_exists(peer.public_key):
             return
         contact = self.get_contact_by_tox_id(peer.public_key)
@@ -458,7 +382,6 @@ class ContactsManager(ToxSave):
 
     def remove_group_peer(self, group_peer_contact):
         contact = self.get_contact_by_tox_id(group_peer_contact.tox_id)
-        if contact:
         self._cleanup_contact_data(contact)
         num = self._contacts.index(contact)
         self._delete_contact(num)
@@ -479,48 +402,38 @@ class ContactsManager(ToxSave):
 
         return suggested_names[0]
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Friend requests
+    # -----------------------------------------------------------------------------------------------------------------
 
-    def send_friend_request(self, sToxPkOrId, message):
+    def send_friend_request(self, tox_id, message):
         """
         Function tries to send request to contact with specified id
-        :param sToxPkOrId: id of new contact or tox dns 4 value
+        :param tox_id: id of new contact or tox dns 4 value
         :param message: additional message
         :return: True on success else error string
         """
-        retval = ''
         try:
             message = message or 'Hello! Add me to your contact list please'
-            if len(sToxPkOrId) == enums.TOX_PUBLIC_KEY_SIZE * 2:  # public key
+            if '@' in tox_id:  # value like groupbot@toxme.io
-                self.add_friend(sToxPkOrId)
+                tox_id = self._tox_dns.lookup(tox_id)
-                title = 'Friend added'
+                if tox_id is None:
-                text = 'Friend added without sending friend request'
+                    raise Exception('TOX DNS lookup failed')
-            else:
+            if len(tox_id) == TOX_PUBLIC_KEY_SIZE * 2:  # public key
-                num = self._tox.friend_add(sToxPkOrId, message.encode('utf-8'))
+                self.add_friend(tox_id)
-                if num < UINT32_MAX:
+                title = util_ui.tr('Friend added')
-                    tox_pk = sToxPkOrId[:enums.TOX_PUBLIC_KEY_SIZE * 2]
+                text = util_ui.tr('Friend added without sending friend request')
-                    self._add_friend(tox_pk)
-                    self.update_filtration()
-                    title = 'Friend added'
-                    text = 'Friend added by sending friend request'
-                    self.save_profile()
-                    retval = True
-                else:
-                    title = 'Friend failed'
-                    text = 'Friend failed sending friend request'
-                    retval = text
-
-        except Exception as ex:  # wrong data
-            title = 'Friend add exception'
-            text = 'Friend request exception with ' + str(ex)
-            self._log(text)
-            LOG.exception(text)
-            LOG.warn(f"DELETE {sToxPkOrId} ?")
-            retval = str(ex)
-        title = util_ui.tr(title)
-        text = util_ui.tr(text)
                 util_ui.message_box(text, title)
-        return retval
+            else:
+                self._tox.friend_add(tox_id, message.encode('utf-8'))
+                tox_id = tox_id[:TOX_PUBLIC_KEY_SIZE * 2]
+                self._add_friend(tox_id)
+                self.update_filtration()
+            self.save_profile()
+            return True
+        except Exception as ex:  # wrong data
+            util.log('Friend request failed with ' + str(ex))
+            return str(ex)
 
     def process_friend_request(self, tox_id, message):
         """
@@ -538,12 +451,14 @@ class ContactsManager(ToxSave):
                 data = self._tox.get_savedata()
                 self._profile_manager.save_profile(data)
         except Exception as ex:  # something is wrong
-            LOG.error('Accept friend request failed! ' + str(ex))
+            util.log('Accept friend request failed! ' + str(ex))
 
     def can_send_typing_notification(self):
         return self._settings['typing_notifications'] and not self.is_active_a_group_chat_peer()
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Contacts numbers update
+    # -----------------------------------------------------------------------------------------------------------------
 
     def update_friends_numbers(self):
         for friend in self._contact_provider.get_all_friends():
@@ -552,17 +467,9 @@ class ContactsManager(ToxSave):
 
     def update_groups_numbers(self):
         groups = self._contact_provider.get_all_groups()
-        LOG.info(f"update_groups_numbers len(groups)={len(groups)}")
-        # Thread 76 "ToxIterateThrea" received signal SIGSEGV, Segmentation fault.
         for i in range(len(groups)):
             chat_id = self._tox.group_get_chat_id(i)
-            if not chat_id:
-                LOG.warn(f"update_groups_numbers {i} chat_id")
-                continue
             group = self.get_contact_by_tox_id(chat_id)
-            if not group:
-                LOG.warn(f"update_groups_numbers {i} group")
-                continue
             group.number = i
         self.update_filtration()
 
@@ -571,22 +478,16 @@ class ContactsManager(ToxSave):
         for group in groups:
             group.remove_all_peers_except_self()
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Private methods
+    # -----------------------------------------------------------------------------------------------------------------
 
     def _load_contacts(self):
         self._load_friends()
         self._load_groups()
         if len(self._contacts):
             self.set_active(0)
-        # filter(lambda c: not c.has_avatar(), self._contacts)
+        for contact in filter(lambda c: not c.has_avatar(), self._contacts):
-        for (i, contact) in enumerate(self._contacts):
-            if contact is None:
-                LOG.warn(f"_load_contacts NULL contact {i}")
-                LOG.info(f"_load_contacts deleting NULL {self._contacts[i]}")
-                del self._contacts[i]
-                #? self.save_profile()
-                continue
-            if contact.has_avatar(): continue
             contact.reset_avatar(self._settings['identicons'])
         self.update_filtration()
 
@@ -596,7 +497,9 @@ class ContactsManager(ToxSave):
     def _load_groups(self):
         self._contacts.extend(self._contact_provider.get_all_groups())
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Current contact subscriptions
+    # -----------------------------------------------------------------------------------------------------------------
 
     def _subscribe_to_events(self, contact):
         contact.name_changed_event.add_callback(self._current_contact_name_changed)

toxygen/contacts/friend.py

@@ -14,7 +14,9 @@ class Friend(contact.Contact):
         self._receipts = 0
         self._typing_notification_handler = common.FriendTypingNotificationHandler(number)
 
+    # -----------------------------------------------------------------------------------------------------------------
     # File transfers support
+    # -----------------------------------------------------------------------------------------------------------------
 
     def insert_inline(self, before_message_id, inline):
         """
@@ -50,17 +52,23 @@ class Friend(contact.Contact):
         self._corr = list(filter(lambda m: not (type(m) is UnsentFileMessage and m.message_id == message_id),
                                  self._corr))
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Full status
+    # -----------------------------------------------------------------------------------------------------------------
 
     def get_full_status(self):
         return self._status_message
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Typing notifications
+    # -----------------------------------------------------------------------------------------------------------------
 
     def get_typing_notification_handler(self):
         return self._typing_notification_handler
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Context menu support
+    # -----------------------------------------------------------------------------------------------------------------
 
     def get_context_menu_generator(self):
         return FriendMenuGenerator(self)

toxygen/contacts/friend_factory.py

@@ -13,26 +13,28 @@ class FriendFactory(ToxSave):
 
     def create_friend_by_public_key(self, public_key):
         friend_number = self._tox.friend_by_public_key(public_key)
+
         return self.create_friend_by_number(friend_number)
 
     def create_friend_by_number(self, friend_number):
         aliases = self._settings['friends_aliases']
-        sToxPk = self._tox.friend_get_public_key(friend_number)
+        tox_id = self._tox.friend_get_public_key(friend_number)
-        assert sToxPk, sToxPk
         try:
-            alias = list(filter(lambda x: x[0] == sToxPk, aliases))[0][1]
+            alias = list(filter(lambda x: x[0] == tox_id, aliases))[0][1]
         except:
             alias = ''
         item = self._create_friend_item()
-        name = alias or self._tox.friend_get_name(friend_number) or sToxPk
+        name = alias or self._tox.friend_get_name(friend_number) or tox_id
         status_message = self._tox.friend_get_status_message(friend_number)
-        message_getter = self._db.messages_getter(sToxPk)
+        message_getter = self._db.messages_getter(tox_id)
-        friend = Friend(self._profile_manager, message_getter, friend_number, name, status_message, item, sToxPk)
+        friend = Friend(self._profile_manager, message_getter, friend_number, name, status_message, item, tox_id)
         friend.set_alias(alias)
 
         return friend
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Private methods
+    # -----------------------------------------------------------------------------------------------------------------
 
     def _create_friend_item(self):
         """

toxygen/contacts/group_chat.py

@@ -1,17 +1,11 @@
-# -*- mode: python; indent-tabs-mode: nil; py-indent-offset: 4; coding: utf-8 -*-
-
 from contacts import contact
 from contacts.contact_menu import GroupMenuGenerator
 import utils.util as util
 from groups.group_peer import GroupChatPeer
-from tox_wrapper import toxcore_enums_and_consts as constants
+from wrapper import toxcore_enums_and_consts as constants
 from common.tox_save import ToxSave
 from groups.group_ban import GroupBan
 
-global LOG
-import logging
-LOG = logging.getLogger(__name__)
-from av.calls import LOG_ERROR, LOG_WARN, LOG_INFO, LOG_DEBUG, LOG_TRACE
 
 class GroupChat(contact.Contact, ToxSave):
 
@@ -31,7 +25,9 @@ class GroupChat(contact.Contact, ToxSave):
     def get_context_menu_generator(self):
         return GroupMenuGenerator(self)
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Properties
+    # -----------------------------------------------------------------------------------------------------------------
 
     def get_is_private(self):
         return self._is_private
@@ -57,7 +53,9 @@ class GroupChat(contact.Contact, ToxSave):
 
     peers_limit = property(get_peers_limit, set_peers_limit)
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Peers methods
+    # -----------------------------------------------------------------------------------------------------------------
 
     def get_self_peer(self):
         return self._peers[0]
@@ -75,20 +73,12 @@ class GroupChat(contact.Contact, ToxSave):
         return self.get_self_role() == constants.TOX_GROUP_ROLE['FOUNDER']
 
     def add_peer(self, peer_id, is_current_user=False):
-        "called from callbacks"
-        if peer_id >  self._peers_limit:
-            LOG_WARN(f"add_peer id={peer_id} > {self._peers_limit}")
-            return
-
-        status_message = f"Private in {self.name}"
-        LOG_TRACE(f"GC.add_peer id={peer_id} status_message={status_message}")
         peer = GroupChatPeer(peer_id,
                              self._tox.group_peer_get_name(self._number, peer_id),
                              self._tox.group_peer_get_status(self._number, peer_id),
                              self._tox.group_peer_get_role(self._number, peer_id),
                              self._tox.group_peer_get_public_key(self._number, peer_id),
-                             is_current_user,
+                             is_current_user)
-                             status_message=status_message)
         self._peers.append(peer)
 
     def remove_peer(self, peer_id):
@@ -96,40 +86,25 @@ class GroupChat(contact.Contact, ToxSave):
             self.remove_all_peers_except_self()
         else:
             peer = self.get_peer_by_id(peer_id)
-            if peer: # broken
             self._peers.remove(peer)
-            else:
-                LOG_WARN(f"remove_peer empty peers for {peer_id}")
 
     def get_peer_by_id(self, peer_id):
         peers = list(filter(lambda p: p.id == peer_id, self._peers))
-        if peers:
+
         return peers[0]
-        else:
-            LOG_WARN(f"get_peer_by_id empty peers for {peer_id}")
-            return None
 
     def get_peer_by_public_key(self, public_key):
         peers = list(filter(lambda p: p.public_key == public_key, self._peers))
-        # DEBUGc: group_moderation #0 mod_id=4294967295 event_type=3
+
-        # WARN_: get_peer_by_id empty peers for 4294967295
-        if peers:
         return peers[0]
-        else:
-            LOG_WARN(f"get_peer_by_public_key empty peers for {public_key}")
-            return None
 
     def remove_all_peers_except_self(self):
         self._peers = self._peers[:1]
 
     def get_peers_names(self):
         peers_names = map(lambda p: p.name, self._peers)
-        if peers_names: # broken
+
         return list(peers_names)
-        else:
-            LOG_WARN(f"get_peers_names empty peers")
-            #? broken
-            return []
 
     def get_peers(self):
         return self._peers[:]
@@ -137,20 +112,21 @@ class GroupChat(contact.Contact, ToxSave):
     peers = property(get_peers)
 
     def get_bans(self):
-        return []
+        ban_ids = self._tox.group_ban_get_list(self._number)
-#        ban_ids = self._tox.group_ban_get_list(self._number)
+        bans = []
-#        bans = []
+        for ban_id in ban_ids:
-#        for ban_id in ban_ids:
+            ban = GroupBan(ban_id,
-#            ban = GroupBan(ban_id,
+                           self._tox.group_ban_get_target(self._number, ban_id),
-#                           self._tox.group_ban_get_target(self._number, ban_id),
+                           self._tox.group_ban_get_time_set(self._number, ban_id))
-#                           self._tox.group_ban_get_time_set(self._number, ban_id))
+            bans.append(ban)
-#            bans.append(ban)
+
-#
+        return bans
-#        return bans
+
-#
     bans = property(get_bans)
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Private methods
+    # -----------------------------------------------------------------------------------------------------------------
 
     @staticmethod
     def _get_default_avatar_path():

toxygen/contacts/group_factory.py

@@ -1,12 +1,7 @@
-# -*- mode: python; indent-tabs-mode: nil; py-indent-offset: 4; coding: utf-8 -*-
-
 from contacts.group_chat import GroupChat
 from common.tox_save import ToxSave
-import tox_wrapper.toxcore_enums_and_consts as constants
+import wrapper.toxcore_enums_and_consts as constants
 
-global LOG
-import logging
-LOG = logging.getLogger(__name__)
 
 class GroupFactory(ToxSave):
 
@@ -17,15 +12,12 @@ class GroupFactory(ToxSave):
         self._db = db
         self._items_factory = items_factory
 
-    def create_group_by_chat_id(self, chat_id):
-        return self.create_group_by_public_key(chat_id)
-
     def create_group_by_public_key(self, public_key):
         group_number = self._get_group_number_by_chat_id(public_key)
+
         return self.create_group_by_number(group_number)
 
     def create_group_by_number(self, group_number):
-        LOG.info(f"create_group_by_number {group_number}")
         aliases = self._settings['friends_aliases']
         tox_id = self._tox.group_get_chat_id(group_number)
         try:
@@ -43,7 +35,9 @@ class GroupFactory(ToxSave):
 
         return group
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Private methods
+    # -----------------------------------------------------------------------------------------------------------------
 
     def _create_group_item(self):
         """
@@ -53,7 +47,7 @@ class GroupFactory(ToxSave):
         return self._items_factory.create_contact_item()
 
     def _get_group_number_by_chat_id(self, chat_id):
-        for i in range(self._tox.group_get_number_groups()+100):
+        for i in range(self._tox.group_get_number_groups()):
             if self._tox.group_get_chat_id(i) == chat_id:
                 return i
         return -1

toxygen/contacts/group_peer_contact.py

@@ -4,9 +4,8 @@ from contacts.contact_menu import GroupPeerMenuGenerator
 
 class GroupPeerContact(contacts.contact.Contact):
 
-    def __init__(self, profile_manager, message_getter, peer_number, name, widget, tox_id, group_pk, status_message=None):
+    def __init__(self, profile_manager, message_getter, peer_number, name, widget, tox_id, group_pk):
-        if status_message is None: status_message=str()
+        super().__init__(profile_manager, message_getter, peer_number, name, str(), widget, tox_id)
-        super().__init__(profile_manager, message_getter, peer_number, name, status_message, widget, tox_id)
         self._group_pk = group_pk
 
     def get_group_pk(self):

toxygen/contacts/group_peer_factory.py

@@ -14,10 +14,7 @@ class GroupPeerFactory(ToxSave):
         item = self._create_group_peer_item()
         message_getter = self._db.messages_getter(peer.public_key)
         group_peer_contact = GroupPeerContact(self._profile_manager, message_getter, peer.id, peer.name,
-                                              item,
+                                              item, peer.public_key, group.tox_id)
-                                              peer.public_key,
-                                              group.tox_id,
-                                              status_message=peer.status_message)
         group_peer_contact.status = peer.status
 
         return group_peer_contact

toxygen/contacts/profile.py

@@ -1,27 +1,19 @@
-# -*- mode: python; indent-tabs-mode: nil; py-indent-offset: 4; coding: utf-8 -*-
 from contacts import basecontact
 import random
 import threading
 import common.tox_save as tox_save
 from middleware.threads import invoke_in_main_thread
 
-iUMAXINT = 4294967295
-iRECONNECT = 50
-
-global LOG
-import logging
-LOG = logging.getLogger('app.'+__name__)
 
 class Profile(basecontact.BaseContact, tox_save.ToxSave):
     """
     Profile of current toxygen user.
     """
-    def __init__(self, profile_manager, tox, screen, contacts_provider, reset_action, app=None):
+    def __init__(self, profile_manager, tox, screen, contacts_provider, reset_action):
         """
         :param tox: tox instance
         :param screen: ref to main screen
         """
-        assert tox
         basecontact.BaseContact.__init__(self,
                                          profile_manager,
                                          tox.self_get_name(),
@@ -35,73 +27,61 @@ class Profile(basecontact.BaseContact, tox_save.ToxSave):
         self._reset_action = reset_action
         self._waiting_for_reconnection = False
         self._timer = None
-        self._app = app
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Edit current user's data
+    # -----------------------------------------------------------------------------------------------------------------
 
-    def change_status(self) -> None:
+    def change_status(self):
         """
         Changes status of user (online, away, busy)
         """
         if self._status is not None:
             self.set_status((self._status + 1) % 3)
 
-    def set_status(self, status) -> None:
+    def set_status(self, status):
         super().set_status(status)
         if status is not None:
             self._tox.self_set_status(status)
         elif not self._waiting_for_reconnection:
             self._waiting_for_reconnection = True
-            self._timer = threading.Timer(iRECONNECT, self._reconnect)
+            self._timer = threading.Timer(50, self._reconnect)
             self._timer.start()
 
-    def set_name(self, value) -> None:
+    def set_name(self, value):
         if self.name == value:
             return
         super().set_name(value)
         self._tox.self_set_name(self._name)
 
-    def set_status_message(self, value) -> None:
+    def set_status_message(self, value):
         super().set_status_message(value)
         self._tox.self_set_status_message(self._status_message)
 
     def set_new_nospam(self):
         """Sets new nospam part of tox id"""
-        self._tox.self_set_nospam(random.randint(0, iUMAXINT))  # no spam - uint32
+        self._tox.self_set_nospam(random.randint(0, 4294967295))  # no spam - uint32
         self._tox_id = self._tox.self_get_address()
-        self._sToxId = self._tox.self_get_address()
-        return self._sToxId
 
+        return self._tox_id
+
+    # -----------------------------------------------------------------------------------------------------------------
     # Reset
+    # -----------------------------------------------------------------------------------------------------------------
 
-    def restart(self) -> None:
+    def restart(self):
         """
         Recreate tox instance
         """
         self.status = None
         invoke_in_main_thread(self._reset_action)
 
-    def _reconnect(self) -> None:
+    def _reconnect(self):
         self._waiting_for_reconnection = False
-        if self._app and self._app.bAppExiting:
-            # dont do anything after the app has been shipped
-            # there's a segv that results
-            return
         contacts = self._contacts_provider.get_all_friends()
         all_friends_offline = all(list(map(lambda x: x.status is None, contacts)))
         if self.status is None or (all_friends_offline and len(contacts)):
             self._waiting_for_reconnection = True
             self.restart()
-            self._timer = threading.Timer(iRECONNECT, self._reconnect)
+            self._timer = threading.Timer(50, self._reconnect)
             self._timer.start()
-
-# Current thread 0x00007901a13ccb80 (most recent call first):
-#   File "/usr/local/lib/python3.11/site-packages/tox_wrapper/tox.py", line 826 in self_get_friend_list_size
-#   File "/usr/local/lib/python3.11/site-packages/tox_wrapper/tox.py", line 838 in self_get_friend_list
-#   File "/mnt/o/var/local/src/toxygen/toxygen/contacts/contact_provider.py", line 45 in get_all_friends
-#   File "/mnt/o/var/local/src/toxygen/toxygen/contacts/profile.py", line 90 in _reconnect
-#   File "/usr/lib/python3.11/threading.py", line 1401 in run
-#   File "/usr/lib/python3.11/threading.py", line 1045 in _bootstrap_inner
-#   File "/usr/lib/python3.11/threading.py", line 1002 in _bootstrap
-#
-

toxygen/file_transfers/file_transfers.py

@@ -1,13 +1,11 @@
-import os
+from wrapper.toxcore_enums_and_consts import TOX_FILE_KIND, TOX_FILE_CONTROL
-from os import chdir, remove, rename
 from os.path import basename, getsize, exists, dirname
+from os import remove, rename, chdir
 from time import time
-
+from wrapper.tox import Tox
 from common.event import Event
 from middleware.threads import invoke_in_main_thread
-from tox_wrapper.tox import Tox
+
-from tox_wrapper.toxcore_enums_and_consts import TOX_FILE_KIND, TOX_FILE_CONTROL
-from middleware.callbacks import LOG_ERROR, LOG_WARN, LOG_INFO, LOG_DEBUG, LOG_TRACE
 
 FILE_TRANSFER_STATE = {
     'RUNNING': 0,
@@ -80,9 +78,6 @@ class FileTransfer:
 
     def get_file_id(self):
         return self._file_id
-    # WTF
-    def get_file_id(self):
-        return self._tox.file_get_file_id(self._friend_number, self._file_number)
 
     file_id = property(get_file_id)
 
@@ -117,6 +112,9 @@ class FileTransfer:
         if self._tox.file_control(self._friend_number, self._file_number, control):
             self.set_state(control)
 
+    def get_file_id(self):
+        return self._tox.file_get_file_id(self._friend_number, self._file_number)
+
     def _signal(self):
         percentage = self._done / self._size if self._size else 0
         if self._creation_time is None or not percentage:
@@ -128,7 +126,9 @@ class FileTransfer:
     def _finished(self):
         self._finished_event(self._friend_number, self._file_number)
 
+# -----------------------------------------------------------------------------------------------------------------
 # Send file
+# -----------------------------------------------------------------------------------------------------------------
 
 
 class SendTransfer(FileTransfer):
@@ -174,14 +174,11 @@ class SendAvatar(SendTransfer):
     """
 
     def __init__(self, path, tox, friend_number):
-        LOG_DEBUG(f"SendAvatar path={path} friend_number={friend_number}")
+        if path is None:
-        if path is None or not os.path.exists(path):
             avatar_hash = None
         else:
             with open(path, 'rb') as fl:
-                data=fl.read()
+                avatar_hash = Tox.hash(fl.read())
-            LOG_DEBUG(f"SendAvatar data={data} type={type(data)}")
-            avatar_hash = tox.hash(data, None)
         super().__init__(path, tox, friend_number, TOX_FILE_KIND['AVATAR'], avatar_hash)
 
 
@@ -223,10 +220,12 @@ class SendFromFileBuffer(SendTransfer):
     def send_chunk(self, position, size):
         super().send_chunk(position, size)
         if not size:
-            os.chdir(dirname(self._path))
+            chdir(dirname(self._path))
-            os.remove(self._path)
+            remove(self._path)
 
+# -----------------------------------------------------------------------------------------------------------------
 # Receive file
+# -----------------------------------------------------------------------------------------------------------------
 
 
 class ReceiveTransfer(FileTransfer):
@@ -316,6 +315,7 @@ class ReceiveAvatar(ReceiveTransfer):
     Get friend's avatar. Doesn't need file transfer item
     """
     MAX_AVATAR_SIZE = 512 * 1024
+
     def __init__(self, path, tox, friend_number, size, file_number):
         full_path = path + '.tmp'
         super().__init__(full_path, tox, friend_number, size, file_number)

toxygen/file_transfers/file_transfers_handler.py

@@ -1,22 +1,11 @@
-# -*- mode: python; indent-tabs-mode: nil; py-indent-offset: 4; coding: utf-8 -*-
-from copy import deepcopy
-
 from messenger.messages import *
-from file_transfers.file_transfers import SendAvatar, is_inline
 from ui.contact_items import *
 import utils.util as util
 from common.tox_save import ToxSave
 
-from middleware.callbacks import LOG_ERROR, LOG_WARN, LOG_INFO, LOG_DEBUG, LOG_TRACE
-
-# LOG=util.log
-global LOG
-import logging
-LOG = logging.getLogger('app.'+__name__)
-log = lambda x: LOG.info(x)
 
 class FileTransfersHandler(ToxSave):
-    lBlockAvatars = []
+
     def __init__(self, tox, settings, contact_provider, file_transfers_message_service, profile):
         super().__init__(tox)
         self._settings = settings
@@ -30,16 +19,16 @@ class FileTransfersHandler(ToxSave):
         # key = (friend number, file number), value - message id
 
         profile.avatar_changed_event.add_callback(self._send_avatar_to_contacts)
-        self. lBlockAvatars = []
         
     def stop(self):
         self._settings['paused_file_transfers'] = self._paused_file_transfers if self._settings['resend_files'] else {}
         self._settings.save()
 
+    # -----------------------------------------------------------------------------------------------------------------
     # File transfers support
+    # -----------------------------------------------------------------------------------------------------------------
 
     def incoming_file_transfer(self, friend_number, file_number, size, file_name):
-        # main thread
         """
         New transfer
         :param friend_number: number of friend who sent file
@@ -48,15 +37,11 @@ class FileTransfersHandler(ToxSave):
         :param file_name: file name without path
         """
         friend = self._get_friend_by_number(friend_number)
-        if friend is None:
-            LOG.info(f'incoming_file_handler Friend NULL friend_number={friend_number}')
-            return None
         auto = self._settings['allow_auto_accept'] and friend.tox_id in self._settings['auto_accept_from_friends']
-        inline = False # ?is_inline(file_name) and self._settings['allow_inline']
+        inline = is_inline(file_name) and self._settings['allow_inline']
         file_id = self._tox.file_get_file_id(friend_number, file_number)
         accepted = True
         if file_id in self._paused_file_transfers:
-            LOG_INFO(f'incoming_file_handler paused friend_number={friend_number}')
             (path, ft_friend_number, is_incoming, start_position) = self._paused_file_transfers[file_id]
             pos = start_position if os.path.exists(path) else 0
             if pos >= size:
@@ -67,22 +52,16 @@ class FileTransfersHandler(ToxSave):
                 friend, accepted, size, file_name, file_number)
             self.accept_transfer(path, friend_number, file_number, size, False, pos)
         elif inline and size < 1024 * 1024:
-            LOG_INFO(f'incoming_file_handler small friend_number={friend_number}')
             self._file_transfers_message_service.add_incoming_transfer_message(
                 friend, accepted, size, file_name, file_number)
             self.accept_transfer('', friend_number, file_number, size, True)
         elif auto:
-            # accepted is really started
-            LOG_INFO(f'incoming_file_handler auto friend_number={friend_number}')
             path = self._settings['auto_accept_path'] or util.curr_directory()
             self._file_transfers_message_service.add_incoming_transfer_message(
                 friend, accepted, size, file_name, file_number)
             self.accept_transfer(path + '/' + file_name, friend_number, file_number, size)
         else:
-            LOG_INFO(f'incoming_file_handler reject friend_number={friend_number}')
             accepted = False
-            # FixME: need GUI ask
-            # accepted is really started
             self._file_transfers_message_service.add_incoming_transfer_message(
                 friend, accepted, size, file_name, file_number)
 
@@ -106,9 +85,7 @@ class FileTransfersHandler(ToxSave):
             self._tox.file_control(friend_number, file_number, TOX_FILE_CONTROL['CANCEL'])
 
     def cancel_not_started_transfer(self, friend_number, message_id):
-        friend = self._get_friend_by_number(friend_number)
+        self._get_friend_by_number(friend_number).delete_one_unsent_file(message_id)
-        if friend is None: return None
-        friend.delete_one_unsent_file(message_id)
 
     def pause_transfer(self, friend_number, file_number, by_friend=False):
         """
@@ -138,7 +115,6 @@ class FileTransfersHandler(ToxSave):
         """
         path = self._generate_valid_path(path, from_position)
         friend = self._get_friend_by_number(friend_number)
-        if friend is None: return None
         if not inline:
             rt = ReceiveTransfer(path, self._tox, friend_number, size, file_number, from_position)
         else:
@@ -169,15 +145,11 @@ class FileTransfersHandler(ToxSave):
 
     def send_inline(self, data, file_name, friend_number, is_resend=False):
         friend = self._get_friend_by_number(friend_number)
-        if friend is None:
-            LOG_WARN("fsend_inline Error friend is None file_name: {file_name}")
-            return
         if friend.status is None and not is_resend:
             self._file_transfers_message_service.add_unsent_file_message(friend, file_name, data)
             return
         elif friend.status is None and is_resend:
-            LOG_WARN("fsend_inline Error friend.status is None file_name: {file_name}")
+            raise RuntimeError()
-            return
         st = SendFromBuffer(self._tox, friend.number, data, file_name)
         self._send_file_add_set_handlers(st, friend, file_name, True)
 
@@ -190,12 +162,11 @@ class FileTransfersHandler(ToxSave):
         :param file_id: file id of transfer
         """
         friend = self._get_friend_by_number(friend_number)
-        if friend is None: return None
         if friend.status is None and not is_resend:
             self._file_transfers_message_service.add_unsent_file_message(friend, path, None)
             return
         elif friend.status is None and is_resend:
-            LOG_WARN('Error in sending')
+            print('Error in sending')
             return
         st = SendTransfer(path, self._tox, friend_number, TOX_FILE_KIND['DATA'], file_id)
         file_name = os.path.basename(path)
@@ -215,27 +186,23 @@ class FileTransfersHandler(ToxSave):
 
     def transfer_finished(self, friend_number, file_number):
         transfer = self._file_transfers[(friend_number, file_number)]
-        friend = self._get_friend_by_number(friend_number)
-        if friend is None: return None
         t = type(transfer)
         if t is ReceiveAvatar:
-            friend.load_avatar()
+            self._get_friend_by_number(friend_number).load_avatar()
         elif t is ReceiveToBuffer or (t is SendFromBuffer and self._settings['allow_inline']):  # inline image
-            LOG.debug('inline')
+            print('inline')
             inline = InlineImageMessage(transfer.data)
             message_id = self._insert_inline_before[(friend_number, file_number)]
             del self._insert_inline_before[(friend_number, file_number)]
-            if friend is None: return None
+            index = self._get_friend_by_number(friend_number).insert_inline(message_id, inline)
-            index = friend.insert_inline(message_id, inline)
             self._file_transfers_message_service.add_inline_message(transfer, index)
         del self._file_transfers[(friend_number, file_number)]
 
     def send_files(self, friend_number):
-        try:
         friend = self._get_friend_by_number(friend_number)
-            if friend is None: return None
         friend.remove_invalid_unsent_files()
         files = friend.get_unsent_files()
+        try:
             for fl in files:
                 data, path = fl.data, fl.path
                 if data is not None:
@@ -244,7 +211,6 @@ class FileTransfersHandler(ToxSave):
                     self.send_file(path, friend_number, True)
             friend.clear_unsent_files()
             for key in self._paused_file_transfers.keys():
-                # RuntimeError: dictionary changed size during iteration
                 (path, ft_friend_number, is_incoming, start_position) = self._paused_file_transfers[key]
                 if not os.path.exists(path):
                     del self._paused_file_transfers[key]
@@ -252,61 +218,30 @@ class FileTransfersHandler(ToxSave):
                     self.send_file(path, friend_number, True, key)
                     del self._paused_file_transfers[key]
         except Exception as ex:
-            LOG_ERROR('send_files EXCEPTION in file sending: ' + str(ex))
+            print('Exception in file sending: ' + str(ex))
 
     def friend_exit(self, friend_number):
-        # RuntimeError: dictionary changed size during iteration
+        for friend_num, file_num in self._file_transfers.keys():
-        lMayChangeDynamically = self._file_transfers.copy()
-        for friend_num, file_num in lMayChangeDynamically:
             if friend_num != friend_number:
                 continue
-            if (friend_num, file_num) not in self._file_transfers:
-                continue
             ft = self._file_transfers[(friend_num, file_num)]
             if type(ft) is SendTransfer:
-                try:
+                self._paused_file_transfers[ft.file_id] = [ft.path, friend_num, False, -1]
-                    file_id = ft.file_id
-                except Exception as e:
-                    LOG_WARN("friend_exit SendTransfer Error getting file_id: {e}")
-                    # drop through
-                else:
-                    self._paused_file_transfers[file_id] = [ft.path, friend_num, False, -1]
             elif type(ft) is ReceiveTransfer and ft.state != FILE_TRANSFER_STATE['INCOMING_NOT_STARTED']:
-                try:
+                self._paused_file_transfers[ft.file_id] = [ft.path, friend_num, True, ft.total_size()]
-                    file_id = ft.file_id
-                except Exception as e:
-                    LOG_WARN("friend_exit ReceiveTransfer Error getting file_id: {e}")
-                    # drop through
-                else:
-                    self._paused_file_transfers[file_id] = [ft.path, friend_num, True, ft.total_size()]
             self.cancel_transfer(friend_num, file_num, True)
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Avatars support
+    # -----------------------------------------------------------------------------------------------------------------
 
     def send_avatar(self, friend_number, avatar_path=None):
         """
         :param friend_number: number of friend who should get new avatar
         :param avatar_path: path to avatar or None if reset
         """
-        return
-        if (avatar_path, friend_number,) in self.lBlockAvatars:
-            return
-        if friend_number is None:
-            LOG_WARN(f"send_avatar friend_number NULL {friend_number}")
-            return
-        if avatar_path and type(avatar_path) != str:
-            LOG_WARN(f"send_avatar avatar_path type {type(avatar_path)}")
-            return
-        LOG_INFO(f"send_avatar avatar_path={avatar_path} friend_number={friend_number}")
-        try:
-            #  self NOT missing - who's self?
         sa = SendAvatar(avatar_path, self._tox, friend_number)
-            LOG_INFO(f"send_avatar avatar_path={avatar_path} sa={sa}")
         self._file_transfers[(friend_number, sa.file_number)] = sa
-        except Exception as e:
-            # ArgumentError('This client is currently not connected to the friend.')
-            LOG_WARN(f"send_avatar EXCEPTION {e}")
-            self.lBlockAvatars.append( (avatar_path, friend_number,) )
 
     def incoming_avatar(self, friend_number, file_number, size):
         """
@@ -316,7 +251,6 @@ class FileTransfersHandler(ToxSave):
         :param size: size of avatar or 0 (default avatar)
         """
         friend = self._get_friend_by_number(friend_number)
-        if friend is None: return None
         ra = ReceiveAvatar(friend.get_contact_avatar_path(), self._tox, friend_number, size, file_number)
         if ra.state != FILE_TRANSFER_STATE['CANCELLED']:
             self._file_transfers[(friend_number, file_number)] = ra
@@ -325,16 +259,16 @@ class FileTransfersHandler(ToxSave):
             friend.reset_avatar(self._settings['identicons'])
 
     def _send_avatar_to_contacts(self, _):
-        # from a callback
         friends = self._get_all_friends()
         for friend in filter(self._is_friend_online, friends):
             self.send_avatar(friend.number)
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Private methods
+    # -----------------------------------------------------------------------------------------------------------------
 
     def _is_friend_online(self, friend_number):
         friend = self._get_friend_by_number(friend_number)
-        if friend is None: return None
 
         return friend.status is not None
 

toxygen/file_transfers/file_transfers_messages_service.py

@@ -2,11 +2,6 @@ from messenger.messenger import *
 import utils.util as util
 from file_transfers.file_transfers import *
 
-global LOG
-import logging
-LOG = logging.getLogger('app.'+__name__)
-
-from av.calls import LOG_ERROR, LOG_WARN, LOG_INFO, LOG_DEBUG, LOG_TRACE
 
 class FileTransfersMessagesService:
 
@@ -17,9 +12,7 @@ class FileTransfersMessagesService:
         self._messages = main_screen.messages
 
     def add_incoming_transfer_message(self, friend, accepted, size, file_name, file_number):
-        assert friend
         author = MessageAuthor(friend.name, MESSAGE_AUTHOR['FRIEND'])
-        # accepted is really started
         status = FILE_TRANSFER_STATE['RUNNING'] if accepted else FILE_TRANSFER_STATE['INCOMING_NOT_STARTED']
         tm = TransferMessage(author, util.get_unix_time(), status, size, file_name, friend.number, file_number)
 
@@ -34,7 +27,6 @@ class FileTransfersMessagesService:
         return tm
 
     def add_outgoing_transfer_message(self, friend, size, file_name, file_number):
-        assert friend
         author = MessageAuthor(self._profile.name, MESSAGE_AUTHOR['ME'])
         status = FILE_TRANSFER_STATE['OUTGOING_NOT_STARTED']
         tm = TransferMessage(author, util.get_unix_time(), status, size, file_name, friend.number, file_number)
@@ -48,19 +40,13 @@ class FileTransfersMessagesService:
         return tm
 
     def add_inline_message(self, transfer, index):
-        """callback"""
         if not self._is_friend_active(transfer.friend_number):
             return
-        if transfer is None or not hasattr(transfer, 'data') or \
-           not transfer.data:
-            LOG_ERROR(f"add_inline_message empty data")
-            return
         count = self._messages.count()
         if count + index + 1 >= 0:
             self._create_inline_item(transfer.data, count + index + 1)
 
     def add_unsent_file_message(self, friend, file_path, data):
-        assert friend
         author = MessageAuthor(self._profile.name, MESSAGE_AUTHOR['ME'])
         size = os.path.getsize(file_path) if data is None else len(data)
         tm = UnsentFileMessage(file_path, data, util.get_unix_time(), author, size, friend.number)
@@ -72,7 +58,9 @@ class FileTransfersMessagesService:
 
         return tm
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Private methods
+    # -----------------------------------------------------------------------------------------------------------------
 
     def _is_friend_active(self, friend_number):
         if not self._contacts_manager.is_active_a_friend():

toxygen/groups/group_peer.py

@@ -5,18 +5,18 @@ class GroupChatPeer:
     Represents peer in group chat.
     """
 
-    def __init__(self, peer_id, name, status, role, public_key, is_current_user=False, is_muted=False, status_message=None):
+    def __init__(self, peer_id, name, status, role, public_key, is_current_user=False, is_muted=False):
         self._peer_id = peer_id
         self._name = name
         self._status = status
-        self._status_message = status_message
         self._role = role
         self._public_key = public_key
         self._is_current_user = is_current_user
         self._is_muted = is_muted
-        self._kind = 'grouppeer'
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Readonly properties
+    # -----------------------------------------------------------------------------------------------------------------
 
     def get_id(self):
         return self._peer_id
@@ -33,12 +33,9 @@ class GroupChatPeer:
 
     is_current_user = property(get_is_current_user)
 
-    def get_status_message(self):
+    # -----------------------------------------------------------------------------------------------------------------
-        return self._status_message
-
-    status_message = property(get_status_message)
-
     # Read-write properties
+    # -----------------------------------------------------------------------------------------------------------------
 
     def get_name(self):
         return self._name

toxygen/groups/groups_service.py

@@ -1,16 +1,9 @@
-# -*- mode: python; indent-tabs-mode: nil; py-indent-offset: 4; coding: utf-8 -*-
-
 import common.tox_save as tox_save
 import utils.ui as util_ui
 from groups.peers_list import PeersListGenerator
 from groups.group_invite import GroupInvite
-import tox_wrapper.toxcore_enums_and_consts as constants
+import wrapper.toxcore_enums_and_consts as constants
-from tox_wrapper.toxcore_enums_and_consts import *
-from tox_wrapper.tox import UINT32_MAX
 
-global LOG
-import logging
-LOG = logging.getLogger('app.'+'gs')
 
 class GroupsService(tox_save.ToxSave):
 
@@ -23,22 +16,18 @@ class GroupsService(tox_save.ToxSave):
         self._widgets_factory_provider = widgets_factory_provider
         self._group_invites = []
         self._screen = None
-        # maybe just use self
-        self._tox = tox
 
     def set_tox(self, tox):
         super().set_tox(tox)
         for group in self._get_all_groups():
             group.set_tox(tox)
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Groups creation
+    # -----------------------------------------------------------------------------------------------------------------
 
     def create_new_gc(self, name, privacy_state, nick, status):
-        try:
         group_number = self._tox.group_new(privacy_state, name, nick, status)
-        except Exception as e:
-            LOG.error(f"create_new_gc {e}")
-            return
         if group_number == -1:
             return
 
@@ -48,34 +37,14 @@ class GroupsService(tox_save.ToxSave):
         self._contacts_manager.update_filtration()
 
     def join_gc_by_id(self, chat_id, password, nick, status):
-        try:
         group_number = self._tox.group_join(chat_id, password, nick, status)
-            assert type(group_number) == int, group_number
-            assert group_number < UINT32_MAX, group_number
-        except Exception as e:
-            # gui
-            title = f"join_gc_by_id {chat_id}"
-            util_ui.message_box(title +'\n' +str(e), title)
-            LOG.error(f"_join_gc_via_id {e}")
-            return
-        LOG.debug(f"_join_gc_via_id {group_number}")
         self._add_new_group_by_number(group_number)
-        group = self._get_group_by_number(group_number)
-        try:
-            assert group and hasattr(group, 'status')
-        except Exception as e:
-            # gui
-            title = f"join_gc_by_id {chat_id}"
-            util_ui.message_box(title +'\n' +str(e), title)
-            LOG.error(f"_join_gc_via_id {e}")
-            return
-        group.status = constants.TOX_USER_STATUS['NONE']
-        self._contacts_manager.update_filtration()
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Groups reconnect and leaving
+    # -----------------------------------------------------------------------------------------------------------------
 
     def leave_group(self, group_number):
-        if type(group_number) == int:
         self._tox.group_leave(group_number)
         self._contacts_manager.delete_group(group_number)
 
@@ -91,25 +60,15 @@ class GroupsService(tox_save.ToxSave):
         group.status = constants.TOX_USER_STATUS['NONE']
         self._clear_peers_list(group)
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Group invites
+    # -----------------------------------------------------------------------------------------------------------------
 
     def invite_friend(self, friend_number, group_number):
-        if self._tox.friend_get_connection_status(friend_number) == TOX_CONNECTION['NONE']:
-            title = f"Error in group_invite_friend {friend_number}"
-            e = f"Friend not connected friend_number={friend_number}"
-            util_ui.message_box(title +'\n' +str(e), title)
-            return
-
-        try:
         self._tox.group_invite_friend(group_number, friend_number)
-        except Exception as e:
-            title = f"Error in group_invite_friend {group_number} {friend_number}"
-            util_ui.message_box(title +'\n' +str(e), title)
 
     def process_group_invite(self, friend_number, group_name, invite_data):
         friend = self._get_friend_by_number(friend_number)
-        # binary  {invite_data}
-        LOG.debug(f"process_group_invite {friend_number} {group_name}")
         invite = GroupInvite(friend.tox_id, group_name, invite_data)
         self._group_invites.append(invite)
         self._update_invites_button_state()
@@ -117,7 +76,6 @@ class GroupsService(tox_save.ToxSave):
     def accept_group_invite(self, invite, name, status, password):
         pk = invite.friend_public_key
         friend = self._get_friend_by_public_key(pk)
-        LOG.debug(f"accept_group_invite {name}")
         self._join_gc_via_invite(invite.invite_data, friend.number, name, status, password)
         self._delete_group_invite(invite)
         self._update_invites_button_state()
@@ -136,7 +94,9 @@ class GroupsService(tox_save.ToxSave):
 
     group_invites_count = property(get_group_invites_count)
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Group info methods
+    # -----------------------------------------------------------------------------------------------------------------
 
     def update_group_info(self, group):
         group.name = self._tox.group_get_name(group.number)
@@ -182,7 +142,9 @@ class GroupsService(tox_save.ToxSave):
         self._tox.group_founder_set_privacy_state(group.number, privacy_state)
         group.is_private = is_private
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Peers list
+    # -----------------------------------------------------------------------------------------------------------------
 
     def generate_peers_list(self):
         if not self._contacts_manager.is_active_a_group():
@@ -200,7 +162,9 @@ class GroupsService(tox_save.ToxSave):
             self._screen = widgets_factory.create_self_peer_screen_window(group)
         self._screen.show()
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Peers actions
+    # -----------------------------------------------------------------------------------------------------------------
 
     def set_new_peer_role(self, group, peer, role):
         self._tox.group_mod_set_role(group.number, peer.id, role)
@@ -219,10 +183,11 @@ class GroupsService(tox_save.ToxSave):
         self_peer.status = status
         self.generate_peers_list()
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Bans support
+    # -----------------------------------------------------------------------------------------------------------------
 
     def show_bans_list(self, group):
-        return
         widgets_factory = self._get_widgets_factory()
         self._screen = widgets_factory.create_groups_bans_screen(group)
         self._screen.show()
@@ -236,10 +201,11 @@ class GroupsService(tox_save.ToxSave):
     def cancel_ban(self, group_number, ban_id):
         self._tox.group_mod_remove_ban(group_number, ban_id)
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Private methods
+    # -----------------------------------------------------------------------------------------------------------------
 
     def _add_new_group_by_number(self, group_number):
-        LOG.debug(f"_add_new_group_by_number group_number={group_number}")
         self._contacts_manager.add_group(group_number)
 
     def _get_group_by_number(self, group_number):
@@ -265,24 +231,9 @@ class GroupsService(tox_save.ToxSave):
         if invite in self._group_invites:
             self._group_invites.remove(invite)
 
-    # status should be dropped
+    def _join_gc_via_invite(self, invite_data, friend_number, nick, status, password):
-    def _join_gc_via_invite(self, invite_data, friend_number, nick, status='', password=''):
+        group_number = self._tox.group_invite_accept(invite_data, friend_number, nick, status, password)
-        LOG.debug(f"_join_gc_via_invite friend_number={friend_number} nick={nick} datalen={len(invite_data)}")
-        if nick is None:
-            nick = ''
-        if invite_data is None:
-            invite_data = b''
-        try:
-            # status should be dropped
-            group_number = self._tox.group_invite_accept(invite_data, friend_number, nick, password=password)
-        except Exception as e:
-            LOG.error(f"_join_gc_via_invite ERROR {e}")
-            return
-        try:
         self._add_new_group_by_number(group_number)
-        except Exception as e:
-            LOG.error(f"_join_gc_via_invite group_number={group_number} {e}")
-            return
 
     def _update_invites_button_state(self):
         self._main_screen.update_gc_invites_button_state()

toxygen/groups/peers_list.py

@@ -1,9 +1,11 @@
 from ui.group_peers_list import PeerItem, PeerTypeItem
-from tox_wrapper.toxcore_enums_and_consts import *
+from wrapper.toxcore_enums_and_consts import *
 from ui.widgets import *
 
 
+# -----------------------------------------------------------------------------------------------------------------
 # Builder
+# -----------------------------------------------------------------------------------------------------------------
 
 
 class PeerListBuilder:
@@ -61,7 +63,9 @@ class PeerListBuilder:
         self._peers[self._index] = peer
         self._index += 1
 
+# -----------------------------------------------------------------------------------------------------------------
 # Generators
+# -----------------------------------------------------------------------------------------------------------------
 
 
 class PeersListGenerator:

toxygen/history/database.py

@@ -1,20 +1,19 @@
-# -*- mode: python; indent-tabs-mode: nil; py-indent-offset: 4; coding: utf-8 -*-
 from sqlite3 import connect
 import os.path
 import utils.util as util
 
-global LOG
-import logging
-LOG = logging.getLogger('app.db')
 
 TIMEOUT = 11
+
 SAVE_MESSAGES = 500
+
 MESSAGE_AUTHOR = {
     'ME': 0,
     'FRIEND': 1,
     'NOT_SENT': 2,
     'GC_PEER': 3
 }
+
 CONTACT_TYPE = {
     'FRIEND': 0,
     'GC_PEER': 1,
@@ -25,33 +24,23 @@ CONTACT_TYPE = {
 class Database:
 
     def __init__(self, path, toxes):
-        self._path = path
+        self._path, self._toxes = path, toxes
-        self._toxes = toxes
         self._name = os.path.basename(path)
-
+        if os.path.exists(path):
-    def open(self):
-        path = self._path
-        toxes = self._toxes
-        if not os.path.exists(path):
-            LOG.warn('Db not found: ' +path)
-            return
             try:
                 with open(path, 'rb') as fin:
                     data = fin.read()
-        except Exception as ex:
-            LOG.error('Db reading error: ' +path +' ' +str(ex))
-            raise
-        try:
                 if toxes.is_data_encrypted(data):
                     data = toxes.pass_decrypt(data)
                     with open(path, 'wb') as fout:
                         fout.write(data)
             except Exception as ex:
-            LOG.error('Db writing error: ' +path +' ' + str(ex))
+                util.log('Db reading error: ' + str(ex))
                 os.remove(path)
-        LOG.info('Db opened: ' +path)
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Public methods
+    # -----------------------------------------------------------------------------------------------------------------
 
     def save(self):
         if self._toxes.has_password():
@@ -69,7 +58,6 @@ class Database:
             data = self._toxes.pass_encrypt(data)
         with open(new_path, 'wb') as fout:
             fout.write(data)
-        LOG.info('Db exported: ' +new_path)
 
     def add_friend_to_db(self, tox_id):
         db = self._connect()
@@ -84,14 +72,11 @@ class Database:
                            '    message_type INTEGER'
                            ')')
             db.commit()
-            return True
+        except:
-        except Exception as e:
+            print('Database is locked!')
-            LOG.error("dd_friend_to_db " +self._name +' Database exception! ' +str(e))
             db.rollback()
-            return False
         finally:
             db.close()
-            LOG.debug(f"add_friend_to_db {tox_id}")
 
     def delete_friend_from_db(self, tox_id):
         db = self._connect()
@@ -99,14 +84,11 @@ class Database:
             cursor = db.cursor()
             cursor.execute('DROP TABLE id' + tox_id + ';')
             db.commit()
-            return True
+        except:
-        except Exception as e:
+            print('Database is locked!')
-            LOG.error("delete_friend_from_db " +self._name +' Database exception! ' +str(e))
             db.rollback()
-            return False
         finally:
             db.close()
-            LOG.debug(f"delete_friend_from_db {tox_id}")
 
     def save_messages_to_db(self, tox_id, messages_iter):
         db = self._connect()
@@ -114,16 +96,13 @@ class Database:
             cursor = db.cursor()
             cursor.executemany('INSERT INTO id' + tox_id +
                                '(message, author_name, author_type, unix_time, message_type) ' +
-                               'VALUES (?, ?, ?, ?, ?);', messages_iter)
+                               'VALUES (?, ?, ?, ?, ?, ?);', messages_iter)
             db.commit()
-            return True
+        except:
-        except Exception as e:
+            print('Database is locked!')
-            LOG.error("" +self._name +' Database exception! ' +str(e))
             db.rollback()
-            return False
         finally:
             db.close()
-            LOG.debug(f"save_messages_to_db {tox_id}")
 
     def update_messages(self, tox_id, message_id):
         db = self._connect()
@@ -132,14 +111,11 @@ class Database:
             cursor.execute('UPDATE id' + tox_id + ' SET author = 0 '
                            'WHERE id = ' + str(message_id) + ' AND author = 2;')
             db.commit()
-            return True
+        except:
-        except Exception as e:
+            print('Database is locked!')
-            LOG.error("" +self._name +' Database exception! ' +str(e))
             db.rollback()
-            return False
         finally:
             db.close()
-            LOG.debug(f"update_messages {tox_id}")
 
     def delete_message(self, tox_id, unique_id):
         db = self._connect()
@@ -147,14 +123,11 @@ class Database:
             cursor = db.cursor()
             cursor.execute('DELETE FROM id' + tox_id + ' WHERE id = ' + str(unique_id) + ';')
             db.commit()
-            return True
+        except:
-        except Exception as e:
+            print('Database is locked!')
-            LOG.error("" +self._name +' Database exception! ' +str(e))
             db.rollback()
-            return False
         finally:
             db.close()
-            LOG.debug(f"delete_message {tox_id}")
 
     def delete_messages(self, tox_id):
         db = self._connect()
@@ -162,21 +135,20 @@ class Database:
             cursor = db.cursor()
             cursor.execute('DELETE FROM id' + tox_id + ';')
             db.commit()
-            return True
+        except:
-        except Exception as e:
+            print('Database is locked!')
-            LOG.error("" +self._name +' Database exception! ' +str(e))
             db.rollback()
-            return False
         finally:
             db.close()
-            LOG.debug(f"delete_messages {tox_id}")
 
     def messages_getter(self, tox_id):
         self.add_friend_to_db(tox_id)
 
         return Database.MessageGetter(self._path, tox_id)
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Messages loading
+    # -----------------------------------------------------------------------------------------------------------------
 
     class MessageGetter:
 
@@ -221,7 +193,9 @@ class Database:
         def _disconnect(self):
             self._db.close()
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Private methods
+    # -----------------------------------------------------------------------------------------------------------------
 
     def _connect(self):
         return connect(self._path, timeout=TIMEOUT)

toxygen/history/history.py

@@ -1,9 +1,5 @@
-# -*- mode: python; indent-tabs-mode: nil; py-indent-offset: 4; coding: utf-8 -*-
 from history.history_logs_generators import *
 
-global LOG
-import logging
-LOG = logging.getLogger('app.db')
 
 class History:
 
@@ -22,14 +18,15 @@ class History:
     def set_contacts_manager(self, contacts_manager):
         self._contacts_manager = contacts_manager
 
+    # -----------------------------------------------------------------------------------------------------------------
     # History support
+    # -----------------------------------------------------------------------------------------------------------------
 
     def save_history(self):
         """
         Save history to db
         """
-        # me a mistake? was _db not _history
+        if self._settings['save_db']:
-        if self._settings['save_history']:
             for friend in self._contact_provider.get_all_friends():
                 self._db.add_friend_to_db(friend.tox_id)
                 if not self._settings['save_unsent_only']:
@@ -60,10 +57,8 @@ class History:
             file_name += '.' + extension
 
         history = self.generate_history(contact, as_text)
-        assert history
         with open(file_name, 'wt') as fl:
             fl.write(history)
-        LOG.info(f"wrote history to {file_name}")
 
     def delete_message(self, message):
         contact = self._contacts_manager.get_curr_contact()
@@ -126,7 +121,9 @@ class History:
 
         return generator.generate()
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Items creation
+    # -----------------------------------------------------------------------------------------------------------------
 
     def _create_message_item(self, message):
         return self._messages_items_factory.create_message_item(message, False)

toxygen/history/history_logs_generators.py

@@ -1,5 +1,5 @@
-import utils.util as util
 from messenger.messages import *
+import utils.util as util
 
 
 class HistoryLogsGenerator:

toxygen/main.py

@@ -0,0 +1,51 @@
+import app
+from user_data.settings import *
+import utils.util as util
+import argparse
+
+
+__maintainer__ = 'Ingvar'
+__version__ = '0.5.0'
+
+
+def clean():
+    """Removes libs folder"""
+    directory = util.get_libs_directory()
+    util.remove(directory)
+
+
+def reset():
+    Settings.reset_auto_profile()
+
+
+def print_toxygen_version():
+    print('Toxygen v' + __version__)
+
+
+def main():
+    parser = argparse.ArgumentParser()
+    parser.add_argument('--version', action='store_true', help='Prints Toxygen version')
+    parser.add_argument('--clean', action='store_true', help='Delete toxcore libs from libs folder')
+    parser.add_argument('--reset', action='store_true', help='Reset default profile')
+    parser.add_argument('--uri', help='Add specified Tox ID to friends')
+    parser.add_argument('profile', nargs='?', default=None, help='Path to Tox profile')
+    args = parser.parse_args()
+
+    if args.version:
+        print_toxygen_version()
+        return
+
+    if args.clean:
+        clean()
+        return
+
+    if args.reset:
+        reset()
+        return
+
+    toxygen = app.App(__version__, args.profile, args.uri)
+    toxygen.main()
+
+
+if __name__ == '__main__':
+    main()

toxygen/messenger/messages.py

@@ -1,8 +1,8 @@
-import os.path
-
 from history.database import MESSAGE_AUTHOR
+import os.path
 from ui.messages_widgets import *
 
+
 MESSAGE_TYPE = {
     'TEXT': 0,
     'ACTION': 1,
@@ -38,8 +38,8 @@ class Message:
 
     MESSAGE_ID = 0
 
-    def __init__(self, message_type, author, iTime):
+    def __init__(self, message_type, author, time):
-        self._time = iTime
+        self._time = time
         self._type = message_type
         self._author = author
         self._widget = None
@@ -66,7 +66,6 @@ class Message:
     message_id = property(get_message_id)
 
     def get_widget(self, *args):
-        # FixMe
         self._widget = self._create_widget(*args)
 
         return self._widget
@@ -82,8 +81,7 @@ class Message:
             self._widget.mark_as_sent()
 
     def _create_widget(self, *args):
-        # overridden
+        pass
-        return None
 
     @staticmethod
     def _get_id():
@@ -97,8 +95,8 @@ class TextMessage(Message):
     Plain text or action message
     """
 
-    def __init__(self, message, owner, iTime, message_type, message_id=0):
+    def __init__(self, message, owner, time, message_type, message_id=0):
-        super().__init__(message_type, owner, iTime)
+        super().__init__(message_type, owner, time)
         self._message = message
         self._id = message_id
 
@@ -121,8 +119,8 @@ class TextMessage(Message):
 
 class OutgoingTextMessage(TextMessage):
 
-    def __init__(self, message, owner, iTime, message_type, tox_message_id=0):
+    def __init__(self, message, owner, time, message_type, tox_message_id=0):
-        super().__init__(message, owner, iTime, message_type)
+        super().__init__(message, owner, time, message_type)
         self._tox_message_id = tox_message_id
 
     def get_tox_message_id(self):
@@ -136,8 +134,8 @@ class OutgoingTextMessage(TextMessage):
 
 class GroupChatMessage(TextMessage):
 
-    def __init__(self, id, message, owner, iTime, message_type, name):
+    def __init__(self, id, message, owner, time, message_type, name):
-        super().__init__(id, message, owner, iTime, message_type)
+        super().__init__(id, message, owner, time, message_type)
         self._user_name = name
 
 
@@ -146,8 +144,8 @@ class TransferMessage(Message):
     Message with info about file transfer
     """
 
-    def __init__(self, author, iTime, state, size, file_name, friend_number, file_number):
+    def __init__(self, author, time, state, size, file_name, friend_number, file_number):
-        super().__init__(MESSAGE_TYPE['FILE_TRANSFER'], author, iTime)
+        super().__init__(MESSAGE_TYPE['FILE_TRANSFER'], author, time)
         self._state = state
         self._size = size
         self._file_name = file_name
@@ -187,10 +185,10 @@ class TransferMessage(Message):
 
     file_name = property(get_file_name)
 
-    def transfer_updated(self, state, percentage, iTime):
+    def transfer_updated(self, state, percentage, time):
         self._state = state
         if self._widget is not None:
-            self._widget.update_transfer_state(state, percentage, iTime)
+            self._widget.update_transfer_state(state, percentage, time)
 
     def _create_widget(self, *args):
         return FileTransferItem(self, *args)
@@ -198,9 +196,9 @@ class TransferMessage(Message):
 
 class UnsentFileMessage(TransferMessage):
 
-    def __init__(self, path, data, iTime, author, size, friend_number):
+    def __init__(self, path, data, time, author, size, friend_number):
         file_name = os.path.basename(path)
-        super().__init__(author, iTime, FILE_TRANSFER_STATE['UNSENT'], size, file_name, friend_number, -1)
+        super().__init__(author, time, FILE_TRANSFER_STATE['UNSENT'], size, file_name, friend_number, -1)
         self._data, self._path = data, path
 
     def get_data(self):
@@ -237,5 +235,5 @@ class InlineImageMessage(Message):
 
 class InfoMessage(TextMessage):
 
-    def __init__(self, message, iTime):
+    def __init__(self, message, time):
-        super().__init__(message, None, iTime, MESSAGE_TYPE['INFO_MESSAGE'])
+        super().__init__(message, None, time, MESSAGE_TYPE['INFO_MESSAGE'])

toxygen/messenger/messenger.py

@@ -1,15 +1,6 @@
-# -*- mode: python; indent-tabs-mode: nil; py-indent-offset: 4; coding: utf-8 -*-
 import common.tox_save as tox_save
-import utils.ui as util_ui
-
 from messenger.messages import *
-from tox_wrapper.tests.support_testing import assert_main_thread
-from tox_wrapper.toxcore_enums_and_consts import TOX_MAX_MESSAGE_LENGTH
 
-global LOG
-import logging
-LOG = logging.getLogger('app.'+__name__)
-log = lambda x: LOG.info(x)
 
 class Messenger(tox_save.ToxSave):
 
@@ -28,9 +19,6 @@ class Messenger(tox_save.ToxSave):
         calls_manager.call_started_event.add_callback(self._on_call_started)
         calls_manager.call_finished_event.add_callback(self._on_call_finished)
 
-    def __repr__(self):
-        return "<Messenger>"
-
     def get_last_message(self):
         contact = self._contacts_manager.get_curr_contact()
         if contact is None:
@@ -38,7 +26,9 @@ class Messenger(tox_save.ToxSave):
 
         return contact.get_last_message_text()
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Messaging - friends
+    # -----------------------------------------------------------------------------------------------------------------
 
     def new_message(self, friend_number, message_type, message):
         """
@@ -61,53 +51,33 @@ class Messenger(tox_save.ToxSave):
             self._screen.messageEdit.clear()
             return
 
-        message_type = TOX_MESSAGE_TYPE['NORMAL']
-        if False: # undocumented
         action_message_prefix = '/me '
         if text.startswith(action_message_prefix):
             message_type = TOX_MESSAGE_TYPE['ACTION']
             text = text[len(action_message_prefix):]
+        else:
+            message_type = TOX_MESSAGE_TYPE['NORMAL']
 
-        if len(text) > TOX_MAX_MESSAGE_LENGTH:
-            text = text[:TOX_MAX_MESSAGE_LENGTH] # 1372
-        try:
         if self._contacts_manager.is_active_a_friend():
             self.send_message_to_friend(text, message_type)
         elif self._contacts_manager.is_active_a_group():
-                self.send_message_to_group('~'+text, message_type)
+            self.send_message_to_group(text, message_type)
         elif self._contacts_manager.is_active_a_group_chat_peer():
             self.send_message_to_group_peer(text, message_type)
-            else:
-                LOG.warn(f'Unknown friend type for Messenger send_message')
-        except Exception as e:
-            LOG.error(f'Messenger send_message {e}')
-            import traceback
-            LOG.warn(traceback.format_exc())
-            title = 'Messenger send_message Error'
-            text = 'Error: ' + str(e)
-            assert_main_thread()
-            util_ui.message_box(text, title)
 
     def send_message_to_friend(self, text, message_type, friend_number=None):
         """
         Send message
         :param text: message text
         :param friend_number: number of friend
-        from Qt callback
         """
-        if not text:
-            return
         if friend_number is None:
             friend_number = self._contacts_manager.get_active_number()
-        if friend_number is None  or friend_number < 0:
+
-            LOG.error(f"No _contacts_manager.get_active_number")
+        if not text or friend_number < 0:
             return
-        assert_main_thread()
 
         friend = self._get_friend_by_number(friend_number)
-        if not friend:
-            LOG.error(f"No self._get_friend_by_number")
-            return
         messages = self._split_message(text.encode('utf-8'))
         t = util.get_unix_time()
         for message in messages:
@@ -136,9 +106,11 @@ class Messenger(tox_save.ToxSave):
                 message_id = self._tox.friend_send_message(friend_number, message.type, message.text.encode('utf-8'))
                 message.tox_message_id = message_id
         except Exception as ex:
-            LOG.warn('Sending pending messages failed with ' + str(ex))
+            util.log('Sending pending messages failed with ' + str(ex))
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Messaging - groups
+    # -----------------------------------------------------------------------------------------------------------------
 
     def send_message_to_group(self, text, message_type, group_number=None):
         if group_number is None:
@@ -169,17 +141,13 @@ class Messenger(tox_save.ToxSave):
         """
         t = util.get_unix_time()
         group = self._get_group_by_number(group_number)
-        if not group:
-            LOG.error(f"FixMe new_group_message _get_group_by_number({group_number})")
-            return
         peer = group.get_peer_by_id(peer_id)
-        if not peer:
-            LOG.error('FixMe new_group_message group.get_peer_by_id ' + str(peer_id))
-            return
         text_message = TextMessage(message, MessageAuthor(peer.name, MESSAGE_AUTHOR['GC_PEER']), t, message_type)
         self._add_message(text_message, group)
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Messaging - group peers
+    # -----------------------------------------------------------------------------------------------------------------
 
     def send_message_to_group_peer(self, text, message_type, group_number=None, peer_id=None):
         if group_number is None or peer_id is None:
@@ -188,36 +156,17 @@ class Messenger(tox_save.ToxSave):
             group = self._get_group_by_public_key(group_peer_contact.group_pk)
             group_number = group.number
 
-        if not text:
+        if not text or group_number < 0 or peer_id < 0:
-            return
-        if group.number < 0:
-            return
-        if peer_id is not None and peer_id < 0:
             return
 
-        assert_main_thread()
+        group_peer_contact = self._contacts_manager.get_or_create_group_peer_contact(group_number, peer_id)
         group = self._get_group_by_number(group_number)
         messages = self._split_message(text.encode('utf-8'))
-
-        # FixMe: peer_id is None?
-        group_peer_contact = self._contacts_manager.get_or_create_group_peer_contact(group_number, peer_id)
-        if group_peer_contact is None:
-            LOG.warn("M.group_send_private_message group_peer_contact is None")
-            return
-        # group_peer_contact now may be None
-
         t = util.get_unix_time()
         for message in messages:
-            bRet = self._tox.group_send_private_message(group_number, peer_id, message_type, message)
+            self._tox.group_send_private_message(group_number, peer_id, message_type, message)
-            if not bRet:
-                LOG.warn("M.group_send_private_messag failed")
-                continue
             message_author = MessageAuthor(group.get_self_name(), MESSAGE_AUTHOR['GC_PEER'])
             message = OutgoingTextMessage(text, message_author, t, message_type)
-            # AttributeError: 'GroupChatPeer' object has no attribute 'append_message'
-            if not hasattr(group_peer_contact, 'append_message'):
-                LOG.warn("M. group_peer_contact has no append_message group_peer_contact={group_peer_contact}")
-            else:
             group_peer_contact.append_message(message)
             if not self._contacts_manager.is_contact_active(group_peer_contact):
                 return
@@ -233,24 +182,22 @@ class Messenger(tox_save.ToxSave):
         t = util.get_unix_time()
         group = self._get_group_by_number(group_number)
         peer = group.get_peer_by_id(peer_id)
-        if not peer:
-            LOG.warn('FixMe new_group_private_message group.get_peer_by_id ' + str(peer_id))
-            return
         text_message = TextMessage(message, MessageAuthor(peer.name, MESSAGE_AUTHOR['GC_PEER']),
                                    t, message_type)
         group_peer_contact = self._contacts_manager.get_or_create_group_peer_contact(group_number, peer_id)
-        if not group_peer_contact:
-            LOG.warn('FixMe new_group_private_message group_peer_contact ' + str(peer_id))
-            return
         self._add_message(text_message, group_peer_contact)
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Message receipts
+    # -----------------------------------------------------------------------------------------------------------------
 
     def receipt(self, friend_number, message_id):
         friend = self._get_friend_by_number(friend_number)
         friend.mark_as_sent(message_id)
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Typing notifications
+    # -----------------------------------------------------------------------------------------------------------------
 
     def send_typing(self, typing):
         """
@@ -268,7 +215,9 @@ class Messenger(tox_save.ToxSave):
         if self._contacts_manager.is_friend_active(friend_number):
             self._screen.typing.setVisible(typing)
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Contact info updated
+    # -----------------------------------------------------------------------------------------------------------------
 
     def new_friend_name(self, friend, old_name, new_name):
         if old_name == new_name or friend.has_alias():
@@ -279,7 +228,9 @@ class Messenger(tox_save.ToxSave):
             friend.actions = True
         self._add_info_message(friend.number, message)
 
+    # -----------------------------------------------------------------------------------------------------------------
     # Private methods
+    # -----------------------------------------------------------------------------------------------------------------
 
     @staticmethod
     def _split_message(message):
@@ -334,29 +285,21 @@ class Messenger(tox_save.ToxSave):
 
     def _add_info_message(self, friend_number, text):
         friend = self._get_friend_by_number(friend_number)
-        assert friend
         message = InfoMessage(text, util.get_unix_time())
         friend.append_message(message)
         if self._contacts_manager.is_friend_active(friend_number):
             self._create_info_message_item(message)
 
     def _create_info_message_item(self, message):
-        assert_main_thread()
         self._items_factory.create_message_item(message)
         self._screen.messages.scrollToBottom()
 
     def _add_message(self, text_message, contact):
-        assert_main_thread()
-        if not contact:
-            LOG.warn("_add_message null contact")
-            return
         if self._contacts_manager.is_contact_active(contact):  # add message to list
-#            LOG.debug("_add_message is_contact_active(contact)")
             self._create_message_item(text_message)
             self._screen.messages.scrollToBottom()
             self._contacts_manager.get_curr_contact().append_message(text_message)
         else:
-#            LOG.debug("_add_message not is_contact_active(contact)")
             contact.inc_messages()
             contact.append_message(text_message)
             if not contact.visibility:

toxygen/middleware/callbacks.py

@@ -1,113 +1,48 @@
-# -*- mode: python; indent-tabs-mode: nil; py-indent-offset: 4; coding: utf-8 -*-
+from PyQt5 import QtGui
-import sys
+from wrapper.toxcore_enums_and_consts import *
-import os
+from wrapper.toxav_enums import *
-import threading
+from wrapper.tox import bin_to_string
-from  qtpy import QtGui
-from tox_wrapper.toxcore_enums_and_consts import *
-from tox_wrapper.toxav_enums import *
-from tox_wrapper.tox import bin_to_string
 import utils.ui as util_ui
 import utils.util as util
+import cv2
+import numpy as np
 from middleware.threads import invoke_in_main_thread, execute
 from notifications.tray import tray_notification
 from notifications.sound import *
-from datetime import datetime
+import threading
-
-iMAX_INT32 = 4294967295
-# callbacks can be called in any thread so were being careful
-def LOG_ERROR(l): print(f"EROR. {l}")
-def LOG_WARN(l):  print(f"WARN. {l}")
-def LOG_INFO(l):
-    bIsVerbose = not hasattr(__builtins__, 'app') or app.oArgs.loglevel <= 20-1
-    if bIsVerbose: print(f"INFO. {l}")
-def LOG_DEBUG(l):
-    bIsVerbose = not hasattr(__builtins__, 'app') or app.oArgs.loglevel <= 10-1
-    if bIsVerbose: print(f"DBUG. {l}")
-def LOG_TRACE(l):
-    bIsVerbose = not hasattr(__builtins__, 'app') or app.oArgs.loglevel < 10-1
-    pass # print(f"TRACE. {l}")
-
-global aTIMES
-aTIMES=dict()
-def bTooSoon(key, sSlot, fSec=10.0):
-    # rate limiting
-    global aTIMES
-    if sSlot not in aTIMES:
-        aTIMES[sSlot] = dict()
-    OTIME = aTIMES[sSlot]
-    now = datetime.now()
-    if key not in OTIME:
-        OTIME[key] = now
-        return False
-    delta = now - OTIME[key]
-    OTIME[key] = now
-    if delta.total_seconds() < fSec: return True
-    return False
 
 # TODO: refactoring. Use contact provider instead of manager
 
+# -----------------------------------------------------------------------------------------------------------------
 # Callbacks - current user
+# -----------------------------------------------------------------------------------------------------------------
 
-global iBYTES
-iBYTES=0
-def sProcBytes(sFile=None):
-    if sys.platform == 'win32': return ''
-    global iBYTES
-    if sFile is None:
-        pid = os.getpid()
-        sFile = f"/proc/{pid}/net/softnet_stat"
-    if os.path.exists(sFile):
-        total = 0
-        with open(sFile, 'r') as iFd:
-            for elt in iFd.readlines():
-                i = elt.find(' ')
-                p = int(elt[:i], 16)
-                total = total + p
-        if iBYTES == 0:
-            iBYTES = total
-            return ''
-        diff = total - iBYTES
-        s = f' {diff // 1024} Kbytes'
-    else:
-        s = ''
-    return s
 
 def self_connection_status(tox, profile):
     """
     Current user changed connection status (offline, TCP, UDP)
     """
-    sSlot = 'self connection status'
     def wrapped(tox_link, connection, user_data):
-        key = f"connection {connection}"
+        print('Connection status: ', str(connection))
-        if bTooSoon(key, sSlot, 10): return
-        s = sProcBytes()
-        try:
         status = tox.self_get_status() if connection != TOX_CONNECTION['NONE'] else None
-            if status:
-                LOG_DEBUG(f"self_connection_status: connection={connection} status={status}" +' '+s)
         invoke_in_main_thread(profile.set_status, status)
-        except Exception as e:
-            LOG_ERROR(f"self_connection_status: {e}")
-            pass
 
     return wrapped
 
 
+# -----------------------------------------------------------------------------------------------------------------
 # Callbacks - friends
+# -----------------------------------------------------------------------------------------------------------------
 
 
 def friend_status(contacts_manager, file_transfer_handler, profile, settings):
-    sSlot = 'friend status'
     def wrapped(tox, friend_number, new_status, user_data):
         """
         Check friend's status (none, busy, away)
         """
-        LOG_INFO(f"Friend's #{friend_number} status changed")
+        print("Friend's #{} status changed!".format(friend_number))
-        key = f"friend_number {friend_number}"
-        if bTooSoon(key, sSlot, 10): return
         friend = contacts_manager.get_friend_by_number(friend_number)
-        if friend.status is None and settings['sound_notifications'] and \
+        if friend.status is None and settings['sound_notifications'] and profile.status != TOX_USER_STATUS['BUSY']:
-          profile.status != TOX_USER_STATUS['BUSY']:
             sound_notification(SOUND_NOTIFICATION['FRIEND_CONNECTION_STATUS'])
         invoke_in_main_thread(friend.set_status, new_status)
 
@@ -126,7 +61,7 @@ def friend_connection_status(contacts_manager, profile, settings, plugin_loader,
         """
         Check friend's connection status (offline, udp, tcp)
         """
-        LOG_DEBUG(f"Friend #{friend_number} connection status: {new_status}")
+        print("Friend #{} connection status: {}".format(friend_number, new_status))
         friend = contacts_manager.get_friend_by_number(friend_number)
         if new_status == TOX_CONNECTION['NONE']:
             invoke_in_main_thread(friend.set_status, None)
@@ -144,35 +79,29 @@ def friend_connection_status(contacts_manager, profile, settings, plugin_loader,
 
 
 def friend_name(contacts_provider, messenger):
-    sSlot = 'friend_name'
     def wrapped(tox, friend_number, name, size, user_data):
         """
         Friend changed his name
         """
-        key = f"friend_number={friend_number}"
+        print('New name friend #' + str(friend_number))
-        if bTooSoon(key, sSlot, 60): return
         friend = contacts_provider.get_friend_by_number(friend_number)
         old_name = friend.name
         new_name = str(name, 'utf-8')
-        LOG_DEBUG(f"get_friend_by_number #{friend_number} {new_name}")
         invoke_in_main_thread(friend.set_name, new_name)
         invoke_in_main_thread(messenger.new_friend_name, friend, old_name, new_name)
 
     return wrapped
 
+
 def friend_status_message(contacts_manager, messenger):
-    sSlot = 'status_message'
     def wrapped(tox, friend_number, status_message, size, user_data):
         """
         :return: function for callback friend_status_message. It updates friend's status message
         and calls window repaint
         """
         friend = contacts_manager.get_friend_by_number(friend_number)
-        key = f"friend_number={friend_number}"
-        if bTooSoon(key, sSlot, 10): return
-
         invoke_in_main_thread(friend.set_status_message, str(status_message, 'utf-8'))
-        LOG_DEBUG(f'User #{friend_number} has new status message')
+        print('User #{} has new status message'.format(friend_number))
         invoke_in_main_thread(messenger.send_messages, friend_number)
 
     return wrapped
@@ -183,19 +112,15 @@ def friend_message(messenger, contacts_manager, profile, settings, window, tray)
         """
         New message from friend
         """
-        LOG_DEBUG(f"friend_message #{friend_number}")
         message = str(message, 'utf-8')
         invoke_in_main_thread(messenger.new_message, friend_number, message_type, message)
         if not window.isActiveWindow():
             friend = contacts_manager.get_friend_by_number(friend_number)
-            if settings['notifications'] \
+            if settings['notifications'] and profile.status != TOX_USER_STATUS['BUSY'] and not settings.locked:
-              and profile.status != TOX_USER_STATUS['BUSY'] \
-              and not settings.locked:
                 invoke_in_main_thread(tray_notification, friend.name, message, tray, window)
             if settings['sound_notifications'] and profile.status != TOX_USER_STATUS['BUSY']:
                 sound_notification(SOUND_NOTIFICATION['MESSAGE'])
             icon = os.path.join(util.get_images_directory(), 'icon_new_messages.png')
-            if tray:
             invoke_in_main_thread(tray.setIcon, QtGui.QIcon(icon))
 
     return wrapped
@@ -206,7 +131,7 @@ def friend_request(contacts_manager):
         """
         Called when user get new friend request
         """
-        LOG_DEBUG(f'Friend request')
+        print('Friend request')
         key = ''.join(chr(x) for x in public_key[:TOX_PUBLIC_KEY_SIZE])
         tox_id = bin_to_string(key, TOX_PUBLIC_KEY_SIZE)
         invoke_in_main_thread(contacts_manager.process_friend_request, tox_id, str(message, 'utf-8'))
@@ -215,12 +140,9 @@ def friend_request(contacts_manager):
 
 
 def friend_typing(messenger):
-    sSlot = "friend_typing"
     def wrapped(tox, friend_number, typing, user_data):
-        key = f"friend_number={friend_number}"
-        if bTooSoon(key, sSlot, 10): return
-        LOG_DEBUG(f"friend_typing #{friend_number}")
         invoke_in_main_thread(messenger.friend_typing, friend_number, typing)
+
     return wrapped
 
 
@@ -231,7 +153,9 @@ def friend_read_receipt(messenger):
     return wrapped
 
 
+# -----------------------------------------------------------------------------------------------------------------
 # Callbacks - file transfers
+# -----------------------------------------------------------------------------------------------------------------
 
 
 def tox_file_recv(window, tray, profile, file_transfer_handler, contacts_manager, settings):
@@ -240,7 +164,7 @@ def tox_file_recv(window, tray, profile, file_transfer_handler, contacts_manager
     """
     def wrapped(tox, friend_number, file_number, file_type, size, file_name, file_name_size, user_data):
         if file_type == TOX_FILE_KIND['DATA']:
-            LOG_INFO(f'file_transfer_handler File friend_number={friend_number}')
+            print('File')
             try:
                 file_name = str(file_name[:file_name_size], 'utf-8')
             except:
@@ -252,18 +176,15 @@ def tox_file_recv(window, tray, profile, file_transfer_handler, contacts_manager
                                   file_name)
             if not window.isActiveWindow():
                 friend = contacts_manager.get_friend_by_number(friend_number)
-                if settings['notifications'] \
+                if settings['notifications'] and profile.status != TOX_USER_STATUS['BUSY'] and not settings.locked:
-                  and profile.status != TOX_USER_STATUS['BUSY'] \
-                  and not settings.locked:
                     file_from = util_ui.tr("File from")
                     invoke_in_main_thread(tray_notification, file_from + ' ' + friend.name, file_name, tray, window)
                 if settings['sound_notifications'] and profile.status != TOX_USER_STATUS['BUSY']:
                     sound_notification(SOUND_NOTIFICATION['FILE_TRANSFER'])
-                if tray:
                 icon = util.join_path(util.get_images_directory(), 'icon_new_messages.png')
                 invoke_in_main_thread(tray.setIcon, QtGui.QIcon(icon))
         else:  # avatar
-            LOG_DEBUG(f'file_transfer_handler Avatar')
+            print('Avatar')
             invoke_in_main_thread(file_transfer_handler.incoming_avatar,
                                   friend_number,
                                   file_number,
@@ -306,7 +227,9 @@ def file_recv_control(file_transfer_handler):
 
     return wrapped
 
+# -----------------------------------------------------------------------------------------------------------------
 # Callbacks - custom packets
+# -----------------------------------------------------------------------------------------------------------------
 
 
 def lossless_packet(plugin_loader):
@@ -331,20 +254,20 @@ def lossy_packet(plugin_loader):
     return wrapped
 
 
+# -----------------------------------------------------------------------------------------------------------------
 # Callbacks - audio
+# -----------------------------------------------------------------------------------------------------------------
 
 def call_state(calls_manager):
-    def wrapped(iToxav, friend_number, mask, user_data):
+    def wrapped(toxav, friend_number, mask, user_data):
         """
         New call state
         """
-        LOG_DEBUG(f"call_state #{friend_number}")
+        print(friend_number, mask)
         if mask == TOXAV_FRIEND_CALL_STATE['FINISHED'] or mask == TOXAV_FRIEND_CALL_STATE['ERROR']:
             invoke_in_main_thread(calls_manager.stop_call, friend_number, True)
         else:
-            # guessing was calls_manager.
+            calls_manager.toxav_call_state_cb(friend_number, mask)
-            #? incoming_call
-            calls_manager._call.toxav_call_state_cb(friend_number, mask)
 
     return wrapped
 
@@ -354,7 +277,7 @@ def call(calls_manager):
         """
         Incoming call from friend
         """
-        LOG_DEBUG(f"Incoming call from {friend_number} {audio} {video}")
+        print(friend_number, audio, video)
         invoke_in_main_thread(calls_manager.incoming_call, audio, video, friend_number)
 
     return wrapped
@@ -365,16 +288,16 @@ def callback_audio(calls_manager):
         """
         New audio chunk
         """
-        LOG_DEBUG(f"callback_audio #{friend_number}")
+        calls_manager.call.audio_chunk(
-        # dunno was .call
-        calls_manager._call.audio_chunk(
             bytes(samples[:audio_samples_per_channel * 2 * audio_channels_count]),
             audio_channels_count,
             rate)
 
     return wrapped
 
+# -----------------------------------------------------------------------------------------------------------------
 # Callbacks - video
+# -----------------------------------------------------------------------------------------------------------------
 
 
 def video_receive_frame(toxav, friend_number, width, height, y, u, v, ystride, ustride, vstride, user_data):
@@ -401,9 +324,6 @@ def video_receive_frame(toxav, friend_number, width, height, y, u, v, ystride, u
 
     It can be created from initial y, u, v using slices
     """
-    LOG_DEBUG(f"video_receive_frame from toxav_video_receive_frame_cb={friend_number}")
-    import cv2
-    import numpy as np
     try:
         y_size = abs(max(width, abs(ystride)))
         u_size = abs(max(width // 2, abs(ustride)))
@@ -429,10 +349,11 @@ def video_receive_frame(toxav, friend_number, width, height, y, u, v, ystride, u
 
         invoke_in_main_thread(cv2.imshow, str(friend_number), frame)
     except Exception as ex:
-        LOG_ERROR(f"video_receive_frame  {ex} #{friend_number}")
+        print(ex)
-        pass
 
+# -----------------------------------------------------------------------------------------------------------------
 # Callbacks - groups
+# -----------------------------------------------------------------------------------------------------------------
 
 
 def group_message(window, tray, tox, messenger, settings, profile):
@@ -440,22 +361,16 @@ def group_message(window, tray, tox, messenger, settings, profile):
     New message in group chat
     """
     def wrapped(tox_link, group_number, peer_id, message_type, message, length, user_data):
-        LOG_DEBUG(f"group_message #{group_number}")
         message = str(message[:length], 'utf-8')
         invoke_in_main_thread(messenger.new_group_message, group_number, message_type, message, peer_id)
         if window.isActiveWindow():
             return
         bl = settings['notify_all_gc'] or profile.name in message
         name = tox.group_peer_get_name(group_number, peer_id)
-        if settings['sound_notifications'] and bl and \
+        if settings['notifications'] and profile.status != TOX_USER_STATUS['BUSY'] and (not settings.locked) and bl:
-           profile.status != TOX_USER_STATUS['BUSY']:
-            sound_notification(SOUND_NOTIFICATION['MESSAGE'])
-        if False and settings['tray_icon'] and tray:
-            if settings['notifications'] and \
-               profile.status != TOX_USER_STATUS['BUSY'] and \
-                   (not settings.locked) and bl:
             invoke_in_main_thread(tray_notification, name, message, tray, window)
-            if tray:
+        if settings['sound_notifications'] and bl and profile.status != TOX_USER_STATUS['BUSY']:
+            sound_notification(SOUND_NOTIFICATION['MESSAGE'])
         icon = util.join_path(util.get_images_directory(), 'icon_new_messages.png')
         invoke_in_main_thread(tray.setIcon, QtGui.QIcon(icon))
 
@@ -467,49 +382,35 @@ def group_private_message(window, tray, tox, messenger, settings, profile):
     New private message in group chat
     """
     def wrapped(tox_link, group_number, peer_id, message_type, message, length, user_data):
-        LOG_DEBUG(f"group_private_message #{group_number}")
         message = str(message[:length], 'utf-8')
         invoke_in_main_thread(messenger.new_group_private_message, group_number, message_type, message, peer_id)
         if window.isActiveWindow():
             return
         bl = settings['notify_all_gc'] or profile.name in message
-        try:
         name = tox.group_peer_get_name(group_number, peer_id)
-        except Exception as e:
+        if settings['notifications'] and profile.status != TOX_USER_STATUS['BUSY'] and (not settings.locked) and bl:
-            LOG_WARN("tox.group_peer_get_name {group_number} {peer_id}")
-            name = ''
-        if settings['notifications'] and settings['tray_icon'] \
-           and profile.status != TOX_USER_STATUS['BUSY'] \
-           and (not settings.locked) and bl:
             invoke_in_main_thread(tray_notification, name, message, tray, window)
         if settings['sound_notifications'] and bl and profile.status != TOX_USER_STATUS['BUSY']:
             sound_notification(SOUND_NOTIFICATION['MESSAGE'])
         icon = util.join_path(util.get_images_directory(), 'icon_new_messages.png')
-        if tray and hasattr(tray, 'setIcon'):
         invoke_in_main_thread(tray.setIcon, QtGui.QIcon(icon))
 
     return wrapped
 
-# Exception ignored on calling ctypes callback function: <function group_invite.<locals>.wrapped at 0x7ffede910700>
+
 def group_invite(window, settings, tray, profile, groups_service, contacts_provider):
     def wrapped(tox, friend_number, invite_data, length, group_name, group_name_length, user_data):
-        LOG_DEBUG(f"group_invite friend_number={friend_number}")
         group_name = str(bytes(group_name[:group_name_length]), 'utf-8')
         invoke_in_main_thread(groups_service.process_group_invite,
                               friend_number, group_name,
                               bytes(invite_data[:length]))
         if window.isActiveWindow():
             return
-        bHasTray = tray and settings['tray_icon']
+        if settings['notifications'] and profile.status != TOX_USER_STATUS['BUSY'] and not settings.locked:
-        if settings['notifications'] \
-           and bHasTray \
-           and profile.status != TOX_USER_STATUS['BUSY'] \
-           and not settings.locked:
             friend = contacts_provider.get_friend_by_number(friend_number)
             title = util_ui.tr('New invite to group chat')
             text = util_ui.tr('{} invites you to group "{}"').format(friend.name, group_name)
             invoke_in_main_thread(tray_notification, title, text, tray, window)
-        if tray:
         icon = util.join_path(util.get_images_directory(), 'icon_new_messages.png')
         invoke_in_main_thread(tray.setIcon, QtGui.QIcon(icon))
 
@@ -517,37 +418,18 @@ def group_invite(window, settings, tray, profile, groups_service, contacts_provi
 
 
 def group_self_join(contacts_provider, contacts_manager, groups_service):
-    sSlot = 'group_self_join'
     def wrapped(tox, group_number, user_data):
-        if group_number is None:
-            LOG_ERROR(f"group_self_join NULL group_number #{group_number}")
-            return
-        LOG_DEBUG(f"group_self_join #{group_number}")
-        key = f"group_number {group_number}"
-        if bTooSoon(key, sSlot, 10): return
         group = contacts_provider.get_group_by_number(group_number)
-        if group is None:
-            LOG_ERROR(f"group_self_join NULL group #{group}")
-            return
         invoke_in_main_thread(group.set_status, TOX_USER_STATUS['NONE'])
         invoke_in_main_thread(groups_service.update_group_info, group)
         invoke_in_main_thread(contacts_manager.update_filtration)
 
     return wrapped
 
+
 def group_peer_join(contacts_provider, groups_service):
-    sSlot = "group_peer_join"
     def wrapped(tox, group_number, peer_id, user_data):
-        key = f"group_peer_join #{group_number} peer_id={peer_id}"
-        if bTooSoon(key, sSlot, 20): return
         group = contacts_provider.get_group_by_number(group_number)
-        if group is None:
-            LOG_ERROR(f"group_peer_join NULL group #{group} group_number={group_number}")
-            return
-        if peer_id > group._peers_limit:
-            LOG_ERROR(key +f" {peer_id} > {group._peers_limit}")
-            return
-        LOG_DEBUG(f"group_peer_join group={group}")
         group.add_peer(peer_id)
         invoke_in_main_thread(groups_service.generate_peers_list)
         invoke_in_main_thread(groups_service.update_group_info, group)
@@ -556,49 +438,29 @@ def group_peer_join(contacts_provider, groups_service):
 
 
 def group_peer_exit(contacts_provider, groups_service, contacts_manager):
-    def wrapped(tox,
+    def wrapped(tox, group_number, peer_id, message, length, user_data):
-                group_number, peer_id,
-                exit_type, name, name_length,
-                message, length,
-                user_data):
         group = contacts_provider.get_group_by_number(group_number)
-        if group:
-            LOG_DEBUG(f"group_peer_exit #{group_number} peer_id={peer_id} exit_type={exit_type}")
         group.remove_peer(peer_id)
         invoke_in_main_thread(groups_service.generate_peers_list)
-        else:
-            LOG_WARN(f"group_peer_exit group not found #{group_number} peer_id={peer_id}")
 
     return wrapped
 
+
 def group_peer_name(contacts_provider, groups_service):
     def wrapped(tox, group_number, peer_id, name, length, user_data):
-        LOG_DEBUG(f"group_peer_name #{group_number} peer_id={peer_id}")
         group = contacts_provider.get_group_by_number(group_number)
         peer = group.get_peer_by_id(peer_id)
-        if peer:
         peer.name = str(name[:length], 'utf-8')
         invoke_in_main_thread(groups_service.generate_peers_list)
-        else:
-            # FixMe: known signal to revalidate roles...
-            #_peers = [(p._name, p._peer_id) for p in group.get_peers()]
-            LOG_TRACE(f"remove_peer group {group!r} has no peer_id={peer_id} in _peers!r")
-            return
 
     return wrapped
 
 
 def group_peer_status(contacts_provider, groups_service):
     def wrapped(tox, group_number, peer_id, peer_status, user_data):
-        LOG_DEBUG(f"group_peer_status #{group_number} peer_id={peer_id}")
         group = contacts_provider.get_group_by_number(group_number)
         peer = group.get_peer_by_id(peer_id)
-        if peer:
         peer.status = peer_status
-        else:
-            # _peers = [(p._name, p._peer_id) for p in group.get_peers()]
-            LOG_TRACE(f"remove_peer group {group!r} has no peer_id={peer_id} in _peers!r")
-        # TODO: add info message
         invoke_in_main_thread(groups_service.generate_peers_list)
 
     return wrapped
@@ -606,62 +468,32 @@ def group_peer_status(contacts_provider, groups_service):
 
 def group_topic(contacts_provider):
     def wrapped(tox, group_number, peer_id, topic, length, user_data):
-        LOG_DEBUG(f"group_topic #{group_number} peer_id={peer_id}")
         group = contacts_provider.get_group_by_number(group_number)
-        if group:
         topic = str(topic[:length], 'utf-8')
         invoke_in_main_thread(group.set_status_message, topic)
-        else:
-            _peers = [(p._name, p._peer_id) for p in group.get_peers()]
-            LOG_WARN(f"group_topic {group!r} has no peer_id={peer_id} in {_peers!r}")
-        # TODO: add info message
 
     return wrapped
 
+
 def group_moderation(groups_service, contacts_provider, contacts_manager, messenger):
+
     def update_peer_role(group, mod_peer_id, peer_id, new_role):
         peer = group.get_peer_by_id(peer_id)
-        if peer:
         peer.role = new_role
         # TODO: add info message
-        else:
-            # FixMe: known signal to revalidate roles...
-            # _peers = [(p._name, p._peer_id) for p in group.get_peers()]
-            LOG_TRACE(f"update_peer_role group {group!r} has no peer_id={peer_id} in _peers!r")
-        # TODO: add info message
 
     def remove_peer(group, mod_peer_id, peer_id, is_ban):
-        peer = group.get_peer_by_id(peer_id)
-        if peer:
         contacts_manager.remove_group_peer_by_id(group, peer_id)
         group.remove_peer(peer_id)
-        else:
-            # FixMe: known signal to revalidate roles...
-            #_peers = [(p._name, p._peer_id) for p in group.get_peers()]
-            LOG_TRACE(f"remove_peer group {group!r} has no peer_id={peer_id} in _peers!r")
         # TODO: add info message
 
-    # source_peer_number, target_peer_number,
     def wrapped(tox, group_number, mod_peer_id, peer_id, event_type, user_data):
-        if mod_peer_id == iMAX_INT32 or peer_id == iMAX_INT32:
-            # FixMe: known signal to revalidate roles...
-            return
-        LOG_DEBUG(f"group_moderation #{group_number} mod_id={mod_peer_id} peer_id={peer_id} event_type={event_type}")
         group = contacts_provider.get_group_by_number(group_number)
-        mod_peer = group.get_peer_by_id(mod_peer_id)
-        if not mod_peer:
-            #_peers = [(p._name, p._peer_id) for p in group.get_peers()]
-            LOG_TRACE(f"remove_peer group {group!r} has no mod_peer_id={mod_peer_id} in _peers!r")
-            return
-        peer = group.get_peer_by_id(peer_id)
-        if not peer:
-            # FixMe: known signal to revalidate roles...
-            #_peers = [(p._name, p._peer_id) for p in group.get_peers()]
-            LOG_TRACE(f"remove_peer group {group!r} has no peer_id={peer_id} in _peers!r")
-            return
 
         if event_type == TOX_GROUP_MOD_EVENT['KICK']:
             remove_peer(group, mod_peer_id, peer_id, False)
+        elif event_type == TOX_GROUP_MOD_EVENT['BAN']:
+            remove_peer(group, mod_peer_id, peer_id, True)
         elif event_type == TOX_GROUP_MOD_EVENT['OBSERVER']:
             update_peer_role(group, mod_peer_id, peer_id, TOX_GROUP_ROLE['OBSERVER'])
         elif event_type == TOX_GROUP_MOD_EVENT['USER']:
@@ -677,7 +509,6 @@ def group_moderation(groups_service, contacts_provider, contacts_manager, messen
 def group_password(contacts_provider):
 
     def wrapped(tox_link, group_number, password, length, user_data):
-        LOG_DEBUG(f"group_password #{group_number}")
         password = str(password[:length], 'utf-8')
         group = contacts_provider.get_group_by_number(group_number)
         group.password = password
@@ -688,7 +519,6 @@ def group_password(contacts_provider):
 def group_peer_limit(contacts_provider):
 
     def wrapped(tox_link, group_number, peer_limit, user_data):
-        LOG_DEBUG(f"group_peer_limit #{group_number}")
         group = contacts_provider.get_group_by_number(group_number)
         group.peer_limit = peer_limit
 
@@ -698,18 +528,19 @@ def group_peer_limit(contacts_provider):
 def group_privacy_state(contacts_provider):
 
     def wrapped(tox_link, group_number, privacy_state, user_data):
-        LOG_DEBUG(f"group_privacy_state #{group_number}")
         group = contacts_provider.get_group_by_number(group_number)
         group.is_private = privacy_state == TOX_GROUP_PRIVACY_STATE['PRIVATE']
 
     return wrapped
 
+# -----------------------------------------------------------------------------------------------------------------
 # Callbacks - initialization
+# -----------------------------------------------------------------------------------------------------------------
 
 
 def init_callbacks(tox, profile, settings, plugin_loader, contacts_manager,
                    calls_manager, file_transfer_handler, main_window, tray, messenger, groups_service,
-                   contacts_provider, ms=None):
+                   contacts_provider):
     """
     Initialization of all callbacks.
     :param tox: Tox instance
@@ -726,7 +557,6 @@ def init_callbacks(tox, profile, settings, plugin_loader, contacts_manager,
     :param groups_service: GroupsService instance
     :param contacts_provider: ContactsProvider instance
     """
-
     # self callbacks
     tox.callback_self_connection_status(self_connection_status(tox, profile))
 

toxygen/middleware/threads.py

@@ -1,170 +1,115 @@
-# -*- mode: python; indent-tabs-mode: nil; py-indent-offset: 4; coding: utf-8 -*-
+from bootstrap.bootstrap import *
-import sys
 import threading
 import queue
-from  qtpy import QtCore
-
-from bootstrap.bootstrap import *
-from bootstrap.bootstrap import download_nodes_list
-from tox_wrapper.toxcore_enums_and_consts import TOX_USER_STATUS, TOX_CONNECTION
-import tox_wrapper.tests.support_testing as ts
 from utils import util
-
 import time
-sleep = time.sleep
+from PyQt5 import QtCore
 
-# LOG=util.log
-global LOG
-import logging
-LOG = logging.getLogger('app.'+'threads')
-# log = lambda x: LOG.info(x)
-
-def LOG_ERROR(l): print('EROR+ '+l)
-def LOG_WARN(l):  print('WARN+ '+l)
-def LOG_INFO(l):  print('INFO+ '+l)
-def LOG_DEBUG(l): print('DBUG+ '+l)
-def LOG_TRACE(l): pass # print('TRACE+ '+l)
-
-iLAST_CONN = 0
-iLAST_DELTA = 60
 
+# -----------------------------------------------------------------------------------------------------------------
 # Base threads
+# -----------------------------------------------------------------------------------------------------------------
 
 class BaseThread(threading.Thread):
 
-    def __init__(self, name=None, target=None):
+    def __init__(self):
+        super().__init__()
         self._stop_thread = False
-        if name:
-            super().__init__(name=name, target=target)
-        else:
-            super().__init__(target=target)
 
-    def stop_thread(self, timeout=-1):
+    def stop_thread(self):
         self._stop_thread = True
-        if timeout < 0:
+        self.join()
-            timeout = ts.iTHREAD_TIMEOUT
+
-        i = 0
-        while i < ts.iTHREAD_JOINS:
-            self.join(timeout)
-            if not self.is_alive(): break
-            i = i + 1
-        else:
-            LOG_WARN(f"BaseThread {self.name} BLOCKED after {ts.iTHREAD_JOINS}")
 
 class BaseQThread(QtCore.QThread):
 
-    def __init__(self, name=None):
+    def __init__(self):
-        # NO name=name
         super().__init__()
         self._stop_thread = False
-        self.name = str(id(self))
 
-    def stop_thread(self, timeout=-1):
+    def stop_thread(self):
         self._stop_thread = True
-        if timeout < 0:
+        self.wait()
-            timeout = ts.iTHREAD_TIMEOUT
-        i = 0
-        while i < ts.iTHREAD_JOINS:
-            self.wait(timeout)
-            if not self.isRunning(): break
-            i = i + 1
-            sleep(ts.iTHREAD_TIMEOUT)
-        else:
-            LOG_WARN(f"BaseQThread {self.name} BLOCKED")
 
+
+# -----------------------------------------------------------------------------------------------------------------
 # Toxcore threads
+# -----------------------------------------------------------------------------------------------------------------
 
 class InitThread(BaseThread):
 
-    def __init__(self, tox, plugin_loader, settings, app, is_first_start):
+    def __init__(self, tox, plugin_loader, settings, is_first_start):
-        super().__init__(name='InitThread')
+        super().__init__()
-        self._tox = tox
+        self._tox, self._plugin_loader, self._settings = tox, plugin_loader, settings
-        self._plugin_loader = plugin_loader
-        self._settings = settings
-        self._app = app
         self._is_first_start = is_first_start
 
     def run(self):
-        # DBUG+ InitThread run: ERROR name 'ts' is not defined
-        import tox_wrapper.tests.support_testing as ts
-        LOG_DEBUG('InitThread run: ')
-        try:
-            if self._is_first_start and ts.bAreWeConnected() and \
-              self._settings['download_nodes_list']:
-                LOG_INFO(f"downloading list of nodes {self._settings['download_nodes_list']}")
-                download_nodes_list(self._settings, oArgs=self._app._args)
-
-            if ts.bAreWeConnected():
-                LOG_INFO(f"calling test_net nodes")
-                self._app.test_net(oThread=self, iMax=4)
-
         if self._is_first_start:
-                LOG_INFO('starting plugins')
+            # download list of nodes if needed
+            download_nodes_list(self._settings)
+            # start plugins
             self._plugin_loader.load()
 
-        except Exception as e:
+        # bootstrap
-            LOG_DEBUG(f"InitThread run: ERROR {e}")
+        try:
-            pass
+            for data in generate_nodes():
-
-        for _ in range(ts.iTHREAD_JOINS):
                 if self._stop_thread:
                     return
-            sleep(ts.iTHREAD_SLEEP)
+                self._tox.bootstrap(*data)
+                self._tox.add_tcp_relay(*data)
+        except:
+            pass
+
+        for _ in range(10):
+            if self._stop_thread:
                 return
+            time.sleep(1)
+
+        while not self._tox.self_get_connection_status():
+            try:
+                for data in generate_nodes(None):
+                    if self._stop_thread:
+                        return
+                    self._tox.bootstrap(*data)
+                    self._tox.add_tcp_relay(*data)
+            except:
+                pass
+            finally:
+                time.sleep(5)
+
 
 class ToxIterateThread(BaseQThread):
 
-    def __init__(self, tox, app=None):
+    def __init__(self, tox):
         super().__init__()
         self._tox = tox
-        self._app = app
 
     def run(self):
-        LOG_DEBUG('ToxIterateThread run: ')
         while not self._stop_thread:
-            try:
-                iMsec = self._tox.iteration_interval()
             self._tox.iterate()
-            except Exception as e:
+            time.sleep(self._tox.iteration_interval() / 1000)
-                # Fatal Python error: Segmentation fault
-                LOG_ERROR(f"ToxIterateThread run: {e}")
-            else:
-                sleep(iMsec / 1000.0)
-
-            global iLAST_CONN
-            if not iLAST_CONN:
-                iLAST_CONN = time.time()
-            # TRAC> TCP_common.c#203:read_TCP_packet recv buffer has 0 bytes, but requested 10 bytes
-
-            # and segv
-            if time.time() - iLAST_CONN > iLAST_DELTA and \
-                ts.bAreWeConnected() and \
-                self._tox.self_get_status() == TOX_USER_STATUS['NONE'] and \
-                self._tox.self_get_connection_status() == TOX_CONNECTION['NONE']:
-                    iLAST_CONN = time.time()
-                    LOG_INFO(f"ToxIterateThread calling test_net")
-                    invoke_in_main_thread(
-                        self._app.test_net, oThread=self, iMax=2)
 
 
 class ToxAVIterateThread(BaseQThread):
+
     def __init__(self, toxav):
         super().__init__()
         self._toxav = toxav
 
     def run(self):
-        LOG_DEBUG('ToxAVIterateThread run: ')
         while not self._stop_thread:
             self._toxav.iterate()
-            sleep(self._toxav.iteration_interval() / 1000)
+            time.sleep(self._toxav.iteration_interval() / 1000)
 
 
+# -----------------------------------------------------------------------------------------------------------------
 # File transfers thread
+# -----------------------------------------------------------------------------------------------------------------
 
 class FileTransfersThread(BaseQThread):
 
     def __init__(self):
-        super().__init__('FileTransfers')
+        super().__init__()
         self._queue = queue.Queue()
         self._timeout = 0.01
 
@@ -179,12 +124,14 @@ class FileTransfersThread(BaseQThread):
             except queue.Empty:
                 pass
             except queue.Full:
-                LOG_WARN('Queue is full in _thread')
+                util.log('Queue is full in _thread')
             except Exception as ex:
-                LOG_ERROR('in _thread: ' + str(ex))
+                util.log('Exception in _thread: ' + str(ex))
 
 
 _thread = FileTransfersThread()
+
+
 def start_file_transfer_thread():
     _thread.start()
 
@@ -197,7 +144,9 @@ def execute(func, *args, **kwargs):
     _thread.execute(func, *args, **kwargs)
 
 
+# -----------------------------------------------------------------------------------------------------------------
 # Invoking in main thread
+# -----------------------------------------------------------------------------------------------------------------
 
 class InvokeEvent(QtCore.QEvent):
     EVENT_TYPE = QtCore.QEvent.Type(QtCore.QEvent.registerEventType())

toxygen/middleware/tox_factory.py

@@ -1,56 +1,27 @@
-# -*- mode: python; indent-tabs-mode: nil; py-indent-offset: 4; coding: utf-8 -*-
-
-import ctypes
-import traceback
-import os
-from ctypes import *
-
 import user_data.settings
-import tox_wrapper.tox
+import wrapper.tox
-import tox_wrapper.toxcore_enums_and_consts as enums
+import wrapper.toxcore_enums_and_consts as enums
-from tox_wrapper.tests import support_testing as ts
+import ctypes
-# callbacks can be called in any thread so were being careful
-# tox.py can be called by callbacks
-from tox_wrapper.tests.support_testing import LOG_ERROR, LOG_WARN, LOG_INFO, LOG_DEBUG, LOG_TRACE
-
-global LOG
-import logging
-LOG = logging.getLogger('app.'+'tox_factory')
-
-from utils import util
-from utils import ui as util_ui
 
 
-def tox_factory(data=None, settings=None, args=None, app=None):
+def tox_factory(data=None, settings=None):
     """
     :param data: user data from .tox file. None = no saved data, create new profile
     :param settings: current profile settings. None = default settings will be used
     :return: new tox instance
     """
-    if not settings:
+    if settings is None:
-        LOG_WARN("tox_factory using get_default_settings")
         settings = user_data.settings.Settings.get_default_settings()
-    else:
-        user_data.settings.clean_settings(settings)
 
-    try:
+    tox_options = wrapper.tox.Tox.options_new()
-        tox_options = tox_wrapper.tox.Tox.options_new()
-        tox_options.contents.ipv6_enabled = settings['ipv6_enabled']
     tox_options.contents.udp_enabled = settings['udp_enabled']
-        tox_options.contents.proxy_type = int(settings['proxy_type'])
+    tox_options.contents.proxy_type = settings['proxy_type']
-        if type(settings['proxy_host']) == str:
     tox_options.contents.proxy_host = bytes(settings['proxy_host'], 'UTF-8')
-        elif type(settings['proxy_host']) == bytes:
+    tox_options.contents.proxy_port = settings['proxy_port']
-            tox_options.contents.proxy_host = settings['proxy_host']
-        else:
-            tox_options.contents.proxy_host = b''
-        tox_options.contents.proxy_port = int(settings['proxy_port'])
     tox_options.contents.start_port = settings['start_port']
     tox_options.contents.end_port = settings['end_port']
     tox_options.contents.tcp_port = settings['tcp_port']
-        tox_options.contents.local_discovery_enabled = settings['local_discovery_enabled']
+    tox_options.contents.local_discovery_enabled = settings['lan_discovery']
-        tox_options.contents.dht_announcements_enabled = settings['dht_announcements_enabled']
-        tox_options.contents.hole_punching_enabled = settings['hole_punching_enabled']
     if data:  # load existing profile
         tox_options.contents.savedata_type = enums.TOX_SAVEDATA_TYPE['TOX_SAVE']
         tox_options.contents.savedata_data = ctypes.c_char_p(data)
@@ -60,30 +31,4 @@ def tox_factory(data=None, settings=None, args=None, app=None):
         tox_options.contents.savedata_data = None
         tox_options.contents.savedata_length = 0
 
-        # overrides
+    return wrapper.tox.Tox(tox_options)
-        tox_options.contents.local_discovery_enabled = False
-        tox_options.contents.ipv6_enabled = False
-        tox_options.contents.hole_punching_enabled = False
-
-        LOG.debug("tox_wrapper.tox.Tox settings: " +repr(settings))
-
-        if 'trace_enabled' in settings and not settings['trace_enabled']:
-            LOG_DEBUG("settings['trace_enabled' disabled" )
-        elif tox_options._options_pointer and \
-          'trace_enabled' in settings and settings['trace_enabled']:
-            ts.vAddLoggerCallback(tox_options)
-            LOG_INFO("c-toxcore trace_enabled enabled" )
-        else:
-            LOG_WARN("No tox_options._options_pointer to add self_logger_cb" )
-
-        retval = tox_wrapper.tox.Tox(tox_options)
-    except Exception as e:
-        if app and hasattr(app, '_log'):
-            pass
-        LOG_ERROR(f"tox_wrapper.tox.Tox failed:  {e}")
-        LOG_WARN(traceback.format_exc())
-        raise
-
-    if app and hasattr(app, '_log'):
-        app._log("DEBUG: tox_wrapper.tox.Tox succeeded")
-    return retval

toxygen/network/tox_dns.py

@@ -1,40 +1,20 @@
-# -*- mode: python; indent-tabs-mode: nil; py-indent-offset: 4; coding: utf-8 -*-
 import json
 import urllib.request
 import utils.util as util
-from  qtpy import QtNetwork, QtCore
+from PyQt5 import QtNetwork, QtCore
-try:
-    import requests
-except ImportError:
-    requests = None
 
-global LOG
-import logging
-LOG = logging.getLogger('app.'+__name__)
 
 class ToxDns:
 
-    def __init__(self, settings, log=None):
+    def __init__(self, settings):
         self._settings = settings
-        self._log = log
 
     @staticmethod
     def _send_request(url, data):
-        if requests:
-            LOG.info('send_request loading with requests: ' + str(url))
-            headers = dict()
-            headers['Content-Type'] = 'application/json'
-            req = requests.get(url, headers=headers)
-            if req.status_code < 300:
-                retval = req.content
-            else:
-                raise LookupError(str(req.status_code))
-        else:
         req = urllib.request.Request(url)
         req.add_header('Content-Type', 'application/json')
         response = urllib.request.urlopen(req, bytes(json.dumps(data), 'utf-8'))
-            retval = response.read()
+        res = json.loads(str(response.read(), 'utf-8'))
-        res = json.loads(str(retval, 'utf-8'))
         if not res['c']:
             return res['tox_id']
         else:
@@ -49,25 +29,12 @@ class ToxDns:
         site = email.split('@')[1]
         data = {"action": 3, "name": "{}".format(email)}
         urls = ('https://{}/api'.format(site), 'http://{}/api'.format(site))
-        if requests:
+        if not self._settings['proxy_type']:  # no proxy
-            for url in urls:
-                LOG.info('TOX nodes loading with requests: ' + str(url))
-                try:
-                    headers = dict()
-                    headers['Content-Type'] = 'application/json'
-                    req = requests.get(url, headers=headers)
-                    if req.status_code < 300:
-                        result = req.content
-                        return result
-                except Exception as ex:
-                    LOG.error('ERROR: TOX DNS loading error with requests: ' + str(ex))
-
-        elif not self._settings['proxy_type']:  # no proxy
             for url in urls:
                 try:
                     return self._send_request(url, data)
                 except Exception as ex:
-                    LOG.error('ERROR: TOX DNS ' + str(ex))
+                    util.log('TOX DNS ERROR: ' + str(ex))
         else:  # proxy
             netman = QtNetwork.QNetworkAccessManager()
             proxy = QtNetwork.QNetworkProxy()
@@ -93,6 +60,6 @@ class ToxDns:
                     if not result['c']:
                         return result['tox_id']
                 except Exception as ex:
-                    LOG.error('ERROR: TOX DNS ' + str(ex))
+                    util.log('TOX DNS ERROR: ' + str(ex))
 
         return None  # error

toxygen/notifications/sound.py

@@ -1,14 +1,8 @@
-import os.path
 import utils.util
 import wave
-
-import tox_wrapper.tests.support_testing as ts
-with ts.ignoreStderr():
 import pyaudio
+import os.path
 
-global LOG
-import logging
-LOG = logging.getLogger('app.'+__name__)
 
 SOUND_NOTIFICATION = {
     'MESSAGE': 0,
@@ -31,19 +25,9 @@ class AudioFile:
 
     def play(self):
         data = self.wf.readframes(self.chunk)
-        try:
         while data:
             self.stream.write(data)
             data = self.wf.readframes(self.chunk)
-        except Exception as e:
-            LOG.error(f"Error during AudioFile play {e!s}")
-            LOG.debug("Error during AudioFile play " \
-                      +' rate=' +str(self.wf.getframerate()) \
-                      + 'format=' +str(self.p.get_format_from_width(self.wf.getsampwidth())) \
-                      +' channels=' +str(self.wf.getnchannels()) \
-                      )
-
-            raise
 
     def close(self):
         self.stream.close()

toxygen/notifications/tray.py

@@ -1,4 +1,4 @@
-from  qtpy import QtCore, QtWidgets
+from PyQt5 import QtCore, QtWidgets
 
 
 def tray_notification(title, text, tray, window):
@@ -10,7 +10,7 @@ def tray_notification(title, text, tray, window):
     :param tray: ref to tray icon
     :param window: main window
     """
-    if tray and QtWidgets.QSystemTrayIcon.isSystemTrayAvailable():
+    if QtWidgets.QSystemTrayIcon.isSystemTrayAvailable():
         if len(text) > 30:
             text = text[:27] + '...'
         tray.showMessage(title, text, QtWidgets.QSystemTrayIcon.NoIcon, 3000)

toxygen/plugin_support/plugin_support.py

@@ -1,4 +1,3 @@
-# -*- mode: python; indent-tabs-mode: nil; py-indent-offset: 4; coding: utf-8 -*-
 import utils.util as util
 import os
 import importlib
@@ -6,14 +5,6 @@ import inspect
 import plugins.plugin_super_class as pl
 import sys
 
-# LOG=util.log
-global LOG
-import logging
-LOG = logging.getLogger('plugin_support')
-def trace(msg, *args, **kwargs): LOG._log(0, msg, [])
-LOG.trace = trace
-
-log = lambda x: LOG.info(x)
 
 class Plugin:
 
@@ -55,49 +46,38 @@ class PluginLoader:
         """
         path = util.get_plugins_directory()
         if not os.path.exists(path):
-            self._app._LOG('WARN: Plugin directory not found: ' + path)
+            util.log('Plugin dir not found')
             return
-
+        else:
             sys.path.append(path)
         files = [f for f in os.listdir(path) if os.path.isfile(os.path.join(path, f))]
         for fl in files:
             if fl in ('plugin_super_class.py', '__init__.py') or not fl.endswith('.py'):
                 continue
-            base_name = fl[:-3]  # module name without .py
+            name = fl[:-3]  # module name without .py
             try:
-                module = importlib.import_module(base_name)  # import plugin
+                module = importlib.import_module(name)  # import plugin
-                LOG.trace('Imported module: ' +base_name +' file: ' +fl)
+            except ImportError:
-            except ImportError as e:
+                util.log('Import error in module ' + name)
-                LOG.warn(f"Import error:  {e}" +' file: ' +fl)
                 continue
             except Exception as ex:
-                LOG.error('importing ' + base_name + ' Exception: ' + str(ex))
+                util.log('Exception in module ' + name + ' Exception: ' + str(ex))
                 continue
             for elem in dir(module):
                 obj = getattr(module, elem)
                 # looking for plugin class in module
                 if not inspect.isclass(obj) or not hasattr(obj, 'is_plugin') or not obj.is_plugin:
                     continue
+                print('Plugin', elem)
                 try:  # create instance of plugin class
-                    instance = obj(self._app) # name, short_name, app
+                    instance = obj(self._app)
-                    # needed by bday...
+                    is_active = instance.get_short_name() in self._settings['plugins']
-                    instance._profile=self._app._ms._profile
-                    instance._settings=self._settings
-                    short_name = instance.get_short_name()
-                    is_active = short_name in self._settings['plugins']
                     if is_active:
-                        try:
                         instance.start()
-                            self._app._log('INFO: Started Plugin ' +short_name)
-                        except Exception as e:
-                            self._app._log.error(f"Starting Plugin ' +short_name +'  {e}")
-                    # else: LOG.info('Defined Plugin ' +short_name)
                 except Exception as ex:
-                    LOG.error('in module ' + short_name + ' Exception: ' + str(ex))
+                    util.log('Exception in module ' + name + ' Exception: ' + str(ex))
                     continue
-                short_name = instance.get_short_name()
+                self._plugins[instance.get_short_name()] = Plugin(instance, is_active)
-                self._plugins[short_name] = Plugin(instance, is_active)
-                LOG.info('Added plugin: ' +short_name +' from file: ' +fl)
                 break
 
     def callback_lossless(self, friend_number, data):
@@ -146,13 +126,7 @@ class PluginLoader:
         """
         Return window or None for specified plugin
         """
-        try:
-            if key in self._plugins and hasattr(self._plugins[key], 'instance'):
         return self._plugins[key].instance.get_window()
-        except Exception as e:
-            self._app._log('WARN: ' +key +' _plugins no slot instance: ' +str(e))
-
-        return None
 
     def toggle_plugin(self, key):
         """
@@ -188,7 +162,6 @@ class PluginLoader:
         for plugin in self._plugins.values():
             if not plugin.is_active:
                 continue
-
             try:
                 result.extend(plugin.instance.get_menu(num))
             except:
@@ -200,10 +173,6 @@ class PluginLoader:
         for plugin in self._plugins.values():
             if not plugin.is_active:
                 continue
-            if not hasattr(plugin.instance, 'get_message_menu'):
-                name = plugin.instance.get_short_name()
-                self._app._log('WARN: get_message_menu not found: ' + name)
-                continue
             try:
                 result.extend(plugin.instance.get_message_menu(menu, selected_text))
             except:
@@ -220,11 +189,6 @@ class PluginLoader:
             del self._plugins[key]
 
     def reload(self):
-        path = util.get_plugins_directory()
+        print('Reloading plugins')
-        if not os.path.exists(path):
-            self._app._log('WARN: Plugin directory not found: ' + path)
-            return
-
         self.stop()
-        self._app._log('INFO: Reloading plugins from ' +path)
         self.load()

toxygen/plugins/README.md

@@ -1,27 +0,0 @@
-# Plugins
-
-Repo with plugins for [Toxygen](https://macaw.me/emdee/toxygen/)
-
-For more info visit [plugins.md](https://macaw.me/emdee/toxygen/blob/master/docs/plugins.md) and [plugin_api.md](https://github.com/toxygen-project[/toxygen/blob/master/docs/plugin-api.md)
-
-# Plugins list:
-
-- ToxId - share your Tox ID and copy friend's Tox ID easily.
-- MarqueeStatus - create ticker from your status message.
-- BirthDay - get notifications on your friends' birthdays.
-- Bot - bot which can communicate with your friends when you are away.
-- SearchPlugin - select text in message and find it in search engine.
-- AutoAwayStatusLinux - sets "Away" status when user is inactive (Linux only).
-- AutoAwayStatusWindows - sets "Away" status when user is inactive (Windows only).
-- Chess - play chess with your friends using Tox.
-- Garland - changes your status like it's garland.
-- AutoAnswer - calls auto answering.
-- uToxInlineSending - send inlines with the same name as uTox does.
-- AvatarEncryption - encrypt all avatars using profile password
-
-## Hard fork
-
-Not all of these are working...
-
-Work on this project is suspended until the
-[MultiDevice](https://git.plastiras.org/emdee/tox_profile/wiki/MultiDevice-Announcements-POC) problem is solved. Fork me!

toxygen/plugins/ae.py

@@ -1,83 +0,0 @@
-# -*- mode: python; indent-tabs-mode: nil; py-indent-offset: 4; coding: utf-8 -*-
-import plugin_super_class
-import json
-from user_data import settings
-import os
-from bootstrap.bootstrap import get_user_config_path
-
-class AvatarEncryption(plugin_super_class.PluginSuperClass):
-
-    def __init__(self, *args):
-        super(AvatarEncryption, self).__init__('AvatarEncryption', 'ae', *args)
-        self._path = os.path.join(get_user_config_path(), 'avatars')
-        self._app = args[0]
-        self._profile = self._app._ms._profile
-        self._window = None
-        #was self._contacts = self._profile._contacts[:]
-        self._contacts = self._profile._contacts_provider.get_all_friends()
-
-    def get_description(self):
-        return QApplication.translate("AvatarEncryption", 'Encrypt all avatars using profile password.')
-
-    def close(self):
-        if not self._encrypt_save.has_password():
-            return
-        i, data = 1, {}
-
-        self.save_contact_avatar(data, self._profile, 0)
-        for friend in self._contacts:
-            self.save_contact_avatar(data, friend, i)
-            i += 1
-        self.save_settings(json.dumps(data))
-
-    def start(self):
-        if not self._encrypt_save.has_password():
-            return
-        data = json.loads(self.load_settings())
-
-        self.load_contact_avatar(data, self._profile)
-        for friend in self._contacts:
-            self.load_contact_avatar(data, friend)
-        self._profile.update()
-
-    def save_contact_avatar(self, data, contact, i):
-        tox_id = contact.tox_id[:64]
-        data[str(tox_id)] = str(i)
-        path = os.path.join(self._path, tox_id + '.png')
-        if os.path.isfile(path):
-            with open(path, 'rb') as fl:
-                avatar = fl.read()
-            encr_avatar = self._encrypt_save.pass_encrypt(avatar)
-            with open(os.path.join(self._path, self._settings.name + '_' + str(i) + '.png'), 'wb') as fl:
-                fl.write(encr_avatar)
-            os.remove(path)
-
-    def load_contact_avatar(self, data, contact):
-        tox_id = str(contact.tox_id[:64])
-        if tox_id not in data:
-            return
-        path = os.path.join(self._path, self._settings.name + '_' + data[tox_id] + '.png')
-        if os.path.isfile(path):
-            with open(path, 'rb') as fl:
-                avatar = fl.read()
-            decr_avatar = self._encrypt_save.pass_decrypt(avatar)
-            with open(os.path.join(self._path, str(tox_id) + '.png'), 'wb') as fl:
-                fl.write(decr_avatar)
-            os.remove(path)
-            contact.load_avatar()
-
-    def load_settings(self):
-        try:
-            with open(plugin_super_class.path_to_data(self._short_name) + self._settings.name + '.json', 'rb') as fl:
-                data = fl.read()
-            return str(self._encrypt_save.pass_decrypt(data), 'utf-8') if data else '{}'
-        except:
-            return '{}'
-
-    def save_settings(self, data):
-        try:
-            data = self._encrypt_save.pass_encrypt(bytes(data, 'utf-8'))
-            with open(plugin_super_class.path_to_data(self._short_name) + self._settings.name + '.json', 'wb') as fl:
-                fl.write(data)
-        except:
-            pass

toxygen/plugins/awayl.py

@@ -1,111 +0,0 @@
-import plugin_super_class
-import threading
-import time
-from  qtpy import QtCore, QtWidgets
-from subprocess import check_output
-import json
-
-
-class InvokeEvent(QtCore.QEvent):
-    EVENT_TYPE = QtCore.QEvent.Type(QtCore.QEvent.registerEventType())
-
-    def __init__(self, fn, *args, **kwargs):
-        QtCore.QEvent.__init__(self, InvokeEvent.EVENT_TYPE)
-        self.fn = fn
-        self.args = args
-        self.kwargs = kwargs
-
-
-class Invoker(QtCore.QObject):
-
-    def event(self, event):
-        event.fn(*event.args, **event.kwargs)
-        return True
-
-_invoker = Invoker()
-
-
-def invoke_in_main_thread(fn, *args, **kwargs):
-    QtCore.QCoreApplication.postEvent(_invoker, InvokeEvent(fn, *args, **kwargs))
-
-
-class AutoAwayStatusLinux(plugin_super_class.PluginSuperClass):
-
-    def __init__(self, *args):
-        super().__init__('AutoAwayStatusLinux', 'awayl', *args)
-        self._thread = None
-        self._exec = None
-        self._active = False
-        self._time = json.loads(self.load_settings())['time']
-        self._prev_status = 0
-        self._app = args[0]
-        self._profile=self._app._ms._profile
-        self._window = None
-
-    def get_description(self):
-        return QApplication.translate("AutoAwayStatusLinux", 'sets "Away" status when user is inactive (Linux only).')
-
-    def close(self):
-        self.stop()
-
-    def stop(self):
-        self._exec = False
-        if self._active:
-            self._thread.join()
-
-    def start(self):
-        self._exec = True
-        self._thread = threading.Thread(target=self.loop)
-        self._thread.start()
-
-    def save(self):
-        self.save_settings('{"time": ' + str(self._time) + '}')
-
-    def change_status(self, status=1):
-        if self._profile.status in (0, 2):
-            self._prev_status = self._profile.status
-        if status is not None:
-            invoke_in_main_thread(self._profile.set_status, status)
-
-    def get_window(self):
-        inst = self
-
-        class Window(QtWidgets.QWidget):
-            def __init__(self):
-                super(Window, self).__init__()
-                self.setGeometry(QtCore.QRect(450, 300, 350, 100))
-                self.label = QtWidgets.QLabel(self)
-                self.label.setGeometry(QtCore.QRect(20, 0, 310, 35))
-                self.label.setText(QtWidgets.QApplication.translate("AutoAwayStatusLinux", "Auto away time in minutes\n(0 - to disable)"))
-                self.time = QtWidgets.QLineEdit(self)
-                self.time.setGeometry(QtCore.QRect(20, 40, 310, 25))
-                self.time.setText(str(inst._time))
-                self.setWindowTitle("AutoAwayStatusLinux")
-                self.ok = QtWidgets.QPushButton(self)
-                self.ok.setGeometry(QtCore.QRect(20, 70, 310, 25))
-                self.ok.setText(
-                    QtWidgets.QApplication.translate("AutoAwayStatusLinux", "Save"))
-                self.ok.clicked.connect(self.update)
-
-            def update(self):
-                try:
-                    t = int(self.time.text())
-                except:
-                    t = 0
-                inst._time = t
-                inst.save()
-                self.close()
-
-        return Window()
-
-    def loop(self):
-        self._active = True
-        while self._exec:
-            time.sleep(5)
-            d = check_output(['xprintidle'])
-            d = int(d) // 1000
-            if self._time:
-                if d > 60 * self._time:
-                    self.change_status()
-                elif self._profile.status == 1:
-                    self.change_status(self._prev_status)