Skip to content

Commit

Permalink
Fix(report): invoices report uses display_name
Browse files Browse the repository at this point in the history
This commit fixes a tiny bug introduced during the transition to
display_name where the user's name moved from the `createdBy` property
to the `display_name` property.
  • Loading branch information
Jonathan Niles committed Sep 6, 2016
1 parent be8591c commit 7c534e9
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 4 deletions.
2 changes: 1 addition & 1 deletion server/config/routes.js
Original file line number Diff line number Diff line change
Expand Up @@ -350,7 +350,7 @@ exports.configure = function configure(app) {

// route for invoice Report
app.get('/invoices/patient/report', patientInvoice.getPatientInvoice);

// reports API: Invoices (receipts)
app.get('/reports/invoices/:uuid', financeReports.receipts.invoices);
app.get('/reports/cash/:uuid', financeReports.receipts.cash);
Expand Down
2 changes: 1 addition & 1 deletion server/controllers/finance/reports/list.handlebars
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@
<td>{{ patientNames }}</td>
<td>{{ currency cost currency_id }}</td>
<td>{{ serviceName }}</td>
<td>{{ createdBy }}</td>
<td>{{ display_name }}</td>
</tr>
{{/each}}
</tbody>
Expand Down
2 changes: 0 additions & 2 deletions server/controllers/finance/reports/list.js
Original file line number Diff line number Diff line change
Expand Up @@ -22,8 +22,6 @@ exports.build = build;
* @return {object} promise
*/
function build(data, request) {


let queryString = request.query;
let renderTarget = queryString && queryString.renderer || defaultRender;

Expand Down

0 comments on commit 7c534e9

Please sign in to comment.