search

OpenPLi / enigma2

Framebuffer based frontend for DVB functions on Linux settop boxes
GNU General Public License v2.0
151 stars 203 forks source link

[ConfigList] sanity check use dummyConfigActions #4041

Closed Dima73 closed 1 month ago

Dima73 commented 1 month ago

File "/usr/lib/enigma2/python/Plugins/SystemPlugins/SatipClient/plugin.py", line 387, in DiscoveryStart File "/usr/lib/enigma2/python/Components/ConfigList.py", line 457, in dummyConfigActions KeyError: 'menuConfigActions'

littlesat commented 1 month ago

How do you trigger this?

littlesat commented 1 month ago

OK… I’m afraid this could be better resolved in that plugin.

Dima73 commented 1 month ago

The problem is not in the plugin, but in the fact that self["menuConfigActions"] and etc is not always initialized at startup, but only under certain conditions. That's why I added an additional check. if "key_menu" not in self and menu_button: self["key_menu"] = StaticText(menu_button.get('text', '')) self["menuConfigActions"] = HelpableActionMap(self, "ConfigListActions", { So,need fullUI=True and "key_menu" not in self and menu_button...

Dima73 commented 1 month ago

Same as self["charConfigActions"]. Where does this even get initialized?

littlesat commented 1 month ago

Put it on the forum needs some investigation It should be declared/initialized. The check is as far I see not a solution.

Huevos commented 1 month ago

https://forums.openpli.org/topic/98908-setup-code-proposal-and-more/page-14#entry1635297

The plugin cannot be calling ConfigListScreen.__init__().