Changeset 539 for trunk/includes/user.php

Timestamp:

May 15, 2013, 12:44:32 AM (11 years ago)

Author:

chronos

Message:

• Upraveno: Sloupce LastIP, LastLogin, LastLogout odděleny z tabulky User do samostatné nové tabulky UserTrace. Tato tabulka slouží pro uchování údajů aktualizovaných při každém zobrazení stránky. Dokud byly tyto údaje aktualizovány přímo v tabulce User, tak docházelo k přegenerování všech indexů tabulky User a tedy výraznému zpomalení provádění SQL dotazů.
• Upraveno: Zobrazení klikacího odkazu na uživatele ve výpisu dalších tabulek.
• Upraveno: Optimalizováno načítání obecného a týmového stavu dokončení překladu.
• Upraveno: Při registraci se volá nově metoda User->Register namísto přímého vkládání položky do databáze.

File:

• trunk/includes/user.php (modified) (3 diffs)

trunk/includes/user.php

@@ -46 +46 @@
   {
     if($this->Role != LICENCE_ANONYMOUS)
-      $this->System->Database->query('UPDATE `User` SET `LastLogout` = NOW() WHERE `ID` = '.$this->Id);
+      $this->System->Database->query('UPDATE `UserTrace` SET '.
+        '`LastLogout` = NOW() WHERE `User` = '.$this->Id);
     $this->SetAnonymous();
   }
@@ -53 +54 @@
   {
     $DbResult = $this->System->Database->query('SELECT * FROM `User` WHERE `ID` = '.$this->Id);
-    $User = $DbResult->fetch_assoc();
-    // Security: Password and Salt hash should not be loaded to variables
-    $this->Id = $User['ID'];
-    $this->Team = $User['Team'];
-    $this->Redirecting = $User['Redirecting'];
-    $this->Language = $User['Language'];
-    $this->Name = $User['Name'];
-    $this->Role = $User['GM'];
-    $this->Email = $User['Email'];
-    $this->Info = $User['Info'];
-    $this->PreferredVersion = $User['PreferredVersion'];
+    if($DbResult->num_rows > 0)
+    {
+      $User = $DbResult->fetch_assoc();
+      // Security: Password and Salt hash should not be loaded to variables
+      $this->Id = $User['ID'];
+      $this->Team = $User['Team'];
+      $this->Redirecting = $User['Redirecting'];
+      $this->Language = $User['Language'];
+      $this->Name = $User['Name'];
+      $this->Role = $User['GM'];
+      $this->Email = $User['Email'];
+      $this->Info = $User['Info'];
+      $this->PreferredVersion = $User['PreferredVersion'];
+    } else $this->SetAnonymous();
   }
   
@@ -113 +117 @@
   function UpdateState()
   {
-    if(array_key_exists('REMOTE_ADDR', $_SERVER)) 
-      $this->System->Database->query('UPDATE `User` SET `LastIP` = "'.$_SERVER['REMOTE_ADDR'].'", `LastLogin` = NOW() WHERE `ID` = '.$this->Id);
+    if(array_key_exists('REMOTE_ADDR', $_SERVER) and ($this->Role != LICENCE_ANONYMOUS)) 
+      $this->System->Database->query('UPDATE `UserTrace` SET '.
+        '`LastIP` = "'.$_SERVER['REMOTE_ADDR'].'", '.
+        '`LastLogin` = NOW(), '.
+        '`UserAgent` = "'.$this->System->Database->real_escape_string($_SERVER['HTTP_USER_AGENT']).'" '.
+        'WHERE `User` = '.$this->Id);
+  }
+  
+  function Register($UserName, $Password, $Email, $Language, $Team, $PreferredVersion)
+  {
+    $Salt = $this->GetPasswordSalt();
+    $this->System->Database->query('INSERT INTO `User` '.
+      '(`Name` , `Pass` , `Salt`, `Email` , `Language` , `Team` , `NeedUpdate`, `RegistrationTime`, `PreferredVersion` ) '.
+      'VALUES ("'.$UserName.'", '.$this->CryptPasswordSQL('"'.$Password.'"', '"'.$Salt.'"').
+      ', "'.$Salt.'", "'.$Email.'", '.$Language.', '.$Team.', 1, NOW(), '.$PreferredVersion.')');
+    $UserId = $this->System->Database->insert_id;
+    $this->System->Database->query('INSERT INTO `UserTrace` (`User`, `LastIP`, `UserAgent`) '.
+        'VALUES ('.$UserId.', "'.$_SERVER['REMOTE_ADDR'].'", '.
+        '"'.$this->System->Database->real_escape_string($_SERVER['HTTP_USER_AGENT']).'")');
   }
 }