timesheet.js icon indicating copy to clipboard operation
timesheet.js copied to clipboard

whats the 39 for ?

Open gautamz07 opened this issue 10 years ago • 1 comments

Not really an issue , this is an awesome plugin ! never knew you could do so much , with so little , one question , what the 39 in the code for ?

https://github.com/sbstjn/timesheet.js/blob/master/source/javascripts/timesheet.js#L35

On the above line i mean.

Thank you.

gautamz07 avatar Sep 15 '15 19:09 gautamz07

The portion of the line is:

Math.round((cur.end-cur.start)/1000/60/60/24/39)

That takes the time in MS, and then divides by 1000 (seconds), 60 (Minutes), 60 (hours), 24 (days), and 39 (months???)

39 is in the place where days in a month should go, but it doesn't make sense. I think this could be a bug.

farfromunique avatar Oct 18 '15 17:10 farfromunique