diff --git a/src/main/webapp/js/goatControllers.js b/src/main/webapp/js/goatControllers.js index 56c705097..2cb5ea69f 100644 --- a/src/main/webapp/js/goatControllers.js +++ b/src/main/webapp/js/goatControllers.js @@ -13,11 +13,11 @@ var goatMenu = function($scope, $http, $modal, $log, $templateCache) { var menuItems = goat.utils.addMenuClasses(goatConstants.menuPrefix.concat(menuData.data)); //top-tier 'categories' for (var i=0;i> id - menuItems[i].children[j].children[k].id = menuItems[i].children[j].children[k].name.replace(/\s|\(|\)/g,''); + menuItems[i].children[j].children[k].id = goat.utils.makeId(menuItems[i].children[j].children[k].name); + //menuItems[i].children[j].children[k].id = menuItems[i].children[j].children[k].name.replace(/\s|\(|\)/g,''); //handle selected Menu state if (menuItems[i].children[j].children[k].id === $scope.curMenuItemSelected) { menuItems[i].children[j].children[k].selectedClass = goatConstants.selectedMenuClass; diff --git a/src/main/webapp/js/goatUtil.js b/src/main/webapp/js/goatUtil.js index fc07803f8..e8a551fb6 100644 --- a/src/main/webapp/js/goatUtil.js +++ b/src/main/webapp/js/goatUtil.js @@ -87,6 +87,9 @@ goat.utils = { $('ul li.selected a.selected').removeClass(goatConstants.selectedMenuClass) $('#'+id).addClass(goatConstants.selectedMenuClass); $('#'+id).parent().addClass(goatConstants.selectedMenuClass); + }, + makeId: function (lessonName) { + return lessonName.replace(/\s|\(|\)|\!|\:|\;|\@|\#|\$|\%|\^|\&|\*/g,'');//TODO move the replace routine into util function } };