CS fixes

Aleksander Machniak

2015-11-22 9f1f754daf4b57a0d0d3aea95d2321716d218cf5

CS fixes

 plugins/acl/acl.php

@@ -123,7 +123,7 @@
                    $group_id = is_array($record[$group_field]) ? $record[$group_field][0] : $record[$group_field];

                    if ($group) {
                        $users[] = array('name' => ($prefix ? $prefix : '') . $group_id, 'display' => $group, 'type' => 'group');
                        $users[] = array('name' => ($prefix ?: '') . $group_id, 'display' => $group, 'type' => 'group');
                        $keys[]  = $group;
                    }
                }

 plugins/attachment_reminder/attachment_reminder.js

old mode 100755
new mode 100644

 plugins/attachment_reminder/attachment_reminder.php

old mode 100755
new mode 100644

 plugins/database_attachments/database_attachments.php

@@ -155,7 +155,7 @@
     */
    protected function _key($args)
    {
        $uname = $args['path'] ? $args['path'] : $args['name'];
        $uname = $args['path'] ?: $args['name'];
        return $args['group'] . md5(time() . $uname . $_SESSION['user_id']);
    }


 plugins/debug_logger/debug_logger.php

 plugins/debug_logger/runlog/runlog.php

@@ -130,7 +130,7 @@
        $this->debug_messages[] = $msg;
        $this->run_log[] = array(
                'type'    => 'note',
                'tag'     => $tag ? $tag:"text",
                'tag'     => $tag ?: 'text',
                'value'   => htmlentities($msg),
                'time'    => microtime(true),
                'parents' => $this->parent_stack,

 plugins/enigma/lib/enigma_error.php

@@ -51,11 +51,6 @@

    function getData($name)
    {
        if ($name) {
            return $this->data[$name];
        }
        else {
            return $this->data;
        }
        return $name ? $this->data[$name] : $this->data;
    }
}

 plugins/enigma/lib/enigma_mime_message.php

@@ -264,7 +264,7 @@
    {
        $this->checkParams();

        $eol = !empty($this->build_params['eol']) ? $this->build_params['eol'] : "\r\n";
        $eol = $this->build_params['eol'] ?: "\r\n";

        // multipart message: and boundary
        if (!empty($this->build_params['boundary'])) {

 plugins/help/help.php

@@ -157,7 +157,8 @@
        // resolve language placeholder
        $rcmail = rcmail::get_instance();
        $langmap = $rcmail->config->get('help_language_map', array('*' => 'en_US'));
        $lang = !empty($langmap[$_SESSION['language']]) ? $langmap[$_SESSION['language']] : $langmap['*'];
        $lang    = $langmap[$_SESSION['language']] ?: $langmap['*'];

        return str_replace('%l', $lang, $path);
    }
}

 plugins/hide_blockquote/hide_blockquote.php

@@ -60,7 +60,7 @@

        $args['blocks']['main']['options']['hide_blockquote_limit'] = array(
            'title' => $this->gettext('quotelimit'),
            'content' => $input->show($limit ? $limit : '')
            'content' => $input->show($limit ?: '')
        );

        return $args;

 plugins/http_authentication/http_authentication.php

 plugins/managesieve/lib/Roundcube/rcube_sieve.php

@@ -111,7 +111,7 @@
     */
    public function error()
    {
        return $this->error ? $this->error : false;
        return $this->error ?: false;
    }

    /**

 plugins/managesieve/lib/Roundcube/rcube_sieve_engine.php

@@ -633,8 +633,8 @@
            foreach ($sizeitems as $item)
                $items[] = $item;

            $this->form['disabled'] = $_POST['_disabled'] ? true : false;
            $this->form['join']     = $join=='allof' ? true : false;
            $this->form['disabled'] = !empty($_POST['_disabled']);
            $this->form['join']     = $join == 'allof';
            $this->form['name']     = $name;
            $this->form['tests']    = array();
            $this->form['actions']  = array();
@@ -1685,7 +1685,7 @@
            $test = $rule['type'];
        }
        else if (in_array($rule['test'], $set)) {
            $test = ($rule['not'] ? 'not' : '') . ($rule['type'] ? $rule['type'] : 'is');
            $test = ($rule['not'] ? 'not' : '') . ($rule['type'] ?: 'is');
        }
        else {
            $test = ($rule['not'] ? 'not' : '') . $rule['test'];
@@ -2358,7 +2358,7 @@
            if (empty($filter['actions'])) {
                continue;
            }
            $fname = $filter['name'] ? $filter['name'] : "#$i";
            $fname = $filter['name'] ?: "#$i";
            $result[] = array(
                'id'    => $idx,
                'name'  => $fname,

 plugins/managesieve/lib/Roundcube/rcube_sieve_vacation.php

@@ -312,7 +312,7 @@
            }
        }

        $this->rc->output->show_message($error ? $error : 'managesieve.saveerror', 'error');
        $this->rc->output->show_message($error ?: 'managesieve.saveerror', 'error');
        $this->rc->output->send();
    }

@@ -327,7 +327,7 @@
        $seconds_extension = in_array('vacation-seconds', $this->exts);

        // build FORM tag
        $form_id = !empty($attrib['id']) ? $attrib['id'] : 'form';
        $form_id = $attrib['id'] ?: 'form';
        $out     = $this->rc->output->request_form(array(
            'id'      => $form_id,
            'name'    => $form_id,
@@ -597,7 +597,7 @@
            }
        }

        return $interval ? $interval : '';
        return $interval ?: '';
    }

    /**

 plugins/managesieve/managesieve.php

@@ -239,7 +239,7 @@
            $include_path .= ini_get('include_path');
            set_include_path($include_path);

            $class_name = 'rcube_sieve_' . ($type ? $type : 'engine');
            $class_name = 'rcube_sieve_' . ($type ?: 'engine');
            $this->engine = new $class_name($this);
        }


 plugins/markasjunk/markasjunk.php

@@ -23,10 +23,13 @@

    if ($rcmail->action == '' || $rcmail->action == 'show') {
      $skin_path = $this->local_skin_path();
      $this->include_script('markasjunk.js');
      if (is_file($this->home . "/$skin_path/markasjunk.css"))
        $this->include_stylesheet("$skin_path/markasjunk.css");

      $this->add_texts('localization', true);
            $this->include_script('markasjunk.js');

            if (is_file($this->home . "/$skin_path/markasjunk.css")) {
                $this->include_stylesheet("$skin_path/markasjunk.css");
            }

      $this->add_button(array(
        'type' => 'link',
@@ -35,7 +38,8 @@
        'class' => 'button buttonPas junk disabled',
        'classact' => 'button junk',
        'title' => 'buttontitle',
        'domain' => 'markasjunk'), 'toolbar');
                    'domain'   => 'markasjunk'
                ),'toolbar');
    }
  }

@@ -71,5 +75,4 @@
    $rcmail->output->command('display_message', $this->gettext('reportedasjunk'), 'confirmation');
    $rcmail->output->send();
  }

}

 plugins/redundant_attachments/redundant_attachments.php

@@ -90,7 +90,7 @@
     */
    private function _key($args)
    {
        $uname = $args['path'] ? $args['path'] : $args['name'];
        $uname = $args['path'] ?: $args['name'];
        return $args['group'] . md5(time() . $uname . $_SESSION['user_id']);
    }


 plugins/subscriptions_option/subscriptions_option.php

@@ -61,7 +61,7 @@
            $rcmail = rcmail::get_instance();
            $use_subscriptions = $rcmail->config->get('use_subscriptions');

            $args['prefs']['use_subscriptions'] = isset($_POST['_use_subscriptions']) ? true : false;
            $args['prefs']['use_subscriptions'] = isset($_POST['_use_subscriptions']);

            // if the use_subscriptions preference changes, flush the folder cache
            if (($use_subscriptions && !isset($_POST['_use_subscriptions'])) ||

 plugins/userinfo/userinfo.php

@@ -27,6 +27,7 @@
  {
    $rcmail = rcmail::get_instance();
    $user = $rcmail->user;
        $identity = $user->get_identity();
    
    $table = new html_table(array('cols' => 2, 'cellpadding' => 3));

@@ -45,11 +46,9 @@
    $table->add('title', rcube::Q($this->gettext('lastlogin')));
    $table->add('', rcube::Q($user->data['last_login']));
    
    $identity = $user->get_identity();
    $table->add('title', rcube::Q($this->gettext('defaultidentity')));
    $table->add('', rcube::Q($identity['name'] . ' <' . $identity['email'] . '>'));
    
    return html::tag('h4', null, rcube::Q('Infos for ' . $user->get_username())) . $table->show();
  }

}

 plugins/zipdownload/zipdownload.php

@@ -156,7 +156,7 @@

        $zip->close();

        $filename = ($message->subject ? $message->subject : 'roundcube') . '.zip';
        $filename = ($message->subject ?: 'roundcube') . '.zip';
        $this->_deliver_zipfile($tmpfname, $filename);

        // delete temporary files from disk
@@ -247,7 +247,7 @@
                    $subject = $this->_convert_filename($subject);
                    $subject = substr($subject, 0, 16);

                    $disp_name = ($subject ? $subject : 'message_rfc822') . ".eml";
                    $disp_name = ($subject ?: 'message_rfc822') . ".eml";
                    $disp_name = $path . $uid . "_" . $disp_name;

                    $tmpfn = tempnam($temp_dir, 'zipmessage');