Skip to content
This repository has been archived by the owner on Feb 26, 2025. It is now read-only.

Commit

Permalink
Issue 635: Add missing "All detail costs" column in yearly costs dash…
Browse files Browse the repository at this point in the history
…board export.

Test plan:

 - Check that the "All detail costs" column is currently missing from yearly
   costs CSV exports (although displayed in the yearly costs page)

 - Apply the patch

 - Check that the column is now in the CSV exports and the values are correct.
  • Loading branch information
veggiematts committed Feb 25, 2020
1 parent baab811 commit df3a4e0
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion resources/dashboard_yearly_costs_exports.php
Original file line number Diff line number Diff line change
Expand Up @@ -63,6 +63,7 @@ function array_to_csv_row($array) {
$columnHeaders[] = $costDetail['shortName'] . " / $i";
}
}
$columnHeaders[] = _("All cost details");
echo array_to_csv_row($columnHeaders);

$count = sizeof($results);
Expand All @@ -89,7 +90,7 @@ function array_to_csv_row($array) {
$dashboardValues[] = $result[$costDetail['shortName'] . " / $i"];
}
}

$dashboardValues[] = $result['costDetailsSum'];
echo array_to_csv_row($dashboardValues);
$currentCount++;
}
Expand Down

0 comments on commit df3a4e0

Please sign in to comment.