Project

General

Profile

Bug #57

The SqliteStorage can't store empty messages thus resulting in a crash

Added by EgS over 12 years ago. Updated over 11 years ago.

Status:
Closed
Priority:
High
Assignee:
Category:
Quassel Core
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:
Version:
0.13.0
OS:
Any

Description

The Current Schema of the SqliteStorage does not allow empty messages (see Add. Info.). But empty messages are not disallowed by IRC protocoll.
Obviously the schema needs to be updated. I'm deferring this until Kaffedoktor has finished the FirstRun Whizzard.

Additional information:

Currentbacklog Table:

CREATE TABLE backlog (
messageid INTEGER NOT NULL PRIMARY KEY AUTOINCREMENT,
time INTEGER NOT NULL,
bufferid INTEGER NOT NULL,
type INTEGER NOT NULL,
flags INTEGER NOT NULL,
senderid INTEGER NOT NULL,
message TEXT NOT NULL)

Associated revisions

Revision 7b645ac7 (diff)
Added by Marcus Eggenberger about 12 years ago

closing BR #57 - handle empty messages properly

History

#1 Updated by EgS over 12 years ago

Ok this basically sucks.

In order to fix this. The definition of backlog.message would have to be altered. But sqlite doesn't support this alteration of tables... -.-

Have to think of a userfriendly solution, of how to address those issues when we have real users...
For now it will probably result in a recreation of the tables.

#2 Updated by EgS about 12 years ago

Fixed in r366

Also available in: Atom PDF