Changeset 900 for trunk/Modules/User

Timestamp:

Apr 10, 2024, 11:49:39 PM (7 months ago)

Author:

chronos

Message:

• Fixed: More form input fields validation.
• Modified: Code cleanup.

Location:

trunk/Modules/User

Files:

• Profile.php (modified) (4 diffs)
• UserList.php (modified) (2 diffs)

trunk/Modules/User/Profile.php

@@ -37 +37 @@
 
     $Output .= '<h3>'.T('Exports').'</h3>'.
-        $PageList['Output'];
+      $PageList['Output'];
 
     $TableColumns = array(
-        array('Name' => 'TimeCreate', 'Title' => T('Creation time')),
-        array('Name' => 'Title', 'Title' => T('Name')),
-        //  array('Name' => 'UserCount', 'Title' => 'Vybraných překladatelů'),
-        //  array('Name' => 'GroupCount', 'Title' => 'Překladových skupin'),
-        array('Name' => 'OutputType', 'Title' => T('Output type')),
-        array('Name' => 'ClientVersion', 'Title' => T('Client version')),
-        array('Name' => 'UsedCount', 'Title' => T('Output inspections')),
-        array('Name' => '', 'Title' => T('Actions')),
+      array('Name' => 'TimeCreate', 'Title' => T('Creation time')),
+      array('Name' => 'Title', 'Title' => T('Name')),
+      //  array('Name' => 'UserCount', 'Title' => 'Vybraných překladatelů'),
+      //  array('Name' => 'GroupCount', 'Title' => 'Překladových skupin'),
+      array('Name' => 'OutputType', 'Title' => T('Output type')),
+      array('Name' => 'ClientVersion', 'Title' => T('Client version')),
+      array('Name' => 'UsedCount', 'Title' => T('Output inspections')),
+      array('Name' => '', 'Title' => T('Actions')),
     );
     $Order = GetOrderTableHeader($TableColumns, 'TimeCreate', 1);
@@ -73 +73 @@
     }
     $Output .= '</table>'.
-        $PageList['Output'];
+      $PageList['Output'];
 
     $Output .= '<div style="text-align: center;"><a href="'.$this->System->Link('/export/').'">'.T('Export page').'</a></div>';
@@ -141 +141 @@
   {
     $Output = '';
+    if (!is_numeric($_GET['user']))
+    {
+      $Output .= ShowMessage('Uživatel nenalezen', MESSAGE_CRITICAL);
+      return $Output;
+    }
     $User = ModuleUser::Cast($this->System->GetModule('User'))->User;
     $Query = 'SELECT `User`.`Name`, `UserTrace`.`LastLogin`, `UserTrace`.`LastIP`, '.
@@ -177 +182 @@
         $Output .= T('Translator is using this rules:').'<br />';
         $Output .= '<ul>';
-        while ($UserTag = $DbResult->fetch_array()) {
+        while ($UserTag = $DbResult->fetch_array())
+        {
           $Output .= ' <li>'.$UserTag['Text'].'</li>';
         }

trunk/Modules/User/UserList.php

@@ -26 +26 @@
         $Output .= '<h3>'.sprintf(T('Users in team %s'), htmlspecialchars($Team['Name'])).'</h3>';
         $TeamFilter = ' AND (`Team`='.$_GET['team'].')';
-      } else {
+      } else
+      {
         $Output .= ShowMessage(sprintf(T('Team %d not found'), $TeamId), MESSAGE_CRITICAL);
       }
@@ -69 +70 @@
     }
     $Output .= '</table>'.
-        $PageList['Output'];
+      $PageList['Output'];
 
     return $Output;