Piwigo Bugtracker

View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0000164Piwigodatabasepublic2005.09.24 17:392005.09.24 21:23
Reporternoiry 
Assigned Toplg 
PrioritynormalSeveritytweakReproducibilityalways
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version1.5.0RC1 
Target VersionFixed in Version1.5.0RC2 
Summary0000164: stockage du nom de categorie en BDD sans cookies
DescriptionLe 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)...
Additional InformationJ'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)
TagsNo tags attached.
browserany
Database engine and version
PHP version
Web serverApache 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


Copyright © 2000 - 2015 MantisBT Team
Contact
Powered by Mantis Bugtracker