More variables renamed to , install/admin_WriteData() is now generic (with open TODO)

[mailer.git] / inc / modules / member / what-surfbar_list.php

diff --git a/inc/modules/member/what-surfbar_list.php b/inc/modules/member/what-surfbar_list.php
index c551004e66dbec16a3e153fa3f76aa389818f2c0..b5aeeba06398c96e6ae7c0e58afe2958be5edff5 100644 (file)
--- a/inc/modules/member/what-surfbar_list.php
+++ b/inc/modules/member/what-surfbar_list.php
@@ -36,31 +36,50 @@ if (!defined('__SECURITY')) {
        $INC = substr(dirname(__FILE__), 0, strpos(dirname(__FILE__), "/inc") + 4) . "/security.php";
        require($INC);
 } elseif (!IS_MEMBER()) {
-       LOAD_URL(URL."/modules.php?module=index");
+       LOAD_URL("modules.php?module=index");
+} elseif ((!EXT_IS_ACTIVE("surfbar")) && (!IS_ADMIN())) {
+       addFatalMessage(getMessage('EXTENSION_PROBLEM_EXT_INACTIVE'), "surfbar");
+       return;
 }
 
 // Add description as navigation point
-ADD_DESCR("member", basename(__FILE__));
+ADD_DESCR("member", __FILE__);
 
 // Load user URLs
 $URLs = SURFBAR_GET_USER_URLS();
 
-// Are there entries?
-if (count($URLs) > 0) {
+// Are there entries or form is submitted?
+if ((isset($_POST['ok'])) && (isset($_POST['action'])) && (isset($_POST['id']))) {
+       // Process the form
+       if (SURFBAR_MEMBER_DO_FORM($_POST, $URLs)) {
+               // Action performed but shall we display it?
+               if ((($_POST['action'] != "edit") && ($_POST['action'] != "delete")) || (isset($_POST['execute']))) {
+                       // Display "action done" message if action is wether 'edit' nor 'delete' or has been executed
+                       LOAD_TEMPLATE("admin_settings_saved", false, getMessage('MEMBER_SURFBAR_ACTION_DONE'));
+               } // END - if
+       } else {
+               // Something went wrong
+               LOAD_TEMPLATE("admin_settings_saved", false, getMessage('MEMBER_SURFBAR_ACTION_FAILED'));
+       }
+} elseif (count($URLs) > 0) {
        // List all URLs
        $OUT = ""; $SW = 2;
        foreach ($URLs as $id => $content) {
                // "Translate"/insert data
                $content['sw'] = $SW;
-               $content['url'] = FRAMETESTER($content['url']);
+               $content['url'] = DEREFERER($content['url']);
                if ($content['views_total'] > 0) {
                        // Include link to stats
-                       $content['views_total'] = "[<a href=\"".URL."/modules.php?module=login&amp;what=surfbar_list\">".TRANSLATE_COMMA($content['views_total'])."</a>]";
+                       $content['views_total'] = "[<a href=\"{!URL!}/modules.php?module=login&amp;what=surfbar_list\">".TRANSLATE_COMMA($content['views_total'])."</a>]";
                } // END - if
                $content['registered']  = MAKE_DATETIME($content['registered'], "2");
                $content['last_locked'] = MAKE_DATETIME($content['last_locked'], "2");
                $content['actions']     = SURFBAR_MEMBER_ACTIONS($content['id'], $content['status']);
                $content['status']      = SURFBAR_TRANSLATE_STATUS($content['status']);
+               if (empty($content['lock_reason'])) {
+                       // Fixes some HTML problems with empty cells
+                       $content['lock_reason'] = "---";
+               } // END - if
 
                // Load row template
                $OUT .= LOAD_TEMPLATE("member_surfbar_list_row", true, $content);
@@ -73,7 +92,7 @@ if (count($URLs) > 0) {
        LOAD_TEMPLATE("member_surfbar_list", false, $OUT);
 } else {
        // No URLs booked so far
-       LOAD_TEMPLATE("admin_settings_saved", false, MEMBER_SURFBAR_NO_URLS_FOUND);
+       LOAD_TEMPLATE("admin_settings_saved", false, getMessage('MEMBER_SURFBAR_NO_URLS_FOUND'));
 }
 
 //