X-Git-Url: http://git.iain.cx/?a=blobdiff_plain;ds=sidebyside;f=lib%2Freport.php;h=999bbc50c4e3c78aa73c1255d6fc421b4b91acf0;hb=refs%2Ftags%2Fuat-2016-03-29;hp=c58864dc770b18ea0669116926a2c8f010836096;hpb=7cbf38d98e84961ccab6cd8213e322fa775a2b9e;p=readifood.git
diff --git a/lib/report.php b/lib/report.php
index c58864d..999bbc5 100644
--- a/lib/report.php
+++ b/lib/report.php
@@ -60,17 +60,23 @@
return true;
}
- function show_order_report(&$order_state_ids) {
+ function show_order_report($from, &$order_state_ids) {
echo "
Orders by week
\n";
+ /* Handle missing weeks at the start of the range. */
+ $dbh = Propel::getConnection();
+ $sth = $dbh->prepare("select yearweek(:from)");
+ $sth->execute(array(':from' => $from));
+ list($first_week) = $sth->fetch();
+ $year_offset = substr($first_week, 0, 4);
+ $week_offset = substr($first_week, 4, 2) - 1;
+
$q = new OrderStateQuery;
$q->filterById($order_state_ids);
$q->withColumn('yearweek(updated)', 'week');
$q->withColumn('count(*)', 'count');
$q->addGroupByColumn('week')->orderByUpdated();
$rows = $q->find();
- $year_offset = 0;
- $week_offset = 0;
$week = 1;
$last_week = 0;
$total = 0;
@@ -81,15 +87,9 @@
Beware that week 201401 comes after 201352.
*/
$yearweek = $row->getWeek();
- if (! $week_offset) {
- $year_offset = substr($yearweek, 0, 4);
- $week_offset = substr($yearweek, 4, 2) - 1;
- }
- else {
- $y = substr($yearweek, 0, 4);
- $w = substr($yearweek, 4, 2);
- $week = (($y - $year_offset) * 52) + ($w - $week_offset);
- }
+ $y = substr($yearweek, 0, 4);
+ $w = substr($yearweek, 4, 2);
+ $week = (($y - $year_offset) * 52) + ($w - $week_offset);
$total += $row->getCount();
/* Fill in missing weeks. XXX */
for ($missing_week = $last_week + 1; $missing_week < $week; $missing_week++) {
@@ -246,7 +246,7 @@
return;
}
- show_order_report($order_state_ids);
+ show_order_report($from, $order_state_ids);
show_postcode_report($order_ids);
show_parcel_report($order_ids);
show_requester_report($order_ids);