reggna / silvertrout

Automatically exported from code.google.com/p/silvertrout
Other
1 stars 0 forks source link

Date and time error in PackageTracker #29

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
The line: 
    188:  if(event.date >= p.lastDate && event.time > p.lastTime)
and
    251:  if(pe.date >= p.lastDate && pe.time > p.lastTime) {
is not a correct way to distinguish whether a date and time is bigger then 
some other date and time.

What if the date is bigger, but not the time? Say that the last update was 
23:59 yesterday, and there is a new update on 00:01 today, that  event would 
not be added.

Original issue reported on code.google.com by reg...@gmail.com on 3 Dec 2009 at 6:13

GoogleCodeExporter commented 9 years ago
Fixed in revision 277

Original comment by tig...@gmail.com on 3 Dec 2009 at 6:57

GoogleCodeExporter commented 9 years ago
Row 252 is still defect.

Original comment by reg...@gmail.com on 3 Dec 2009 at 7:21

GoogleCodeExporter commented 9 years ago
This issue was closed by revision r278.

Original comment by tig...@gmail.com on 3 Dec 2009 at 9:02