search

dctrwatson / znc-fish

ZNC FiSH module
15 stars 15 forks source link

Remove prefixes #11

Open ipimpat opened 9 years ago

ipimpat commented 9 years ago

How to "remove" the message prefixes when:

/msg *fish SetConfig prefix_encrypted (+)
/msg *fish SetConfig prefix_decrypted (-)

Doesn't allow to set "empty" values?

DanielOaks commented 9 years ago

Oh, I see. This is caused by my silly comparison at https://github.com/dctrwatson/znc-fish/blob/master/fish.cpp#L582 and assuming that nobody would want to set an empty value.

In that if statement, I'll probably just check the value (prefix_en/decrypted) and if it's a valid config option, set the appropriate 'empty' value (empty string for prefix_encrypted/decrypted, whatever else for other values)

Sorry for the trouble! I'll try to get around to it sometime soon.

ipimpat commented 9 years ago

Currently I work around it by editing the .registry file for each user and restarting ZNC

DanielOaks commented 9 years ago

Yep, that makes sense and is probably the only way to do it for now. I haven't been doing as much ZNC coding recently but I'll try to get 'round to it in the next few days and submit a PR. Thanks for submitting the issue.