Skip to content
This repository has been archived by the owner on Jan 9, 2023. It is now read-only.

#1146 - Fixed the issue of number of patients under Reports not showi… #1151

Merged
merged 1 commit into from
Aug 31, 2017
Merged

#1146 - Fixed the issue of number of patients under Reports not showi… #1151

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
#1146 - Fixed the issue of number of patients under Reports not showi… 
…ng as whole number.
  • Loading branch information
@SiddharthaSarma
SiddharthaSarma committed Aug 26, 2017
commit 7010f53765684e81d73d7b3010f438f33f1cde36
6 changes: 3 additions & 3 deletions app/patients/reports/controller.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -660,16 +660,16 @@ export default AbstractReportController.extend(PatientDiagnosis, PatientVisits,
patientName: visit.get('patient.displayName'),
admissionDate: visit.get('startDate'),
dischargeDate: visit.get('endDate'),
patientDays: daysDiff
patientDays: this._numberFormat(daysDiff, true)
}, false, reportColumns);
}
return previousValue += daysDiff;
}.bind(this), 0);
if (detailed) {
this._addReportRow({ patientDays: `Total: ${this._numberFormat(patientDays)}` }, true, reportColumns);
this._addReportRow({ patientDays: `Total: ${this._numberFormat(patientDays, true)}` }, true, reportColumns);

} else {
this._addReportRow({ total: patientDays }, false, reportColumns);
this._addReportRow({ total: this._numberFormat(patientDays, true) }, false, reportColumns);
}
this._finishReport(reportColumns);
},
Expand Down