Ticket #7 (closed defect: fixed)

Opened 1 year ago

Last modified 4 months ago

L'affichage de la date d'une sortie est incorrect dans les pages de diff.

Reported by: alex Assigned to: somebody
Priority: minor Milestone:
Component: component1 Version:
Keywords: Cc:
Triage Stage: Unreviewed State of Approval: Unnecessary
Attached Patches: None Complexity: Unknown
Compatibility: Unknown Specification: Unnecessary

Description

Pour une raison inconnue le format du champ "date" des sorties est incorrect lorsqu'on veut l'afficher dans les pages de diff : pb pour la comparaison (pas de différence meme si les dates des 2 versions diffèrent dans la BD) et pb dans la recapitulation de la version la plus récente des 2 dont on fait le diff.

La date est OK dans les actions view et preview...

Change History

10/17/07 19:51:30 changed by alex

  • priority changed from blocker to minor.

07/15/08 20:17:38 changed by xbrrr

  • status changed from new to closed.
  • resolution set to fixed.

J'ai identifié l'origine du bug #7. (il m'aura bien fait courir pour le coup...)

Je passe les détails, mais en gros, à un moment on récupère la date de la sortie de la bdd ('2008-06-13' par exemple), et comme il existe une fonction 'filterSetDate' pour la classe Outings, on le passe à la moulinette. (on a des sets car on utilise la fonction createFromArchive qui peuple les champs a coups de set() ) Et c'est là qu'est le problème, la fonction attend quelquechose du genre Array{ 'year' => 2008, 'month' => 6, 'day' => 13 } et le convertit en YYYY-MM-DD. Du coup, on se retrouve toujours avec '2-02-02', soit le 2 février 2002....

A noter qu'il ne faut pas bazarder la fonction, vu qu'elle fonctionne comme il faut dans le cas où on édite une sortie (l'entrée est un tableau)

La solution que j'ai utilisé c'est de convertir les dates de string en array. (voir r234)

07/19/08 14:24:45 changed by xbrrr

  • status changed from closed to reopened.
  • resolution deleted.

Bizarre, ça marchait bien en local chez moi, mais ce n'est bon ni sur le site de test, ni sur la prod, je rouvre le ticket

07/22/08 19:37:27 changed by xbrrr

  • status changed from reopened to closed.
  • resolution set to fixed.

r253 (apparemment, c2cpc23 est avec php 5.2.0 qui ne supporte pas l'utilisation des sous-masques nommés)