Add Chat Messages to DBLog Plugin

[ect0s]

Add chat message table to DBLog

Schema includes:

Server
match
Time
Chat(teamchat/squadchat)
message
steamid

I did not create a foreign key dependence on steamuser, as I had issues when we received a chat message but had not received and created the steamuser via PLAYER_CONNECTED.

[werewolfboy13]

@ect0s has then been checked to make sure the tables are created properly in the DB?

[ect0s]

@werewolfboy13

@ect0s has then been checked to make sure the tables are created properly in the DB?

Yes, though I did just update the Schema.

+---------+--------------+------+-----+---------+----------------+
| Field   | Type         | Null | Key | Default | Extra          |
+---------+--------------+------+-----+---------+----------------+
| id      | int          | NO   | PRI | NULL    | auto_increment |
| time    | datetime     | YES  |     | NULL    |                |
| chat    | varchar(255) | YES  |     | NULL    |                |
| message | text         | YES  |     | NULL    |                |
| steamid | varchar(255) | YES  |     | NULL    |                |
| server  | int          | NO   | MUL | NULL    |                |
| match   | int          | YES  | MUL | NULL    |                |
+---------+--------------+------+-----+---------+----------------+

[werewolfboy13]

Should be good, unable to test internally on my end.

[ect0s]

@werewolfboy13

I've had this running for 4 days so far, no issues.

I've also deleted and recreated the DB multiple times, so as long as the SquadJS user people have configured in their DB can create the table, this should be good to merge.

[reck1610]

@ect0s could you add the team id and squad id of the player too?
That would allow a complete recreation of the chat history.

[ect0s]

@ect0s could you add the team id and squad id of the player too? That would allow a complete recreation of the chat history.

Sure.

Would team faction maybe matter here as well? Or the Squads Name at the time?

Just if I'm going to open and edit this PR, I'd like to nail down everything at once

[reck1610]

The name of the squad could be useful, because it can change if a squad gets opened and closed.
The faction should be resolvable using the match. I don't think we need to duplicate the information.

[ect0s]

@reck1610 Implemented.

Will require an alter to the table if you've already taken these changes.

+-----------+--------------+------+-----+---------+----------------+
| Field     | Type         | Null | Key | Default | Extra          |
+-----------+--------------+------+-----+---------+----------------+
| id        | int          | NO   | PRI | NULL    | auto_increment |
| time      | datetime     | YES  |     | NULL    |                |
| chat      | varchar(255) | YES  |     | NULL    |                |
| message   | text         | YES  |     | NULL    |                |
| steamid   | varchar(255) | YES  |     | NULL    |                |
| squadName | varchar(255) | YES  |     | NULL    |                |
| squadID   | int          | YES  |     | NULL    |                |
| team      | int          | YES  |     | NULL    |                |
| server    | int          | NO   | MUL | NULL    |                |
| match     | int          | YES  | MUL | NULL    |                |
+-----------+--------------+------+-----+---------+----------------+

[werewolfboy13]

Someone have the ALTER command so we can do some documentation for it?

[fantinodavide]

Someone have the ALTER command so we can do some documentation for it?

The ALTER command is not needed because the changes add a new table, but won't modify the previously existing tables and the foreign keys are automatically updated by sequelize calling the .sync() method.

In case we really want to make sure, we can replace .sync() with .sync({ alter: true })

Also, I have another modification of this DBLog plugin to include tickets and some more details about the teams. Everything get stored at the end of each match.
It would be a nice addition, but it would also require altering the Match table to include some extra fields
Specifically, here are the new fields:

• tickets
• winnerTeam
• winnerTeamID
• team1
• team2
• isDraw