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

Feature Request: Support for different Data Format #61

Closed
Pisisip opened this issue Jan 7, 2016 · 8 comments
Closed

Feature Request: Support for different Data Format #61

Pisisip opened this issue Jan 7, 2016 · 8 comments

Comments

@Pisisip
Copy link

Pisisip commented Jan 7, 2016

I can't sync with EDSM because it say wrong data format. I have the Italian locale so the format of the date is DD/MM/YYYY and EDSM refuse the log with an error.

I don't know if it can be considered a feature that should implement the EDSM site or is better by your program.

@AnthorNet
Copy link

Hi,
Proper date format should be implemented in EDD and in UTC.

@gregmalcolm
Copy link
Contributor

The Italian format is the same format as British format. No complaints from there as far as I can tell. Not sure what else it could be though.

@Pisisip
Copy link
Author

Pisisip commented Jan 7, 2016

I have this error:

EDSM: Sending41 flightlog entries
EDSM sync ERROR:206:Date is not in the format YYYY-MM-DD HH:MM:SS
EDSM sync Done

cattura

@Pisisip
Copy link
Author

Pisisip commented Jan 7, 2016

It appears also this exception:

GetEDSMDistances exception:Stringa non riconosciuta come valore DateTime valido.
in System.DateTimeParse.Parse(String s, DateTimeFormatInfo dtfi, DateTimeStyles styles)
in EDDiscovery.DB.DistanceClass.ParseEDSM(String json, String& date)
in EDDiscovery.EDDiscoveryForm.GetEDSMDistances()

@finwen
Copy link
Contributor

finwen commented Jan 7, 2016

Can you go to the settings tab and enable EDSM logging.

And then add logfile here. You will find it in %localappdata%\EDDiscovery\Log

@Pisisip
Copy link
Author

Pisisip commented Jan 9, 2016

Here the log.
01.48.43; GET http://www.edsm.net/api-logs-v1/get-logs?startdatetime=2011-01-01+00.00.00&apiKey=*********&commanderName=Pisisip; 01.48.44; {"msgnum":100,"msg":"OK","logs":[]}; 01.48.44; GET http://www.edsm.net/api-logs-v1/set-log?systemName=V2578+Ophiuchii&commanderName=Pisisip&apiKey=******&dateVisited=2016-01-03+13.15.39; 01.48.44; {"msgnum":206,"msg":"Date is not in the format YYYY-MM-DD HH:MM:SS"}; 01.50.12; ExceptionRichiesta annullata: Timeout dell'operazione..;

----Messaggio originale----

Da: notifications@github.com

Data: 07/01/2016 23.46

A: "EDDiscovery/EDDiscovery"EDDiscovery@noreply.github.com

Cc: "Pisisip"pisi78@inwind.it

Ogg: Re: [EDDiscovery] Feature Request: Support for different Data Format (#61)

Can you go to the settings tab and enable EDSM logging.

And then add logfile here. You will find it in %localappdata%\EDDiscovery\Log


Reply to this email directly or view it on GitHub.

@finwen
Copy link
Contributor

finwen commented Jan 9, 2016

Thanks. I see the error now. Should be solved easy in next version

2016-01-09 1:54 GMT+01:00 Pisisip notifications@github.com:

Here the log.
01.48.43; GET
http://www.edsm.net/api-logs-v1/get-logs?startdatetime=2011-01-01+00.00.00&apiKey=cf47f429cb8c99a51d97b0f53e408a836bfbbd86&commanderName=Pisisip;
01.48.44; {"msgnum":100,"msg":"OK","logs":[]}; 01.48.44; GET
http://www.edsm.net/api-logs-v1/set-log?systemName=V2578+Ophiuchii&commanderName=Pisisip&apiKey=cf47f429cb8c99a51d97b0f53e408a836bfbbd86&dateVisited=2016-01-03+13.15.39;
01.48.44; {"msgnum":206,"msg":"Date is not in the format YYYY-MM-DD
HH:MM:SS"}; 01.50.12; ExceptionRichiesta annullata: Timeout
dell'operazione..;

----Messaggio originale----

Da: notifications@github.com

Data: 07/01/2016 23.46

A: "EDDiscovery/EDDiscovery"EDDiscovery@noreply.github.com

Cc: "Pisisip"pisi78@inwind.it

Ogg: Re: [EDDiscovery] Feature Request: Support for different Data Format
(#61)

Can you go to the settings tab and enable EDSM logging.

And then add logfile here. You will find it in
%localappdata%\EDDiscovery\Log


Reply to this email directly or view it on GitHub.


Reply to this email directly or view it on GitHub
#61 (comment)
.

@finwen
Copy link
Contributor

finwen commented Jan 9, 2016

starttime.ToString("yyyy-MM-dd HH:mm:ss", CultureInfo.InvariantCulture))

Needed to add Invariant culture. Otherwise time part was translated to ex 12.23.45 for cultures with . for time format.

@finwen finwen closed this as completed in 288c886 Jan 9, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants