Merge pull request silverstripe#824 from nyeholt/tab_null_pointer

BUG Fixed possible reference to null string
chillu · Sep 26, 2012 · 6114883 · 6114883
2 parents 33f67e1 + da70dc4
commit 6114883
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/admin/javascript/LeftAndMain.js b/admin/javascript/LeftAndMain.js
@@ -945,8 +945,8 @@ jQuery.noConflict();
 			 */
 			rewriteHashlinks: function() {
 				$(this).find('ul a').each(function() {
-					var href = $(this).attr('href').replace(/.*(#.*)/, '$1');
-					if(href) $(this).attr('href', href);
+					var href = $(this).attr('href');
+					if(href) $(this).attr('href', href.replace(/.*(#.*)/, '$1'));
 				});
 			}
 		});