move logout_user() to UserHelper

classes/api.php

@@ -81,7 +81,7 @@ class API extends Handler {
 	}
 
 	function logout() {
-		Pref_Users::logout_user();
+		UserHelper::logout();
 		$this->wrap(self::STATUS_OK, array("status" => "OK"));
 	}
 

classes/handler/public.php

@@ -286,7 +286,7 @@ class Handler_Public extends Handler {
 
 	function logout() {
 		if (validate_csrf($_POST["csrf_token"])) {
-			Pref_Users::logout_user();
+			UserHelper::logout();
 			header("Location: index.php");
 		} else {
 			header("Content-Type: text/json");

classes/pref/users.php

@@ -418,15 +418,4 @@ class Pref_Users extends Handler_Protected {
 				return $default;
 		}
 
-	static function logout_user() {
-		if (session_status() === PHP_SESSION_ACTIVE)
-			session_destroy();
-
-		if (isset($_COOKIE[session_name()])) {
-		   setcookie(session_name(), '', time()-42000, '/');
-
-		}
-		session_commit();
-	}
-
 }

classes/userhelper.php

@@ -105,7 +105,7 @@ class UserHelper {
 				}
 
 				if (empty($_SESSION["uid"])) {
-					Pref_Users::logout_user();
+					UserHelper::logout();
 
 					Handler_Public::render_login_form();
 					exit;
@@ -157,4 +157,16 @@ class UserHelper {
 
 		return false;
 	}
+
+	static function logout() {
+		if (session_status() === PHP_SESSION_ACTIVE)
+			session_destroy();
+
+		if (isset($_COOKIE[session_name()])) {
+		   setcookie(session_name(), '', time()-42000, '/');
+
+		}
+		session_commit();
+	}
+
 }