search

Letractively / eude

Automatically exported from code.google.com/p/eude
0 stars 0 forks source link

Modification des planètes inoccupé en planète colonisable #140

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
Description de la fonctionnalité demandé:
Modifier la suppression des planètes inoccupé

Informations complémentaires:
Suite à quelques discutions avec sam renfou, et au vu des infos que donne
looki, il serait bien de ne plus supprimer les planètes devenu inoccupé
mais de les passer en planète à colo afin de pouvoir garder le % d'eau de
la planète, vu que looki ne donne pas l'info si planète pas colonisé.

Je sais pas ce que tu en penses ?

Original issue reported on code.google.com by XCharism...@gmail.com on 16 May 2010 at 10:44

GoogleCodeExporter commented 8 years ago 
[deleted comment]
GoogleCodeExporter commented 8 years ago 
Why not.

Original comment by Alex10336@gmail.com on 17 May 2010 at 11:29

GoogleCodeExporter commented 8 years ago

Original comment by XCharism...@gmail.com on 20 May 2010 at 8:16

GoogleCodeExporter commented 8 years ago 
This issue was updated by revision r293.

Modif à tester, de mon côté après plusieurs test aucun soucis.

http://code.google.com/p/eude/source/diff?spec=svn293&r=293&format=side&path=/tr
unk/last/class/cartographie.class.php

A alex, contrôle la ligne 112. Je suis pas sur de cette modif.

Original comment by XCharism...@gmail.com on 20 May 2010 at 8:22

GoogleCodeExporter commented 8 years ago 
Cette ligne est un truc de 'sécurité'. A priori jamais utilisé.

Si l'ajout des ressources d'une planète dans la seconde table échoue, son 
référant 
disparait également. (doublon toussa...)

Original comment by Alex10336@gmail.com on 20 May 2010 at 8:27

GoogleCodeExporter commented 8 years ago 
Sert à rien donc, par contre, si tu as une idée pour renvoyer un - à la 
place du %
quand water retourne rien.

Original comment by XCharism...@gmail.com on 20 May 2010 at 8:29

GoogleCodeExporter commented 8 years ago 
Je vois pas trop pourquoi la suppression de %%infos%% puisque les requêtes ne 
le reset 
pas ;)

http://code.google.com/p/eude/source/diff?
spec=svn293&old=289&r=293&format=unidiff&path=/trunk/last/tpl/cartographie.tpl.p
hp

Original comment by Alex10336@gmail.com on 20 May 2010 at 8:32

GoogleCodeExporter commented 8 years ago 
Ba comme une planète à colo n'a pas de nom à la base, j'ai pas vu 
l'intérêt de le laisser

Original comment by XCharism...@gmail.com on 20 May 2010 at 8:34

GoogleCodeExporter commented 8 years ago 
- "Sert à rien donc,"
Oui, et non,
Sauf sur une base de donnée ayant des doublons, sa peut planter juste. (ma 
base en a)
deux requête les 'affichant'

ALTER TABLE  `SQL_PREFIX_Coordonnee` ADD UNIQUE  `coords` (  `POSIN` ,  
`COORDET` )

SELECT COUNT( * ) , CONCAT_WS(  '-',  `POSIN` ,  `COORDET` ) 
FROM  `SQL_PREFIX_Coordonnee` 
GROUP BY CONCAT_WS(  '-',  `POSIN` ,  `COORDET` ) 
ORDER BY COUNT( * ) DESC

Un truc de ce genre du coup:
UPDATE SQL_PREFIX_Coordonnee SET Type='2', infos='' where Type in (0,3,5) AND 
POSIN='{$uni}' AND COORDET='{$sys}'
UPDATE SQL_PREFIX_Coordonnee SET Type='2', infos='' where Type in (0,3,5) AND 
POSIN='{$cur_ss}' AND COORDET in ({$del_planet})"

Original comment by Alex10336@gmail.com on 20 May 2010 at 8:46

GoogleCodeExporter commented 8 years ago 
Ba 2 choix possible, soit laissé comme j'ai modif et ne pas renvoyer 'infos', 
soit le
supprimer, mais dans les 2 cas ca revient au même vu que lorsque le jour où la
planète sera colo, le nom de la planète changera.

Comme tu le sent après, et donc remettre la requête anti doublon. Bien que 
dans la
logique où les planètes ne sont plus éjecté mais constamment passé de 
planète à
joueur, l'id ne changent plus ainsi le pid peut rester constamment et donc
logiquement plus de doublon.

Original comment by XCharism...@gmail.com on 20 May 2010 at 8:53

GoogleCodeExporter commented 8 years ago 
vi en même temps... je pensais qu'il y avais quelqu'un d'autre a utiliser la 
tpl 
'TypeA'.

Je viens de relooker au cas où, a priori, nikel ;)

Original comment by Alex10336@gmail.com on 20 May 2010 at 9:03

GoogleCodeExporter commented 8 years ago 
Hiaaaaaaa je suis trop fort :p

Concernant le % après water, tu as une solution déjà intégré au DE pour le 
remplacer
par un - quand la water est vide ?

Original comment by XCharism...@gmail.com on 20 May 2010 at 9:08

GoogleCodeExporter commented 8 years ago 
le caractère '%' est dans la bdd ?

Original comment by Alex10336@gmail.com on 20 May 2010 at 9:13

GoogleCodeExporter commented 8 years ago 
http://code.google.com/p/eude/source/browse/trunk/last/tpl/cartographie.tpl.php?
spec=svn293&r=293#218

Non

Original comment by XCharism...@gmail.com on 20 May 2010 at 9:16

GoogleCodeExporter commented 8 years ago 
si j'ai bien compris: r296

Original comment by Alex10336@gmail.com on 20 May 2010 at 9:22

GoogleCodeExporter commented 8 years ago 
http://code.google.com/p/eude/source/detail?r=297

Parfais maintenant. Bon ba issue done.

Original comment by XCharism...@gmail.com on 20 May 2010 at 9:32

GoogleCodeExporter commented 8 years ago 
Ah non la requête à remettre

Original comment by XCharism...@gmail.com on 20 May 2010 at 9:34

GoogleCodeExporter commented 8 years ago 
Voilà :)

Original comment by XCharism...@gmail.com on 20 May 2010 at 9:40

GoogleCodeExporter commented 8 years ago

Original comment by Alex10336@gmail.com on 20 May 2010 at 9:43