Roundcube Community Forum

 

addressbook in compose window

Started by corbosman, October 28, 2009, 03:42:19 PM

Previous topic - Next topic

jeffshead

Quote from: sckramer;33961confirmed that it shows Username <> in...

Can you please provide a fix for 0.5.2?

corbosman

Did you try the latest version to see if it works?

jeffshead

Quote from: corbosman;34561Did you try the latest version to see if it works?

Do you mean try compose_addressbook-5.0beta1.tgz with RoundCube 0.5.2 or try RoundCube 0.6?

I did not want to use RC 0.6 because I believe some of the other plugins I use have not been updated to work with 0.6.

I did not try compose_addressbook-5.0beta1.tgz with RoundCube 0.5.2 because of your warning (only use with roundcube 0.6 svn+, do NOT use with roundcube 0.5.x)

corbosman

You can try compose_addressbook-5.0beta1.tgz to see if it works for 0.5.2. Probably not.

and compose_addressbook-4.0 doesnt work?  What's not working?

ABerglund

compose_addressbook-4.0 is working for me just fine in roundcube 0.5.2.
Arne Berglund
SysAdmin, Internet Services
Lane Education Service District
Eugene, OR, USA

jeffshead

#185
Quote from: corbosman;34568You can try compose_addressbook-5.0beta1.tgz to see if it works for 0.5.2. Probably not.

and compose_addressbook-4.0 doesnt work?  What's not working?

When using the 'compose_addressbook-4.0.tgz' with RC 0.5.2 trunk 4713, I have the same issue as others. When I select someone from the address book, it adds Displayname<> to the To field. It does not add the email address.

I just tested 'compose_addressbook-5.0beta1.tgz' with RC 0.5.2 trunk 4713 and that seems to have fixed that particular issue but I'm wondering if any other issues will become evident since this version is designed for RC 0.6.

corbosman

4713 is newer than 0.5.2 (4699). You must have gotten 0.6 beta code, because the only changes between 4.0 and 5.0 of my plugin are vcard changes that were done in RC 0.6. If 5.0beta1 works, then you're safe to use it.

jeffshead

Quote from: corbosman;345764713 is newer than 0.5.2 (4699). You must have gotten 0.6 beta code, because the only changes between 4.0 and 5.0 of my plugin are vcard changes that were done in RC 0.6. If 5.0beta1 works, then you're safe to use it.

Thanks! I'll keep testing.

corbosman

I uploaded a new version of compose_addressbook, 5.0b2 that uses roundcube's version of jqueryui instead of my version by requiring it as per the new plugin_require function. No need to configure jqueryui in the main config anymore. Remember to stop using my jqueryui plugin, and use the one from roundcube core instead!

You can get the new compose_addressbook here:  Downloads - roundcube-plugins - Roundcube Plugins - Google Project Hosting

ABerglund

#189
Quote from: ABerglund;34570compose_addressbook-4.0 is working for me just fine in roundcube 0.5.2.
Just a quick note - while compose_addressbook-4.0 does function for me in RC 0.5.2, further testing has revealed one anomaly. Once the "Loading..." dialog drops down when the search begins, it never goes away throughout the remainder of the compose session.

I'm not certain if this is strictly a compose_addressbook issue, or part of a larger bug in RC 0.5.2. In general, I see a lot more "Loading", "Moving" and "Deleting" dialogs that are reluctant to go away in RC 0.5.2. And I've already gone into the code and disabled a few of the dialog prompts.

EDIT - I think this is in RC, not the plugin. I get the hanging "Loading..." too often in other places as well.
Arne Berglund
SysAdmin, Internet Services
Lane Education Service District
Eugene, OR, USA

corbosman

Try version 4.1, is that better?

corbosman

Oh, btw, it is the plugin. Multiple plugins. Every plugin that tries to show the spinning busy ball. This is because RC a few versions ago changed the way you have to call that function, and in the process broke every plugin using it, resulting in the sticky loading msg.

Old code:

    rcmail.set_busy(true, 'loading');
    rcmail.http_post('plugin.get_addressbook', '', true);


New code:

    lock = rcmail.set_busy(true, 'loading');
    rcmail.http_post('plugin.get_addressbook', '', lock);

ABerglund

Ahh! That makes sense. I searched my install for the rcmail.set_busy string, and found it in two plugins. The version of compose_addressbook I'm using had two instance that were correct, but one that was not. After changing that, it works perfectly.

I also found the old code in contextmenu. I corrected that one too, we'll see if that makes me any happier with 0.5.2 in general. The spinning wheel has been very annoying.
Arne Berglund
SysAdmin, Internet Services
Lane Education Service District
Eugene, OR, USA

cityhunter

thanks for the nice plugin , have plan to add check box for address list choice?

corbosman

I dont understand what you mean.