Changeset 800 for trunk/Modules/ClientVersion

Timestamp:

Mar 16, 2014, 11:15:43 AM (11 years ago)

Author:

chronos

Message:

• Added: Module Redirection responsible for redirection of old pages to new valid address.
• Fixed: Error on client version item display if id was not specified.
• Fixed: Error on silent application module reregistration.
• Removed: White space on end of lines in some files.

File:

• trunk/Modules/ClientVersion/ClientVersion.php (modified) (6 diffs)

trunk/Modules/ClientVersion/ClientVersion.php

@@ -13 +13 @@
     $this->Dependencies = array('');
   }
-  
+
   function Start()
   {
@@ -38 +38 @@
                 return($Output);
         }
-        
+
         function ShowItem()
         {
-     $YesNo = array('Ne', 'Ano');
-     $DbResult = $this->System->Database->query('SELECT * FROM ClientVersion WHERE Id='.($_GET['id']*1));
-     if($DbResult->num_rows > 0)
-     {
-         $Version = $DbResult->fetch_assoc();
-    
-     $Output = '<h3>'.T('Client version').'</h3>';
-     $Output .= '<table class="BaseTable">'.
+    if(array_key_exists('id', $_GET))
+    {
+      $YesNo = array('Ne', 'Ano');
+      $DbResult = $this->System->Database->query('SELECT * FROM ClientVersion WHERE Id='.($_GET['id']*1));
+      if($DbResult->num_rows > 0)
+      {
+        $Version = $DbResult->fetch_assoc();
+
+      $Output = '<h3>'.T('Client version').'</h3>';
+      $Output .= '<table class="BaseTable">'.
         '<tr><td>'.T('Version').'</td><td>'.$Version['Version'].'</td></tr>'.
         '<tr><td>'.T('More information').'</td><td><a href="http://www.wowwiki.com/Patch_'.$Version['Version'].'">wowwiki.com'.
@@ -58 +60 @@
         '</table>';
         $Output .= '<div><a href="?">'.T('All versions list').'</a></div>';
-        if($Version['Imported']) 
+        if($Version['Imported'])
           $Output .= '<div><a href="'.$this->System->Link('/progress/?Version='.$Version['Version']).'">'.T('Progress').'</a></div>';
-     } else $Output = ShowMessage(T('Item not found'), MESSAGE_CRITICAL);
+      } else $Output = ShowMessage(T('Item not found'), MESSAGE_CRITICAL);
+    } else $Output = ShowMessage(T('Item not found'), MESSAGE_CRITICAL);
                 return($Output);
         }
-        
+
         function ShowList()
-        {               
+        {
                 $this->Title = T('Game version');
     $DbResult = $this->System->Database->query('SELECT COUNT(*) FROM `ClientVersion`');
     $DbRow = $DbResult->fetch_row();
-    $PageList = GetPageList($DbRow[0]);    
+    $PageList = GetPageList($DbRow[0]);
 
     $Output = '<h3>'.T('Game version').'</h3>'.
@@ -75 +78 @@
 
     $TableColumns = array(
-      array('Name' => 'Version', 'Title' => T('Version')), 
-      array('Name' => 'BuildNumber', 'Title' => T('Build')), 
-      array('Name' => 'ReleaseDate', 'Title' => T('Release date')), 
-      array('Name' => 'Title', 'Title' => T('Title')), 
+      array('Name' => 'Version', 'Title' => T('Version')),
+      array('Name' => 'BuildNumber', 'Title' => T('Build')),
+      array('Name' => 'ReleaseDate', 'Title' => T('Release date')),
+      array('Name' => 'Title', 'Title' => T('Title')),
       array('Name' => 'Imported', 'Title' => T('Imported')),
     );
@@ -84 +87 @@
     $Output .= '<table class="BaseTable">'.
       $Order['Output'];
-      
+
     $YesNo = array('Ne', 'Ano');
 
@@ -97 +100 @@
     $Output .= '</table>'.
       $PageList['Output'];
-    return($Output);        
+    return($Output);
   }
 }