Changeset 28651

Timestamp:

Jun 9, 2014, 7:20:43 PM (10 years ago)

Author:

mistic100

Message:

feature 3076: Enhance plugin update system

Location:

trunk

Files:

• admin/include/plugins.class.php (modified) (10 diffs)
• admin/include/themes.class.php (modified) (1 diff)
• include/functions_plugins.inc.php (modified) (7 diffs)

trunk/admin/include/plugins.class.php

@@ -56 +56 @@
     }
   }
+  function update($old_version, $new_version, &$errors=array()) {}
 }
 
@@ -86 +87 @@
   private static function build_maintain_class($plugin_id)
   {
-    $file_to_include = PHPWG_PLUGINS_PATH . $plugin_id . '/maintain.inc.php';
+    $file_to_include = PHPWG_PLUGINS_PATH . $plugin_id . '/maintain';
     $classname = $plugin_id.'_maintain';
 
-    if (file_exists($file_to_include))
-    {
-      include_once($file_to_include);
+    if (file_exists($file_to_include.'.class.php'))
+    {
+      include_once($file_to_include.'.class.php');
+      return new $classname($plugin_id);
+    }
+
+    if (file_exists($file_to_include.'.inc.php'))
+    {
+      include_once($file_to_include.'.inc.php');
 
       if (class_exists($classname))
       {
-        $plugin_maintain = new $classname($plugin_id);
-      }
-      else
-      {
-        $plugin_maintain = new DummyPlugin_maintain($plugin_id);
-      }
-    }
-    else
-    {
-      $plugin_maintain = new DummyPlugin_maintain($plugin_id);
-    }
-
-    return $plugin_maintain;
+        return new $classname($plugin_id);
+      }
+    }
+
+    return new DummyPlugin_maintain($plugin_id);
   }
 
@@ -122 +121 @@
       $crt_db_plugin = $this->db_plugins_by_id[$plugin_id];
     }
-    
+
     $plugin_maintain = self::build_maintain_class($plugin_id);
 
@@ -188 +187 @@
 ;';
         pwg_query($query);
-        
+
         $plugin_maintain->deactivate();
         break;
@@ -207 +206 @@
 ;';
         pwg_query($query);
-        
+
         $plugin_maintain->uninstall();
         break;
@@ -236 +235 @@
   /**
    * Get plugins defined in the plugin directory
-   */  
+   */
   function get_fs_plugins()
   {
@@ -257 +256 @@
               'author'=>'',
             );
-          $plg_data = implode( '', file($path.'/main.inc.php') );
+          $plg_data = file_get_contents($path.'/main.inc.php', null, null, 0, 2048);
 
           if ( preg_match("|Plugin Name: (.*)|", $plg_data, $val) )
@@ -327 +326 @@
   {
     global $conf;
-    
+
     $versions_to_check = array();
     $url = PEM_URL . '/api/get_version_list.php?category_id='. $conf['pem_plugins_category'] .'&format=php';
@@ -424 +423 @@
       return false;
     }
-    
+
     global $conf;
 
@@ -478 +477 @@
     return false;
   }
-  
+
   /**
    * Sort $server_plugins

trunk/admin/include/themes.class.php

@@ -87 +87 @@
       if (class_exists($classname))
       {
-        $theme_maintain = new $classname($theme_id);
-      }
-      else
-      {
-        $theme_maintain = new DummyTheme_maintain($theme_id);
-      }
-    }
-    else
-    {
-      $theme_maintain = new DummyTheme_maintain($theme_id);
-    }
-
-    return $theme_maintain;
+        return new $classname($theme_id);
+      }
+    }
+    
+    return new DummyTheme_maintain($theme_id);
   }
 

trunk/include/functions_plugins.inc.php

@@ -36 +36 @@
  * Used to declare maintenance methods of a plugin.
  */
-class PluginMaintain 
+class PluginMaintain
 {
   /** @var string $plugin_id */
@@ -66 +66 @@
 
   /**
-   * Tests if the plugin needs to be updated and call an update function
-   *
-   * @param string $version version exposed by the plugin (potentially new)
-   * @param string $on_update name of a method to call when an update is needed
-   *          it receives the previous version as first parameter
-   */
-  function autoUpdate($version, $on_update=null)
-  {
-    global $pwg_loaded_plugins;
-    
-    $current_version = $pwg_loaded_plugins[$this->plugin_id]['version'];
-    
-    if ( $version == 'auto' or $current_version == 'auto'
-        or safe_version_compare($current_version, $version, '<')
-      )
-    {
-      if (!empty($on_update))
-      {
-        call_user_func(array(&$this, $on_update), $current_version);
-      }
-      
-      if ($version != 'auto')
-      {
-        $query = '
-UPDATE '. PLUGINS_TABLE .'
-  SET version = "'. $version .'"
-  WHERE id = "'. $this->plugin_id .'"
-;';
-        pwg_query($query);
-        
-        $pwg_loaded_plugins[$this->plugin_id]['version'] = $version;
-      }
-    }
-  }
+   * @param string $old_version
+   * @param string $new_version
+   * @param array &$errors - used to return error messages
+   */
+  function update($old_version, $new_version, &$errors=array()) {}
 }
 
@@ -105 +76 @@
  * Used to declare maintenance methods of a theme.
  */
-class ThemeMaintain 
+class ThemeMaintain
 {
   /** @var string $theme_id */
@@ -127 +98 @@
 
   function delete() {}
-  
-  /**
-   * Tests if the theme needs to be updated and call an update function
-   *
-   * @param string $version version exposed by the theme (potentially new)
-   * @param string $on_update name of a method to call when an update is needed
-   *          it receives the previous version as first parameter
-   */
-  function autoUpdate($version, $on_update=null)
-  {
-    $query = '
-SELECT version
-  FROM '. THEMES_TABLE .'
-  WHERE id = "'. $this->theme_id .'"
-;';
-    list($current_version) = pwg_db_fetch_row(pwg_query($query));
-    
-    if ( $version == 'auto' or $current_version == 'auto'
-        or safe_version_compare($current_version, $version, '<')
-      )
-    {
-      if (!empty($on_update))
-      {
-        call_user_func(array(&$this, $on_update), $current_version);
-      }
-      
-      if ($version != 'auto')
-      {
-        $query = '
-UPDATE '. THEMES_TABLE .'
-  SET version = "'. $version .'"
-  WHERE id = "'. $this->theme_id .'"
-;';
-        pwg_query($query);
-      }
-    }
-  }
 }
 
@@ -251 +185 @@
  * @param mixed $data data to transmit to all handlers
  * @param mixed $args,... optional arguments
- * @return mixed $data 
+ * @return mixed $data
  */
 function trigger_change($event, $data=null)
@@ -400 +334 @@
   WHERE '. implode(' AND ', $clauses);
   }
-  
+
   return query2array($query);
 }
 
 /**
- * Loads a plugin, it includes the main.inc.php file and updates _$pwg_loaded_plugins_.
+ * Loads a plugin in memory.
+ * It performs autoupdate, includes the main.inc.php file and updates *$pwg_loaded_plugins*.
  *
  * @param string $plugin
@@ -412 +347 @@
 {
   $file_name = PHPWG_PLUGINS_PATH.$plugin['id'].'/main.inc.php';
-  if ( file_exists($file_name) )
-  {
+  if (file_exists($file_name))
+  {
+    autoupdate_plugin($plugin);
     global $pwg_loaded_plugins;
     $pwg_loaded_plugins[ $plugin['id'] ] = $plugin;
-    include_once( $file_name );
+    include_once($file_name);
+  }
+}
+
+/**
+ * Performs update task of a plugin.
+ * Autoupdate is only performed if the plugin has a maintain.class.php file.
+ *
+ * @since 2.7
+ *
+ * @param array &$plugin (id, version, state) will be updated if version changes
+ */
+function autoupdate_plugin(&$plugin)
+{
+  $maintain_file = PHPWG_PLUGINS_PATH.$plugin['id'].'/maintain.class.php';
+
+  // autoupdate is applicable only to plugins with 2.7 architecture
+  if (file_exists($maintain_file))
+  {
+    // try to find the filesystem version in lines 2 to 10 of main.inc.php
+    $fh = fopen(PHPWG_PLUGINS_PATH.$plugin['id'].'/main.inc.php', 'r');
+    $fs_version = null;
+    $i = -1;
+
+    while (($line = fgets($fh))!==false && $fs_version==null && $i<10)
+    {
+      $i++;
+      if ($i < 2) continue; // first lines are typically "<?php" and "/*"
+
+      if (preg_match('#Version: ([\\w.-]+)#', $line, $matches))
+      {
+        $fs_version = $matches[1];
+      }
+    }
+
+    fclose($fh);
+
+    if ($fs_version != null)
+    {
+      global $pwg_loaded_plugins, $page;
+
+      // if version is auto (dev) or superior
+      if (
+          $fs_version == 'auto' or $plugin['version'] == 'auto'
+          or safe_version_compare($plugin['version'], $fs_version, '<')
+        )
+      {
+        // call update method
+        include_once($maintain_file);
+
+        $classname = $plugin['id'].'_maintain';
+        $plugin_maintain = new $classname($plugin['id']);
+        $plugin_maintain->update($plugin['version'], $fs_version, $page['errors']);
+
+        $plugin['version'] = $fs_version;
+
+        // update database (only on production)
+        if ($plugin['version'] != 'auto')
+        {
+          $query = '
+UPDATE '. PLUGINS_TABLE .'
+  SET version = "'. $plugin['version'] .'"
+  WHERE id = "'. $plugin['id'] .'"
+;';
+          pwg_query($query);
+        }
+      }
+    }
   }
 }