Closed (fixed)
Project:
Project
Version:
5.x-1.x-dev
Component:
Usage statistics
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
13 Aug 2007 at 23:58 UTC
Updated:
28 Aug 2007 at 03:33 UTC
The project_usage_process_weekly() function called by the cron job expects that the array of timestamps returned by project_usage_get_weeks_since() will begin with an index of 0. But instead it is returning an array with an initial index of 0 due to a misunderstanding on my part. I thought that $foo[$i] = $i++ would give array(0 => 0); rather than array(1 => 0);. As a result the first week in the week's worth of stats wasn't being computed correctly.
| Comment | File | Size | Author |
|---|---|---|---|
| usage_usage_wrong_offset.patch | 744 bytes | drewish |
Comments
Comment #1
drewish commentedI really would like to get this committed and on d.o before we try to run the weekly stats for the first time.
Comment #2
dwwLooked reasonable, and you say it's RTBC, so I committed to HEAD and installed on d.o.
Comment #3
(not verified) commented