[Deluge] #3538: Adding host in console-UI fails.

[Deluge]

#3538: Adding host in console-UI fails.
----------------------------+------------------------
Reporter: mhertz | Type: bug
Status: new | Priority: minor
Milestone: needs verified | Component: Console UI
Version: develop | Keywords:
----------------------------+------------------------
As title, currently it's not possible to add new host in console-UI as
states port need be integer.

I added some debug-prints and found the 'result' dict indeed saves port as
string, e.g. I tested it worked here just changing line 107 of
connectionmanager.py from/to:
{{{
result['port']['value'],
}}}
{{{
int(result['port']['value']),
}}}
Also, currently when having input new host data and pressed return then
need also press escape to get out of connection-manager, which i just
changed in same file line 135 from/to:
{{{
self.update_select_host_popup()
}}}
{{{
self.pop_popup()
}}}
Thanks in advance.

--
Ticket URL: <https://dev.deluge-torrent.org/ticket/3538>
Deluge <https://deluge-torrent.org/>
Deluge Project

[Deluge]

#3538: Adding host in console-UI fails.

-------------------------+----------------------------
Reporter: mhertz | Owner:

Type: bug | Status: new
Priority: minor | Milestone: needs verified
Component: Console UI | Version: develop

Resolution: | Keywords:
-------------------------+----------------------------

Comment (by DjLegolas):

PR: https://github.com/deluge-torrent/deluge/pull/393

--
Ticket URL: <https://dev.deluge-torrent.org/ticket/3538#comment:1>

[Deluge]

#3538: Adding host in console-UI fails.

-------------------------+---------------------
Reporter: mhertz | Owner:
Type: bug | Status: closed
Priority: minor | Milestone: 2.1.2

Component: Console UI | Version: develop

Resolution: Fixed | Keywords:
-------------------------+---------------------
Changes (by Cas):

* status: new => closed
* resolution: => Fixed
* milestone: needs verified => 2.1.2

Comment:

Fixed [543fce4f2]

--
Ticket URL: <https://dev.deluge-torrent.org/ticket/3538#comment:2>