#9 Database setup #11
Conversation
| import org.jooq.impl.DSL; | ||
| import org.togetherjava.discordbot.commands.CommandContext; | ||
|
|
||
| public abstract class SimpleRepository<T> implements Repository<T> { |
There was a problem hiding this comment.
Since this klass seems to be a baseclass of Jooq implementations. I'd rename this class so that the name reflects what this class represents.
src/main/resources/sample-config.yml
Outdated
| @@ -3,6 +3,8 @@ | |||
| # The command prefixes | |||
| prefixes: ["!", "?"] | |||
| botToken: "your token" | |||
| # the url for the database connection | |||
| dburl: "jdbc:sqlite:yourfile.sqlite" | |||
There was a problem hiding this comment.
Enter the same db url as in the pom. Since it did not seem to work with 2 different urls.
|
|
||
| @ActiveCommand(name = "example", parentClass = BasePrefixCommand.class) | ||
| @SuppressWarnings("unused") | ||
| public class ExampleCommandDB extends CommandNode<CommandContext> { |
There was a problem hiding this comment.
I understand that you want to show how it works but this command should be disabled in the actual bot. It could lead to people abusing it and trying to fill the server harddrive unecessary.
| ### Database Access | ||
| 1. Create a schema in resources/db, be sure to start it with a correct version number i.e. V1__example | ||
| ```sqlite | ||
| CREATE TABLE IF NOT EXISTS EXAMPLE | ||
| ( | ||
| ID INTEGER AUTO_INCREMENT PRIMARY KEY, | ||
| MEMBER varchar(30) NOT NULL, | ||
| TEXT varchar(30) NOT NULL | ||
| ); | ||
| ``` | ||
| 2. Make a new class in org.togetherjava.db.repositories that extends JooqRepository. | ||
| 3. Create a Constructor that accepts CommandContext as a parameter. | ||
| 4. You can then either create a DAO instance using Jooqs generated DAO class that is generated from your schema using | ||
| the DSLContext for configuration provided in the JooqRepository class. Or you can directly use the DSLContext to create SQL queries | ||
| ```java | ||
| ExampleDao dao = new ExampleDao(dslContext.configuration()); | ||
| ``` |
There was a problem hiding this comment.
This seems good, but I do not use databases enough to understand some of this terminology. Could someone else review this and confirm it makes sense. @AzatothTheAwakened can you take a look? Would be nice to finish up this PR and get it merged!
SizableShrimp
added
documentation
|
A JOOQ Db sounds nice. We now also have some DI container that can keep track of it. |
Created the initial database with a simple repository system.