Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update server.lua and DataBase #1

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

plysken
Copy link

@plysken plysken commented Apr 20, 2017

Better to use identifier than id

Better to use identifier than id
Some people have a column ID witch is the primarykey, lot of people, so i think you should make your database in that way.
@plysken plysken changed the title Update server.lua Update server.lua and DataBase Apr 20, 2017
@vodkhard
Copy link
Owner

Merci pour ton aide surtout pour l'organisation des fichiers mais je préfère garder cette structure de base de données (juste changer le utf-8)

@plysken
Copy link
Author

plysken commented Apr 20, 2017

Tu es français, ça va me faciliter la tâche pour communiquer.
Pas de soucis, chacun code à sa manière et je respecte ça.

On peu se parler en live (discord par exemple ?)

Sinon ça te dérange si je continue a t’envoyer des suggestions de modifications ? Je vais pas te spammer car c'est ton 1er script et je le trouve déjà super. Juste je vais le "customiser" et peu être tu aimerais en profiter. ?

Copy link

@rudzboy rudzboy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ouais en gros c'est ça qu'il faut faire !

Mais 😉 Il faut conserver les FOREIGN KEY par contre ;) + ajouter DELETE ON CASCADE
Ajouter une contrainte d'unicité du couple (player_id +item_id)
Et ça devrait le faire

@Kaotic
Copy link

Kaotic commented May 9, 2017

Ceci fait une query en plus, pas dutout opti

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants