[Bug] Inconsistencies in the Expected Format Provider in Converters
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 68/100
Piste de recherche
Start with SingleToStringTypeConverter.cs, the double and decimal converter files, and the cited DoubleToStringTypeConverterTests.cs cases. Run ReactiveUI.Binding.Tests under fr-FR and compare the converter comments, behavior, and test expectations across cultures. Decide on consistent culture behavior, then align the converters, comments, and tests; done when the suite passes under the affected cultures.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Prerequisites
- I have searched this repository's open and closed issues for duplicates.
- I am using the latest stable release.
What happens
There are many inconsistencies regarding the expected format provider in converters.
Concretely, SingleToStringTypeConverter indicates that the conversion is performed "using the invariant culture", but only does that for this one case, while other converters for double, decimal, ..., indicate that the conversion is performed "using the current culture".
To make it even more confusing, there are unit tests to verify both behaviors, which is impossible in many cultures. For example, double to string tests have checks expecting both cultures at the same time (TryConvert_WithConversionHint_FormatsCorrectly, TryConvert_NegativeValue_Succeeds).
What is the expected behavior?
What you expected instead
Converters with consistent behavior, comments, and tests.
Minimal reproduction
N/A
Steps to reproduce
- Run
ReactiveUI.Binding.Testsusingfr-FRculture, for example. - Tests fail, and a deeper search through the code base shows all the inconsistencies mentioned above.
Exception and stack trace
Package and version
Latest main (c79c70ccff42edbd97fe7e1dcac6edabc563dd52)
UI framework
None, this is a class library or console app
Operating system
Windows
Target framework
net10.0
Device and OS version
No response
IDE
No response
Did this work in an earlier version?
No response
Workaround
No response
Additional information
No response
- Langage dominant
- C#
- Étoiles
- 9
- Forks
- 2
- Merge moyen
- 7 h 7 min
- PR mergées (30 j)
- 75
Préparer son environnement
Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de reactiveui/ReactiveUI.Binding.SourceGenerators
-
Dependency DashboardOuverte
Difficulté 1/5 Moins d'une heure Accessibilité débutants 30/100
reactiveui/ReactiveUI.Binding.SourceGenerators#3 ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de reactiveui/ReactiveUI.Binding.SourceGenerators
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
Les mainteneurs répondent en général sous 1 jour
-
enhancement
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
Les mainteneurs répondent en général sous 1 jour
-
area:re1 bounty help wanted needs-game
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
StackExchange/StackExchange.Redis#3274 ·
Les mainteneurs répondent en général sous 1 jour
-
affected/v17 affected/v18 area/frontend type/bug
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
umbraco/Umbraco-CMS#24186 ·
Les mainteneurs répondent en général sous 2 jours