Data Sources reports bug fixes (#2972)

* Fix frequency chart bug
* Default loading spinner to false. It is set to true if there is something to load.
* This solves the loading spinner issue specifically for the data source reports.

js/components/reports/classes/Treemap.js

@@ -146,10 +146,10 @@ define([
 			data
 		}) {
 			const normalizedData = atlascharts.chart.normalizeDataframe(ChartUtils.normalizeArray(data, true));
+			let tableData = [];
 
 			if (!normalizedData.empty) {
 				let distinctConceptIds = new Set([]);
-				let tableData = [];
 				// Make the values unique per https://github.com/OHDSI/Atlas/issues/913
 				normalizedData.conceptPath.forEach((d, i) => {
 					if (!distinctConceptIds.has(normalizedData.conceptId[i])) {
@@ -165,14 +165,13 @@ define([
 						});
 					}
 				});
-				this.tableData(tableData); 
 				this.treeData(normalizedData);
-
-				return {
-					data
-				};
 			}
+			this.tableData(tableData);
 
+			return {
+				data
+			};
 		}
 
 		getData() {

js/components/reports/reportDrilldown.js

@@ -234,7 +234,7 @@ define([
 					this.chartFormats.frequencyDistribution.yMax = yScaleMax;
 					this.chartFormats.frequencyDistribution.xLabel = ko.pureComputed(function () {
 						return ko.i18n('dataSources.drilldown.chartFormat.frequencyDistribution.xLabel1', 'Count ("x" or more ')() +
-							report() +
+							report +
 							ko.i18n('dataSources.drilldown.chartFormat.frequencyDistribution.xLabel2', 's)')();
 					});
 					this.chartFormats.frequencyDistribution.ticks = Math.min(5, frequencyHistogram.INTERVALS);