Changeset 1091 for trunk/wifidog-auth/wifidog/classes/Network.php

Timestamp:

09/06/06 10:58:06 (5 years ago)

Author:

benoitg

Message:

* Remove most inclusions of MainUI in classes. That completely
broke authentication (circular dependencies). Fixes 245.

Unfixes 242. Exceptions should not be caught in the context

they are thrown in. Else there no point in throwing exception in the
first place.
* HyperLink?.php: Fix bug where a link with identical text as
it's link would see the text replaced by the clickthrough-tracked
equivalent.

Files:

• trunk/wifidog-auth/wifidog/classes/Network.php (modified) (3 diffs)

trunk/wifidog-auth/wifidog/classes/Network.php

@@ -54 +54 @@
 require_once('classes/Cache.php');
 require_once('classes/ThemePack.php');
-require_once('classes/InterfaceElements.php');
-require_once('classes/MainUI.php');
+
 
 /**
@@ -1491 +1490 @@
     public function getAdminUI()
     {
+        require_once('classes/InterfaceElements.php');
         // Init values
         $html = '';
@@ -1695 +1695 @@
         $user = User::getCurrentUser();
 
-        try {
             if (!$this->hasAdminAccess($user)) {
                 throw new Exception(_('Access denied!'));
             }
-        } catch (Exception $e) {
-            $ui = new MainUI();
-            $ui->setToolSection('ADMIN');
-            $ui->displayError($e->getMessage(), false);
-            exit;
-        }
-
+            
         // Content management
         $name = "network_".$this->id."_content";