From: avl <alex.litovchenko@gmail.com>
Date: Fri, 28 Nov 2008 14:28:53 +0000 (+0200)
Subject: refactor: move authAdmin into static class
X-Git-Tag: 0.5~448
X-Git-Url: https://git.andy128k.dev/?a=commitdiff_plain;h=48fc809a62b6e2400a8bf4c113692d7a5afd785e;p=ipf.git

refactor: move authAdmin into static class
---

diff --git a/ipf/admin/app.php b/ipf/admin/app.php
index ba5a905..94c75a8 100644
--- a/ipf/admin/app.php
+++ b/ipf/admin/app.php
@@ -18,4 +18,18 @@ class IPF_Admin_App extends IPF_Application{
             array('regex'=>'logout/$#i', 'func'=>'IPF_Admin_Views_Logout'),
         );
     }
+
+	static function checkAdminAuth($request){
+	    $ok = true;
+	    if ($request->user->isAnonymous())
+	        $ok = false;
+	    elseif ( (!$request->user->is_staff) && (!$request->user->is_superuser) )
+	        $ok = false;
+
+	    if ($ok)
+	        return true;
+	    else
+	        return new IPF_HTTP_Response_Redirect(IPF_HTTP_URL_urlForView('IPF_Admin_Views_Login'));
+	}
+
 }
\ No newline at end of file
diff --git a/ipf/admin/views.php b/ipf/admin/views.php
index 9d1d72f..331eaa7 100644
--- a/ipf/admin/views.php
+++ b/ipf/admin/views.php
@@ -1,20 +1,7 @@
 <?php
 
-function checkAdminAuth($request){
-    $ok = true;
-    if ($request->user->isAnonymous())
-        $ok = false;
-    elseif ( (!$request->user->is_staff) && (!$request->user->is_superuser) )
-        $ok = false;
-
-    if ($ok)
-        return true;
-    else
-        return new IPF_HTTP_Response_Redirect(IPF_HTTP_URL_urlForView('IPF_Admin_Views_Login'));
-}
-
 function IPF_Admin_Views_Index($request, $match){
-    $ca = checkAdminAuth($request);
+    $ca = IPF_Admin_App::checkAdminAuth($request);
     if ($ca!==true) return $ca;
 
     $apps = array();
@@ -66,7 +53,7 @@ function IPF_Admin_Views_Index($request, $match){
 }
 
 function IPF_Admin_Views_ListItems($request, $match){
-    $ca = checkAdminAuth($request);
+    $ca = IPF_Admin_App::checkAdminAuth($request);
     if ($ca!==true) return $ca;
 
     $lmodel = $match[2];
@@ -83,7 +70,7 @@ function IPF_Admin_Views_ListItems($request, $match){
 }
 
 function IPF_Admin_Views_EditItem($request, $match){
-    $ca = checkAdminAuth($request);
+    $ca = IPF_Admin_App::checkAdminAuth($request);
     if ($ca!==true) return $ca;
 
     $lapp = $match[1];
@@ -104,7 +91,7 @@ function IPF_Admin_Views_EditItem($request, $match){
 }
 
 function IPF_Admin_Views_DeleteItem($request, $match){
-    $ca = checkAdminAuth($request);
+    $ca = IPF_Admin_App::checkAdminAuth($request);
     if ($ca!==true) return $ca;
 
     $lapp = $match[1];
@@ -126,7 +113,7 @@ function IPF_Admin_Views_DeleteItem($request, $match){
 
 
 function IPF_Admin_Views_AddItem($request, $match){
-    $ca = checkAdminAuth($request);
+    $ca = IPF_Admin_App::checkAdminAuth($request);
     if ($ca!==true) return $ca;
 
     $lapp = $match[1];
@@ -144,7 +131,7 @@ function IPF_Admin_Views_AddItem($request, $match){
 }
 
 function IPF_Admin_Views_ChangePassword($request, $match){
-    $ca = checkAdminAuth($request);
+    $ca = IPF_Admin_App::checkAdminAuth($request);
     if ($ca!==true) return $ca;
 
     $lapp = 'auth';