Mark mime_magic option with old default value as obsolete

Thomas Bruederli

2013-01-10 25a31031b00bd4eadc71d85cc0e0d7b4837a6a4e

 installer/rcube_install.php

@@ -70,20 +70,20 @@
    $this->step = intval($_REQUEST['_step']);
    $this->is_post = $_SERVER['REQUEST_METHOD'] == 'POST';
  }
  

  /**
   * Singleton getter
   */
  function get_instance()
  {
    static $inst;
    

    if (!$inst)
      $inst = new rcube_install();
    

    return $inst;
  }
  

  /**
   * Read the default config files and store properties
   */
@@ -109,19 +109,19 @@
   */
  function _load_config($suffix)
  {
    if (is_readable($main_inc = RCMAIL_CONFIG_DIR . '/main.inc' . $suffix)) {
    if (is_readable($main_inc = RCUBE_CONFIG_DIR . 'main.inc' . $suffix)) {
      include($main_inc);
      if (is_array($rcmail_config))
        $this->config += $rcmail_config;
    }
    if (is_readable($db_inc = RCMAIL_CONFIG_DIR . '/db.inc'. $suffix)) {
    if (is_readable($db_inc = RCUBE_CONFIG_DIR . 'db.inc'. $suffix)) {
      include($db_inc);
      if (is_array($rcmail_config))
        $this->config += $rcmail_config;
    }
  }
  
  


  /**
   * Getter for a certain config property
   *
@@ -132,10 +132,10 @@
  function getprop($name, $default = '')
  {
    $value = $this->config[$name];
    

    if ($name == 'des_key' && !$this->configured && !isset($_REQUEST["_$name"]))
      $value = rcube_install::random_key(24);
    

    return $value !== null && $value !== '' ? $value : $default;
  }

@@ -149,7 +149,7 @@
   */
  function create_config($which, $force = false)
  {
    $out = @file_get_contents(RCMAIL_CONFIG_DIR . "/{$which}.inc.php.dist");
    $out = @file_get_contents(RCUBE_CONFIG_DIR . $which . '.inc.php.dist');

    if (!$out)
      return '[Warning: could not read the config template file]';
@@ -193,16 +193,16 @@
        $value = '%p';
      }
      else if ($prop == 'default_folders') {
       $value = array();
       foreach ($this->config['default_folders'] as $_folder) {
         switch ($_folder) {
         case 'Drafts': $_folder = $this->config['drafts_mbox']; break;
         case 'Sent':   $_folder = $this->config['sent_mbox']; break;
         case 'Junk':   $_folder = $this->config['junk_mbox']; break;
         case 'Trash':  $_folder = $this->config['trash_mbox']; break;
        $value = array();
        foreach ($this->config['default_folders'] as $_folder) {
          switch ($_folder) {
          case 'Drafts': $_folder = $this->config['drafts_mbox']; break;
          case 'Sent':   $_folder = $this->config['sent_mbox']; break;
          case 'Junk':   $_folder = $this->config['junk_mbox']; break;
          case 'Trash':  $_folder = $this->config['trash_mbox']; break;
          }
       if (!in_array($_folder, $value))
         $value[] = $_folder;
        if (!in_array($_folder, $value))
          $value[] = $_folder;
        }
      }
      else if (is_bool($default)) {
@@ -241,14 +241,14 @@
    $this->config = array();
    $this->load_defaults();
    $defaults = $this->config;
    

    $this->load_config();
    if (!$this->configured)
      return null;
    

    $out = $seen = array();
    $required = array_flip($this->required_config);
    

    // iterate over the current configuration
    foreach ($this->config as $prop => $value) {
      if ($replacement = $this->replaced_config[$prop]) {
@@ -260,7 +260,12 @@
        $seen[$prop] = true;
      }
    }
    

    // the old default mime_magic reference is obsolete
    if ($this->config['mime_magic'] == '/usr/share/misc/magic') {
        $out['obsolete'][] = array('prop' => 'mime_magic', 'explain' => "Set value to null in order to use system default");
    }

    // iterate over default config
    foreach ($defaults as $prop => $value) {
      if (!isset($seen[$prop]) && isset($required[$prop]) && !(is_bool($this->config[$prop]) || strlen($this->config[$prop])))
@@ -280,7 +285,7 @@
              'explain' => "You are missing pspell support for language $lang ($descr)");
      }
    }
    

    if ($this->config['log_driver'] == 'syslog') {
      if (!function_exists('openlog')) {
        $out['dependencies'][] = array('prop' => 'log_driver',
@@ -291,7 +296,7 @@
          'explain' => 'Using <tt>syslog</tt> for logging requires a syslog ID to be configured');
      }
    }
    

    // check ldap_public sources having global_search enabled
    if (is_array($this->config['ldap_public']) && !is_array($this->config['autocomplete_addressbooks'])) {
      foreach ($this->config['ldap_public'] as $ldap_public) {
@@ -301,11 +306,11 @@
        }
      }
    }
    

    return $out;
  }
  
  


  /**
   * Merge the current configuration with the defaults
   * and copy replaced values to the new options.
@@ -327,11 +332,11 @@
      }
      unset($current[$prop]);
    }
    

    foreach ($this->obsolete_config as $prop) {
      unset($current[$prop]);
    }
    

    // add all ldap_public sources having global_search enabled to autocomplete_addressbooks
    if (is_array($current['ldap_public'])) {
      foreach ($current['ldap_public'] as $key => $ldap_public) {
@@ -341,9 +346,6 @@
        }
      }
    }
    
    if ($current['keep_alive'] && $current['session_lifetime'] < $current['keep_alive'])
      $current['session_lifetime'] = max(10, ceil($current['keep_alive'] / 60) * 2);

    $this->config  = array_merge($this->config, $current);

@@ -351,7 +353,7 @@
      $this->config['ldap_public'][$key] = $current['ldap_public'][$key];
    }
  }
  

  /**
   * Compare the local database schema with the reference schema
   * required for this version of Roundcube
@@ -363,11 +365,11 @@
  {
    if (!$this->configured)
      return false;
    

    // read reference schema from mysql.initial.sql
    $db_schema = $this->db_read_schema(INSTALL_PATH . 'SQL/mysql.initial.sql');
    $errors = array();
    

    // check list of tables
    $existing_tables = $DB->list_tables();

@@ -454,7 +456,8 @@
        '0.5-beta', '0.5', '0.5.1',
        '0.6-beta', '0.6',
        '0.7-beta', '0.7', '0.7.1', '0.7.2', '0.7.3',
        '0.8-beta', '0.8-rc', '0.8.0', '0.8.1',
        '0.8-beta', '0.8-rc', '0.8.0', '0.8.1', '0.8.2', '0.8.3', '0.8.4',
        '0.9-beta',
    ));
    return $select;
  }
@@ -636,8 +639,8 @@
   */
  function update_db($DB, $version)
  {
    $version = strtolower($version);
    $engine = isset($this->db_map[$DB->db_provider]) ? $this->db_map[$DB->db_provider] : $DB->db_provider;
    $version = version_parse(strtolower($version));
    $engine  = isset($this->db_map[$DB->db_provider]) ? $this->db_map[$DB->db_provider] : $DB->db_provider;

    // read schema file from /SQL/*
    $fname = INSTALL_PATH . "SQL/$engine.update.sql";
@@ -646,7 +649,7 @@
      foreach ($lines as $line) {
        $is_comment = preg_match('/^--/', $line);
        if (!$from && $is_comment && preg_match('/from version\s([0-9.]+[a-z-]*)/', $line, $m)) {
          $v = strtolower($m[1]);
          $v = version_parse(strtolower($m[1]));
          if ($v == $version || version_compare($version, $v, '<='))
            $from = true;
        }