diff --git a/main/inc/lib/main_api.lib.php b/main/inc/lib/main_api.lib.php
index 0d21ea1c36..f1e4cf4538 100644
--- a/main/inc/lib/main_api.lib.php
+++ b/main/inc/lib/main_api.lib.php
@@ -1394,18 +1394,13 @@ function api_display_tool_view_option()
 	// Exceptions apply for all multi-frames pages
 	if (strpos($_SERVER['REQUEST_URI'],'chat/chat_banner.php')!==false)
 	{	//the chat is a multiframe bit that doesn't work too well with the student_view, so do not show the link
-		$sourceurl = '';
+		return '';
 	}
 	if(strpos($_SERVER['REQUEST_URI'],'document/headerpage.php')!==false)
 	{
 		//$sourceurl = str_replace('document/headerpage.php','document/showinframes.php',$_SERVER['REQUEST_URI']);
 		//showinframes doesn't handle student view anyway...
-		$sourceurl = '';
-	}
-	//exit if sourceurl is empty
-	if(empty($sourceurl))
-	{
-		return $sourceurl;
+		return '';
 	}
 
 	// check if the $_SERVER['REQUEST_URI'] contains already url parameters (thus a questionmark)