search

justinpenner / TalkingLeaves

A GlyphsApp plugin to help you explore the world’s languages and writing systems
MIT License
27 stars 2 forks source link

crash #20

Closed justanotherfoundry closed 1 month ago

justanotherfoundry commented 1 month ago

The Plugin TalkingLeaves.glyphsPlugin has caused a crash. Please update it by opening the Plugin Manager or re-downloading it from the original source.

Details: Traceback (most recent call last):

File "vanillaBase.py", line 539, in action_ self.callback(sender)

File "TalkingLeaves.py", line 744, in showUnsupportedCallback self.refreshLangs(sender)

File "TalkingLeaves.py", line 389, in refreshLangs self.currentScript = self.scriptsTable.get()[self.scriptsTable.getSelectedIndexes()[0]]['Script']



IndexError: list index out of range

App:3.3-3309 Plugin:(null)-(null) System:14.5.0
justinpenner commented 1 month ago

I'm not sure what's causing this, as it seems to indicate that the scripts table has nothing selected, which shouldn't be possible unless line 389 is somehow being run before the table loads. In either case 4506391 should fix it since we're no longer assuming that scriptsTable always has one item selected.

justanotherfoundry commented 1 month ago

Seems like this error occurs because the script aborts the method after the other crash I reported, which means some of the code has not been executed. The dialogue does show, however, but with some initialization missing, I guess. In other words, the fix for the other crash also fixes this one.