✨ Numeric Date Entry - enable custom culture #145
Conversation
| @@ -65,6 +65,21 @@ public ICommand ErrorCommand | |||
| set => SetValue(ErrorCommandProperty, value); | |||
| } | |||
|
|
|||
| private CultureInfo _culture; | |||
There was a problem hiding this comment.
Pour résumer, l'objectif c'est que dans l'appli on utilise ce composant systématiquement avec une culture à "calendrier grégorien" ?
There was a problem hiding this comment.
Pas tout a fait. Ce composant est le date input de SDET. Je rajoute à ce composant la possibilité de spécifier une culture différente de la culture par défaut du device. Une fois que ce sera fait, je pourrais retourner dans l'app xamarin et mettre à jour partout où ce composant est utilisé.
There was a problem hiding this comment.
Hmm, je ne comprends pas la nuance. On est bien d'accord que l'objectif c'est que dans l'app, on n'utilise plus ce composant que dans sa version "avec spécification d'une culture" ?
There was a problem hiding this comment.
Ah, oui, j'avais mal lu la question. J'avais compris "on utilise systématiquement ce composant quand on veut afficher une date" ><
Oui, on utilisera systématiquement avec une culture spécifiée
There was a problem hiding this comment.
Par contre je me rends compte que ce n'est pas exactement ce que j'avais compris ce matin. Je pensais que dans le ToString on pourrait préciser un type de calendrier, pas une culture. Du coup par exemple on passera quoi comme "culture" dans l'app pour in fine avoir un calendrier grégorien ?
There was a problem hiding this comment.
Par contre je me rends compte que ce n'est pas exactement ce que j'avais compris ce matin. Je pensais que dans le ToString on pourrait préciser un type de calendrier, pas une culture
Je pensais aussi ça ce matin ^^ Sauf que pour formatter un datetime, il n'attend pas juste un calendrier mais un IFormatProvider (que CultureInfo implémente) car il y a d'autres paramètres (comme l'ordre JOUR / MOIS / ANNEE de certains affichages).
Ce que je suis en train de faire côté app xamarin, c'est que quand c'est la culture thai, je fournit une autre culture au toString (pour l'instant je suis partit sur la français, mais c'est le temps de mes tests. Il faudra qu eje trouve la plus adaptée) et pour toutes les autres cultures, je fournis la culture du device. Comme ça on impacte uniquement le thai
There was a problem hiding this comment.
Ah ! Bon ça me rassure j'étais pas à côté de la plaque x)
C'est dommage ça nous fait faire vraiment un gros hack mais tant pis... Si déjà on n'envoie une culture que dans les cas ou c'est du thaï c'est un moindre mal
JosselinTILLAY
force-pushed
the
task/PROD-194-numericdateentry-enable-custom-culture
branch
2 times, most recently
from
18a90b2 to
9bc1853
Compare
JosselinTILLAY
force-pushed
the
task/PROD-194-numericdateentry-enable-custom-culture
branch
from
9bc1853 to
e21f3a6
Compare
Ayé, tests rajoutés |
| namespace Smartway.UiComponent.UnitTests.Inputs.NumericDateInput | ||
| { | ||
| class TestableNumericDateEntry: NumericDateEntry | ||
| { | ||
| public new DateTime GetFilledDate() => base.GetFilledDate(); | ||
| } | ||
| } |
There was a problem hiding this comment.
C'est quoi l'interet de cette classe si la seule méthode que tu créees c'est la même méthode que la classe mère? ^^
There was a problem hiding this comment.
Je la rend public
There was a problem hiding this comment.
Haha le fourbe, je connaissais pas cette méthode, j'approuve!
JosselinTILLAY
deleted the
task/PROD-194-numericdateentry-enable-custom-culture
branch
No description provided.