Piwigo Bugtracker

Viewing Issue Advanced Details Jump to Notes ] Issue History ] Print ]
ID Category Severity Reproducibility Date Submitted Last Update
0000164 [Piwigo] database tweak always 2005.09.24 17:39 2005.09.24 21:23
Reporter noiry View Status public  
Assigned To plg
Priority normal Resolution fixed Platform
Status closed   OS
Projection none   OS Version
ETA none Fixed in Version 1.5.0RC2 Product Version 1.5.0RC1
  Target Version Product Build
Summary 0000164: stockage du nom de categorie en BDD sans cookies
Description Le lien HyperText est stocké avec le nom de la categorie en BDD

On obtient :

INSERT INTO `phpwebgallery_history` VALUES ('2005-09-24 19:22:38', 'user', '82.224.57.158', './category.php?cat=4 / ./category.php?cat=12', 'picture', '001.jpg');

A priori pas de probleme (on a meme un lien hypertext dans l'historique detaillé) sauf si cela dépasse 150 caractères (taille du champs en BDD)...
Steps To Reproduce
Additional Information J'ai pas de solution cette fois-ci :)
(j'ai mis à 1024 le champs en BDD, mais c'est pas la meilleure des choses a faire)
Tags No tags attached.
browser any
Database engine and version
PHP version
Web server Apache 1.3.x
Attached Files

- Relationships

-  Notes
(0000261)
plg (manager)
2005.09.24 19:30

Stocker le lien hypertexte, c'est une connerie. Ou un bug si tu préfères. Une vraie belle solution serait de stocker dans la table history : datetime, user_id, category_id, picture_id. La table history serait moins sujète à l'obésité.

Sauf que cela veut dire que si la catégorie 12 disparaît, alors il n'y aura plus de libellé à y associer. Idem pour les utilisateurs. Il me semble hors de propos de conserver en base les catégories et utilisateurs supprimés.

C'est un choix conceptuel à faire. A mon avis, la situation actuelle est mauvaise et il faudrait faire plus propre (avec des identifiants faisant référence à d'autres tables).

Plus globalement, au niveau modèle de données, il faudrait prévoir un stockage des données brutes (une ligne par page category.php/picture.php affichée) et des tables d'agrégations. La table des données brutes serait vidable une fois les informations agrégées. Mais c'est un autre débat, il a sa place parmi les réflexions d'évolution de la branche 1.6. Je compte sur toi pour y participer :-)

Pour notre problème immédiat, je propose un simple strip_tags, tu en penses quoi ?
(0000263)
plg (manager)
2005.09.24 21:23

Ok, donc suite à échange par email, noiry est d'accord pour utiliser strip_tags, correction faite dans la version de développement.

- Issue History
Date Modified Username Field Change
2005.09.24 17:39 noiry New Issue
2005.09.24 17:39 noiry browser => any
2005.09.24 17:39 noiry Web server => Apache 1.3.x
2005.09.24 19:30 plg Note Added: 0000261
2005.09.24 19:32 plg Assigned To => plg
2005.09.24 19:32 plg Severity minor => tweak
2005.09.24 19:32 plg Status new => feedback
2005.09.24 21:23 plg Note Added: 0000263
2005.09.24 21:23 plg Status feedback => closed
2005.09.24 21:23 plg Resolution open => fixed
2005.09.24 21:23 plg Fixed in Version => 1.5.0RC2


Mantis 1.1.6[^]
Copyright © 2000 - 2008 Mantis Group
Contact
Powered by Mantis Bugtracker