[résolu] Fermeture automatique ?

Problèmes, bugs et difficultés rencontrés sur le site.
Avatar du membre
yip
Messages : 387
Enregistré le : 08 mars 2004, 23:32

Message par yip »

sly a écrit : 1 "on ne sais pas si c'est fermé"
2 "on sait que c'est ouvert"
3 "on sait que c'est fermé"
4 "on sait que c'est une ruine"
5 "on sait qu'elle est détruite"

La modification d'une fiche donnant elle la possibilité de choisir de 1 à 5

Pour les développeurs : ok, c'est pas forcément beau dans le code, je suis tout ouïe pour une solution qui pourrait être plus propre et généraliste mais en gardant cette fonctionnalité.
C'est pas mal du tout ! le stocker en int; en utilisant 0 pour ""on ne sais pas si c'est fermé"" et -1 pour l'ouverture, on gagne encore un peu:
>0: fermé
0 inconnu
<0: ouvert (ou l'inverse)
Mais comment gère tu les sources "taries" ?
Il faut une correspondance avec le texte externe ?

Et puis :calimero: j'ai merdé, il n'y a plus de "inconnu" en base. J'ai regroupé les "inconnus" et "ouvert" l'autre jour sans faire gaffe....il y a des backups pour récupérer la feature ? si oui je peux préparer l'injection.
Avatar du membre
sly
Messages : 5041
Enregistré le : 29 févr. 2004, 17:59
Localisation : Chambéry - Savoie

Message par sly »

yip a écrit : C'est pas mal du tout ! le stocker en int; en utilisant 0 pour ""on ne sais pas si c'est fermé"" et -1 pour l'ouverture, on gagne encore un peu:
>0: fermé
0 inconnu
<0: ouvert (ou l'inverse)
V'la une bonne idée.
Pour éviter toutefois de faire des

if (4)
print("détruit")
if (5)
print(ruine)

à trop d'endroit dans le code, on peut se stoker la correspondance dans un tableau du config.
Ce qui permettrait un truc genre :

Code : Tout sélectionner

if &#40;$etat>0&#41;
print&#40;$config&#91;'etat_fermeture'&#93;&#91;$x&#93;&#41;;
Mais comment gère tu les sources "taries" ?
Il faut une correspondance avec le texte externe ?
Hmmm, ma foi oui, j'avais oublié ce truc des champs mutants...
ça sera pas bien pire qu'aujourd'hui de toute façon, donc un truc genre :

Code : Tout sélectionner

if &#40;$etat>0&#41;
&#123;
  if &#40;$etat==1&#41;
    print&#40;$point->champ_fermer_mutant&#41;
  else
    print&#40;$config&#91;'etat_fermeture'&#93;&#91;$x&#93;&#41;;
&#125;
Ou alors, tout foutre en l'air ce système de champs mutant et mettre ça dans le $config
$config['noms_champs_points']['ferme'][$id_point_eau]="Tarie définitivement";
$config['noms_champs_points']['ferme'][$id_cabane]="Fermée";

Ouais, sauf que ça fait des id à la main, ouais ben à ce compte là bazarder la table type_point et mettre ça dans le config..

Ouais oulla bon, je devrais pas écrire au fûr et à mesure que je pense.

Ma pensée d'après, c'est évidement une nouvelle table "noms_propriété_points"
ou on trouve :
id type point | champs_point | valeur | affichage | format
7 | ferme | 1 | fermée | checkbox
7 | ferme | 2 | ruine | checkbox
7 | ferme | 3 | détruite | checkbox
12 | ferme | 1 | tarie définitivement | checkbox
12 | remarq | * | Remarques | textarea

et là, à nous le formulaire générique dynamiquement généré et et et oulla là, je crois que je m'emballe encore trop, mais ça plairait peut-être à dominique, lui qui trouvait que le formulaire de saisie de point était trop compliqué à maintenir ;-)
Et puis :calimero: j'ai merdé, il n'y a plus de "inconnu" en base. J'ai regroupé les "inconnus" et "ouvert" l'autre jour sans faire gaffe....il y a des backups pour récupérer la feature ? si oui je peux préparer l'injection.
tu as converti les ferme='non' en ferme='' plutôt ?
refuges=> select count(*),ferme from points group by ferme;
count | ferme
-------+---------
15 | ruine
2 | non
14 | detruit
2360 |
125 | oui
Seulement 2 cabanes déclarées ouvertes ça semble peu en effet.

J'ai récupéré un dump avec ce qui semble être mieux et j'ai mis ça dans une base test2
impossible d'écraser la table points actuel car mon dump est trop vieux, mais on peut au moins récupérer l'info. Je te laisse y faire ?
Avatar du membre
yip
Messages : 387
Enregistré le : 08 mars 2004, 23:32

Message par yip »

Merci je ferais ça.

Quand on rentre un point sans savoir s'il est ouvert ou non, c'est qu'on a trop bu la veille. ;)
C'est pour ça que j'ai pas trop regretté la boulette.. pensant que ca pourrait passer ...

Ah, la simplification se transforme toujours en complexification ... ;)
plutot que faire une nouvelle table je prefererais autant abandonner les int et revenir au:
-NULL // no se
-'' // ouvert
-'raison fermeture' // peut donc etre "tarie" pour les mutants comme "ruine"
qui s'adapte a tout s'en trop de souci. ( la seule commande empty() ) renvoie si c'est ouvert ou No Se)

OK je répare ça. merci pour le dump.
Avatar du membre
yip
Messages : 387
Enregistré le : 08 mars 2004, 23:32

Message par yip »

C'est réparé pour les anciens, il y en a de nouveau 740 en "fermé inconnu"

Fort possible qu'il y ait encore des bugs dans ce secteur là.
Charlinette
Messages : 941
Enregistré le : 22 janv. 2012, 18:30
Localisation : Ardèche centre

Message par Charlinette »

J'ai hésité à créer un autre fil de discussion mais c'est peut-être le même bug...
Cette cabane à qui il ne manque pas un mur s’affiche pourtant comme si...
La fiche est juste, l'icône est faux
http://www.refuges.info/point/3807/caba ... ransureux/
Avatar du membre
Dominique
Messages : 3705
Enregistré le : 08 avr. 2006, 21:58

Message par Dominique »

Charlinette a écrit :J'ai hésité à créer un autre fil de discussion mais c'est peut-être le même bug...
Cette cabane à qui il ne manque pas un mur s’affiche pourtant comme si...
La fiche est juste, l'icône est faux
http://www.refuges.info/point/3807/caba ... ransureux/
On affiche un abri précaire quand il y a 0 places pour dormir
(une histoire à dormir debout)
Charlinette
Messages : 941
Enregistré le : 22 janv. 2012, 18:30
Localisation : Ardèche centre

Message par Charlinette »

Ah oui... mais "très large place pour dormir" pas facile à quantifier... je mets un chiffre bidon ?
J'ai mis 5... :?: