diff options
author | Aleksander Machniak <alec@alec.pl> | 2013-05-07 15:12:22 +0200 |
---|---|---|
committer | Aleksander Machniak <alec@alec.pl> | 2013-05-07 15:12:22 +0200 |
commit | 3725cfb245bfae3a77baf857ea5403e8064b84b9 (patch) | |
tree | 91a918c017e5c351d6175e9c416c702b08007dac /program/steps/settings | |
parent | 2193f6a1301edcb62de6f0cf338acccdbf5ec2f1 (diff) |
Avoid uninitialized/unused variables
Diffstat (limited to 'program/steps/settings')
-rw-r--r-- | program/steps/settings/edit_prefs.inc | 2 | ||||
-rw-r--r-- | program/steps/settings/folders.inc | 3 |
2 files changed, 2 insertions, 3 deletions
diff --git a/program/steps/settings/edit_prefs.inc b/program/steps/settings/edit_prefs.inc index 971ed60b6..468e4994d 100644 --- a/program/steps/settings/edit_prefs.inc +++ b/program/steps/settings/edit_prefs.inc @@ -40,7 +40,7 @@ function rcmail_user_prefs_form($attrib) $out = $form_start; - foreach ($SECTIONS[$CURR_SECTION]['blocks'] as $idx => $block) { + foreach ($SECTIONS[$CURR_SECTION]['blocks'] as $block) { if (!empty($block['options'])) { $table = new html_table(array('cols' => 2)); diff --git a/program/steps/settings/folders.inc b/program/steps/settings/folders.inc index 0c7d9063f..34a72dd95 100644 --- a/program/steps/settings/folders.inc +++ b/program/steps/settings/folders.inc @@ -283,7 +283,6 @@ function rcube_subscription_form($attrib) $noselect = false; $classes = array($i%2 ? 'even' : 'odd'); - $folder_js = Q($folder['id']); $folder_utf8 = rcube_charset_convert($folder['id'], 'UTF7-IMAP'); $display_folder = str_repeat(' ', $folder['level']) . Q($protected ? rcmail_localize_foldername($folder['id']) : $folder['name']); @@ -394,7 +393,7 @@ function rcmail_rename_folder($oldname, $newname) $a_threaded = (array) $RCMAIL->config->get('message_threading', array()); $oldprefix = '/^' . preg_quote($oldname . $delimiter, '/') . '/'; - foreach ($a_threaded as $key => $val) { + foreach (array_keys($a_threaded) as $key) { if ($key == $oldname) { unset($a_threaded[$key]); $a_threaded[$newname] = true; |