deluge gtk no options working

[renato]

with the gtk code move it's not possible to open any option and neither close or shutdown from the menu or from the gtk icons. the pref win is not loaading and no option is working, the icon is the only way to close it

[s0undt3ch]

Might be fixed in current master. Please report back.

[renato]

No, still not working.

[Calum]

broken options

[Calum]

Firefox if you are going to comment on tickets please be more descriptive than 'doesn't work' as it is not helpful.

I have attached a screenshot with specific places that still need fixed.

I found that messing with options will delete torrents from deluge, not ideal!

also this error message appeared:

15:20:16.208 [ERROR   ][deluge.core.eventmanager               :61  ] Event handler TorrentStateChangedEvent failed in <bound method TorrentView.on_torrentstatechanged_event of <deluge.ui.gtkui.torrentview.TorrentView object at 0x1a8b950>>

[s0undt3ch]

Replying to Cas:

Firefox if you are going to comment on tickets please be more descriptive than 'doesn't work' as it is not helpful.

I have attached a screenshot with specific places that still need fixed.

I found that messing with options will delete torrents from deluge, not ideal!

also this error message appeared:

15:20:16.208 [ERROR   ][deluge.core.eventmanager               :61  ] Event handler TorrentStateChangedEvent failed in <bound method TorrentView.on_torrentstatechanged_event of <deluge.ui.gtkui.torrentview.TorrentView object at 0x1a8b950>>

Regarding the options, yes, I found that out already, another thing lost in the migration, or different behaviour between libglade and gtkbuilder.

Regarding the other issue, I'll investigate.

Anyway, this is when I find the time for it in the next few days, maybe tomorrow. I already fixed the .ui file locally, just need to fix the event handling in deluge's gtk code.

Firefox, regarding your issue with the toolbar not working, I really don't know what's up with your setup since neither me or Cas can reproduce it, ie, it's working for both of us.

[s0undt3ch]

Ok, since the work was partially done, let's see if it's fixed. Just pull from master and try.

Firefox:

git fetch
git checkout 4daa7e24706d6db8e2513e7e37b57d8683e78256

[s0undt3ch]

Still need to investigate signals regarding both changing value by hand, manual typing a number, and, changing value by clicking arrows in spinbutton.

[renato]

Really i dont like to say but on this side nothing is changed,i pulled it down and installed it, but nothing changed seems something is screwing it up,but what...423431105096535d6d0b9fe352441058bfa9cc24 is fine so cant be anything to do with the os.

[s0undt3ch]

Some script searching for previous and current signal handlers:

import os, re

PREV = '/tmp/deluge/deluge/ui/gtkui/glade/'
CURR = '/home/vampas/projects/Deluge/deluge/ui/gtkui/glade/'

PREV_HANDLERS = []
CURR_HANDLERS = []

signal_re = re.compile(r'(?:<signal name=")(?P<name>.*)" handler="(?P<handler>.*)"/>')

for fname in os.listdir(PREV):
    if not fname.endswith('.glade'):
        continue

    for line in open(os.path.join(PREV, fname)).readlines():
        match = signal_re.search(line)
        if not match:
            continue

        PREV_HANDLERS.append(match.group('handler'))
#        print '   Signal Name:', match.group('name')
#        print 'Signal Handler:', match.group('handler')

for fname in os.listdir(PREV):
    if not fname.endswith('.glade'):
        continue

    for line in open(os.path.join(PREV, fname)).readlines():
        match = signal_re.search(line)
        if not match:
            continue

        CURR_HANDLERS.append(match.group('handler'))
#        print '   Signal Name:', match.group('name')
#        print 'Signal Handler:', match.group('handler')


print "Previous handlers count", len(PREV_HANDLERS), "... Removing duplicates", len(set(PREV_HANDLERS))
print "Current handlers count", len(CURR_HANDLERS), "... Removing duplicates", len(set(CURR_HANDLERS))

print "All previous signals are handled in current code?", sorted(PREV_HANDLERS) == sorted(CURR_HANDLERS)

print set(PREV_HANDLERS).difference(CURR_HANDLERS)

which outputs:

Previous handlers count 165 ... Removing duplicates 121
Current handlers count 165 ... Removing duplicates 121
All previous signals are handled in current code? True
set([])

So, it should be fixed now. Not closing it yet, need to investigate Cas's issue reported here.

[s0undt3ch]

I wasn't able to reproduce Cas's issue.

Cas, I'm closing this ticket, if this still happens on your end, please re-open a new one if you find that it's not that related to what was handled in this ticket.

Fixed in current master.

[Calum]

Milestone renamed