Bug #1600
In ignore highlight rules channels should be separated by commas not semicolons
0%
Description
It makes more sense to separate channels with commas here because semicolons could actually be part of a channel name whereas commas I believe are always used for a list of channels.
History
#1 Updated by digitalcircuit over 4 years ago
I'd agree with this; the IRC specifications mark , as a reserved character.
I think my reasoning for going with semicolons was due to Quassel 0.12's Ignore List Manager already using semicolons as separators, so I'd either need to break backwards compatibility, or highlight rules would be using a different separator than ignore rules.
Ideally, the Quassel core should have some sort of database settings version (not the same as database schema version!) so it can apply migrations on start, akin to how the Quassel client has handled preserving compatibility across otherwise breaking changes. Given the complications of two database backends, I haven't yet tried to implement this - certainly open for ideas or help, though! This would enable other sorts of small core-side cleanups as well.