diff --git a/code/GroupSubsites.php b/code/GroupSubsites.php index f398f58..64cb1df 100644 --- a/code/GroupSubsites.php +++ b/code/GroupSubsites.php @@ -99,10 +99,10 @@ class GroupSubsites extends DataObjectDecorator implements PermissionProvider { */ function alternateTreeTitle() { if($this->owner->AccessAllSubsites) { - return $this->owner->Title . ' (global group)'; + return htmlspecialchars($this->owner->Title, ENT_QUOTES) . ' (global group)'; } else { $subsites = Convert::raw2xml(implode(", ", $this->owner->Subsites()->column('Title'))); - return $this->owner->Title . " ($subsites)"; + return htmlspecialchars($this->owner->Title) . " ($subsites)"; } } diff --git a/code/LeftAndMainSubsites.php b/code/LeftAndMainSubsites.php index a1835ad..e7a0f4d 100644 --- a/code/LeftAndMainSubsites.php +++ b/code/LeftAndMainSubsites.php @@ -20,7 +20,7 @@ class LeftAndMainSubsites extends Extension { */ function getCMSTreeTitle() { $subsite = Subsite::currentSubSite(); - return $subsite ? $subsite->Title : null; + return $subsite ? htmlspecialchars($subsite->Title, ENT_QUOTES) : 'Site Content'; } function updatePageOptions(&$fields) { @@ -99,7 +99,7 @@ class LeftAndMainSubsites extends Extension { foreach($list as $subsite) { $selected = $subsite->ID == $currentSubsiteID ? ' selected="selected"' : ''; - $output .= "\n"; + $output .= "\n"; } $output .= '';