Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CalculateCaseTotalsOnSuite causes FormatException #52

Open
sweco-sejsau-zz opened this issue Jul 11, 2016 · 2 comments
Open

CalculateCaseTotalsOnSuite causes FormatException #52

sweco-sejsau-zz opened this issue Jul 11, 2016 · 2 comments
Assignees
Labels

Comments

@sweco-sejsau-zz
Copy link
Contributor

Hey,

Seems like the CalculateCaseTotalsOnSuite is dependent on what decimal separator you have configured on your Windows (I'm from Sweden and we have ',' as default). Happens on line 351.

Here's a suggestion for solve this problem:
suite.time = cases.Sum(x => Convert.ToDouble(x.time, CultureInfo.InvariantCulture)).ToString();

Cheers, Joel

@sharadkjaiswal
Copy link
Contributor

@sweco-sejsau, thanks for looking into it and highlighting the issue. Please propose a pull request for your suggested changes and we would merge it.

@sharadkjaiswal sharadkjaiswal self-assigned this Jul 11, 2016
@sweco-sejsau-zz
Copy link
Contributor Author

Ok done!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants