lholoubek / solo-toolkit

Collection of utilities for the 3DR Solo drone in a desktop app
MIT License
7 stars 4 forks source link

Logs folder should have a unique name #3

Open nickspeal opened 8 years ago

nickspeal commented 8 years ago

SoloLogPull created folders called: /Users/Nick/Documents/logs/2016-06-23_15_56_logs

Solo ToolKit v0.1.3 creates folders called: /Users/Nick/Documents/logs/2016_7_5_logs

As a result, if I try to pull logs twice in one day, I get an error.

Bonus: Sometimes with SoloLogPull, I'd try to pull logs, fail, then try again within a minute. In that case the log folder would have the same name. It'd be nice to append some suffix if the log folder is not unique (rather than popping the error that you do)

nickspeal commented 8 years ago

This should work here

return (date.getMonth() + 1).toString() + "-" + date.getDate() + "__" + date.getHours() + "-" + date.getMinutes() + "-" + date.getSeconds();

Not yet tested.

nickspeal commented 8 years ago

TODO, check if the time is in local time

nickspeal commented 8 years ago

Still not resolved in v0.1.8

bbasso commented 8 years ago

This is fixed!