Skip to content

A
atlas
Commit f92e06cd by gutkaBinit Committed by nixonrodrigues
Options

ATLAS-2945 UI - If a Term is assigned to an Category then to user is able to…

ATLAS-2945 UI - If a Term is assigned to an Category then to user is able to assign the same term again Signed-off-by: 's avatarnixonrodrigues <nixon@apache.org>
parent 4c7c911c
Showing with 29 additions and 40 deletions
dashboardv2/public/js/utils/CommonViewFunction.js
......@@ -16,7 +16,7 @@
* limitations under the License.
*/
define(['require', 'utils/Utils', 'modules/Modal', 'utils/Messages', 'utils/Globals', 'utils/Enums', 'moment'], function(require, Utils, Modal, Messages, Globals, Enums, moment) {
define(['require', 'utils/Utils', 'modules/Modal', 'utils/Messages', 'utils/Enums', 'moment'], function(require, Utils, Modal, Messages, Enums, moment) {
'use strict';
var CommonViewFunction = {};
......@@ -335,18 +335,6 @@ define(['require', 'utils/Utils', 'modules/Modal', 'utils/Messages', 'utils/Glob
}
if (!Enums.entityStateReadOnly[obj.status]) {
if (obj.guid) {
var newD = { "guid": obj.guid, "termLinks": obj.meanings };
Globals.termMeanings = Globals.termMeanings ? Globals.termMeanings : [];
if (Globals.termMeanings.length > 0) {
for (var x in Globals.termMeanings) {
if (Globals.termMeanings[x]['guid'] == obj.guid) {
Globals.termMeanings[x].termLinks = obj.meanings;
}
}
}
if (newD.termLinks.length > 0 && Globals.termMeanings == 0) {
Globals.termMeanings.push(newD);
}
addTerm += '<a href="javascript:void(0)" data-id="addTerm" class="btn btn-action btn-sm assignTag" data-guid="' + obj.guid + '" ><i class="fa fa-plus"></i></a>';
} else {
addTerm += '<a href="javascript:void(0)" data-id="addTerm" class="btn btn-action btn-sm assignTag"><i style="right:0" class="fa fa-plus"></i></a>';
......
dashboardv2/public/js/views/detail_page/DetailPageLayoutView.js
......@@ -198,7 +198,7 @@ define(['require',
this.generateTag([]);
}
if (collectionJSON.relationshipAttributes && collectionJSON.relationshipAttributes.meanings) {
this.generateTerm(collectionJSON.relationshipAttributes.meanings, collectionJSON);
this.generateTerm(collectionJSON.relationshipAttributes.meanings);
}
if (Globals.entityTypeConfList && _.isEmptyArray(Globals.entityTypeConfList)) {
this.ui.editButtonContainer.html(ButtonsTmpl({ btn_edit: true }));
......@@ -403,21 +403,9 @@ define(['require',
this.ui.propagatedTagList.html(propagatedTagListData);
},
generateTerm: function(data, obj) {
generateTerm: function(data) {
var that = this,
termData = "";
var newD = { "guid": obj.guid, "termLinks": obj.relationshipAttributes.meanings };
Globals.termMeanings = Globals.termMeanings ? Globals.termMeanings : [];
if (Globals.termMeanings.length > 0) {
for (var x in Globals.termMeanings) {
if (Globals.termMeanings[x]['guid'] == obj.guid) {
Globals.termMeanings[x].termLinks = obj.relationshipAttributes.meanings;
}
}
}
if (newD.termLinks.length > 0 && Globals.termMeanings == 0) {
Globals.termMeanings.push(newD);
}
_.each(data, function(val) {
if (val.relationshipStatus == "ACTIVE") {
termData += '<span class="btn btn-action btn-sm btn-icon btn-blue" title=' + val.displayText + ' data-id="termClick"><span>' + val.displayText + '</span><i class="fa fa-close" data-id="deleteTerm" data-guid="' + val.guid + '" data-type="term" title="Remove Term"></i></span>';
......@@ -458,13 +446,18 @@ define(['require',
});
},
onClickAddTermBtn: function(e) {
var that = this;
var that = this,
entityGuid = that.id,
associatedTerms = this.collection.first().get('entity').relationshipAttributes.meanings;
require(['views/glossary/AssignTermLayoutView'], function(AssignTermLayoutView) {
var view = new AssignTermLayoutView({
guid: that.id,
callback: function() {
that.fetchCollection();
},
associatedTerms: associatedTerms,
showLoader: that.showLoader.bind(that),
hideLoader: that.hideLoader.bind(that),
glossaryCollection: that.glossaryCollection
......
dashboardv2/public/js/views/glossary/AssignTermLayoutView.js
......@@ -59,7 +59,7 @@ define(['require',
* @constructs
*/
initialize: function(options) {
_.extend(this, _.pick(options, 'glossaryCollection', 'guid', 'callback', 'hideLoader', 'isCategoryView', 'categoryData', 'isTermView', 'termData', 'isAttributeRelationView', 'selectedTermAttribute'));
_.extend(this, _.pick(options, 'glossaryCollection', 'guid', 'callback', 'hideLoader', 'isCategoryView', 'categoryData', 'isTermView', 'termData', 'isAttributeRelationView', 'selectedTermAttribute', 'associatedTerms'));
var that = this;
this.options = options;
if (!this.isCategoryView && !this.isTermView && !this.isAttributeRelationView) {
......@@ -207,7 +207,8 @@ define(['require',
"isAssignCategoryView": that.isTermView,
"isAssignEntityView": that.isEntityView,
"isAssignAttributeRelationView": that.isAttributeRelationView,
"glossary": that.glossary
"glossary": that.glossary,
"associatedTerms": that.associatedTerms
}, that.options)));
});
},
......
dashboardv2/public/js/views/glossary/GlossaryDetailLayoutView.js
......@@ -320,8 +320,10 @@ define(['require',
});
}
}
var view = new AssignTermLayoutView({
categoryData: that.data,
associatedTerms: that.data && that.data.terms && that.data.terms.length > 0 ? that.data.terms : [],
isCategoryView: that.isCategoryView,
callback: function() {
that.getData();
......
dashboardv2/public/js/views/glossary/GlossaryLayoutView.js
......@@ -88,7 +88,7 @@ define(['require',
* @constructs
*/
initialize: function(options) {
_.extend(this, _.pick(options, 'guid', 'value', 'glossaryCollection', 'glossary', 'isAssignTermView', 'isAssignCategoryView', 'isAssignEntityView', 'isAssignAttributeRelationView'));
_.extend(this, _.pick(options, 'associatedTerms', 'guid', 'value', 'glossaryCollection', 'glossary', 'isAssignTermView', 'isAssignCategoryView', 'isAssignEntityView', 'isAssignAttributeRelationView'));
this.viewType = "term";
this.isAssignView = this.isAssignTermView || this.isAssignCategoryView || this.isAssignEntityView || this.isAssignAttributeRelationView;
this.bindEvents();
......@@ -249,7 +249,9 @@ define(['require',
generateData: function(opt) {
var that = this,
selectedGuid = that.guid,
associatedTerms = that.associatedTerms,
type = opt.type;
if (opt.type == this.viewType) {
this.query[opt.type].isNodeNotFoundAtLoad = true;
}
......@@ -339,16 +341,18 @@ define(['require',
});
}
if (type == "term" && obj.terms) {
var theTerm = _.find(Globals.termMeanings, function(obj, index) {
if (obj.guid == selectedGuid) {
_.each(obj.terms, function(term) {
if (associatedTerms) {
var associatedTermFound = _.find(associatedTerms, function(obj, index) {
if ((obj.termGuid ? obj.termGuid : obj.guid) == term.termGuid) {
return obj;
}
});
_.each(obj.terms, function(term) {
var includedTerms = _.map(theTerm && theTerm.termLinks, function(obj, index) {
return obj.termGuid || obj.guid;
});
if ((!includedTerms.includes(term.termGuid))) {
if (associatedTermFound) {
return;
}
}
var typeName = term.typeName || "GlossaryTerm",
guid = term.termGuid,
termObj = {
......@@ -370,7 +374,6 @@ define(['require',
})
parent.children.push(termObj);
}
});
}
return parent;
......
dashboardv2/public/js/views/search/SearchResultLayoutView.js
......@@ -944,10 +944,12 @@ define(['require',
},
onClickAddTermBtn: function(e) {
var that = this,
entityGuid = $(e.currentTarget).data("guid");
entityGuid = $(e.currentTarget).data("guid"),
associatedTerms = this.searchCollection.find({ guid: entityGuid }).get('meanings');
require(['views/glossary/AssignTermLayoutView'], function(AssignTermLayoutView) {
var view = new AssignTermLayoutView({
guid: entityGuid,
associatedTerms: associatedTerms,
callback: function() {
that.fetchCollection();
},
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment