Changeset 1092 for trunk/wifidog-auth/wifidog/classes/User.php

Timestamp:

09/07/06 04:38:25 (6 years ago)

Author:

benoitg

Message:

• Fix #63
• Fix problem with my Splash only user fix (I used match on the

user_id instead of the username).

• Prevent the Splash only user from logging out, wrecking havock

in the tokens.

• Do not allow SplashOnlyUser? to set his preferences.
• Assigning a reusable content to a contentgroup element would

make it non-reusable.
Usability and UI fixes:

• Allow other users to log in when they are at a splash only

node.

• Make isSimpleContent class available for Simple content types.

Make it borderless and backgroundless in the default theme_pack.

• Simplify HTML output for simple content.
• Fix IE layout bug in node admin
• Content linked to nodes and networks now edits in a new

window.

• Remove "Add new node" menu item for consistency.

Files:

• trunk/wifidog-auth/wifidog/classes/User.php (modified) (45 diffs)

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

@@ -1 +1 @@
 <?php
+
 
 /* vim: set expandtab tabstop=4 shiftwidth=4 softtabstop=4: */
@@ -45 +46 @@
  * Load required classes
  */
-require_once('classes/Network.php');
-require_once('classes/Mail.php');
-require_once('classes/InterfaceElements.php');
+require_once ('classes/Network.php');
+require_once ('classes/Mail.php');
+require_once ('classes/InterfaceElements.php');
 
 /**
@@ -56 +57 @@
  * @copyright  2005-2006 Benoit Grégoire, Technologies Coeus inc.
  */
-class User implements GenericObject
-{
+class User implements GenericObject {
     private $mRow;
     private $id;
@@ -65 +65 @@
      * @return a User object, or null if there was an error
      */
-    public static function getObject($id)
-    {
+    public static function getObject($id) {
         $object = null;
         $object = new self($id);
@@ -72 +71 @@
     }
 
-    static function createNewObject()
-    {
+    static function createNewObject() {
         echo "<h1>Use User::createUser() instead</h1>";
     }
@@ -79 +77 @@
     * @return html markup
     */
-    public static function getCreateNewObjectUI()
-    {
+    public static function getCreateNewObjectUI() {
         return null;
     }
@@ -90 +87 @@
      * @return the node object or null if no new node was created.
      */
-    static function processCreateNewObjectUI()
-    {
+    static function processCreateNewObjectUI() {
         return self :: createNewObject();
     }
@@ -98 +94 @@
      *
      * @return mixed A User object, or null if there was an error
-
-     */
-    public static function getCurrentUser()
-    {
-        require_once('classes/Session.php');
+    
+     */
+    public static function getCurrentUser() {
+        require_once ('classes/Session.php');
         $session = new Session();
         $user = null;
-        try
-        {
+        try {
             $user = self :: getObject($session->get(SESS_USER_ID_VAR));
             //$user = new User($session->get(SESS_USER_ID_VAR));
-        }
-        catch (Exception $e)
-        {
+        } catch (Exception $e) {
             /**If any problem occurs, the user should be considered logged out*/
             $session->set(SESS_USER_ID_VAR, null);
@@ -126 +118 @@
      *
      * @return bool True if everything went well setting the session
-
-     */
-    public static function setCurrentUser(User $user)
-    {
-        try
-        {
+    
+     */
+    public static function setCurrentUser(User $user) {
+        try {
             $session = new Session();
             $session->set(SESS_USER_ID_VAR, $user->getId());
             $session->set(SESS_PASSWORD_HASH_VAR, $user->getPasswordHash());
             return true;
-        }
-        catch (Exception $e)
-        {
+        } catch (Exception $e) {
             return false;
         }
@@ -147 +135 @@
      *
      * @return string Hostname of server
-
-     */
-    public static function getCurrentServer()
-    {
+    
+     */
+    public static function getCurrentServer() {
         return $_SERVER['SERVER_NAME'];
     }
@@ -159 +146 @@
      * @return a User object, or null if there was an error
      */
-    public static function getUserByUsernameAndOrigin($username, Network $account_origin)
-    {
+    public static function getUserByUsernameAndOrigin($username, Network $account_origin) {
         global $db;
         $object = null;
@@ -178 +164 @@
      * @return a User object, or null if there was an error
      */
-    public static function getUserByEmailAndOrigin($email, Network $account_origin)
-    {
+    public static function getUserByEmailAndOrigin($email, Network $account_origin) {
         global $db;
         $object = null;
@@ -195 +180 @@
      * @return The 32 character hash.
      */
-    public static function passwordHash($password)
-    {
+    public static function passwordHash($password) {
         /**
          * utf8_decode is used for backward compatibility with old passwords
@@ -209 +193 @@
      * @return the newly created User object, or null if there was an error
      */
-    static function createUser($id, $username, Network $account_origin, $email, $password)
-    {
+    static function createUser($id, $username, Network $account_origin, $email, $password) {
         global $db;
 
@@ -229 +212 @@
     }
 
-/*    public static function purgeUnvalidatedUsers($days_since_creation)
-    {
-        global $db;
-        $days_since_creation = $db->escapeString($days_since_creation);
-
-        //$db->execSqlUpdate("INSERT INTO users (user_id,username, account_origin,email,pass,account_status,validation_token,reg_date) VALUES ('$id_str','$username_str','$account_origin_str','$email_str','$password_hash','$status','$token',NOW())");
-    }*/
+    /*    public static function purgeUnvalidatedUsers($days_since_creation)
+        {
+            global $db;
+            $days_since_creation = $db->escapeString($days_since_creation);
+    
+            //$db->execSqlUpdate("INSERT INTO users (user_id,username, account_origin,email,pass,account_status,validation_token,reg_date) VALUES ('$id_str','$username_str','$account_origin_str','$email_str','$password_hash','$status','$token',NOW())");
+        }*/
 
     /** @param $object_id The id of the user */
-    function __construct($object_id)
-    {
+    function __construct($object_id) {
         global $db;
         $this->mDb = & $db;
@@ -245 +227 @@
         $sql = "SELECT * FROM users WHERE user_id='{$object_id_str}'";
         $db->execSqlUniqueRes($sql, $row, false);
-        if ($row == null)
-        {
+        if ($row == null) {
             throw new Exception(sprintf(_("User id: %s could not be found in the database"), $object_id_str));
         }
@@ -253 +234 @@
     } //End class
 
-    function getId()
-    {
+    function getId() {
         return $this->id;
     }
@@ -261 +241 @@
      * @return Network object (never returns null)
      */
-    public function getNetwork()
-    {
+    public function getNetwork() {
         return Network :: getObject($this->mRow['account_origin']);
     }
 
     /** Get a user display suitable for a user list.  Will include link to the user profile. */
-    function getListUI()
-    {
+    function getListUI() {
         /*    $roles = array ();
-
-    if ($current_node->isOwner($online_user)) {
+        
+        if ($current_node->isOwner($online_user)) {
         $roles[] = _("owner");
-    }
-
-    if ($current_node->isTechnicalOfficer($online_user)) {
+        }
+        
+        if ($current_node->isTechnicalOfficer($online_user)) {
         $roles[] = _("technical officer");
-    }
-
-    if ($roles) {
+        }
+        
+        if ($roles) {
         $rolenames = join($roles, ",");
-    }*/
+        }*/
         $html = '';
-        if($this->isSplashOnlyUser())
-        {
-                $html .= _("Guest");
+        if ($this->isSplashOnlyUser()) {
+            $html .= _("Guest");
         }
         $html .= $this->getUserName();
@@ -291 +268 @@
     }
 
-    function getUsername()
-    {
+    function getUsername() {
         return $this->mRow['username'];
     }
-    
-     /** Set the user's username
-     * @param $value The new value
-     * @return true on success, false on failure
-     * @throws exception if the user tries to set a duplicate username
-     */
+
+    /** Set the user's username
+    * @param $value The new value
+    * @return true on success, false on failure
+    * @throws exception if the user tries to set a duplicate username
+    */
     function setUsername($value) {
         $retval = true;
@@ -306 +282 @@
             global $db;
             $value = $db->escapeString($value);
-            $retval = @$db->execSqlUpdate("UPDATE users SET username = '{$value}' WHERE user_id='{$this->id}'", false);
-            if(!$retval)
-            {
-                throw new exception (sprintf(_("Sorry, the username %s is not available"), $value));
+            $retval = @ $db->execSqlUpdate("UPDATE users SET username = '{$value}' WHERE user_id='{$this->id}'", false);
+            if (!$retval) {
+                throw new exception(sprintf(_("Sorry, the username %s is not available"), $value));
             }
             $this->refresh();
@@ -316 +291 @@
     }
 
-    public function getEmail()
-    {
+    public function getEmail() {
         return $this->mRow['email'];
     }
 
-    public function setEmail($email)
-    {
-      $email_str = $this->mDb->escapeString($email);
-      if (!($update = $this->mDb->execSqlUpdate("UPDATE users SET email='{$email_str}' WHERE user_id='{$this->id}'")))
-        {
-          throw new Exception(_("Could not update email address."));
-        }
-      $this->mRow['email'] = $email; // unescaped
-    }
-
-    function setIsInvisible($value)
-    {
-      $retval = true;
-      if ($value != $this->isAdvertised())
-        {
-          global $db;
-          $value ? $value = 'TRUE' : $value = 'FALSE';
-          $retval = $db->execSqlUpdate("UPDATE users SET is_invisible = {$value} WHERE user_id = '{$this->getId()}'", false);
-          $this->refresh();
-        }
-      return $retval;
-    }
-
-    public function isInvisible()
-    {
-      return (($this->mRow['is_invisible'] == 't') ? true : false);
+    public function setEmail($email) {
+        $email_str = $this->mDb->escapeString($email);
+        if (!($update = $this->mDb->execSqlUpdate("UPDATE users SET email='{$email_str}' WHERE user_id='{$this->id}'"))) {
+            throw new Exception(_("Could not update email address."));
+        }
+        $this->mRow['email'] = $email; // unescaped
+    }
+
+    function setIsInvisible($value) {
+        $retval = true;
+        if ($value != $this->isAdvertised()) {
+            global $db;
+            $value ? $value = 'TRUE' : $value = 'FALSE';
+            $retval = $db->execSqlUpdate("UPDATE users SET is_invisible = {$value} WHERE user_id = '{$this->getId()}'", false);
+            $this->refresh();
+        }
+        return $retval;
+    }
+
+    public function isInvisible() {
+        return (($this->mRow['is_invisible'] == 't') ? true : false);
     }
 
     /**What locale (language) does the user prefer? */
-    public function getPreferedLocale()
-    {
-      global $session;
-      $locale = $this->mRow['prefered_locale'];
-      if (empty($locale) && !empty($session))
-        $locale = $session->get(SESS_LANGUAGE_VAR);
-      if (empty($locale))
-        $locale = DEFAULT_LANG;
-      return $locale;
-    }
-
-    public function setPreferedLocale($locale)
-    {
-      $locale_str = $this->mDb->escapeString($locale);
-      if (!($update = $this->mDb->execSqlUpdate("UPDATE users SET prefered_locale='{$locale_str}' WHERE user_id='{$this->id}'")))
-        {
-          throw new Exception(_("Could not update username locale."));
-        }
-      $this->mRow['prefered_locale'] = $locale;
+    public function getPreferedLocale() {
+        global $session;
+        $locale = $this->mRow['prefered_locale'];
+        if (empty ($locale) && !empty ($session))
+            $locale = $session->get(SESS_LANGUAGE_VAR);
+        if (empty ($locale))
+            $locale = DEFAULT_LANG;
+        return $locale;
+    }
+
+    public function setPreferedLocale($locale) {
+        $locale_str = $this->mDb->escapeString($locale);
+        if (!($update = $this->mDb->execSqlUpdate("UPDATE users SET prefered_locale='{$locale_str}' WHERE user_id='{$this->id}'"))) {
+            throw new Exception(_("Could not update username locale."));
+        }
+        $this->mRow['prefered_locale'] = $locale;
     }
 
     /** get the hashed password stored in the database */
-    public function getPasswordHash()
-    {
+    public function getPasswordHash() {
         return $this->mRow['pass'];
     }
@@ -380 +345 @@
      * @return Possible values are listed in common.php
     */
-    function getAccountStatus()
-    {
+    function getAccountStatus() {
         return $this->mRow['account_status'];
     }
 
-    function setAccountStatus($status)
-    {
+    function setAccountStatus($status) {
         global $db;
 
         $status_str = $db->escapeString($status);
-        if (!($update = $db->execSqlUpdate("UPDATE users SET account_status='{$status_str}' WHERE user_id='{$this->id}'")))
-        {
+        if (!($update = $db->execSqlUpdate("UPDATE users SET account_status='{$status_str}' WHERE user_id='{$this->id}'"))) {
             throw new Exception(_("Could not update status."));
         }
@@ -399 +361 @@
     /** Is the user valid?  Valid means that the account is validated or hasn't exhausted it's validation period.
      $errmsg: Returs the reason why the account is or isn't valid */
-    function isUserValid(& $errmsg = null)
-    {
+    function isUserValid(& $errmsg = null) {
         global $db;
         $retval = false;
         $account_status = $this->getAccountStatus();
-        if ($account_status == ACCOUNT_STATUS_ALLOWED)
-        {
+        if ($account_status == ACCOUNT_STATUS_ALLOWED) {
             $retval = true;
-        }
-        else
-            if ($account_status == ACCOUNT_STATUS_VALIDATION)
-            {
+        } else
+            if ($account_status == ACCOUNT_STATUS_VALIDATION) {
                 $sql = "SELECT CASE WHEN ((NOW() - reg_date) > networks.validation_grace_time) THEN true ELSE false END AS validation_grace_time_expired, networks.validation_grace_time FROM users  JOIN networks ON (users.account_origin = networks.network_id) WHERE (user_id='{$this->id}')";
                 $db->execSqlUniqueRes($sql, $user_info, false);
 
-                if ($user_info['validation_grace_time_expired'] == 't')
-                {
-                    $errmsg = sprintf(_("Sorry, your %s minutes grace period to retrieve your email and validate your account has now expired. You will have to connect to the internet and validate your account from another location or create a new account. For help, please %s click here %s."), $user_info['validation_grace_time_expired'], '<a href="'.BASE_URL_PATH.'faq.php'.'">', '</a>');
+                if ($user_info['validation_grace_time_expired'] == 't') {
+                    $errmsg = sprintf(_("Sorry, your %s minutes grace period to retrieve your email and validate your account has now expired. You will have to connect to the internet and validate your account from another location or create a new account. For help, please %s click here %s."), $user_info['validation_grace_time_expired'], '<a href="' . BASE_URL_PATH . 'faq.php' . '">', '</a>');
                     $retval = false;
-                }
-                else
-                {
+                } else {
                     $errmsg = _("Your account is currently valid.");
                     $retval = true;
                 }
-            }
-            else
-            {
-                $errmsg = _("Sorry, your account is not valid: ").$account_status_to_text[$account_status];
+            } else {
+                $errmsg = _("Sorry, your account is not valid: ") . $account_status_to_text[$account_status];
                 $retval = false;
             }
@@ -433 +386 @@
     }
 
-    public function isSuperAdmin()
-    {
+    public function isSuperAdmin() {
         global $db;
         //$this->session->dump();
 
         $db->execSqlUniqueRes("SELECT * FROM users NATURAL JOIN administrators WHERE (users.user_id='$this->id')", $user_info, false);
-        if (!empty ($user_info))
-        {
+        if (!empty ($user_info)) {
             return true;
-        }
-        else
-        {
+        } else {
             return false;
         }
@@ -453 +402 @@
      * Tells if the current user is owner of at least one hotspot.
      */
-    public function isOwner()
-    {
+    public function isOwner() {
         global $db;
         $db->execSql("SELECT * FROM node_stakeholders WHERE is_owner = true AND user_id='{$this->getId()}'", $row, false);
@@ -463 +411 @@
     }
 
-    public function isNobody()
-        {
-                global $db;
-                $db->execSqlUniqueRes("SELECT DISTINCT user_id FROM (SELECT user_id FROM network_stakeholders WHERE user_id='{$this->getId()}' UNION SELECT user_id FROM node_stakeholders WHERE user_id='{$this->getId()}' UNION SELECT user_id FROM administrators WHERE user_id='{$this->getId()}') as tmp", $row, false);
-                if ($row == null)
-                        return true;
-                return false;
-        }
-
-/** Is this user the Splash Only User() */
-    public function isSplashOnlyUser()
-        {
-if($this->mRow['user_id'] == "SPLASH_ONLY_USER") {
-return true;
-}
-else {
-return false;
-}
-        }
-        
-    function getValidationToken()
-    {
+    public function isNobody() {
+        global $db;
+        $db->execSqlUniqueRes("SELECT DISTINCT user_id FROM (SELECT user_id FROM network_stakeholders WHERE user_id='{$this->getId()}' UNION SELECT user_id FROM node_stakeholders WHERE user_id='{$this->getId()}' UNION SELECT user_id FROM administrators WHERE user_id='{$this->getId()}') as tmp", $row, false);
+        if ($row == null)
+            return true;
+        return false;
+    }
+
+    /** Is this user the Splash Only User() */
+    public function isSplashOnlyUser() {
+        if ($this->mRow['username'] == "SPLASH_ONLY_USER") {
+            return true;
+        } else {
+            return false;
+        }
+    }
+
+    function getValidationToken() {
         return $this->mRow['validation_token'];
     }
@@ -491 +435 @@
     @return true on success, false on failure
     */
-    function generateConnectionToken()
-    {
-        if ($this->isUserValid())
-        {
+    function generateConnectionToken() {
+        if ($this->isUserValid()) {
             global $db;
             global $session;
 
             $token = self :: generateToken();
-            if ($_SERVER['REMOTE_ADDR'])
-            {
+            if ($_SERVER['REMOTE_ADDR']) {
                 $node_ip = $db->escapeString($_SERVER['REMOTE_ADDR']);
             }
 
-            if ($session && $node_ip && $session->get(SESS_GW_ID_VAR))
-            {
+            if ($session && $node_ip && $session->get(SESS_GW_ID_VAR)) {
                 $node_id = $db->escapeString($session->get(SESS_GW_ID_VAR));
-                $db->execSqlUpdate("INSERT INTO connections (user_id, token, token_status, timestamp_in, node_id, node_ip, last_updated) VALUES ('".$this->getId()."', '$token', '".TOKEN_UNUSED."', NOW(), '$node_id', '$node_ip', NOW())", false);
+                $db->execSqlUpdate("INSERT INTO connections (user_id, token, token_status, timestamp_in, node_id, node_ip, last_updated) VALUES ('" . $this->getId() . "', '$token', '" . TOKEN_UNUSED . "', NOW(), '$node_id', '$node_ip', NOW())", false);
                 $retval = $token;
-            }
-            else
+            } else
                 $retval = false;
-        }
-        else
-        {
+        } else {
             $retval = false;
         }
@@ -520 +457 @@
     }
 
-    function setPassword($password)
-    {
+    function setPassword($password) {
         global $db;
 
         $new_password_hash = User :: passwordHash($password);
-        if (!($update = $db->execSqlUpdate("UPDATE users SET pass='$new_password_hash' WHERE user_id='{$this->id}'")))
-        {
+        if (!($update = $db->execSqlUpdate("UPDATE users SET pass='$new_password_hash' WHERE user_id='{$this->id}'"))) {
             throw new Exception(_("Could not change user's password."));
         }
@@ -532 +467 @@
     }
 
-    function getAccountOrigin()
-    {
+    function getAccountOrigin() {
         return $this->mRow['account_origin'];
     }
@@ -539 +473 @@
     /** Return all the users
      */
-    static function getAllUsers()
-    {
+    static function getAllUsers() {
         global $db;
 
         $db->execSql("SELECT * FROM users", $objects, false);
-        if ($objects == null)
-        {
+        if ($objects == null) {
             throw new Exception(_("No users could not be found in the database"));
         }
@@ -551 +483 @@
     }
 
-    function sendLostUsername()
-    {
+    function sendLostUsername() {
         $network = $this->getNetwork();
         $mail = new Mail();
@@ -558 +489 @@
         $mail->setSenderEmail($network->getValidationEmailFromAddress());
         $mail->setRecipientEmail($this->getEmail());
-        $mail->setMessageSubject($network->getName()._(" lost username request"));
-        $mail->setMessageBody(_("Hello,\nYou have requested that the authentication server send you your username:\nUsername: ").$this->getUsername()._("\n\nHave a nice day,\nThe Team"));
+        $mail->setMessageSubject($network->getName() . _(" lost username request"));
+        $mail->setMessageBody(_("Hello,\nYou have requested that the authentication server send you your username:\nUsername: ") . $this->getUsername() . _("\n\nHave a nice day,\nThe Team"));
         $mail->send();
     }
 
-    function sendValidationEmail()
-    {
-        if ($this->getAccountStatus() != ACCOUNT_STATUS_VALIDATION)
-        {
+    function sendValidationEmail() {
+        if ($this->getAccountStatus() != ACCOUNT_STATUS_VALIDATION) {
             throw new Exception(_("The user is not in validation period."));
-        }
-        else
-        {
-            if ($this->getValidationToken() == "")
-            {
+        } else {
+            if ($this->getValidationToken() == "") {
                 throw new Exception(_("The validation token is empty."));
-            }
-            else
-            {
+            } else {
                 $network = $this->getNetwork();
 
@@ -583 +507 @@
                 $mail->setSenderEmail($network->getValidationEmailFromAddress());
                 $mail->setRecipientEmail($this->getEmail());
-                $mail->setMessageSubject($network->getName()._(" new user validation"));
-                $url = BASE_SSL_PATH . "validate.php?user_id=".$this->getId()."&token=".$this->getValidationToken();
-                $mail->setMessageBody(_("Hello,\nPlease follow the link below to validate your account.\n").$url._("\n\nThank you,\nThe Team."));
+                $mail->setMessageSubject($network->getName() . _(" new user validation"));
+                $url = BASE_SSL_PATH . "validate.php?user_id=" . $this->getId() . "&token=" . $this->getValidationToken();
+                $mail->setMessageBody(_("Hello,\nPlease follow the link below to validate your account.\n") . $url . _("\n\nThank you,\nThe Team."));
                 $mail->send();
             }
@@ -591 +515 @@
     }
 
-    function sendLostPasswordEmail()
-    {
+    function sendLostPasswordEmail() {
         $network = $this->getNetwork();
         $new_password = $this->randomPass();
@@ -601 +524 @@
         $mail->setSenderEmail($network->getValidationEmailFromAddress());
         $mail->setRecipientEmail($this->getEmail());
-        $mail->setMessageSubject($network->getName()._(" new password request"));
-        $mail->setMessageBody(_("Hello,\nYou have requested that the authentication server send you a new password:\nUsername: ").$this->getUsername()._("\nPassword: ").$new_password._("\n\nHave a nice day,\nThe Team"));
+        $mail->setMessageSubject($network->getName() . _(" new password request"));
+        $mail->setMessageBody(_("Hello,\nYou have requested that the authentication server send you a new password:\nUsername: ") . $this->getUsername() . _("\nPassword: ") . $new_password . _("\n\nHave a nice day,\nThe Team"));
         $mail->send();
     }
 
-    static function userExists($id)
-    {
+    static function userExists($id) {
         global $db;
         $id_str = $db->escapeString($id);
@@ -615 +537 @@
     }
 
-    public static function emailExists($id)
-    {
+    public static function emailExists($id) {
         global $db;
         $id_str = $db->escapeString($id);
@@ -624 +545 @@
     }
 
-    public static function randomPass()
-    {
+    public static function randomPass() {
         $rand_pass = ''; // makes sure the $pass var is empty.
-        for ($j = 0; $j < 3; $j ++)
-        {
-            $startnend = array ('b', 'c', 'd', 'f', 'g', 'h', 'j', 'k', 'l', 'm', 'n', 'p', 'q', 'r', 's', 't', 'v', 'w', 'x', 'y', 'z',);
-            $id = array ('a', 'e', 'i', 'o', 'u', 'y',);
+        for ($j = 0; $j < 3; $j++) {
+            $startnend = array (
+                'b',
+                'c',
+                'd',
+                'f',
+                'g',
+                'h',
+                'j',
+                'k',
+                'l',
+                'm',
+                'n',
+                'p',
+                'q',
+                'r',
+                's',
+                't',
+                'v',
+                'w',
+                'x',
+                'y',
+                'z',
+                
+            );
+            $id = array (
+                'a',
+                'e',
+                'i',
+                'o',
+                'u',
+                'y',
+                
+            );
             $count1 = count($startnend) - 1;
             $count2 = count($id) - 1;
 
-            for ($i = 0; $i < 3; $i ++)
-            {
-                if ($i != 1)
-                {
+            for ($i = 0; $i < 3; $i++) {
+                if ($i != 1) {
                     $rand_pass .= $startnend[rand(0, $count1)];
-                }
-                else
-                {
+                } else {
                     $rand_pass .= $id[rand(0, $count2)];
                 }
@@ -649 +595 @@
     }
 
-    public static function generateToken()
-    {
+    public static function generateToken() {
         return md5(uniqid(rand(), 1));
     }
@@ -663 +608 @@
      *
      * @return string HTML markup
-
-     */
-    public static function getSelectUserUI($user_prefix, $add_button_name = null, $add_button_value = null)
-    {
+    
+     */
+    public static function getSelectUserUI($user_prefix, $add_button_name = null, $add_button_value = null) {
         // Define globals
         global $db;
 
-        $_networkSelector = InterfaceElements::generateDiv(Network::getSelectNetworkUI($user_prefix), "admin_section_network_selector", "admin_section_network_selector_" . $user_prefix);
+        $_networkSelector = InterfaceElements :: generateDiv(Network :: getSelectNetworkUI($user_prefix), "admin_section_network_selector", "admin_section_network_selector_" . $user_prefix);
 
         // Check if we need to add an "add" button
         if ($add_button_name && $add_button_value) {
-            $_userSelector = _("Username") . ": " .  InterfaceElements::generateInputText("select_user_" . $user_prefix . "_username", "", "", "input_text", array("onkeypress" => "if ((event.which ? event.which : event.keyCode) == 13) {form.$add_button_name.click() }"));
-            $_userSelector .= InterfaceElements::generateInputSubmit($add_button_name, $add_button_value);
+            $_userSelector = _("Username") . ": " . InterfaceElements :: generateInputText("select_user_" . $user_prefix . "_username", "", "", "input_text", array (
+                "onkeypress" => "if ((event.which ? event.which : event.keyCode) == 13) {form.$add_button_name.click() }"
+            ));
+            $_userSelector .= InterfaceElements :: generateInputSubmit($add_button_name, $add_button_value);
         } else {
-            $_userSelector = _("Username") . ": " .  InterfaceElements::generateInputText("select_user_" . $user_prefix . "_username");
-        }
-        $_html = InterfaceElements::generateDiv($_networkSelector . $_userSelector, 'user_select_user_ui_container');
+            $_userSelector = _("Username") . ": " . InterfaceElements :: generateInputText("select_user_" . $user_prefix . "_username");
+        }
+        $_html = InterfaceElements :: generateDiv($_networkSelector . $_userSelector, 'user_select_user_ui_container');
 
         return $_html;
@@ -688 +634 @@
      * @return the User object, or null if the user is invalid or none was selected
      */
-    static function processSelectUserUI($user_prefix)
-    {
+    static function processSelectUserUI($user_prefix) {
         $object = null;
-        try
-        {
+        try {
             $network = Network :: processSelectNetworkUI($user_prefix);
             $name = "select_user_{$user_prefix}_username";
-            if (!empty ($_REQUEST[$name]))
-            {
+            if (!empty ($_REQUEST[$name])) {
                 $username = $_REQUEST[$name];
                 return self :: getUserByUsernameAndOrigin($username, $network);
-            }
-            else
+            } else
                 return null;
-        }
-        catch (Exception $e)
-        {
+        } catch (Exception $e) {
             return null;
         }
     }
 
-    public function getAdminUI()
-    {
-        global $db;
-        $currentUser= self::getCurrentUser();
+    public function getAdminUI() {
+        global $db;
+        $currentUser = self :: getCurrentUser();
         $html = '';
-        $html .= "<fieldset class='admin_container ".get_class($this)."'>\n";
+        $html .= "<fieldset class='admin_container " . get_class($this) . "'>\n";
         $html .= "<ul class='admin_element_list'>\n";
-if($this==$currentUser || $this->getNetwork()->hasAdminAccess($currentUser))
-{
-    //username
-        $html .= "<li class='admin_element_item_container'>\n";
-        $html .= "<div class='admin_element_label'>"._("Username")." : </div>\n";
-        $html .= "<div class='admin_element_data'>\n";
-        $name = "user_".$this->getId()."_username";
-        $html .= "<input type='text' name='$name' value='".htmlentities($this->getUsername())."' size=30>\n";
-        $html .= _("Be carefull when changing this: it's the username you use to log in!");
-        $html .= "</div>\n";
-        $html .= "</li>\n";
-}
-/*
-        $html .= "<li class='admin_element_item_container'>\n";
-        $html .= "<div class='admin_element_label'>"._("Real name")." : </div>\n";
-        $html .= "<div class='admin_element_data'>\n";
-        $name = "user_".$this->getId()."_real_name";
-        $html .= "<input type='text' name='$name' value='".htmlentities($this->getRealName())."' size=30 readonly>\n";
-        $html .= "</div>\n";
-        $html .= "</li>\n";
-
-        $html .= "<li class='admin_element_item_container'>\n";
-        $html .= "<div class='admin_element_label'>"._("Website URL")." : </div>\n";
-        $html .= "<div class='admin_element_data'>\n";
-        $name = "user_".$this->getId()."_website";
-        $html .= "<input type='text' name='$name' value='".htmlentities($this->getWebsiteURL())."' size=30 readonly>\n";
-        $html .= "</div>\n";
-        $html .= "</li>\n";
-*/
+        if (($this == $currentUser && !$this->isSplashOnlyUser() )|| $this->getNetwork()->hasAdminAccess($currentUser)) {
+            //username
+            $html .= "<li class='admin_element_item_container'>\n";
+            $html .= "<div class='admin_element_label'>" . _("Username") . " : </div>\n";
+            $html .= "<div class='admin_element_data'>\n";
+            $name = "user_" . $this->getId() . "_username";
+            $html .= "<input type='text' name='$name' value='" . htmlentities($this->getUsername()) . "' size=30>\n";
+            $html .= _("Be carefull when changing this: it's the username you use to log in!");
+            $html .= "</div>\n";
+            $html .= "</li>\n";
+        }
+        /*
+                $html .= "<li class='admin_element_item_container'>\n";
+                $html .= "<div class='admin_element_label'>"._("Real name")." : </div>\n";
+                $html .= "<div class='admin_element_data'>\n";
+                $name = "user_".$this->getId()."_real_name";
+                $html .= "<input type='text' name='$name' value='".htmlentities($this->getRealName())."' size=30 readonly>\n";
+                $html .= "</div>\n";
+                $html .= "</li>\n";
+        
+                $html .= "<li class='admin_element_item_container'>\n";
+                $html .= "<div class='admin_element_label'>"._("Website URL")." : </div>\n";
+                $html .= "<div class='admin_element_data'>\n";
+                $name = "user_".$this->getId()."_website";
+                $html .= "<input type='text' name='$name' value='".htmlentities($this->getWebsiteURL())."' size=30 readonly>\n";
+                $html .= "</div>\n";
+                $html .= "</li>\n";
+        */
         $html .= "</fieldset>\n";
         return $html;
     }
 
-    public function processAdminUI()
-    {
-                global $db;
-        $currentUser= self::getCurrentUser();
-        if($this==$currentUser || $this->getNetwork()->hasAdminAccess($currentUser))
-{
-    //username
-        $name = "user_".$this->getId()."_username";
-        $this->setUsername($_REQUEST[$name]);
- }
-    }
-
-    public function delete(& $errmsg)
-    {
-    }
-
-    public function getUserUI()
-        {
+    public function processAdminUI() {
+        global $db;
+        $currentUser = self :: getCurrentUser();
+        if ($this == $currentUser || $this->getNetwork()->hasAdminAccess($currentUser)) {
+            //username
+            $name = "user_" . $this->getId() . "_username";
+            $this->setUsername($_REQUEST[$name]);
+        }
+    }
+
+    public function delete(& $errmsg) {
+    }
+
+    public function getUserUI() {
         $html = "";
         $html .= $this->getRealName();
 
         return $html;
-        }
+    }
 
     /** Add content to this user ( subscription ) */
-    public function addContent(Content $content)
-    {
+    public function addContent(Content $content) {
         global $db;
         $content_id = $db->escapeString($content->getId());
@@ -784 +717 @@
 
     /** Remove content from this node */
-    public function removeContent(Content $content)
-    {
+    public function removeContent(Content $content) {
         global $db;
         $content_id = $db->escapeString($content->getId());
@@ -795 +727 @@
     /**Get an array of all Content linked to this node
     * @return an array of Content or an empty arrray */
-    function getAllContent()
-    {
+    function getAllContent() {
         global $db;
         $retval = array ();
         $sql = "SELECT * FROM user_has_content WHERE user_id='$this->id' ORDER BY subscribe_timestamp";
         $db->execSql($sql, $content_rows, false);
-        if ($content_rows != null)
-        {
-            foreach ($content_rows as $content_row)
-            {
+        if ($content_rows != null) {
+            foreach ($content_rows as $content_row) {
                 $retval[] = Content :: getObject($content_row['content_id']);
             }
@@ -812 +741 @@
 
     /** Reloads the object from the database.  Should normally be called after a set operation */
-    protected function refresh()
-    {
-      $this->__construct($this->id);
+    protected function refresh() {
+        $this->__construct($this->id);
     }
 
     /** Set Smarty template values.  Standardization routine. */
-        public static function assignSmartyValues($smarty, $user=null) {
-                if (!$user) $user = User::getCurrentUser();
-                $smarty->assign('username', $user ? $user->getUsername() : '');
+    public static function assignSmartyValues($smarty, $user = null) {
+        if (!$user)
+            $user = User :: getCurrentUser();
+        $smarty->assign('username', $user ? $user->getListUI() : '');
         $smarty->assign('userId', $user ? $user->getId() : '');
-                /**
-                 * Define user security levels for the template
-                 *
-                 * These values are used in the default template of WiFoDog but could be
-                 * used in a customized template to restrict certain links to specific
-                 * user access levels.
-                 */
-                $smarty->assign('isValidUser', $user ? true : false);
-                $smarty->assign('isSuperAdmin', $user && $user->isSuperAdmin());
-                $smarty->assign('isOwner', $user && $user->isOwner());
+        /**
+         * Define user security levels for the template
+         *
+         * These values are used in the default template of WiFoDog but could be
+         * used in a customized template to restrict certain links to specific
+         * user access levels.
+         */
+        $smarty->assign('isValidUser', $user && !$user->isSplashOnlyUser() ? true : false);
+        $smarty->assign('isSuperAdmin', $user && $user->isSuperAdmin());
+        $smarty->assign('isOwner', $user && $user->isOwner());
 
         if (isset ($_REQUEST['debug_request']) && ($user && $user->isSuperAdmin())) {
@@ -838 +767 @@
             $smarty->assign('debugOutput', print_r($_REQUEST, true));
         }
-        }
+    }
 }
 
@@ -848 +777 @@
  * End:
  */
-