CS fixes

Aleksander Machniak

2016-03-14 3a13b5dab88346ca6fe73a95037fb9ee73dd9b06

CS fixes

 program/include/rcmail_install.php

@@ -5,7 +5,7 @@
 | rcmail_install.php                                                    |
 |                                                                       |
 | This file is part of the Roundcube Webmail package                    |
 | Copyright (C) 2008-2014, The Roundcube Dev Team                       |
 | Copyright (C) 2008-2016, The Roundcube Dev Team                       |
 |                                                                       |
 | Licensed under the GNU General Public License version 3 or            |
 | any later version with exceptions for skins & plugins.                |
@@ -22,19 +22,19 @@
 */
class rcmail_install
{
  var $step;
  var $is_post = false;
  var $failures = 0;
  var $config = array();
  var $configured = false;
  var $legacy_config = false;
  var $last_error = null;
  var $email_pattern = '([a-z0-9][a-z0-9\-\.\+\_]*@[a-z0-9]([a-z0-9\-][.]?)*[a-z0-9])';
  var $bool_config_props = array();
    public $step;
    public $last_error;
    public $is_post           = false;
    public $failures          = 0;
    public $config            = array();
    public $configured        = false;
    public $legacy_config     = false;
    public $email_pattern     = '([a-z0-9][a-z0-9\-\.\+\_]*@[a-z0-9]([a-z0-9\-][.]?)*[a-z0-9])';
    public $bool_config_props = array();

  var $local_config = array('db_dsnw', 'default_host', 'support_url', 'des_key', 'plugins');
  var $obsolete_config = array('db_backend', 'db_max_length', 'double_auth');
  var $replaced_config = array(
    public $local_config    = array('db_dsnw', 'default_host', 'support_url', 'des_key', 'plugins');
    public $obsolete_config = array('db_backend', 'db_max_length', 'double_auth');
    public $replaced_config = array(
    'skin_path'            => 'skin',
    'locale_string'        => 'language',
    'multiple_identities'  => 'identities_level',
@@ -47,7 +47,7 @@
  );

  // list of supported database drivers
  var $supported_dbs = array(
    public $supported_dbs = array(
    'MySQL'               => 'pdo_mysql',
    'PostgreSQL'          => 'pdo_pgsql',
    'SQLite'              => 'pdo_sqlite',
@@ -61,7 +61,7 @@
  /**
   * Constructor
   */
  function __construct()
    public function __construct()
  {
    $this->step = intval($_REQUEST['_step']);
    $this->is_post = $_SERVER['REQUEST_METHOD'] == 'POST';
@@ -70,12 +70,13 @@
  /**
   * Singleton getter
   */
  static function get_instance()
    public static function get_instance()
  {
    static $inst;

    if (!$inst)
        if (!$inst) {
      $inst = new rcmail_install();
        }

    return $inst;
  }
@@ -83,7 +84,7 @@
  /**
   * Read the local config files and store properties
   */
  function load_config()
    public function load_config()
  {
    // defaults
    if ($config = $this->load_config_file(RCUBE_CONFIG_DIR . 'defaults.inc.php')) {
@@ -102,6 +103,7 @@
        $this->config = array_merge($this->config, $config);
        $this->legacy_config = true;
      }

      if ($config = $this->load_config_file(RCUBE_CONFIG_DIR . 'db.inc.php')) {
        $this->config = array_merge($this->config, $config);
        $this->legacy_config = true;
@@ -116,7 +118,10 @@
   */
  public function load_config_file($file)
  {
    if (is_readable($file)) {
        if (!is_readable($file)) {
            return;
        }

      include $file;

      // read comments from config file
@@ -124,9 +129,10 @@
        $tokens = token_get_all(file_get_contents($file));
        $in_config = false;
        $buffer = '';

        for ($i=0; $i < count($tokens); $i++) {
          $token = $tokens[$i];
          if ($token[0] == T_VARIABLE && $token[1] == '$config' || $token[1] == '$rcmail_config') {
                if ($token[0] == T_VARIABLE && ($token[1] == '$config' || $token[1] == '$rcmail_config')) {
            $in_config = true;
            if ($buffer && $tokens[$i+1] == '[' && $tokens[$i+2][0] == T_CONSTANT_ENCAPSED_STRING) {
              $propname = trim($tokens[$i+2][1], "'\"");
@@ -148,25 +154,25 @@

      return $config;
    }
  }

  /**
   * Getter for a certain config property
   *
   * @param string Property name
   * @param string Default value
     *
   * @return string The property value
   */
  function getprop($name, $default = '')
    public function getprop($name, $default = '')
  {
    $value = $this->config[$name];

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

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


  /**
   * Create configuration file that contains parameters
@@ -174,7 +180,7 @@
   *
   * @return string The complete config file content
   */
  function create_config()
    public function create_config()
  {
    $config = array();

@@ -183,40 +189,48 @@
      $value      = !$is_default || $this->bool_config_props[$prop] ? $_POST["_$prop"] : $default;

      // always disable installer
      if ($prop == 'enable_installer')
            if ($prop == 'enable_installer') {
        $value = false;
            }

      // reset useragent to default (keeps version up-to-date)
      if ($prop == 'useragent' && stripos($value, 'Roundcube Webmail/') !== false)
            if ($prop == 'useragent' && stripos($value, 'Roundcube Webmail/') !== false) {
        $value = $this->defaults[$prop];
            }

      // generate new encryption key, never use the default value
      if ($prop == 'des_key' && $value == $this->defaults[$prop])
            if ($prop == 'des_key' && $value == $this->defaults[$prop]) {
        $value = rcube_utils::random_bytes(24);
            }

      // convert some form data
      if ($prop == 'debug_level' && !$is_default) {
        if (is_array($value)) {
          $val = 0;
          foreach ($value as $dbgval)
                    foreach ($value as $dbgval) {
            $val += intval($dbgval);
                    }
          $value = $val;
        }
      }
      else if ($prop == 'db_dsnw' && !empty($_POST['_dbtype'])) {
        if ($_POST['_dbtype'] == 'sqlite')
          $value = sprintf('%s://%s?mode=0646', $_POST['_dbtype'], $_POST['_dbname']{0} == '/' ? '/' . $_POST['_dbname'] : $_POST['_dbname']);
        else if ($_POST['_dbtype'])
                if ($_POST['_dbtype'] == 'sqlite') {
                    $value = sprintf('%s://%s?mode=0646', $_POST['_dbtype'],
                        $_POST['_dbname']{0} == '/' ? '/' . $_POST['_dbname'] : $_POST['_dbname']);
                }
                else if ($_POST['_dbtype']) {
          $value = sprintf('%s://%s:%s@%s/%s', $_POST['_dbtype'], 
            rawurlencode($_POST['_dbuser']), rawurlencode($_POST['_dbpass']), $_POST['_dbhost'], $_POST['_dbname']);
                }
      }
      else if ($prop == 'smtp_auth_type' && $value == '0') {
        $value = '';
      }
      else if ($prop == 'default_host' && is_array($value)) {
        $value = self::_clean_array($value);
        if (count($value) <= 1)
                if (count($value) <= 1) {
          $value = $value[0];
                }
      }
      else if ($prop == 'mail_pagesize' || $prop == 'addressbook_pagesize') {
        $value = max(2, intval($value));
@@ -236,15 +250,19 @@
      else if ($prop == 'plugins' && !empty($_POST['submit'])) {
        $value = array();
        foreach (array_keys($_POST) as $key) {
          if (preg_match('/^_plugins_*/', $key))
                    if (preg_match('/^_plugins_*/', $key)) {
            array_push($value, $_POST[$key]);
                    }
        }
      }

      // skip this property
      if (($value == $this->defaults[$prop]) && !in_array($prop, $this->local_config)
            if ($value == $this->defaults[$prop]
                && (!in_array($prop, $this->local_config)
          || in_array($prop, array_merge($this->obsolete_config, array_keys($this->replaced_config)))
          || preg_match('/^db_(table|sequence)_/', $prop)) {
                    || preg_match('/^db_(table|sequence)_/', $prop)
                )
            ) {
        continue;
      }

@@ -255,6 +273,7 @@

    $out = "<?php\n\n";
    $out .= "/* Local configuration for Roundcube Webmail */\n\n";

    foreach ($config as $prop => $value) {
      // copy option descriptions from existing config or defaults.inc.php
      $out .= $this->comments[$prop];
@@ -264,13 +283,12 @@
    return $out;
  }


  /**
   * save generated config file in RCUBE_CONFIG_DIR
   *
   * @return boolean True if the file was saved successfully, false if not
   */
  function save_configfile($config)
    public function save_configfile($config)
  {
    if (is_writable(RCUBE_CONFIG_DIR)) {
      return file_put_contents(RCUBE_CONFIG_DIR . 'config.inc.php', $config);
@@ -285,12 +303,12 @@
   *
   * @return array List with problems detected
   */
  function check_config()
    public function check_config()
  {
    $this->load_config();

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

    $out = $seen = array();
@@ -309,31 +327,45 @@

    // 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");
            $out['obsolete'][] = array(
                'prop'    => 'mime_magic',
                'explain' => "Set value to null in order to use system default"
            );
    }

    // check config dependencies and contradictions
    if ($this->config['enable_spellcheck'] && $this->config['spellcheck_engine'] == 'pspell') {
      if (!extension_loaded('pspell')) {
        $out['dependencies'][] = array('prop' => 'spellcheck_engine',
          'explain' => 'This requires the <tt>pspell</tt> extension which could not be loaded.');
                $out['dependencies'][] = array(
                    'prop'    => 'spellcheck_engine',
                    'explain' => "This requires the <tt>pspell</tt> extension which could not be loaded."
                );
      }
      else if (!empty($this->config['spellcheck_languages'])) {
        foreach ($this->config['spellcheck_languages'] as $lang => $descr)
          if (!@pspell_new($lang))
            $out['dependencies'][] = array('prop' => 'spellcheck_languages',
              'explain' => "You are missing pspell support for language $lang ($descr)");
                foreach ($this->config['spellcheck_languages'] as $lang => $descr) {
                    if (!@pspell_new($lang)) {
                        $out['dependencies'][] = array(
                            'prop'    => 'spellcheck_languages',
                            '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',
          'explain' => 'This requires the <tt>syslog</tt> extension which could not be loaded.');
                $out['dependencies'][] = array(
                    'prop'    => 'log_driver',
                    'explain' => "This requires the <tt>syslog</tt> extension which could not be loaded."
                );
      }

      if (empty($this->config['syslog_id'])) {
        $out['dependencies'][] = array('prop' => 'syslog_id',
          'explain' => 'Using <tt>syslog</tt> for logging requires a syslog ID to be configured');
                $out['dependencies'][] = array(
                    'prop'    => 'syslog_id',
                    'explain' => "Using <tt>syslog</tt> for logging requires a syslog ID to be configured"
                );
      }
    }

@@ -341,7 +373,10 @@
    if (is_array($this->config['ldap_public']) && !is_array($this->config['autocomplete_addressbooks'])) {
      foreach ($this->config['ldap_public'] as $ldap_public) {
        if ($ldap_public['global_search']) {
          $out['replaced'][] = array('prop' => 'ldap_public::global_search', 'replacement' => 'autocomplete_addressbooks');
                    $out['replaced'][] = array(
                        'prop'        => 'ldap_public::global_search',
                        'replacement' => 'autocomplete_addressbooks'
                    );
          break;
        }
      }
@@ -350,25 +385,28 @@
    return $out;
  }


  /**
   * Merge the current configuration with the defaults
   * and copy replaced values to the new options.
   */
  function merge_config()
    public function merge_config()
  {
    $current = $this->config;
    $this->config = array();

    foreach ($this->replaced_config as $prop => $replacement) {
      if (isset($current[$prop])) {
        if ($prop == 'skin_path')
                if ($prop == 'skin_path') {
          $this->config[$replacement] = preg_replace('#skins/(\w+)/?$#', '\\1', $current[$prop]);
        else if ($prop == 'multiple_identities')
                }
                else if ($prop == 'multiple_identities') {
          $this->config[$replacement] = $current[$prop] ? 2 : 0;
        else
                }
                else {
          $this->config[$replacement] = $current[$prop];
      }
            }

      unset($current[$prop]);
    }

@@ -401,10 +439,11 @@
   *
   * @return boolean True if the schema is up-to-date, false if not or an error occurred
   */
  function db_schema_check($DB)
    public function db_schema_check($DB)
  {
    if (!$this->configured)
        if (!$this->configured) {
      return false;
        }

    // read reference schema from mysql.initial.sql
    $db_schema = $this->db_read_schema(INSTALL_PATH . 'SQL/mysql.initial.sql');
@@ -415,14 +454,17 @@

    foreach ($db_schema as $table => $cols) {
      $table = $this->config['db_prefix'] . $table;

      if (!in_array($table, $existing_tables)) {
        $errors[] = "Missing table '".$table."'";
      }
      else {  // compare cols
        $db_cols = $DB->list_cols($table);
        $diff = array_diff(array_keys($cols), $db_cols);
        if (!empty($diff))

                if (!empty($diff)) {
          $errors[] = "Missing columns in table '$table': " . join(',', $diff);
                }
      }
    }

@@ -437,13 +479,15 @@
    $lines = file($schemafile);
    $table_block = false;
    $schema = array();
        $keywords    = array('PRIMARY','KEY','INDEX','UNIQUE','CONSTRAINT','REFERENCES','FOREIGN');

    foreach ($lines as $line) {
      if (preg_match('/^\s*create table `?([a-z0-9_]+)`?/i', $line, $m)) {
        $table_block = $m[1];
      }
      else if ($table_block && preg_match('/^\s*`?([a-z0-9_-]+)`?\s+([a-z]+)/', $line, $m)) {
        $col = $m[1];
        if (!in_array(strtoupper($col), array('PRIMARY','KEY','INDEX','UNIQUE','CONSTRAINT','REFERENCES','FOREIGN'))) {
                if (!in_array(strtoupper($col), $keywords)) {
          $schema[$table_block][$col] = $m[2];
        }
      }
@@ -455,8 +499,9 @@
  /**
   * Try to detect some file's mimetypes to test the correct behavior of fileinfo
   */
  function check_mime_detection()
    public function check_mime_detection()
  {
        $errors = array();
    $files = array(
      'skins/larry/images/roundcube_logo.png' => 'image/png',
      'program/resources/blank.tif' => 'image/tiff',
@@ -464,7 +509,6 @@
      'skins/larry/README' => 'text/plain',
    );

    $errors = array();
    foreach ($files as $path => $expected) {
      $mimetype = rcube_mime::file_content_type(INSTALL_PATH . $path, basename($path));
      if ($mimetype != $expected) {
@@ -478,8 +522,9 @@
  /**
   * Check the correct configuration of the 'mime_types' mapping option
   */
  function check_mime_extensions()
    public function check_mime_extensions()
  {
        $errors = array();
    $types = array(
      'application/zip'   => 'zip',
      'application/x-tar' => 'tar',
@@ -488,7 +533,6 @@
      'image/svg+xml' => 'svg',
    );

    $errors = array();
    foreach ($types as $mimetype => $expected) {
      $ext = rcube_mime::get_mime_extensions($mimetype);
      if (!in_array($expected, (array) $ext)) {
@@ -504,25 +548,25 @@
   *
   * @return string Error message or null if none exists
   */
  function get_error()
    public function get_error()
  {
      return $this->last_error['message'];
  }


  /**
   * Return a list with all imap hosts configured
   *
   * @return array Clean list with imap hosts
   */
  function get_hostlist()
    public function get_hostlist()
  {
    $default_hosts = (array)$this->getprop('default_host');
    $out = array();

    foreach ($default_hosts as $key => $name) {
      if (!empty($name))
            if (!empty($name)) {
        $out[] = rcube_utils::parse_host(is_numeric($key) ? $name : $key);
            }
    }

    return $out;
@@ -531,7 +575,7 @@
  /**
   * Create a HTML dropdown to select a previous version of Roundcube
   */
  function versions_select($attrib = array())
    public function versions_select($attrib = array())
  {
    $select = new html_select($attrib);
    $select->add(array(
@@ -546,21 +590,24 @@
        '0.9-beta', '0.9-rc', '0.9-rc2',
        // Note: Do not add newer versions here
    ));

    return $select;
  }

  /**
   * Return a list with available subfolders of the skin directory
   */
  function list_skins()
    public function list_skins()
  {
    $skins = array();
    $skindir = INSTALL_PATH . 'skins/';

    foreach (glob($skindir . '*') as $path) {
      if (is_dir($path) && is_readable($path)) {
        $skins[] = substr($path, strlen($skindir));
      }
    }

    return $skins;
  }

@@ -568,7 +615,7 @@
  * Return a list with available subfolders of the plugins directory
  * (with their associated description in composer.json)
  */
  function list_plugins()
    public function list_plugins()
  {
    $plugins = array();
    $plugin_dir = INSTALL_PATH . 'plugins/';
@@ -603,12 +650,11 @@
   * @param string Test name
   * @param string Confirm message
   */
  function pass($name, $message = '')
    public function pass($name, $message = '')
  {
    echo rcube::Q($name) . ':&nbsp; <span class="success">OK</span>';
    $this->_showhint($message);
  }


  /**
   * Display an error status and increase failure count
@@ -618,7 +664,7 @@
   * @param string URL for details
   * @param bool   Do not count this failure
   */
  function fail($name, $message = '', $url = '', $optional=false)
    public function fail($name, $message = '', $url = '', $optional=false)
  {
    if (!$optional) {
      $this->failures++;
@@ -628,7 +674,6 @@
    $this->_showhint($message, $url);
  }


  /**
   * Display an error status for optional settings/features
   *
@@ -636,12 +681,11 @@
   * @param string Error message
   * @param string URL for details
   */
  function optfail($name, $message = '', $url = '')
    public function optfail($name, $message = '', $url = '')
  {
    echo rcube::Q($name) . ':&nbsp; <span class="na">NOT OK</span>';
    $this->_showhint($message, $url);
  }


  /**
   * Display warning status
@@ -650,43 +694,44 @@
   * @param string Warning message
   * @param string URL for details
   */
  function na($name, $message = '', $url = '')
    public function na($name, $message = '', $url = '')
  {
    echo rcube::Q($name) . ':&nbsp; <span class="na">NOT AVAILABLE</span>';
    $this->_showhint($message, $url);
  }


  function _showhint($message, $url = '')
    private function _showhint($message, $url = '')
  {
    $hint = rcube::Q($message);

    if ($url)
        if ($url) {
      $hint .= ($hint ? '; ' : '') . 'See <a href="' . rcube::Q($url) . '" target="_blank">' . rcube::Q($url) . '</a>';

    if ($hint)
      echo '<span class="indent">(' . $hint . ')</span>';
  }

        if ($hint) {
            echo '<span class="indent">(' . $hint . ')</span>';
        }
    }

  static function _clean_array($arr)
    private static function _clean_array($arr)
  {
    $out = array();

    foreach (array_unique($arr) as $k => $val) {
      if (!empty($val)) {
        if (is_numeric($k))
                if (is_numeric($k)) {
          $out[] = $val;
        else
                }
                else {
          $out[$k] = $val;
                }
      }
    }

    return $out;
  }


  static function _dump_var($var, $name=null)
    private static function _dump_var($var, $name=null)
  {
    // special values
    switch ($name) {
@@ -696,15 +741,17 @@
                    136 => 'LOG_LOCAL1', 144 => 'LOG_LOCAL2', 152 => 'LOG_LOCAL3',
                    160 => 'LOG_LOCAL4', 168 => 'LOG_LOCAL5', 176 => 'LOG_LOCAL6',
                    184 => 'LOG_LOCAL7', 48 => 'LOG_LPR', 16 => 'LOG_MAIL',
                    56 => 'LOG_NEWS', 40 => 'LOG_SYSLOG', 8 => 'LOG_USER', 64 => 'LOG_UUCP');
      if ($val = $list[$var])
                56 => 'LOG_NEWS', 40 => 'LOG_SYSLOG', 8 => 'LOG_USER', 64 => 'LOG_UUCP'
            );

            if ($val = $list[$var]) {
        return $val;
            }
      break;

    case 'mail_header_delimiter':
      $var = str_replace(array("\r", "\n"), array('\r', '\n'), $var);
      return '"' . $var. '"';
      break;
/*
    // RCMAIL_VERSION is undefined here
    case 'useragent':
@@ -728,14 +775,14 @@
          }
        }

        if ($isnum)
                if ($isnum) {
          return 'array(' . join(', ', array_map(array('rcmail_install', '_dump_var'), $var)) . ')';
                }
      }
    }

    return var_export($var, true);
  }


  /**
   * Initialize the database with the according schema
@@ -743,7 +790,7 @@
   * @param object rcube_db Database connection
   * @return boolen True on success, False on error
   */
  function init_db($DB)
    public function init_db($DB)
  {
    $engine = $DB->db_provider;

@@ -766,7 +813,6 @@
    return true;
  }


  /**
   * Update database schema
   *
@@ -774,17 +820,16 @@
   *
   * @return boolen True on success, False on error
   */
  function update_db($version)
    public function update_db($version)
  {
    return rcmail_utils::db_update(INSTALL_PATH . 'SQL', 'roundcube', $version,
        array('quiet' => true));
        return rcmail_utils::db_update(INSTALL_PATH . 'SQL',
            'roundcube', $version, array('quiet' => true));
  }


  /**
   * Handler for Roundcube errors
   */
  function raise_error($p)
    public function raise_error($p)
  {
      $this->last_error = $p;
  }