Page 1 of 1

onNotification interface change

Posted: Thursday 16 February 2017 14:42
by Dnpwwo
I've completed work on sending notifications through to plugins so that they can handle them if they want to in the next beta.

Although no plugins are using it yet, some do have an 'onNotification' function defined that could not get an error like:

Code: Select all

 2017-02-16 21:03:12.040 Error: (Test Kodi) 'onNotification' failed 'TypeError':'onNotification() takes 1 positional argument but 7 were given'.
because I had to change the interface.

The plugins will still work as before, but to remove the error simply remove the old onNotification call back from the plugin or change the definition to be:

Code: Select all

def onNotification(Name, Subject, Text, Status, Priority, Sound, ImageFile):