From 540e13b8d50a52e9cb479e36bc6d1e16275a2cd5 Mon Sep 17 00:00:00 2001 From: Aleksander Machniak Date: Tue, 13 Nov 2012 19:44:52 +0100 Subject: Fix warning when 'autovalues' property isn't set --- program/include/rcube_ldap.php | 3 ++- program/steps/mail/func.inc | 1 + 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/program/include/rcube_ldap.php b/program/include/rcube_ldap.php index 90ce73ae2..7cef25556 100644 --- a/program/include/rcube_ldap.php +++ b/program/include/rcube_ldap.php @@ -160,7 +160,8 @@ class rcube_ldap extends rcube_addressbook } // make sure LDAP_rdn field is required - if (!empty($this->prop['LDAP_rdn']) && !in_array($this->prop['LDAP_rdn'], $this->prop['required_fields']) && !in_array($this->prop['LDAP_rdn'], array_keys($this->prop['autovalues']))) { + if (!empty($this->prop['LDAP_rdn']) && !in_array($this->prop['LDAP_rdn'], $this->prop['required_fields']) + && !in_array($this->prop['LDAP_rdn'], array_keys((array)$this->prop['autovalues']))) { $this->prop['required_fields'][] = $this->prop['LDAP_rdn']; } diff --git a/program/steps/mail/func.inc b/program/steps/mail/func.inc index 5e24a4311..01d95e059 100644 --- a/program/steps/mail/func.inc +++ b/program/steps/mail/func.inc @@ -1110,6 +1110,7 @@ function rcmail_message_body($attrib) if (!empty($MESSAGE->parts)) { foreach ($MESSAGE->parts as $i => $part) { + console($part); if ($part->type == 'headers') $out .= rcmail_message_headers(sizeof($header_attrib) ? $header_attrib : NULL, $part->headers); else if ($part->type == 'content') { -- cgit v1.2.3 From 275728ab0511c5e89c372aaa5508dc0040c146b2 Mon Sep 17 00:00:00 2001 From: Aleksander Machniak Date: Tue, 13 Nov 2012 19:47:16 +0100 Subject: Remove debug code from last commit --- program/steps/mail/func.inc | 1 - 1 file changed, 1 deletion(-) diff --git a/program/steps/mail/func.inc b/program/steps/mail/func.inc index 01d95e059..5e24a4311 100644 --- a/program/steps/mail/func.inc +++ b/program/steps/mail/func.inc @@ -1110,7 +1110,6 @@ function rcmail_message_body($attrib) if (!empty($MESSAGE->parts)) { foreach ($MESSAGE->parts as $i => $part) { - console($part); if ($part->type == 'headers') $out .= rcmail_message_headers(sizeof($header_attrib) ? $header_attrib : NULL, $part->headers); else if ($part->type == 'content') { -- cgit v1.2.3 From e30500643fbe0c9f9237570b73e59992f776cf0c Mon Sep 17 00:00:00 2001 From: Thomas Bruederli Date: Wed, 14 Nov 2012 10:28:34 +0100 Subject: Enable default behavior of the browser when shift/ctrl-click task links (e.g. open in new window/tab) --- program/include/rcube_output_html.php | 2 +- program/js/app.js | 7 ++++++- 2 files changed, 7 insertions(+), 2 deletions(-) diff --git a/program/include/rcube_output_html.php b/program/include/rcube_output_html.php index db947bd2b..ac07d58e9 100644 --- a/program/include/rcube_output_html.php +++ b/program/include/rcube_output_html.php @@ -1087,7 +1087,7 @@ class rcube_output_html extends rcube_output // make valid href to specific buttons if (in_array($attrib['command'], rcmail::$main_tasks)) { $attrib['href'] = $this->app->url(array('task' => $attrib['command'])); - $attrib['onclick'] = sprintf("%s.command('switch-task','%s',null,event); return false", rcmail::JS_OBJECT_NAME, $attrib['command']); + $attrib['onclick'] = sprintf("return %s.command('switch-task','%s',this,event)", rcmail::JS_OBJECT_NAME, $attrib['command']); } else if ($attrib['task'] && in_array($attrib['task'], rcmail::$main_tasks)) { $attrib['href'] = $this->app->url(array('action' => $attrib['command'], 'task' => $attrib['task'])); diff --git a/program/js/app.js b/program/js/app.js index 0f5a60c6f..2a170b258 100644 --- a/program/js/app.js +++ b/program/js/app.js @@ -507,6 +507,11 @@ function rcube_webmail() if (this.busy) return false; + // let the browser handle this click (shift/ctrl usually opens the link in a new window/tab) + if ((obj && (obj.href.indexOf(location.href) < 0)) && rcube_event.get_modifier(event)) { + return true; + } + // command not supported or allowed if (!this.commands[command]) { // pass command to parent window @@ -562,7 +567,7 @@ function rcube_webmail() break; case 'about': - location.href = '?_task=settings&_action=about'; + this.redirect('?_task=settings&_action=about', false); break; case 'permaurl': -- cgit v1.2.3