Fixed display of scraped results

close #10
jwillmer · Jun 3, 2017 · cb3d802 · cb3d802
1 parent e0ad5fd
commit cb3d802
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 4 deletions.
diff --git a/extension/scripts/Controller.js b/extension/scripts/Controller.js
@@ -1089,9 +1089,14 @@ SitemapController.prototype = {
 		$("#scrape-sitemap-config input").prop('disabled', true);
 
         chrome.runtime.sendMessage(request, function (selectors) {
-            // table selector can dynamically add columns (addMissingColumns Feature)
-            sitemap.selectors = selectors;
-			this.browseSitemapData();
+            // table selector can dynamically add columns
+            // replace current selector (columns) with the dynamicly created once
+            sitemap.selectors = [];
+            for (var n = 0; n < selectors.length; n++){
+                var selector = selectors[n];
+                sitemap.selectors.push(new Selector(selector));
+            }
+            this.browseSitemapData();
 		}.bind(this));
 		return false;
 	},

diff --git a/extension/scripts/Sitemap.js b/extension/scripts/Sitemap.js
@@ -184,7 +184,7 @@ Sitemap.prototype = {
 
         var uniqueColumns = [];
         $.each(columns, function (i, e) {
-            if ($.inArray(e, result) == -1) result.push(e);
+            if ($.inArray(e, uniqueColumns) == -1) uniqueColumns.push(e);
         });
 
         return uniqueColumns;