Skip to content
Snippets Groups Projects
Commit f83e67c4 authored by juanf's avatar juanf
Browse files

SSDM-4453 : Links are now forced to use plain text display names, solves...

SSDM-4453 : Links are now forced to use plain text display names, solves issues with HTML/Javascript injection and webpage refreshes.

SVN: 37421
parent 8b20b0db
No related branches found
No related tags found
No related merge requests found
...@@ -822,7 +822,8 @@ var FormUtil = new function() { ...@@ -822,7 +822,8 @@ var FormUtil = new function() {
var click = function() { var click = function() {
mainController.changeView(view, permIdOrIdentifier, true); mainController.changeView(view, permIdOrIdentifier, true);
} }
var link = $("<a>", { "href" : href, "class" : "browser-compatible-javascript-link" }).append(displayName); displayName = String(displayName).replace(/<(?:.|\n)*?>/gm, ''); //Clean any HTML tags
var link = $("<a>", { "href" : href, "class" : "browser-compatible-javascript-link" }).text(displayName);
link.click(click); link.click(click);
return link; return link;
} }
......
...@@ -179,7 +179,8 @@ function SideMenuWidgetView(sideMenuWidgetController, sideMenuWidgetModel) { ...@@ -179,7 +179,8 @@ function SideMenuWidgetView(sideMenuWidgetController, sideMenuWidgetModel) {
this.getLinkForNode = function(displayName, menuId, view, viewData) { this.getLinkForNode = function(displayName, menuId, view, viewData) {
var href = Util.getURLFor(menuId, view, viewData); var href = Util.getURLFor(menuId, view, viewData);
var $menuItemLink = $("<a>", {"href": href, "class" : "browser-compatible-javascript-link browser-compatible-javascript-link-tree" }).append(displayName); displayName = String(displayName).replace(/<(?:.|\n)*?>/gm, ''); //Clean any HTML tags
var $menuItemLink = $("<a>", {"href": href, "class" : "browser-compatible-javascript-link browser-compatible-javascript-link-tree" }).text(displayName);
return $menuItemLink[0].outerHTML; return $menuItemLink[0].outerHTML;
} }
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment