Changeset 1027

Timestamp:

Feb 6, 2006, 10:52:16 PM (18 years ago)

Author:

plg

Message:

improvement: upgrades id retrieving in include/common.inc.php and
upgrade_feed.php are now made by dedicated function
get_available_upgrade_ids.

bug fixed: after an installation, you had to play all available upgrades,
which was wrong. install.php inserts informations related to all available
upgrades at installation time. Thus avoiding automatic upgrades.

Location:

trunk

Files:

• include/common.inc.php (modified) (1 diff)
• include/functions.inc.php (modified) (2 diffs)
• install.php (modified) (1 diff)
• upgrade_feed.php (modified) (1 diff)

trunk/include/common.inc.php

@@ -135 +135 @@
 if ($conf['check_upgrade_feed'])
 {
-  define('PREFIX_TABLE', $prefixeTable);
-  define('UPGRADES_PATH', PHPWG_ROOT_PATH.'install/db');
-  
   // retrieve already applied upgrades
   $query = '
 SELECT id
-  FROM '.PREFIX_TABLE.'upgrade
+  FROM '.UPGRADE_TABLE.'
 ;';
   $applied = array_from_query($query, 'id');
 
   // retrieve existing upgrades
-  $existing = array();
-
-  if ($contents = opendir(UPGRADES_PATH))
-  {
-    while (($node = readdir($contents)) !== false)
-    {
-      if (is_file(UPGRADES_PATH.'/'.$node)
-          and preg_match('/^(.*?)-database\.php$/', $node, $match))
-      {
-        array_push($existing, $match[1]);
-      }
-    }
-  }
-  natcasesort($existing);
+  $existing = get_available_upgrade_ids();
 
   // which upgrades need to be applied?

trunk/include/functions.inc.php

@@ -489 +489 @@
 function redirect( $url )
 {
-  global $user, $template, $lang_info, $conf, $lang, $t2, $page;
+  global $user, $template, $lang_info, $conf, $lang, $t2, $page, $debug;
 
   // $refresh, $url_link and $title are required for creating an automated
@@ -973 +973 @@
   return $email;
 }
+
+/**
+ * which upgrades are available ?
+ *
+ * @return array
+ */
+function get_available_upgrade_ids()
+{
+  $upgrades_path = PHPWG_ROOT_PATH.'install/db';
+
+  $available_upgrade_ids = array();
+  
+  if ($contents = opendir($upgrades_path))
+  {
+    while (($node = readdir($contents)) !== false)
+    {
+      if (is_file($upgrades_path.'/'.$node)
+          and preg_match('/^(.*?)-database\.php$/', $node, $match))
+      {
+        array_push($available_upgrade_ids, $match[1]);
+      }
+    }
+  }
+  natcasesort($available_upgrade_ids);
+
+  return $available_upgrade_ids;
+}
 ?>

trunk/install.php

@@ -339 +339 @@
 ;';
     mysql_query($query);
+
+    // Available upgrades must be ignored after a fresh installation. To
+    // make PWG avoid upgrading, we must tell it upgrades have already been
+    // made.
+    foreach (get_available_upgrade_ids() as $upgrade_id)
+    {
+      $query = '
+INSERT INTO '.UPGRADE_TABLE.'
+  (id, applied, description)
+  VALUES
+  ('.$upgrade_id.', NOW(), \'upgrade included in installation\')
+';
+      mysql_query($query);
+    }
   }
 }

trunk/upgrade_feed.php

@@ -57 +57 @@
 
 // retrieve existing upgrades
-$existing = array();
-
-if ($contents = opendir(UPGRADES_PATH))
-{
-  while (($node = readdir($contents)) !== false)
-  {
-    if (is_file(UPGRADES_PATH.'/'.$node)
-        and preg_match('/^(.*?)-database\.php$/', $node, $match))
-    {
-      array_push($existing, $match[1]);
-    }
-  }
-}
-natcasesort($existing);
+$existing = get_available_upgrade_ids();
 
 // which upgrades need to be applied?