Missing arguments in some event handler functions

[Samuel Mehrbrodt]

I recently upgraded to Ubuntu 14.04 (Python 3.4, Qt 4.8, PyQt 4.10) and now get exceptions on certain actions.
For example when sending a song to live, I get the following exception:

Traceback (most recent call last):
  File "/home/samuel/programming/openlp/trunk/openlp/core/common/openlpmixin.py", line 67, in wrapped
    raise e
  File "/home/samuel/programming/openlp/trunk/openlp/core/common/openlpmixin.py", line 63, in wrapped
    return func(*args, **kwargs)
TypeError: on_go_live() takes 1 positional argument but 2 were given

The same happens with some other functions that are bound to Qt Events and don't expect any parameters.

[Samuel Mehrbrodt (sam92)]

This happens in trunk btw.

[Tim Bentley (trb143)]

just add "field=none" to the method. That fixes it.

[Samuel Mehrbrodt (sam92)]

Hm this happens only when I start OpenLP with "./openlp.py -l debug".
Without the debug option I don't have any problems.

Any idea why?

[Tim Bentley (trb143)]

No it has baffled. me.

OpenLPMixin dynamically adds debug wrappers to all (except black list) methods in the class.
No more Log.debug.

This only happens if debug is enabled. This works fine for all methods except pyqt ones which need the extra parameter.
You must have found the odd one I messed in testing.

[Erik Lundin (erik-lundin)]

The problem seems to be that the signal QAction::triggered emits with one argument (checked):

http://qt-project.org/doc/qt-4.8/qaction.html#triggered