summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--program/steps/settings/edit_prefs.inc19
-rw-r--r--program/steps/settings/func.inc70
2 files changed, 45 insertions, 44 deletions
diff --git a/program/steps/settings/edit_prefs.inc b/program/steps/settings/edit_prefs.inc
index b5fd0b95b..5a6e7e8fa 100644
--- a/program/steps/settings/edit_prefs.inc
+++ b/program/steps/settings/edit_prefs.inc
@@ -34,27 +34,30 @@ function rcmail_user_prefs_form($attrib)
$RCMAIL->output->add_label('nopagesizewarning');
unset($attrib['form']);
-
+
list($form_start, $form_end) = get_form_tags($attrib, 'save-prefs', null,
array('name' => '_section', 'value' => $CURR_SECTION));
$out = $form_start;
foreach ($SECTIONS[$CURR_SECTION]['blocks'] as $idx => $block) {
- if ($block['options']) {
+ if (!empty($block['options'])) {
$table = new html_table(array('cols' => 2));
foreach ($block['options'] as $option) {
if ($option['advanced'])
- $table->set_row_attribs('advanced');
-
+ $table->set_row_attribs('advanced');
+
$table->add('title', $option['title']);
- $table->add(null, $option['content']);
- }
-
- $out .= html::tag('fieldset', null, html::tag('legend', null, $block['name']) . $table->show($attrib));
+ $table->add(null, $option['content']);
}
+
+ $out .= html::tag('fieldset', null, html::tag('legend', null, $block['name']) . $table->show($attrib));
+ }
+ else if (!empty($block['content'])) {
+ $out .= html::tag('fieldset', null, html::tag('legend', null, $block['name']) . $block['content']);
}
+ }
return $out . $form_end;
}
diff --git a/program/steps/settings/func.inc b/program/steps/settings/func.inc
index b07c042b3..c8e46a32e 100644
--- a/program/steps/settings/func.inc
+++ b/program/steps/settings/func.inc
@@ -35,7 +35,7 @@ function rcmail_preferences_frame($attrib)
$OUTPUT->set_env('contentframe', $attrib['name']);
$OUTPUT->set_env('blankpage', $attrib['src'] ? $OUTPUT->abs_url($attrib['src']) : 'program/blank.gif');
-
+
return html::iframe($attrib);
}
@@ -43,13 +43,13 @@ function rcmail_preferences_frame($attrib)
function rcmail_sections_list($attrib)
{
global $RCMAIL;
-
+
// add id to message list table if not specified
if (!strlen($attrib['id']))
$attrib['id'] = 'rcmsectionslist';
list($list, $cols) = rcmail_user_prefs();
-
+
// create XHTML table
$out = rcube_table_output($attrib, $list, $cols, 'id');
@@ -96,7 +96,7 @@ function get_form_tags($attrib, $action, $id = null, $hidden = null)
global $EDIT_FORM, $RCMAIL;
$form_start = $form_end = '';
-
+
if (empty($EDIT_FORM)) {
$request_key = $action . (isset($id) ? '.'.$id : '');
$form_start = $RCMAIL->output->request_form(array(
@@ -107,12 +107,12 @@ function get_form_tags($attrib, $action, $id = null, $hidden = null)
'request' => $request_key,
'noclose' => true
) + $attrib);
-
+
if (is_array($hidden)) {
$hiddenfields = new html_hiddenfield($hidden);
$form_start .= $hiddenfields->show();
}
-
+
$form_end = !strlen($attrib['form']) ? '</form>' : '';
$EDIT_FORM = !empty($attrib['form']) ? $attrib['form'] : 'form';
@@ -139,26 +139,26 @@ function rcmail_user_prefs($current=null)
array('list' => $sections, 'cols' => array('section')));
$sections = $plugin['list'];
-
+
$config = $RCMAIL->config->all();
$no_override = array_flip($RCMAIL->config->get('dont_override', array()));
-
+
foreach ($sections as $idx => $sect) {
-
+
if ($current && $sect['id'] != $current)
continue;
-
+
$blocks = array();
-
+
switch ($sect['id']) {
// general
case 'general':
-
+
$blocks = array(
'main' => array('name' => Q(rcube_label('mainoptions'))),
'list' => array('name' => Q(rcube_label('listoptions'))),
);
-
+
// language selection
if (!isset($no_override['language'])) {
$a_lang = $RCMAIL->list_languages();
@@ -275,12 +275,12 @@ function rcmail_user_prefs($current=null)
);
}
}
-
- break;
-
+
+ break;
+
// Mailbox view (mail screen)
case 'mailbox':
-
+
$blocks = array(
'main' => array('name' => Q(rcube_label('mainoptions'))),
'new_message' => array('name' => Q(rcube_label('newmessage'))),
@@ -302,7 +302,7 @@ function rcmail_user_prefs($current=null)
if (!isset($no_override['preview_pane_mark_read'])) {
// apply default if config option is not set at all
$config['preview_pane_mark_read'] = $RCMAIL->config->get('preview_pane_mark_read', 0);
-
+
$field_id = 'rcmfd_preview_pane_mark_read';
$select_delay = new html_select(array('name' => '_preview_pane_mark_read', 'id' => $field_id,
'disabled' => $config['preview_pane']?0:1));
@@ -342,7 +342,7 @@ function rcmail_user_prefs($current=null)
$select_autoexpand_threads->add(rcube_label('never'), 0);
$select_autoexpand_threads->add(rcube_label('do_expand'), 1);
$select_autoexpand_threads->add(rcube_label('expand_only_unread'), 2);
-
+
$blocks['main']['options']['autoexpand_threads'] = array(
'title' => html::label($field_id, Q(rcube_label('autoexpand_threads'))),
'content' => $select_autoexpand_threads->show($config['autoexpand_threads']),
@@ -386,10 +386,10 @@ function rcmail_user_prefs($current=null)
}
break;
-
+
// Message viewing
case 'mailview':
-
+
$blocks = array(
'main' => array('name' => Q(rcube_label('mainoptions'))),
);
@@ -452,10 +452,10 @@ function rcmail_user_prefs($current=null)
}
break;
-
+
// Mail composition
case 'compose':
-
+
$blocks = array(
'main' => array('name' => Q(rcube_label('mainoptions'))),
'sig' => array('name' => Q(rcube_label('signatureoptions'))),
@@ -558,10 +558,10 @@ function rcmail_user_prefs($current=null)
}
break;
-
+
// Special IMAP folders
case 'folders':
-
+
$blocks = array(
'main' => array('name' => Q(rcube_label('mainoptions'))),
);
@@ -605,10 +605,10 @@ function rcmail_user_prefs($current=null)
}
break;
-
+
// Server settings
case 'server':
-
+
$blocks = array(
'main' => array('name' => Q(rcube_label('mainoptions'))),
'maintenance' => array('name' => Q(rcube_label('maintenance'))),
@@ -676,20 +676,18 @@ function rcmail_user_prefs($current=null)
'content' => $input_expunge->show($config['logout_expunge']?1:0),
);
}
-
+
break;
}
$data = $RCMAIL->plugins->exec_hook('preferences_list', array('section' => $sect['id'], 'blocks' => $blocks));
$found = false;
-
+
// create output
foreach ($data['blocks'] as $block) {
- if ($block['options']) {
- foreach ($block['options'] as $option) {
- $found = true;
- break 2;
- }
+ if (!empty($block['content']) || !empty($block['options'])) {
+ $found = true;
+ break;
}
}
@@ -709,10 +707,10 @@ function rcmail_get_skins()
$skins = array();
$dir = opendir($path);
-
+
if (!$dir)
return false;
-
+
while (($file = readdir($dir)) !== false)
{
$filename = $path.'/'.$file;