From bb7f3598e4f04ca67a419c01966dd6b8bd0561ee Mon Sep 17 00:00:00 2001 From: Benedikt Magnus <magnus@magnuscraft.de> Date: Sat, 13 Nov 2021 22:43:33 +0100 Subject: [PATCH] Smaller changes --- locale/de-DE.commands.json | 2 +- scripts/wichtelbot/database/database.ts | 43 ------------------- .../message/modules/assignmentModule.ts | 2 +- tests/performance/assignment.ts | 2 +- .../wichtelbot.message.module.assignment.ts | 22 +++++----- 5 files changed, 14 insertions(+), 57 deletions(-) diff --git a/locale/de-DE.commands.json b/locale/de-DE.commands.json index 46f6fe3..f5db414 100644 --- a/locale/de-DE.commands.json +++ b/locale/de-DE.commands.json @@ -9,7 +9,7 @@ "Rufe Hilfe", "Ruf Hilfe" ], - "info": "Schicke eine Nachricht an die Wichtelorga. Es wird sich dann jemand bei dir melden und dir bei deinem Problem helfen." + "info": "Schickt eine Nachricht an die Wichtelorga. Es wird sich dann jemand bei dir melden und dir bei deinem Problem helfen." }, "changeInformation": { "commands": [ diff --git a/scripts/wichtelbot/database/database.ts b/scripts/wichtelbot/database/database.ts index e69178b..22158cd 100644 --- a/scripts/wichtelbot/database/database.ts +++ b/scripts/wichtelbot/database/database.ts @@ -670,28 +670,6 @@ export function Nutzerzahlen (Callback) ); } -/** - * Liest alle eingetragenen Ausschlüsse aus der Datenbank. - * @param {Number} NutzerId Die Discord-ID des Nutzers. - * @param {Function} Callback Callback, der nach dem Laden der Ausschlüsse ausgeführt wird. Parameter: {Array} Ausschlusswichtel. - * / -export function Ausschlüsse (NutzerId, Callback) -{ - mainDatabase.all( - 'SELECT * FROM Ausschluesse WHERE NutzerId = ? ORDER BY Zeit', - NutzerId, - function (Fehler, Reihen) - { - Fehlerbehandlung(Fehler); - - if (Fehler) - Reihen = []; - - Callback(Reihen); - } - ); -} - /** * Liest alle Steamnamen aus der Datenbank. * @param {Function} Callback Callback, der nach dem Laden der Ausschlüsse ausgeführt wird. Parameter: {Array} Ausschlusswichtel. @@ -734,27 +712,6 @@ export function WichtelOhneVerschicktesPaket (Callback) ); } -/** - * Trägt eine Liste an Nutzer-Wichtel-Zuordnungen in die Datenbank ein. - * @param {Array} Zuordnungen Eine Liste von Objekten mit .Nutzer.Id und .Wichtel.Id. - * @param {Function} Callback Callback, der nach dem Eintragen der Wichtel ausgeführt wird. - * / -export function WichtelEintragen (Zuordnungen, Callback) -{ - let Transaktion = NeueTransaktion(mainDatabase.Name); - - let Vorgang = Transaktion.prepare("INSERT INTO Wichtel (NutzerId, WichtelId) VALUES (?, ?)", Transaktion.Fehlerbehandlung); - - for (let Zuordnung of Zuordnungen) - Vorgang.run(Zuordnung.Nutzer.Id, Zuordnung.Wichtel.Id, Transaktion.Fehlerbehandlung); - - Vorgang.finalize(function () - { - Callback(Transaktion.Schließen()); - } - ); -} - /** * Setzt den Status einer Liste von Nutzern (Teilnehmern) auf "Wichtel". * @param {Array} Teilnehmerliste Eine Liste von Nutzer-IDs, dessen Status zu "Wichtel" geändert werden soll. diff --git a/scripts/wichtelbot/message/modules/assignmentModule.ts b/scripts/wichtelbot/message/modules/assignmentModule.ts index a78d584..535dda3 100644 --- a/scripts/wichtelbot/message/modules/assignmentModule.ts +++ b/scripts/wichtelbot/message/modules/assignmentModule.ts @@ -40,7 +40,7 @@ export class AssignmentModule * Run the assignment. * @returns True if successful, false otherwise. */ - public assign (): boolean // TODO: Return the reason for failure. + public runAssignment (): boolean // TODO: Return the reason for failure. { const members = this.database.getWaitingMember(); diff --git a/tests/performance/assignment.ts b/tests/performance/assignment.ts index 71ce59a..5665327 100644 --- a/tests/performance/assignment.ts +++ b/tests/performance/assignment.ts @@ -53,7 +53,7 @@ function performanceTest (this: Mocha.Suite): void it('members.', function () { - assignmentModule.assign(); + assignmentModule.runAssignment(); } ); } diff --git a/tests/tests/wichtelbot.message.module.assignment.ts b/tests/tests/wichtelbot.message.module.assignment.ts index 4eb5c4a..20acf74 100644 --- a/tests/tests/wichtelbot.message.module.assignment.ts +++ b/tests/tests/wichtelbot.message.module.assignment.ts @@ -3,11 +3,11 @@ import { assert } from 'chai'; import { AssignmentModule } from '../../scripts/wichtelbot/message/modules/assignmentModule'; import ContactTestUtility from '../utility/contact'; import Database from '../../scripts/wichtelbot/database/database'; +import { ExclusionData } from '../../scripts/wichtelbot/classes/exclusion'; +import { ExclusionReason } from '../../scripts/wichtelbot/types/exclusionReason'; import GiftType from '../../scripts/wichtelbot/types/giftType'; import Member from '../../scripts/wichtelbot/classes/member'; import { RelationshipTestUtility } from '../utility/relationship'; -import { ExclusionData } from '../../scripts/wichtelbot/classes/exclusion'; -import { ExclusionReason } from '../../scripts/wichtelbot/types/exclusionReason'; describe('assignment module', function () @@ -45,7 +45,7 @@ describe('assignment module', database.saveMember(newMember); } - const successful = assignmentModule.assign(); + const successful = assignmentModule.runAssignment(); assert.isTrue(successful); @@ -73,7 +73,7 @@ describe('assignment module', database.saveMember(newMember); } - const successful = assignmentModule.assign(); + const successful = assignmentModule.runAssignment(); assert.isTrue(successful); @@ -101,7 +101,7 @@ describe('assignment module', database.saveMember(newMember); } - const successful = assignmentModule.assign(); + const successful = assignmentModule.runAssignment(); assert.isTrue(successful); @@ -146,7 +146,7 @@ describe('assignment module', database.saveMember(newMember); } - const successful = assignmentModule.assign(); + const successful = assignmentModule.runAssignment(); assert.isTrue(successful); @@ -198,7 +198,7 @@ describe('assignment module', database.saveUserExclusions(exclusions); - const successful = assignmentModule.assign(); + const successful = assignmentModule.runAssignment(); assert.isTrue(successful); @@ -214,7 +214,7 @@ describe('assignment module', it('cannot assign with zero members', function () { - const successful = assignmentModule.assign(); + const successful = assignmentModule.runAssignment(); assert.isFalse(successful); } @@ -228,7 +228,7 @@ describe('assignment module', database.saveContact(newMember); database.saveMember(newMember); - const successful = assignmentModule.assign(); + const successful = assignmentModule.runAssignment(); assert.isFalse(successful); } @@ -248,7 +248,7 @@ describe('assignment module', database.saveMember(newMember); } - const successful = assignmentModule.assign(); + const successful = assignmentModule.runAssignment(); assert.isFalse(successful); } @@ -279,7 +279,7 @@ describe('assignment module', database.saveMember(newMember); } - const successful = assignmentModule.assign(); + const successful = assignmentModule.runAssignment(); assert.isFalse(successful); } -- GitLab