- Dev - Roundcube mailing lists

append tel url in addressbook
by nyx-network 16 Dec '13

16 Dec '13

Hello, i have modified the show.inc file for append the support for tel://url in addressbook. If you want add the modification to your code, it's ok for me. Cheers, --- old/program/steps/addressbook/show.inc +++ new/program/steps/addressbook/show.inc @@ -137,7 +137,7 @@ 'name' => rcube_label('properties'), 'content' => array( 'email' => array('size' => $i_size, 'render_func' => 'rcmail_render_email_value'), - 'phone' => array('size' => $i_size), + 'phone' => array('size' => $i_size, 'render_func' => 'rcmail_render_tel_value'), 'address' => array(), 'website' => array('size' => $i_size, 'render_func' => 'rcmail_render_url_value'), 'im' => array('size' => $i_size), @@ -187,6 +187,14 @@ ), Q($email)); } +function rcmail_render_tel_value($tel, $col) +{ + return html::a(array( + 'href' => 'tel:' . $tel, + 'class' => 'tel', + ), Q($tel)); +} + function rcmail_render_url_value($url, $col) { -- Mickaël Winschel Ingénieur systèmes et réseaux Site : http://www.nyx-network.com

3 4

Login as another user -> Calendars not shown
by Henning 09 Dec '13

09 Dec '13

Hi, logged in for another user, you get errors viewing the calendars. No calendars are shown. IMAP Error: Login failed for user(a)domain.com from 80.80.80.80. AUTHENTICATE PLAIN: authentication failure in /usr/share/roundcubemail/program/lib/Roundcube/rcube_imap.php on line 184 (GET /roundcubemail/?_task=calendar) regards, Henning

1 0

code bug ?
by Sébastien BLAISOT 08 Dec '13

08 Dec '13

Hi everybody, since RC 0.9b, there is a bug in my automatic_addressbook plugin when editing a contact with th move_to config option set to 1 This means that when editing a contact in the collected_abook, it is saved in the default abook and removed from its original collected_abook (calling $CONTACTS->insert and setting abort to true and result to the new cid) the log is showing : [07-Dec-2013 14:18:16 Europe/Paris] PHP Fatal error: Cannot use object of type rcube_result_set as array in /var/www/vhosts/ssl_common/webmail/test/rc09b/program/steps/addressbook/save.inc on line 134 this seams to come from the following lines of code in program/steps/addressbook/save : $record = $CONTACTS->get_record($newcid ? $newcid : $cid, true); $record['email'] = reset($CONTACTS->get_col_values('email', $record, true)); it seems to me that get_record return in general a record, but can return in some circonstances a rcube_result_set object and the next line of code can't handle this (array assignation on a rcube_result_set object). Can somebody confirme this ? What are the condition for get_record to return an rcube_result_set with only 1 object in it ? I don't already understand very well what happens but I can make additionnal tests to help debug this. If you can confirme this seems to be a bug, I'll file a bug report. regards, sebastien

2 4

Re: [RCD] [Svn] [roundcube/roundcubemail] ac3cdd: Skip charset (or use US_ASCII) intead of UTF-8 if ...
by Michael Heydekamp 02 Dec '13

02 Dec '13

Am 25.11.2013 15:54, schrieb GitHub: > Log Message: > ----------- > Skip charset (or use US_ASCII) intead of UTF-8 if body contains only ASCII characters Yoohoo! :) This is the first test. Any hope that this logic will be extended to ISO-8859-1/15 (and probably Windows-1252) in the future? Thanks and cheers, -- Michael Heydekamp Co-Admin freexp.de Duesseldorf/Germany

2 2

Roundcube version 1.0-beta available for download
by Thomas Bruederli 01 Dec '13

01 Dec '13

Hello folks We’re proud to announce that the long awaited version 1.0 of Roundcube webmail is now available as beta release for download and testing. With this milestone we introduce some important changes to the configuration as well as some nice new features: Cleaned up the configuration into a single file Importing email messages and contact group assignments Advanced LDAP address book functionality A toggle to switch between HTML and plaintext view Save drafts in local storage for recovery Canned responses to save and recall boilerplate texts Improved keyboard navigation in messages list Attachment reminder plugin And of course plenty of small improvements and bug fixes. See the complete Changelog at http://trac.roundcube.net/wiki/Changelog and download the new packages from http://roundcube.net/download. WARNING: the old configuration files named main.inc.php and db.inc.php are now deprecated and should be replaced with one single config.inc.php file. Run the ./bin/update.sh script to get this conversion done or manually merge the files. NOTE: the new config.inc.php should only contain options that differ from the ones listed in defaults.inc.php. Please note that this is a beta release and we recommend to test it on a separate environment. Have fun! Thomas

2 1

Larry: Improved #message element behavior/location
by Thomas Bruederli 24 Nov '13

24 Nov '13

Hello devs and list lurkers One of the remaining big tasks we have scheduled for Roundcube 1.0 is the re-location of the status messages: http://trac.roundcube.net/ticket/1488974 Many people complained that the location at the bottom of the screen is somewhat out of the user's sight and the messages itself are not visible enough. Please find attached a suggestion to refactor the status messages into stacked popups but still located at the bottom of the window. What do thou think? Kind regards, Thomas

7 30

Problems with sending messages IE8/1.0-git (Javascript error)
by Michael Heydekamp 24 Nov '13

24 Nov '13

Two problems I just realized in IE8 when I want to send a message with IE8/1.0-git in a separate window: 1) A few seconds after the window has been opened the following error message appears in the lower left corner of the status bar: ----------------------------------------------------------------------- > Meldung: Das Objekt unterstützt diese Eigenschaft oder Methode nicht. > Zeile: 3712 > Zeichen: 9 > Code: 0 > URI: http://***.de/program/js/app.js?s=1384978220 ----------------------------------------------------------------------- The result aifter clicking the send button is that the message IS being sent, but the window stays on the screen and Roundcube "thinks" that it has NOT been sent. 2) Clicking on the Cancel button and on the OK button in the consecutive message box does NOT cancel the message - the window stays on the screen in this scenario as well. Not sure yet if it makes a difference when sending a message in the same window... Cheers, -- Michael Heydekamp Co-Admin freexp.de Düsseldorf/Germany

2 4

Re: [RCD] [Svn] [roundcube/roundcubemail] f8101f: Accept text/html parts inside multipart/mixed parts as HTML contents
by Michael Heydekamp 17 Nov '13

17 Nov '13

Thanks for this fix - works for me! Nonetheless there is still another issue with such kind of messages: In the message list, attachments are not indicated (the 'paper clip' icon is missing). Cheers, -- Michael Heydekamp Co-Admin freexp.de Düsseldorf/Germany Am 15.11.2013 18:58, schrieb GitHub: > Branch: refs/heads/master > Home: https://github.com/roundcube/roundcubemail > Commit: f8101fff1ec4a52bde553e77a120fd8618c025aa > > https://github.com/roundcube/roundcubemail/commit/f8101fff1ec4a52bde553e77a… > Author: Thomas Bruederli <thomas(a)roundcube.net> > Date: 2013-11-15 (Fri, 15 Nov 2013) > > Changed paths: > M program/lib/Roundcube/rcube_message.php > > Log Message: > ----------- > Accept text/html parts inside multipart/mixed parts as HTML contents > > > > > _______________________________________________ > Roundcube SVN commits mailing list > svn(a)lists.roundcube.net > http://lists.roundcube.net/mailman/listinfo/svn

2 2

Hooks 'message_compose' and 'message_compose_body'
by Rosali 14 Nov '13

14 Nov '13

In current 1.0-git (master) and in 0.9.5 I see that $isHTML is set to false if we pre-define body content by hooks 'message_compose' and/or 'message_compose_body' regardless whether or not user preference is 'HTML': ./program/steps/mail/compose.inc ::: lines 643 ff. else if ($COMPOSE['param']['body']) { $body = $COMPOSE['param']['body']; $isHtml = false; } Is there a special reason for this behavior? I guess it is a bug. At least it prevents me to pre-define HTML compose bodies. If there is a special reason for this fallback to text bodies then please note that the HTML Editor select box is set accordingly; it is set accordingly to user's preference. I have already contacted Alec and he replied: "$COMPOSE['param']s in general are for text only, because they are URL parameters. If you want to inject any body use message_compose_body hook. Feel free to propose fix/better solution via pull request." I think Alec is not right. $COMPOSE['param'] is also set by the hook 'message_compose'. See ./program/steps/mail/compose.inc ::: lines 450 ff. // pipe compose parameters thru plugins $plugin = $RCMAIL->plugins->exec_hook('message_compose', $COMPOSE); $COMPOSE['param'] = array_merge($COMPOSE['param'], $plugin['param']); So, the simple solution to allow plugin developers to pre-define HTML bodies is: ./program/steps/mail/compose.inc ::: lines 643 ff. else if ($COMPOSE['param']['body']) { $body = $COMPOSE['param']['body']; $isHtml = $COMPOSE['param']['html'] ? true : false; }

2 1

saving drafts to localstorage
by Charles McNulty 10 Nov '13

10 Nov '13

Is this feature request to store drafts in localstorage defined in any detail anywhere? Is it partially implemented anywhere, or part of a plugin? I looked and couldn't find anything other than a couple of messages in the dev archives. My main question is whether we're going to save the drafts as unencrypted data? Is this something that people are comfortable with? Especially given that we would not store this as session-only data (because we presumably would want to restore data from an interrupted session), it seems like storing it as unencrypted might represent problems (at least for some more sensitive e-mails). The obvious solution would be for the server to store a private key and to send a public key to javascript for encrypting localstorage. That raises issues of where to store the private key, and how/when to generate it. It also starts to overlap with Alec's GPG plugin. Would we want to use the same GPG key that we use for sending encrypted messages (essentially sending them to ourselves), or would this be a completely separate key just for localstorage that we could potentially generate at setup and just store along with the userdata in the database. If I can get some answers to these questions, I'd love to start developing this. -Charles

3 3

Re: [RCD] [Svn] [roundcube/roundcubemail] b972b4: Add an option to disable smart Reply-List behaviour - reply_all_mode (#1488734)
by Michael Heydekamp 08 Nov '13

08 Nov '13

I think these options under "Default action of [Reply all] button" in context with the commit message "Add an option to disable smart Reply-List behaviour"... - reply to all - reply to mailing list only (if found) ...require either some explanation or different phrasing. The latter option sounds to me as if any reply-all action would be disabled when replying to a non-list message. But I dare to believe that this isn't true, right? (Untested) Furhermore, the Reply-All button has two options anway. To which of these two options does this change apply? Or is it just a toggle which of these two options of the button will be taken by default when just clicking on the button without explicitely selecting one of those options? If so, then it's even more confusing. Cheers, -- Michael Heydekamp Co-Admin freexp.de Düsseldorf/Germany Am 30.10.2013 12:04, schrieb GitHub: > Branch: refs/heads/master > Home: https://github.com/roundcube/roundcubemail > Commit: b972b4c4c079898250912588d1eb1d8907dfe868 > > https://github.com/roundcube/roundcubemail/commit/b972b4c4c079898250912588d… > Author: Aleksander Machniak <alec(a)alec.pl> > Date: 2013-10-30 (Wed, 30 Oct 2013) > > Changed paths: > M CHANGELOG > M config/defaults.inc.php > M program/js/app.js > M program/localization/en_US/labels.inc > M program/steps/mail/func.inc > M program/steps/settings/func.inc > M program/steps/settings/save_prefs.inc > M skins/classic/functions.js > M skins/larry/ui.js > > Log Message: > ----------- > Add an option to disable smart Reply-List behaviour - reply_all_mode (#1488734) > > > > > _______________________________________________ > Roundcube SVN commits mailing list > svn(a)lists.roundcube.net > http://lists.roundcube.net/mailman/listinfo/svn

2 1

Re: [RCD] [Svn] [roundcube/roundcubemail] 0f4c9b: check if the new password matched the requirements
by Michael Heydekamp 08 Nov '13

08 Nov '13

Just wanted to confirm that the commit below appears to work (see screenshot attached). :) Just one thing: Is there any way to keep those AJAX messages (or however they are called) on the screen until they have been "clicked away" (or confirmed with the Enter key) by the user? Especially in this case that would make most sense as it takes some time to read (and understand) the message, but it does appear on the screen for just 7 seconds or so. Cheers, -- Michael Heydekamp Co-Admin freexp.de Düsseldorf/Germany Am 04.11.2013 17:53, schrieb GitHub: > Branch: refs/heads/master > Home: https://github.com/roundcube/roundcubemail > Commit: 0f4c9b850f665f0fdafb00809b4b9f100c59f331 > > https://github.com/roundcube/roundcubemail/commit/0f4c9b850f665f0fdafb00809… > Author: Till Krüss <me(a)tillkruess.com> > Date: 2013-10-15 (Tue, 15 Oct 2013) > > Changed paths: > M plugins/password/drivers/domainfactory.php > > Log Message: > ----------- > check if the new password matched the requirements; code cleanup > > > Commit: 910a3581be47da1d19eb15a68ea6b51c2c31ef67 > > https://github.com/roundcube/roundcubemail/commit/910a3581be47da1d19eb15a68… > Author: Till Krüss <me(a)tillkruess.com> > Date: 2013-10-15 (Tue, 15 Oct 2013) > > Changed paths: > M plugins/password/drivers/domainfactory.php > > Log Message: > ----------- > check if password was changed successfully, before looking for error messages > > > Commit: d4f27d90d57b62a833e643fa9e823d4a8aaa1dcf > > https://github.com/roundcube/roundcubemail/commit/d4f27d90d57b62a833e643fa9… > Author: Aleksander Machniak <alec(a)alec.pl> > Date: 2013-11-04 (Mon, 04 Nov 2013) > > Changed paths: > M plugins/password/drivers/domainfactory.php > > Log Message: > ----------- > Merge pull request #136 from tillkruess/extend-df-password-driver > > Extend domainfactory password driver > > > Compare: > https://github.com/roundcube/roundcubemail/compare/a8b004e8d8f0...d4f27d90d… > > _______________________________________________ > Roundcube SVN commits mailing list > svn(a)lists.roundcube.net > http://lists.roundcube.net/mailman/listinfo/svn

2 3

Re: [RCD] [Svn] [roundcube/roundcubemail] e769a7: need to reset shift_start on control select
by Michael Heydekamp 07 Nov '13

07 Nov '13

Works on IE10. :) -- Michael Heydekamp Co-Admin freexp.de Düsseldorf/Germany Am 05.11.2013 08:58, schrieb GitHub: > Branch: refs/heads/master > Home: https://github.com/roundcube/roundcubemail > Commit: e769a7b588dde028a41cde482995601b63ad87b2 > > https://github.com/roundcube/roundcubemail/commit/e769a7b588dde028a41cde482… > Author: cmcnulty <cmcnulty(a)kznf.com> > Date: 2013-10-30 (Wed, 30 Oct 2013) > > Changed paths: > M program/js/list.js > > Log Message: > ----------- > need to reset shift_start on control select > > reproduce steps: shift-select some messages, hold down control and > select another message. hold down shift+control and select more > messages - desired behavior, two separate ranges of selected messages. > (consistent with outlook/thunderbird/windows explorer) > > > Commit: 5e3e4fe0c6f8e8ab1ff29ec685f127281700c953 > > https://github.com/roundcube/roundcubemail/commit/5e3e4fe0c6f8e8ab1ff29ec68… > Author: Aleksander Machniak <alec(a)alec.pl> > Date: 2013-11-04 (Mon, 04 Nov 2013) > > Changed paths: > M program/js/list.js > > Log Message: > ----------- > Merge pull request #152 from cmcnulty/control-shift > > Reset shift_start on control select > > > Compare: > https://github.com/roundcube/roundcubemail/compare/04fbc555b939...5e3e4fe0c… > > _______________________________________________ > Roundcube SVN commits mailing list > svn(a)lists.roundcube.net > http://lists.roundcube.net/mailman/listinfo/svn

1 0

Re: [RCD] [Svn] [roundcube/roundcubemail] a8b004: Improve identity selection based on From: header (#1489378)
by Michael Heydekamp 07 Nov '13

07 Nov '13

This one appears to work as well now. Thanks a lot! As we finally seem to be done with all those different identity selection scenarios upon replying to a message: Is there any hope that we will be able to assign a default identity to an entry in the address book? Example: When I compose a new message to any Roundcube list, I still have to change the identity manually (and am sometimes missing that, so the message won't be delivered). Cheers, -- Michael Heydekamp Co-Admin freexp.de Düsseldorf/Germany Am 04.11.2013 11:20, schrieb GitHub: > Branch: refs/heads/master > Home: https://github.com/roundcube/roundcubemail > Commit: a8b004e8d8f040d868e4b19da9527c177be9959d > > https://github.com/roundcube/roundcubemail/commit/a8b004e8d8f040d868e4b19da… > Author: Aleksander Machniak <alec(a)alec.pl> > Date: 2013-11-04 (Mon, 04 Nov 2013) > > Changed paths: > M CHANGELOG > M program/steps/mail/func.inc > M tests/MailFunc.php > > Log Message: > ----------- > Improve identity selection based on From: header (#1489378) > > > > > _______________________________________________ > Roundcube SVN commits mailing list > svn(a)lists.roundcube.net > http://lists.roundcube.net/mailman/listinfo/svn

1 0

Keyboard navigation "fixes"
by Michael Heydekamp 04 Nov '13

04 Nov '13

I'm not sure what the intention of those several Ctrl+click fixes is and what the space bar should now do exactly, but Ctrl+click in the message list doesn't work in latest 1.0-git - at least not with IE10. It does still work in 0.9.5. Why don't you just leave it as it was? ;) Cheers, -- Michael Heydekamp Co-Admin freexp.de Düsseldorf/Germany Am 30.10.2013 14:45, schrieb GitHub: > Branch: refs/heads/master > Home: https://github.com/roundcube/roundcubemail > Commit: bc35e865ae700b21ce6085aa33ba8354a2b09ac9 > > https://github.com/roundcube/roundcubemail/commit/bc35e865ae700b21ce6085aa3… > Author: Aleksander Machniak <alec(a)alec.pl> > Date: 2013-10-30 (Wed, 30 Oct 2013) > > Changed paths: > M program/js/list.js > > Log Message: > ----------- > Fix Ctrl+click selection after recent refactorings of keyboard nawigation on list widget > > > > > _______________________________________________ > Roundcube SVN commits mailing list > svn(a)lists.roundcube.net > http://lists.roundcube.net/mailman/listinfo/svn Am 30.10.2013 12:53, schrieb GitHub: > Branch: refs/heads/master > Home: https://github.com/roundcube/roundcubemail > Commit: 9806c72764e4c3f82f3b249b5456137662211e2d > > https://github.com/roundcube/roundcubemail/commit/9806c72764e4c3f82f3b249b5… > Author: cmcnulty <cmcnulty(a)kznf.com> > Date: 2013-10-29 (Tue, 29 Oct 2013) > > Changed paths: > M program/js/list.js > > Log Message: > ----------- > replace space bar with arrow key thread handling and switch spacebar to select messages > > > Commit: a8876fba93a1e52dd0daab6fa3b7b74fedbdc252 > > https://github.com/roundcube/roundcubemail/commit/a8876fba93a1e52dd0daab6fa… > Author: cmcnulty <cmcnulty(a)kznf.com> > Date: 2013-10-29 (Tue, 29 Oct 2013) > > Changed paths: > M program/js/list.js > > Log Message: > ----------- > Revert "simplify home/end key behavior" > > This reverts commit 3bef61232bc2e0e470a66dda6d88248100fdc36f. > > > Commit: 0fb8940d066fc6ecac5a8c97385380c152b69acd > > https://github.com/roundcube/roundcubemail/commit/0fb8940d066fc6ecac5a8c973… > Author: Aleksander Machniak <alec(a)alec.pl> > Date: 2013-10-30 (Wed, 30 Oct 2013) > > Changed paths: > M program/js/list.js > > Log Message: > ----------- > Merge pull request #151 from cmcnulty/new-key-navigation > > Spacebar for message selection, left/right arrow for thread collapse/expand > > > Compare: > https://github.com/roundcube/roundcubemail/compare/c11a3f81acdc...0fb8940d0… > > _______________________________________________ > Roundcube SVN commits mailing list > svn(a)lists.roundcube.net > http://lists.roundcube.net/mailman/listinfo/svn

3 30

Purpose of change_identity JavaScript event trigger? (was: [Svn] [roundcube/roundcubemail] af61b9: Adds change_identity JavaScript event trigger)
by Michael Heydekamp 04 Nov '13

04 Nov '13

Hi there, may I ask what the intention of the commit quoted below exactly is...? Here it changed the behaviour of select_identity apparently in so far as the address in the From: header is now being selected as the identity, even if it does not match the full mailbox spec (name+address). This is in contradiction to what Alec recently wrote: ---------------------------------------------------------------------------- Am 14.10.2013 11:03, schrieb A.L.E.C: > On 10/14/2013 10:47 AM, Michael Heydekamp wrote: > >> Thanks. But are you sure about "From"? During my tests I didn't manage to >> select the identity in the From: header at all. The default identity had >> been selected instead (unless From: and Return-path: were identical). > > Well, From header check matches full mailbox spec., e.g. "Roundcube > Users <dev(a)lists.roundcube.net>". > > This is because we want to handle correctly situation when user has many > identities with same address but different name. It's not perfect and > I'm not saying we don't need an improvement here. ---------------------------------------------------------------------------- And an hour later, he wrote: ---------------------------------------------------------------------------- Am 14.10.2013 12:03, schrieb A.L.E.C: > For To/Cc the matching is different. We check addresses, but give bigger > prio to identity where address+name matches. So, here we're safe. > I suppose we could do the same for From with some more coding. Feel free > to create a ticket. ---------------------------------------------------------------------------- So I created a ticket: http://trac.roundcube.net/ticket/1489378 Is the commit below now the reaction to this ticket? Most likely not, as the ticket hasn't been closed yet. But the commit serves at least a part of the purpose of this ticket (= use From: header as identity even if it doesn't match full mailbox spec). So I'm not sure what's this all about, and if the result that I'm seeing is indeed intentional. Or am I wrong and the behaviour described above is related to a totally different commit...? Cheers, Michael (confused) -- Michael Heydekamp Co-Admin freexp.de Düsseldorf/Germany Am 18.10.2013 21:40, schrieb GitHub: > Branch: refs/heads/master > Home: https://github.com/roundcube/roundcubemail > Commit: af61b9705e40efa938bd38d95173a41cf210e6dc > > https://github.com/roundcube/roundcubemail/commit/af61b9705e40efa938bd38d95… > Author: Niklas Femerstrand <nik(a)qnrq.se> > Date: 2013-10-18 (Fri, 18 Oct 2013) > > Changed paths: > M program/js/app.js > > Log Message: > ----------- > Adds change_identity JavaScript event trigger > > > Commit: 771a82df9031485b3f9e7decf6b263c6847507b1 > > https://github.com/roundcube/roundcubemail/commit/771a82df9031485b3f9e7decf… > Author: Thomas B. <thomas(a)roundcube.net> > Date: 2013-10-18 (Fri, 18 Oct 2013) > > Changed paths: > M program/js/app.js > > Log Message: > ----------- > Merge pull request #139 from qnrq/master > > Adds change_identity JavaScript event trigger > > > Compare: > https://github.com/roundcube/roundcubemail/compare/1498ac81c946...771a82df9… > > _______________________________________________ > Roundcube SVN commits mailing list > svn(a)lists.roundcube.net > http://lists.roundcube.net/mailman/listinfo/svn

3 3

Plugins, $config and RC 1.0
by Michael Heydekamp 04 Nov '13

04 Nov '13

Hi there, we're unsure and a bit confused about how to handle the plugin configs in our 1.0-git test environment: 1) Almost all plugins do still use the legacy "$rcmail_config" array in their .dist file, rather than "$config". Do we need to manually run a search/replace across all config.inc.php(.dist)? 2) The plugin 'password' for instance does already use the "$config" array in it's config.php.dist. But this is not being reflected in config.inc.php (shouldn't there be some sort of automatism?). 3) Furthermore, the same plugin 'password' does contain a number of new config settings in .dist - but also those did not find their way into the .php file. So we have to merge them into .php as well? If so: Hmmm, I have seen (and even developed) more convenient and automated ways of updating configs already, but that was decades ago under DOS/16... ;) JFTR: The 'password' plugin is just an arbitrary example, the same may/will most likely apply to other plugins as well. But 'password' is just the first one where we recognized that "$config" is being used at least in the .dist file already. Any helpful advice appreciated. Cheers, -- Michael Heydekamp Co-Admin freexp.de Düsseldorf/Germany

3 12

Regression upon right-clicking on message
by Michael Heydekamp 04 Nov '13

04 Nov '13

See attached: In 0.9.5, a sub-menu opens upon right-clicking and hovering the mouse on the "Mehr..." item. In 1.0-git, no such sub-menu opens. But it did still open a few days ago. <sigh> (Win7/64, IE10) Cheers, -- Michael Heydekamp Co-Admin freexp.de Düsseldorf/Germany

3 3

state of unit tests
by Charles McNulty 31 Oct '13

31 Oct '13

What is the current state of the unit tests? I am getting many failures of both the PHP and Selenium tests, but I was running them off on master, not a released version. Part of the problem was that I was hitting the 120 second PHP timeout that's hardcoded into iniset.php. When you run the tests do you manually change that or is there some other way of running the tests that bypasses the execution timeout? Are tests being added to and maintained? I've become used to test-driven development with jQuery, and it feels a little dirty to submit a bug or a pull request without a corresponding unit test. -Charles -- Sent using Roundcube http://roundcube.net/

2 3

focused messages - dotted outline
by Charles McNulty 30 Oct '13

30 Oct '13

Hello team, I spent quite a few hours back in 2006 "scratching my own itch" in the roundcube code. I haven't had much time to work on it since then, but I happen to have some time at the moment to dedicate to the project. I just submitted a pull request that will make "control-click" and "control-arrow-key" behave more like a native application. One thing that has always bothered my about the roundcube layout is that there's no way to know which message in the list has the "focus" in cases where the focus is separate from the selected message. For instance if you select a message, and then CTRL-click that same message, it will be unselected, but it still has the focus, because if you then click the down arrow it will select the next message. The css "outline" attribute would work for this, but there are some inconsistencies with where exactly each browser places the outline - and IE (including IE10) does not support the outline-offset attribute at all, which means that the outline doesn't work very well in IE (no surprise). To see what I mean, you can insert the following style into styles.css: .records-table tr.focused { outline:1px dotted black; outline-offset: -1px; } I'd appreciate any thoughts on the issue! -Charles McNulty

3 7

keyboard documentation and further ideas
by Charles McNulty 30 Oct '13

30 Oct '13

I scoured the code for keyboard handling and here's what I found: Message & Contact List (unless otherwise noted): Up and down arrow keys: Select next/previous item When selecting a message in a thread: Plus/minus keys: Open/close the thread Spacebar: Toggles thread open and closed +/= key: Open thread (firefox - not chrome) Page-up/page-down: Load previous/next page Home/end: Select first/last item Enter key: Show message (not applicable to contacts) Escape key: Cancel mouse drag (mid-drag) Delete key: Delete selected items Backspace key: Delete selected messages only (does not work for contacts) It doesn't look like this is documented anywhere. Anyone care to add this to the wiki? I'd be happy to create/manage a wiki page for key-bindings, or it could just be added to the User_Guide page. Personally, I think they could be tweaked though, and maybe that should happen before documenting the behavior. 1) The (+/=) key support is browser dependent, and could probably just be dropped. 2) The backspace key works for messages but not contacts and seems redundant with the delete key - additionally some people use the backspace key for "back" in a browser. I'm not suggesting we support using the backspace key for browser-back, but I am suggesting that we might not want to delete messages if a user presses it intending to go back. 3) I think that we should add left/right cursor (arrow) key support to expand/collapse threads. -Charlie

6 6

problem latest git and mobile devices
by Cor Bosman 29 Oct '13

29 Oct '13

Hey all, ive gotten some complaints regarding a recent git version with a test user base. It seems you can no longer scroll a message in the messageview. I can reproduce this on my iphone with today's git version but ive also had android complaints. This is a pretty serious issue as it makes rc unusable on mobile devices. Reports show that that is on average about 20% of a site's userbase. Cor

5 12

Re: [RCD] [RCU] roundcube 0.9.3 - Invalid request. no data saved
by Reindl Harald 29 Oct '13

29 Oct '13

Am 29.10.2013 14:53, schrieb Igor Zinovik: > My roundcube does not allow me to login into my mailbox. It just writes: > " invalid request no data was saved " > > I enabled all logging settings in main.inc.php > $rcmail_config['smtp_log'] = true; > $rcmail_config['log_logins'] = true; > $rcmail_config['log_session'] = true; > $rcmail_config['sql_debug'] = true; > $rcmail_config['imap_debug'] = true; > > My Apache is able to successfully save sessions to /var/lib/php5/ that's nice but roundcube still insists to not use the php session-handler otherwise i would no longer need to carry the patch from months ago in the RPM-SPEC Patch2: roundcubemail_native_session_handler.patch [builduser@buildserver:/rpmbuild/SPECS]$ cat ../SOURCES/roundcubemail_native_session_handler.patch diff --git a/config/main.inc.php.dist b/config/main.inc.php.dist index 4a73ff1..58332cb 100644 --- a/config/main.inc.php.dist +++ b/config/main.inc.php.dist @@ -254,9 +254,10 @@ $rcmail_config['session_name'] = null; // Session path. Defaults to PHP session.cookie_path setting. $rcmail_config['session_path'] = null; -// Backend to use for session storage. Can either be 'db' (default) or 'memcache' -// If set to memcache, a list of servers need to be specified in 'memcache_hosts' +// Backend to use for session storage. Can either be 'db' (default), 'memcache' or 'php' +// If set to 'memcache', a list of servers need to be specified in 'memcache_hosts' // Make sure the Memcache extension (http://pecl.php.net/package/memcache) version >= 2.0.0 is installed +// Setting this value to 'php' will use the default session save handler configured in PHP $rcmail_config['session_storage'] = 'db'; // Use these hosts for accessing memcached diff --git a/program/include/rcmail.php b/program/include/rcmail.php index 89f2f96..a7ddb1e 100644 --- a/program/include/rcmail.php +++ b/program/include/rcmail.php @@ -746,7 +746,7 @@ class rcmail extends rcube // before closing the database connection, write session data if ($_SERVER['REMOTE_ADDR'] && is_object($this->session)) { - session_write_close(); + $this->session->write_close(); } // write performance stats to logs/console diff --git a/program/lib/Roundcube/rcube.php b/program/lib/Roundcube/rcube.php index 4471ace..4e3f8fc 100644 --- a/program/lib/Roundcube/rcube.php +++ b/program/lib/Roundcube/rcube.php @@ -457,7 +457,6 @@ class rcube ini_set('session.name', $sess_name ? $sess_name : 'roundcube_sessid'); ini_set('session.use_cookies', 1); ini_set('session.use_only_cookies', 1); - ini_set('session.serialize_handler', 'php'); ini_set('session.cookie_httponly', 1); // use database for storing session data @@ -471,7 +470,7 @@ class rcube // start PHP session (if not in CLI mode) if ($_SERVER['REMOTE_ADDR']) { - session_start(); + $this->session->start(); } } diff --git a/program/lib/Roundcube/rcube_session.php b/program/lib/Roundcube/rcube_session.php index dedde22..4e06827 100644 --- a/program/lib/Roundcube/rcube_session.php +++ b/program/lib/Roundcube/rcube_session.php @@ -42,6 +42,7 @@ class rcube_session private $secret = ''; private $ip_check = false; private $logging = false; + private $storage; private $memcache; @@ -59,11 +60,14 @@ class rcube_session $this->set_lifetime($lifetime); // use memcache backend - if ($config->get('session_storage', 'db') == 'memcache') { + $this->storage = $config->get('session_storage', 'db'); + if ($this->storage == 'memcache') { $this->memcache = rcube::get_instance()->get_memcache(); // set custom functions for PHP session management if memcache is available if ($this->memcache) { + ini_set('session.serialize_handler', 'php'); + session_set_save_handler( array($this, 'open'), array($this, 'close'), @@ -79,7 +83,9 @@ class rcube_session true, true); } } - else { + else if ($this->storage != 'php') { + ini_set('session.serialize_handler', 'php'); + // set custom functions for PHP session management session_set_save_handler( array($this, 'open'), @@ -92,6 +98,22 @@ class rcube_session } + /** + * Wrapper for session_start() + */ + public function start() + { + session_start(); + + // copy some session properties to object vars + if ($this->storage == 'php') { + $this->key = session_id(); + $this->ip = $_SESSION['__IP']; + $this->changed = $_SESSION['__MTIME']; + } + } + + public function open($save_path, $session_name) { return true; @@ -116,6 +138,20 @@ class rcube_session /** + * Wrapper for session_write_close() + */ + public function write_close() + { + if ($this->storage == 'php') { + $_SESSION['__IP'] = $this->ip; + $_SESSION['__MTIME'] = time(); + } + + session_write_close(); + } + + + /** * Read session data from database * * @param string Session ID

1 0

Post hooks for DB operations
by Gus 29 Oct '13

29 Oct '13

Hi all, I'm currently writing a plugin that needs to act after an identity has been inserted into database. There is an identity_create hook that is called before the identity is inserted. How about adding a hook (whose argument would be data for the newly created identity, including the identity_id), that would be called after the database record has been created? That way you could write a plugin that could, e.g. 1. use identity_create to start a DB transaction and possibly modify identity data 2. use the new identity_stored (or something similar) hook to write it's own record into its own database table (with a constraint referencing the new identity) and commit the transaction (or rollback if something went wrong) Similar to those, I would also propose post-insert/update hooks for other events that could prove useful, so: identity_stored ( $record ) for after an identity was stored (created or updated) user_created ( $record ) for after a new user has been created contact_stored ( $record ) for after a contact has been stored (created or updated) group_stored ( $record ) for after a group has been stored (created, updated, renamed, members added, members deleted) preferences_stored ( $record ) for after user's preferences have been saved Is this acceptable? It's not too complicated to add those and I'd be willing to contribute the code and document the changes on the wiki. If not, I'd appreciate if you could clue me into how I might be able to write a plugin that needs to do something similar as described above (i.e. create records in additional tables that keep referential integrity with records in Roundcube's own tables). Regards, Tin

1 0

feedback regarding larry
by Cor Bosman 24 Oct '13

24 Oct '13

Hey all, ive been getting some feedback regarding the larry theme. Most of it i can handle myself, but one specifically keeps coming back. People dont realise that this button does anything, and miss all the feature that are accessed through it: http://grab.by/rkkc It's in a bit of a strange place. You click on the left, but it opens something underneath. It doesnt really look very clear what it's supposed to do. I would actually prefer to start it out in open, and let people figure out how to close it, that would sure save us a lot of support calls. Would be even better if it could be more intuitive about that it's supposed to do. Cor

5 8

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

Dev