Project

General

Profile

Bug #1600

In ignore highlight rules channels should be separated by commas not semicolons

Added by knivey 19 days ago. Updated 15 days ago.

Status:
New
Priority:
Low
Assignee:
-
Category:
-
Target version:
-
Start date:
09/10/2020
Due date:
% Done:

0%

Estimated time:
Version:
0.13.0
OS:
Any

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 15 days 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.

Also available in: Atom PDF