r2379 - drop user_enabled column and all related stuff
authorDenis Ovsienko <infrastation@yandex.ru>
Sat, 31 Jan 2009 17:28:29 +0000 (17:28 +0000)
committerDenis Ovsienko <infrastation@yandex.ru>
Sat, 31 Jan 2009 17:28:29 +0000 (17:28 +0000)
ChangeLog
README
inc/auth.php
inc/database.php
inc/interface.php
inc/navigation.php
inc/ophandlers.php
install.php
install/init-auth.sql
install/init-structure.sql
upgrade.php

index cbb7336..69dc298 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -15,6 +15,7 @@
        update: enable username handling in Cisco switch connector
        update: introduce icons for routers and SLB
        update: display and edit the dictionary chapter by chapter (ticket:151)
+       update: 'enabled' user acount flag was dropped
        bugfix: show error message if snmp module isn't loaded.  closes ticket 43 (by Aaron)
        bugfix: show error message for duplicate IPv4 network record
        bugfix: it was possible to access the system as any existing
diff --git a/README b/README
index 2fdf18c..c94fc4f 100644 (file)
--- a/README
+++ b/README
@@ -137,3 +137,8 @@ in the database.  There are several settings in php.ini which you may need to mo
     upload_max_filesize - max size for uploaded files
     post_max_size       - max size of all form data submitted via POST (including files)
 
+User accounts used to have 'enabled' flag, which allowed individual blocking and
+unblocking of each account. This flag was dropped in favor of existing mean of access
+setup (RackCode). Systems, which had blocked user accounts, have to be reviewed for
+consistency.
+
index 65614d7..5d4053d 100644 (file)
@@ -60,8 +60,6 @@ function authenticate ()
        }
        if ($require_valid_user and !isset ($accounts[$remote_username]))
                dieWith401();
-       if (isset ($accounts[$remote_username]) and $accounts[$remote_username]['user_enabled'] != 'yes')
-               dieWith401();
        $remote_displayname = $remote_username;
        switch (TRUE)
        {
index 1774fd0..ee8596e 100644 (file)
@@ -1545,21 +1545,6 @@ function commitUpdateUserAccount ($id, $new_username, $new_realname, $new_passwo
        return TRUE;
 }
 
-function commitEnableUserAccount ($id, $new_enabled_value)
-{
-       global $dbxlink;
-       $query =
-               "update UserAccount set user_enabled = '${new_enabled_value}' " .
-               "where user_id = ${id} limit 1";
-       $result = $dbxlink->query ($query);
-       if ($result == NULL)
-       {
-               showError ('SQL query failed', __FUNCTION__);
-               die;
-       }
-       return TRUE;
-}
-
 // This function returns an array of all port type pairs from PortCompat table.
 function getPortCompat ()
 {
index 1b1e156..943b226 100644 (file)
@@ -96,8 +96,6 @@ $image['inservice']['height'] = 16;
 $image['notinservice']['path'] = 'pix/tango-dialog-error.png';
 $image['notinservice']['width'] = 16;
 $image['notinservice']['height'] = 16;
-$image['blockuser'] = $image['inservice'];
-$image['unblockuser'] = $image['notinservice'];
 $image['find']['path'] = 'pix/tango-system-search.png';
 $image['find']['width'] = 16;
 $image['find']['height'] = 16;
@@ -1362,8 +1360,8 @@ function printLog ($log)
                                38 => array ('code' => 'success', 'format' => '%u real server(s) were successfully (de)activated'),
                                39 => array ('code' => 'success', 'format' => 'User account %s updated.'),
                                40 => array ('code' => 'success', 'format' => 'User account %s created.'),
-                               41 => array ('code' => 'success', 'format' => 'User account disabled.'),
-                               42 => array ('code' => 'success', 'format' => 'User account enabled.'),
+// ...
+// ...
                                43 => array ('code' => 'success', 'format' => 'Saved successfully.'),
                                44 => array ('code' => 'success', 'format' => '%s failures and %s successfull changes.'),
                                45 => array ('code' => 'success', 'format' => "Attribute '%s' created."),
@@ -1404,9 +1402,9 @@ function printLog ($log)
                                102 => array ('code' => 'error', 'format' => "Error creating user account '%s'"),
                                103 => array ('code' => 'error', 'format' => 'getHashByID() failed'),
                                104 => array ('code' => 'error', 'format' => "Error updating user account '%s'"),
-                               105 => array ('code' => 'error', 'format' => 'Error enabling user account.'),
-                               106 => array ('code' => 'error', 'format' => 'Error disabling user account.'),
-                               107 => array ('code' => 'error', 'format' => 'Admin account cannot be disabled'),
+// ...
+// ...
+// ...
                                108 => array ('code' => 'error', 'format' => '%u failures and %u successfull changes.'),
                                109 => array ('code' => 'error', 'format' => 'Update failed!'),
                                110 => array ('code' => 'error', 'format' => 'Supplement failed!'),
@@ -3359,7 +3357,7 @@ function renderUserListEditor ()
        function printNewItemTR ()
        {
                printOpFormIntro ('createUser');
-               echo "<tr><td>&nbsp;</td><td><input type=text size=16 name=username tabindex=100></td>\n";
+               echo "<tr><td><input type=text size=16 name=username tabindex=100></td>\n";
                echo "<td><input type=text size=24 name=realname tabindex=101></td>";
                echo "<td><input type=password size=64 name=password tabindex=102></td><td>";
                printImageHREF ('create', 'Add new account', TRUE, 103);
@@ -3369,28 +3367,13 @@ function renderUserListEditor ()
        startPortlet ('User accounts');
        showMessageOrError();
        echo "<table cellspacing=0 cellpadding=5 align=center class=widetable>\n";
-       echo "<tr><th>status (click to change)</th><th>Username</th><th>Real name</th><th>Password</th><th>&nbsp;</th></tr>\n";
+       echo "<tr><th>Username</th><th>Real name</th><th>Password</th><th>&nbsp;</th></tr>\n";
        if (getConfigVar ('ADDNEW_AT_TOP') == 'yes')
                printNewItemTR();
        foreach ($accounts as $account)
        {
                printOpFormIntro ('updateUser', array ('user_id' => $account['user_id']));
-               echo "<tr><td>";
-               if ($account['user_enabled'] == 'yes' && $account['user_id'] != 1)
-               {
-                       echo "<a href='${root}process.php?op=disableUser&page=${pageno}&tab=${tabno}&user_id=${account['user_id']}'>";
-                       printImageHREF ('blockuser', 'disable account');
-                       echo "</a>\n";
-               }
-               if ($account['user_enabled'] == 'no' && $account['user_id'] != 1)
-               {
-                       echo "<a href='${root}process.php?op=enableUser&page=${pageno}&tab=${tabno}&user_id=${account['user_id']}'>";
-                       printImageHREF ('unblockuser', 'enable account');
-                       echo "</a>\n";
-               }
-               // Otherwise skip icon.
-               echo "</td>";
-               echo "<td><input type=text name=username value='${account['user_name']}' size=16></td>";
+               echo "<tr><td><input type=text name=username value='${account['user_name']}' size=16></td>";
                echo "<td><input type=text name=realname value='${account['user_realname']}' size=24></td>";
                echo "<td><input type=password name=password value='${account['user_password_hash']}' size=64></td><td>";
                printImageHREF ('save', 'Save changes', TRUE);
@@ -5392,13 +5375,6 @@ function renderUser ($user_id)
        echo '<table border=0 align=center>';
        echo "<tr><th class=tdright>Account name:</th><td class=tdleft>${username}</td></tr>";
        echo '<tr><th class=tdright>Real name:</th><td class=tdleft>' . $accounts[$username]['user_realname'] . '</td></tr>';
-       echo '<tr><th class=tdright>Enabled:</th><td class=tdleft>';
-       // This is weird, some other image titles have to be used.
-       if ($accounts[$username]['user_enabled'] == 'yes')
-               printImageHREF ('blockuser', 'enabled');
-       else
-               printImageHREF ('unblockuser', 'disabled');
-       echo '</td></tr>';
        // Using printTagTRs() is inappropriate here, because autotags will be filled with current user's
        // data, not the viewed one.
 //     printTagTRs ("${root}?page=userlist&");
index 9f8963f..ea7f7f6 100644 (file)
@@ -430,18 +430,11 @@ $tabhandler['userlist']['default'] = 'renderUserList';
 $tabhandler['userlist']['edit'] = 'renderUserListEditor';
 $ophandler['userlist']['edit']['updateUser'] = 'updateUser';
 $ophandler['userlist']['edit']['createUser'] = 'createUser';
-$ophandler['userlist']['edit']['disableUser'] = 'disableUser';
-$ophandler['userlist']['edit']['enableUser'] = 'enableUser';
 $msgcode['updateUser']['OK'] = 39;
 $msgcode['updateUser']['ERR1'] = 103;
 $msgcode['updateUser']['ERR1'] = 104;
 $msgcode['createUser']['OK'] = 40;
 $msgcode['createUser']['ERR'] = 102;
-$msgcode['disableUser']['OK'] = 41;
-$msgcode['disableUser']['ERR1'] = 107;
-$msgcode['disableUser']['ERR2'] = 106;
-$msgcode['enableUser']['OK'] = 42;
-$msgcode['enableUser']['ERR'] = 105;
 
 $page['user']['title_handler'] = 'dynamic_title_user';
 $page['user']['parent'] = 'userlist';
index 715e911..2c1a6b8 100644 (file)
@@ -485,26 +485,6 @@ function updateUser ()
                return buildRedirectURL (__FUNCTION__, 'ERR2', array ($username));
 }
 
-function enableUser ()
-{
-       assertUIntArg ('user_id', __FUNCTION__);
-       if (commitEnableUserAccount ($_REQUEST['user_id'], 'yes') == TRUE)
-               return buildRedirectURL (__FUNCTION__, 'OK');
-       else
-               return buildRedirectURL (__FUNCTION__, 'ERR');
-}
-
-function disableUser ()
-{
-       assertUIntArg ('user_id', __FUNCTION__);
-       if ($_REQUEST['user_id'] == 1)
-               return buildRedirectURL (__FUNCTION__, 'ERR1');
-       if (commitEnableUserAccount ($_REQUEST['user_id'], 'no'))
-               return buildRedirectURL (__FUNCTION__, 'OK');
-       else
-               return buildRedirectURL (__FUNCTION__, 'ERR2');
-}
-
 // This function find differences in users's submit and PortCompat table
 // and modifies database accordingly.
 function savePortMap ()
index 2ec1a6f..4640540 100644 (file)
@@ -348,8 +348,8 @@ function init_database_dynamic ()
        }
        else
        {
-               $query = "INSERT INTO `UserAccount` (`user_id`, `user_name`, `user_enabled`, `user_password_hash`, `user_realname`) " .
-                       "VALUES (1,'admin','yes',sha1('${_REQUEST['password']}'),'RackTables Administrator')";
+               $query = "INSERT INTO `UserAccount` (`user_id`, `user_name`, `user_password_hash`, `user_realname`) " .
+                       "VALUES (1,'admin',sha1('${_REQUEST['password']}'),'RackTables Administrator')";
                $result = $dbxlink->exec ($query);
                echo "Administrator password has been set successfully.<br>";
                return TRUE;
index aebff11..9991b17 100644 (file)
@@ -1,5 +1,5 @@
-INSERT INTO `UserAccount` (`user_id`, `user_name`, `user_enabled`, `user_password_hash`, `user_realname`)
-VALUES (1,'admin','yes',sha1(
+INSERT INTO `UserAccount` (`user_id`, `user_name`, `user_password_hash`, `user_realname`)
+VALUES (1,'admin',sha1(
 
 # Uncomment and change the next line to your password, e.g.:
 # 'mysecretpassword'
index 0638a22..55840dd 100644 (file)
@@ -295,7 +295,6 @@ CREATE TABLE `TagTree` (
 CREATE TABLE `UserAccount` (
   `user_id` int(10) unsigned NOT NULL auto_increment,
   `user_name` char(64) NOT NULL,
-  `user_enabled` enum('yes','no') NOT NULL default 'no',
   `user_password_hash` char(128) default NULL,
   `user_realname` char(64) default NULL,
   PRIMARY KEY  (`user_id`),
index a334cb1..bcfa843 100644 (file)
@@ -161,6 +161,7 @@ CREATE TABLE `FileLink` (
                        $query[] = "INSERT INTO `Config` (varname, varvalue, vartype, emptyok, is_hidden, description) VALUES ('PREVIEW_TEXT_COLS','80','uint','yes','no','Columns for text file preview')";
                        $query[] = "INSERT INTO `Config` (varname, varvalue, vartype, emptyok, is_hidden, description) VALUES ('PREVIEW_IMAGE_MAXPXS','320','uint','yes','no','Max pixels per axis for image file preview')";
                        $query[] = "alter table TagTree add column valid_realm set('file','ipv4net','ipv4vs','ipv4rspool','object','rack','user') not null default 'file,ipv4net,ipv4vs,ipv4rspool,object,rack,user' after parent_id";
+                       $query[] = "alter table UserAccount drop column user_enabled";
                        $query[] = "UPDATE Config SET varvalue = '0.17.0' WHERE varname = 'DB_VERSION'";
                        break;
                default: