ueffel / Keypirinha-Plugin-Kill

Provides a command to kill a running processes.
MIT License
25 stars 0 forks source link

In case of an invalid _default_action #1

Closed polyvertex closed 8 years ago

polyvertex commented 8 years ago

There's a potential bug at line 192 in case self._default_action is not found in self._actions. You probably want to correct that at line 33 in _read_config. For that matter, to call settings.get_enum should be more appropriate in your case.

ueffel commented 8 years ago

Thanks for reporting that. I see what you mean, i cant be sure what a user would put in the config and if it is one of my valid actions.