Kodi Community Forum
Script.Favorites error - Printable Version

+- Kodi Community Forum (http://forum.kodi.tv)
+-- Forum: Development (/forumdisplay.php?fid=32)
+--- Forum: Python Add-on Development (/forumdisplay.php?fid=26)
+--- Thread: Script.Favorites error (/showthread.php?tid=78892)



Script.Favorites error - mcborzu - 2010-08-13 04:41

This seems to be the relevant section:

PHP Code:
22:04:16 T:4016 M:1232875520    INFO: -->Python script returned the following error<--
22:04:16 T:4016 M:1232875520   ERRORError Typeexceptions.SystemError
22
:04:16 T:4016 M:1232875520   ERRORError ContentsErrorWindow is NULLthis is not possible :-)
22:04:16 T:4016 M:1232863232   ERRORTraceback (most recent call last):
                                              
File "C:\Program Files\XBMC\addons\script.favourites\default.py"line 58in ?
                                                
Main()
                                              
File "C:\Program Files\XBMC\addons\script.favourites\default.py"line 17in __init__
                                                dialog
.close()
                                            
SystemErrorErrorWindow is NULLthis is not possible :-)
22:04:16 T:4016 M:1232863232    INFO: -->End of Python script error report<-- 


Heres the whole PASTEBIN, and link to script.favourites if needed

The script works it's just spitting out an error which in turns activates the DialogKiaToast warning. I'll need a little help as I barely know any python, thx...


Deleting line 14 works, stops the error and script still works:
PHP Code:
dialog.close() 
Just don't if it's kosher to actually delete that line.


- spiff - 2010-08-13 11:02

was the dialog ever opened? i have a feeling this is caused by sloppy scripting.


- mcborzu - 2010-08-13 15:15

Again I don't know much about python but it used to run without error. I traced it back to builds 32393 and above spits out that error if it helps....

Or if just deleting
PHP Code:
dialog.close() 
works I'll just go with that...


- spiff - 2010-08-13 15:33

yes, we changed from silently ignoring such malusage (calling methods in uninited ui elements) to actually tossing the errors. removing that line is probably not the proper solution, since i have a feeling it might be needed under certain conditions...


- spiff - 2010-08-13 16:00

can you please verify http://www.math.ntnu.no/~arnemort/stuf/fixfavorites.diff

fuck it, it's so trivial, i pushed it.