search

lzear / LeekWars_Kikimeter

5 stars 3 forks source link

Ajout de champ pour l'export vers une page externe #19

Open Syxxxx opened 9 years ago

Syxxxx commented 9 years ago

Ajout des champs :

Changement de la clé primaire (fightid + leekid)

Attention :

lzear commented 9 years ago

Je ne peux pas accéder à mon FTP depuis ma connexion 3G (j'ai que ça). Ça marche impec' ? Je Merge dès que j'ai une confirmation. Aussi, je pense qu'il serait judicieux de poster le "alter table" ici et sur le forum.

Je me demande s'il y a un autre moyen de signaler aux utilisateurs qu'il faut changer la BDD. En l’occurrence je pense que quasiment personne utilise la BDD, mais je pense que ça pourrait être gênant si des joueurs autoUpdatent le script et qu'il ne sont pas avertis que leur PHP/SQL est obsolète.

Syxxxx commented 9 years ago

Un test exterieur serai le bienvenu (vu que j'ai recopié mes modifs d'une version anterieur à celle sur Github).

pour le alter table j'ai pas trop de solution à part ajouter un fichier d'update sql et d'afficher un warning à la 1ere execution du scprit après update.

si tu as d'autres idées je suis preneur

yLark commented 9 years ago

Je n'utilise pas les fonctions php, donc je ne peux guère vous aider... En plus, depuis j'ai fait des modifs, donc va falloir résoudre les conflits après le merge. @AlexClaw, tu utilises les fonctions php ?

abourdin commented 9 years ago

Au début oui, mais plus maintenant faute d'utilité concrète. Le code est toujours déployé sur heroku mais ne sert pas à grand chose pour l'instant.

lzear commented 9 years ago

J'ai Internet (la fibre!!) et un peu de temps libre maintenant. Je peux m'occuper du merge n'importe quand. Mais le soucis c'est qu'il y a eu pas mal de modifications entre temps. Je sais pas trop quelle est la solution appropriée à ce genre de problèmes. Je te propose d'insérer tes modifications dans la version la plus récente du plugin et re-faire un pull-request. On mettra ton amélioration dans une branche à part pour pouvoir travailler dessus. Car il faut ajouter pas mal de champs maintenant que Lark a traité les données JSON : Force, Agilité, Fréquence, Coeurs, TotalLife (avant tout blindage), PT, PM... et surement d'autres choses encore. Aussi, il faut voir si la derniere MAJ a une influence sur les données à enregistrer

Après, j'essaierai d'améliorer ce qu'il se passe coté serveur.

(Je t'ai mis en contributeur)

Syxxxx commented 9 years ago

Félicitation pour la fibre :) oui, je vais refaire mes modifs sur la derniere version sa sera plus simple. je n'ai pas eu le temps de regarder cela depuis mon pull request.

sinon pour evité les bug lors des maj, je crois que le plus simple est de faire un menu de config comme celuis qui existe deja mais pour l'export des données. par défaut les champs de base et si on ajoute des champs on propose un fichier avec la structure de la base. qu'en pense tu ?

Merci pour l'ajout des contributeurs. Bonne soirée