Fixes OHDSI#209

1. explicitly declare variables when "use strict" is applied. 2. Fixed cohort report's report names so that they are found properly when doing the report render.
tnovo · Sep 22, 2016 · 6e1521e · 6e1521e
1 parent c483c78
commit 6e1521e
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/js/app.js b/js/app.js
@@ -867,7 +867,7 @@ define(['jquery', 'knockout', 'jnj_chart', 'd3', 'ohdsi.util', 'appConfig', 'fac
 			reportKey: 'Condition Eras',
 			analyses: [1001, 1000, 1007, 1006, 1004, 1002, 116, 117, 1]
         }, {
-			name: "Conditions By Index",
+			name: "Conditions by Index",
 			reportKey: 'Conditions by Index',
 			analyses: [1700, 1800, 1801, 1802, 1803, 1804, 1805, 1806, 1807, 1808, 1809, 1810, 1811, 1812, 1813, 1814, 1815, 1816, 1820, 1821, 1830, 1831, 1840, 1841, 1850, 1851, 1860, 1861, 1870, 1871, 116, 117, 1]
         }, {

diff --git a/js/jnj.chart.js b/js/jnj.chart.js
@@ -990,7 +990,7 @@
 			};
 			var options = $.extend({}, defaults, options);
 
-			tooltipBuilder = line_defaultTooltip(options.xLabel || "x", options.xFormat, function (d) {
+			var tooltipBuilder = line_defaultTooltip(options.xLabel || "x", options.xFormat, function (d) {
 					return d[options.xValue];
 				},
 				options.yLabel || "y", options.yFormat,
@@ -2975,7 +2975,7 @@
 
 				$('.grouper').remove();
 
-				top_nodes = treemap.nodes(target)
+				var top_nodes = treemap.nodes(target)
 					.filter(function (d) {
 						return d.parent == target;
 					});