search

sbstjn / timesheet.js

JavaScript library for HTML5 & CSS3 time sheets
https://sbstjn.github.io/timesheet.js
MIT License
6.97k stars 549 forks source link

whats the 39 for ? #45

Open gautamz07 opened 9 years ago

gautamz07 commented 9 years ago

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.

farfromunique commented 8 years ago

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.