Merge pull request silverstripe#3418 from halkyon/remove_supression

Remove @ suppression operator on session_start()
mertenhanisch · Aug 22, 2014 · c8c1833 · c8c1833
2 parents 0d169d5 + 0b5599e
commit c8c1833
Showing 1 changed file with 1 addition and 3 deletions.
diff --git a/control/Session.php b/control/Session.php
@@ -376,10 +376,8 @@ public function inst_start($sid = null) {
 			// seperate (less secure) session for non-HTTPS requests
 			if($secure) session_name('SECSESSID');
 
-			// @ is to supress win32 warnings/notices when session wasn't cleaned up properly
-			// There's nothing we can do about this, because it's an operating system function!
 			if($sid) session_id($sid);
-			@session_start();
+			session_start();
 
 			$this->data = isset($_SESSION) ? $_SESSION : array();
 		}