jceddy / DailyArenaDeckAdvisor

A Deck Advisor companion App for Magic Arena.
GNU General Public License v3.0
18 stars 7 forks source link

Unhandled Exception in loadTask (Window_Loaded - Main Application) #247

Closed jceddy closed 3 years ago

jceddy commented 3 years ago

An unhandled exception was detected.

Exception:

System.AggregateException: One or more errors occurred. ---> System.FormatException: String was not recognized as a valid DateTime. at System.DateTimeParse.Parse(String s, DateTimeFormatInfo dtfi, DateTimeStyles styles) at DailyArena.Common.Core.Database.CardDatabase.UpdateCardDatabase() at DailyArena.Common.Core.Database.CardDatabase.Initialize(Boolean useProtectedData) at DailyArena.DeckAdvisor.MainWindow.b94_0() at System.Threading.Tasks.Task.Execute() --- End of inner exception stack trace --- ---> (Inner Exception #0) System.FormatException: String was not recognized as a valid DateTime. at System.DateTimeParse.Parse(String s, DateTimeFormatInfo dtfi, DateTimeStyles styles) at DailyArena.Common.Core.Database.CardDatabase.UpdateCardDatabase() at DailyArena.Common.Core.Database.CardDatabase.Initialize(Boolean useProtectedData) at DailyArena.DeckAdvisor.MainWindow.b94_0() at System.Threading.Tasks.Task.Execute()<---

%log2021032421.txt%

%Player.log%

zdeyn commented 3 years ago

I'm getting the same thing

frederiktoft commented 3 years ago

Also just got the same issue as soon as the program starts up.

jceddy commented 3 years ago

Can one of you attach your Player.log file here?

frederiktoft commented 3 years ago

Where is that log located?

jceddy commented 3 years ago

C:\Users\[UserName]\AppData\LocalLow\Wizards Of The Coast\MTGA

frederiktoft commented 3 years ago

@jceddy Here you go.

Player.log

sepam21 commented 3 years ago

The same thing here This is my log Player.log

KatieA414 commented 3 years ago

I am new to Daily Deck Builder and just now installed it. In investigating all of the areas of it, when I clicked on Historic (bo3) I was thrown into an error message that sent me to this web site. I cannot get past this error each time I reload the program. What do I do now to get the program back in operation again?

DeadDraw64 commented 3 years ago

Hey. i am having issues with not being able to load anything.

i used this before on windows 7 but this was about a year ago around m21 release date no problem. but now i can't use. Log20210420_123709.log

love the program by the way really helpful. wish i could still use it

jceddy commented 3 years ago

Sorry I've been super busy folks, and haven't gotten a chance to debug this with the logs people have given me. I promise I will do so this week.

jceddy commented 3 years ago

@jceddy Here you go.

Player.log

I don't see any collection information in this Player.log file.

Do you have Detailed Logs turned on in Arena?

jceddy commented 3 years ago

I have not been able to replicate this issue, can I ask one of you to try deleting all the files in this location:

C:\Users\{User}\AppData\LocalLow\DailyArena\DailyArenaDeckAdvisor

And then try running the program again?

DeadDraw64 commented 3 years ago

Hello, thanks for the reply.

As soon as i start up the deck advisor i get the handling exception error would ypy like a new log

Get Outlook for iOShttps://aka.ms/o0ukef


From: jceddy @.> Sent: Thursday, April 22, 2021 8:53:25 PM To: jceddy/DailyArenaDeckAdvisor @.> Cc: DeadDraw64 @.>; Comment @.> Subject: Re: [jceddy/DailyArenaDeckAdvisor] Unhandled Exception in loadTask (Window_Loaded - Main Application) (#247)

I have not been able to replicate this issue, can I ask one of you to try deleting all the files in this location:

C:\Users{User}\AppData\LocalLow\DailyArena\DailyArenaDeckAdvisor

And then running the program again?

— You are receiving this because you commented. Reply to this email directly, view it on GitHubhttps://github.com/jceddy/DailyArenaDeckAdvisor/issues/247#issuecomment-825141615, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ATYKMJWWTETE6QAQ4J53ZADTKB5DLANCNFSM4ZYEBTUQ.

DeadDraw64 commented 3 years ago

Hello Mate.

I tried your suggestion still unhandled exception sending data to server

Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10

From: @.> Sent: 22 April 2021 20:53 To: @.> Cc: @.>; @.> Subject: Re: [jceddy/DailyArenaDeckAdvisor] Unhandled Exception in loadTask (Window_Loaded - Main Application) (#247)

I have not been able to replicate this issue, can I ask one of you to try deleting all the files in this location:

C:\Users{User}\AppData\LocalLow\DailyArena\DailyArenaDeckAdvisor

And then running the program again?

— You are receiving this because you commented. Reply to this email directly, view it on GitHubhttps://github.com/jceddy/DailyArenaDeckAdvisor/issues/247#issuecomment-825141615, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ATYKMJWWTETE6QAQ4J53ZADTKB5DLANCNFSM4ZYEBTUQ.

jefersonnavarro commented 3 years ago

I have not been able to replicate this issue, can I ask one of you to try deleting all the files in this location:

C:\Users{User}\AppData\LocalLow\DailyArena\DailyArenaDeckAdvisor

And then try running the program again?

I did it but the problem persists.

sepam21 commented 3 years ago

firstChanceExceptions2021042308.txt log2021042308.txt

I deleted the entire folder C:\Users{User}\AppData\LocalLow\DailyArena\DailyArenaDeckAdvisor and start again:

Actually this is the first time I downloaded your program and it turns out that I have not been able to use it successfully yet.

P.S. I use Russian in windows 10, but the language in MTGA is English.

frederiktoft commented 3 years ago

I deleted the files here as advised :

C:\Users{User}\AppData\LocalLow\DailyArena\DailyArenaDeckAdvisor

Re-ran MTG Arena with detailed logs on. Then ran Daily Arena, but still the same problem. Here is the most current Player.log: Player.log

frederiktoft commented 3 years ago

Decided to run the code locally and debug it, and figured out it was luckily a simple issue: DailyArena.Common.Core.CardDatabase.cs, line 379: set.Value["rotation"] == null ? DateTime.MaxValue : DateTime.Parse((string)set.Value["rotation"]), Should be changed to: set.Value["rotation"] == null ? DateTime.MaxValue : DateTime.Parse((string)set.Value["rotation"], System.Globalization.CultureInfo.InvariantCulture), That fixes the issue and the application boots up fine afterwards. Should I do a direct commit, or do a pull request?

You should use InvariantCulture in more places where you parse dates, otherwise you could run into the same issue again.

jceddy commented 3 years ago

thanks, man, you can do a pull request, or I will just go through and change that anywhere I find it

jceddy commented 3 years ago

Actually, I just pushed this change to the common project

frederiktoft commented 3 years ago

Great! 😎🦄

jceddy commented 3 years ago

Update to the main program solution is pushed now, too...will be putting up a new version ASAP

jceddy commented 3 years ago

Version 1.0.9.1 has been pushed, can folks run the new version and see if it fixes their issues?

sepam21 commented 3 years ago

My problem is gone - everything works great. Thanks to all :)

DeadDraw64 commented 3 years ago

All sorted just like the old days 😊

Thanks a Billion Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10

From: @.> Sent: 23 April 2021 14:13 To: @.> Cc: @.>; @.> Subject: Re: [jceddy/DailyArenaDeckAdvisor] Unhandled Exception in loadTask (Window_Loaded - Main Application) (#247)

Update to the main program solution is pushed now, too...will be putting up a new version ASAP

— You are receiving this because you commented. Reply to this email directly, view it on GitHubhttps://github.com/jceddy/DailyArenaDeckAdvisor/issues/247#issuecomment-825649639, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ATYKMJSLB7CHL7MZQXQTHU3TKFXARANCNFSM4ZYEBTUQ.

jceddy commented 3 years ago

Cool. I'm going to close this issue, please open a new one if you run into any issues.