Faulty parsing of config file

[scorp]

Hi,

I'm using minimal system with openbox and I've tested oblogout yesterday.
I found a bug in parsing config file (commands part). I've never used Python
yet, but I could find a faulty line, so here is a bugfix:

In oblogout/__init__.py line 225
- if key[1] in ['logout', 'restart', 'shutdown', 'suspend', 'hibernate', 'safesuspend', 'lock', 'switch']:
---
+ if key[0] in ['logout', 'restart', 'shutdown', 'suspend', 'hibernate', 'safesuspend', 'lock', 'switch']:

[Andrew Williams (nikdoof)]

Interesting, what distribution are you using? What version of Python is included within that?

[scorp (scorp)]

I'm using ArchLinux with Python 2.6. I have no gnome, so I wanted to change commands (xlock, pm-suspend and s.o.). It didn't work, so I've looked in the accordant code.

One more bug in the same area is, faulty commands execution. I didn't have time yet to
look deeper, but I set own suspend and xlock command, but if I select suspend in oblogout, xlock is started.

[scorp (scorp)]

Forget the last comment about a bug. I've looked through the code and I see, that lock is
executed on every selection of suspend, hibernate or lock. So the behavior is OK, it's just
the dbus suspend problem. But the faulty line in the configuration reading remains. I've looked in the repo for the line, before I notified the bug, it's still not fixed.

[Andrew Williams (nikdoof)]

Commited in 0.3, rev 96