x/stackalytics
Code Issues Proposed changes

Added fix for BUG 1697585. Modified the logic to remove repeated module

Browse Source 
names from the response array.

Closes Bug: 1697585
Modified File: dashboard/templates/_macros/module_details.html
Bug: The module names are repeated if the "Release" is selected as "All"
and any Official Module is selected.
Solution: In the response modules array the module names were repeated. 
Removed the duplicates from the array.

Change-Id: I91317961a24614b90106ea84eb61178aa7255788
This commit is contained in:
Arunkumar Ramadass 2017-06-19 16:09:12 +05:30
parent fa0eaf06a1
commit c8555000c4
1 changed files with 8 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
stackalytics/dashboard/templates/_macros/module_details.html
View File

@ -6,10 +6,17 @@
url: makeURI("/api/1.0/modules/{{ module }}", extra_options),
dataType: "json",
success: function (data) {
var obj = {};
var unique_array = [];
var module = data["module"];
for (var i=0; i < module.modules.length; i++) {
module.modules[i].uri = makeURI('/', {module: module.modules[i].module_name});
if(!obj.hasOwnProperty(module.modules[i].module_name)){
module.modules[i].uri = makeURI('/', {module: module.modules[i].module_name});
unique_array.push(module.modules[i]);
}
obj[module.modules[i].module_name] = module.modules[i];
}
module.modules=unique_array;
$("#module_details_template").tmpl(data["module"]).appendTo("#module_details_container");
}
});