fireout / keepasssequencer

Word sequencer for KeePass
65 stars 12 forks source link

Unhandled exception for invalid strings in item configuration #54

Open fritzophrenic opened 8 years ago

fritzophrenic commented 8 years ago

When configuring capitalization, substitution, percentage, etc. for a given item, try entering an invalid string. E.g. "Proper" for substitutions, or "aardvark" for capitalization. An unhandled exception message pops up.

We should catch this error and prompt for a new value or explain why it is wrong. Or implement a dialog that only allows choosing from valid entries.

fritzophrenic commented 8 years ago

Note #46 may make this issue obsolete.

fireout commented 8 years ago

Suggestion may work, but we have to see what happens if an incorrect string is set directly in the definition file. Maybe it would be better to have a common behavior (like a fallback value maybe)

In any case, a feedback dialog would be required