After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 489043 - Meeting Attendees doesn't work properly
Meeting Attendees doesn't work properly
Status: RESOLVED FIXED
Product: evolution
Classification: Applications
Component: Calendar
2.12.x (obsolete)
Other All
: High critical
: ---
Assigned To: evolution-calendar-maintainers
Evolution QA team
Depends on:
Blocks:
 
 
Reported: 2007-10-22 15:07 UTC by David Ronis
Modified: 2008-08-27 16:39 UTC
See Also:
GNOME target: ---
GNOME version: 2.21/2.22


Attachments
script output from terminal session (22.25 KB, text/plain)
2007-10-24 16:36 UTC, David Ronis
  Details
Screenshot of dialog adding the 5th attendee (86.43 KB, image/png)
2007-11-09 17:14 UTC, David Ronis
  Details
Screenshot adding the 6th attendee (85.50 KB, image/png)
2007-11-09 17:15 UTC, David Ronis
  Details
Contact manager GUI before pressing OK (46.36 KB, image/png)
2008-02-01 18:14 UTC, David Ronis
  Details
Contact manager GUI after pressing "OK" (33.46 KB, image/png)
2008-02-01 18:16 UTC, David Ronis
  Details
Broken contact list saved as vcard (272 bytes, text/plain)
2008-02-01 18:30 UTC, David Ronis
  Details
proposed eds patch (contact list) (1.27 KB, patch)
2008-02-04 22:09 UTC, Milan Crha
committed Details | Review

Description David Ronis 2007-10-22 15:07:03 UTC
Please describe the problem:
I've set up a meeting with several attendees and want to add one new attendee.  I 
open up the meeting dialog and click on the Add button.  I get a new attendee line, but can't add any information to it (like an e-mail address).  As soon as I click on the new line, it disappears.

I also tried managing the attendee list by clicking on the attendee button (at the top left).  Again I get the correct popup, but now, ALL the existing attendees are deleted.  I presume that I can add them again, but don't want to go through the bother.

Clearly something is broken.  I should be able to add an atteendee either way, and once added, only the Remove button should delete them.

Steps to reproduce:
See above

 


Actual results:


Expected results:


Does this happen every time?
I've tried the described scenarios several times.  I get the same behavior each time.

Other information:
I'm using garnome's svn trunk, on a slackware 11.0 (heavily modified) and have compiled evo-2.12.1 using gcc-4.2.2.
Comment 1 David Ronis 2007-10-24 15:40:57 UTC
I also seem to have a related problem with my contact list management.  Basically, now when I try to add a contact, it is either garbled (the contact is obtained correctly from exchange) or one of the exiting contacts is deleted.

This is breaking all kinds of things here.

I tried raising the priority of this bug but wasn't allowed to; IMHO it should be otherwise users will unknowingly break their contact lists.


Comment 2 David Ronis 2007-10-24 16:36:47 UTC
Created attachment 97788 [details]
script output from terminal session

Following a suggestion of andre on #evolution, I ran evo and e-d-s from a terminal.  The attachement is the record of the session
Comment 3 Karsten Bräckelmann 2007-11-02 20:02:30 UTC
(In reply to comment #1)
> I tried raising the priority of this bug but wasn't allowed to; IMHO it should
> be otherwise users will unknowingly break their contact lists.

Priority High...
Comment 4 David Ronis 2007-11-02 20:14:41 UTC
Yet another observation.  It seems that the contact list problem may be related to exchange.  The lists I've been using are "populated" with e-mail adresses that I obtain from exchange's global address list.  I made up a contact list comprised of e-mails from my local personal addresses and this seems to work, both when I create the list initially and when I re-open it to modify things.

Comment 5 David Ronis 2007-11-06 16:22:16 UTC
One more observation:

I've just tried creating a totally new contact list, one with two names on it.  The dialog looks like it is working properly; however, after I push 'OK', the resulting contact (which has the correct name) contains no real entries, just one line containing EMAIL, I tried again and got X-EVOLUTION-LIST-SHOW_ADDRESSES
(I had to reopen the contact to see this).  Evo showed "FILE" or "FALSE" in it's main window.

Comment 6 André Klapper 2007-11-06 16:41:21 UTC
there's two issues in the output:

impl_GNOME_Evolution_Addressbook_Book_getContactList
(evolution-data-server-1.12:31860): libebook-WARNING **: invalid character found in parameter spec


(evolution:31908): libecal-WARNING **: e-cal.c:318: Unexpected response
cal = 0x8256600
cal = 0x80922c0
libecal-Message: e_cal_recur_generate_instances_of_rule(): bogus component, does not have DTSTART.  Skipping...
Comment 7 David Ronis 2007-11-06 17:21:48 UTC
Following more discussion in #evolution, I was pointed to Bug 380644, which sounds similar.  I've just downloaded the svn sources, applied 380644's patch and reinstalled.  My bug with the contact list editor seems to still be there.  Manually added email addresses appear, but those brought in via exchange's gal, while initially appearing correctly in the dialog, end up getting corrupted once 'OK' is set.  Reopening the dialog only leads to more breakage.

The problem with editing the attendee list is also still there.

(I'm starting to wonder if the patched files were really rebuilt).



Comment 8 David Ronis 2007-11-06 19:33:13 UTC
Further discussion on #evolution led me to discover the following:

In updating the attendee list, if after pressing "add" I simply start typing the new address, it works.  I don't see the change until after I click on the new highlighted field, but I seem to be able to add entries.

Comment 9 David Ronis 2007-11-09 17:00:54 UTC
 Here's yet another observation. If I create a new meeting, and start adding people, the dialog works as expected until the 6th invitee (including me).  Basically, by the the 5th has partially scrolled outside the window and that triggers part of the problem (the widget doesn't auto-scroll to keep the active text-input field in full view)

Having seen this, I'm pretty sure that this explains why simply going ahead
and typing still works, and probably has little to do with the contact list
problem.


Comment 10 David Ronis 2007-11-09 17:14:18 UTC
Created attachment 98828 [details]
Screenshot of dialog adding the 5th attendee
Comment 11 David Ronis 2007-11-09 17:15:05 UTC
Created attachment 98829 [details]
Screenshot adding the 6th attendee
Comment 12 Srinivasa Ragavan 2007-11-10 05:37:59 UTC
Can you look at it suman?
Comment 13 Milan Crha 2007-11-27 17:37:30 UTC
This contains 3 different problems, some of them are solved :) I'm going to look at it, if Suman is not against.
Comment 14 Srinivasa Ragavan 2007-11-27 17:56:24 UTC
Suman wont be. He would be busy on MAPI for atleast another 2 months :)
Comment 15 Milan Crha 2007-11-27 17:59:33 UTC
Oh, months, Jesus :)
Comment 16 David Ronis 2008-02-01 17:59:47 UTC
I've not been able to test this for some time as GAL lookups under exchange have been broken.  They are fixed as of 2.21.90, but the problem with attendee's being pulled in by the GAL still exists.  The existing attendees (e.g., those set by "hand") get trashed and replaced by weird strings as described in comments #5 and #7.

As before, everything looks right while I'm adding names to the list in the GUI
(i.e., the lookup works, and then gets moved to the list in the lower window).  Once I save the list it comes back trashed.

Comment 17 David Ronis 2008-02-01 18:14:45 UTC
Created attachment 104214 [details]
Contact manager GUI before pressing OK

This shows the contact list manager before I press "OK".  All three entries were retrieved from our GAL.
Comment 18 David Ronis 2008-02-01 18:16:48 UTC
Created attachment 104215 [details]
Contact manager GUI after pressing "OK"

In addition, the entry (list view) shows the wrong leading e-mail address.
Comment 19 David Ronis 2008-02-01 18:30:36 UTC
Created attachment 104216 [details]
Broken contact list saved as vcard
Comment 20 Srinivasa Ragavan 2008-02-04 06:20:26 UTC
David, you are trying to create contact lists in Exchange addressbook? (I know GAL is readonly, but asking on others).

If so, AFAIK, it awaits implementation. I'm wondering how is that enabled for you.
Comment 21 David Ronis 2008-02-04 17:49:01 UTC
RE: comment #20.  I don't think so.  I'm in evo's contact lists, and am trying to create a new contact list in a subfolder of "On this computer".  The GAL is being queried for e-mail addresses, and appears to be doing it's job correctly.  The problem seems to occur once I save the list.
Comment 22 Milan Crha 2008-02-04 22:09:52 UTC
Created attachment 104438 [details] [review]
proposed eds patch (contact list)

for evolution-data-server;

David sent me some example contacts and he has there a quotes in the field, and because the quote is a special character, then it cannot be saved there (one cannot escape it either), so here's the patch. Just notice the quotation character is removed when storing. It's what RFC says.
Comment 23 Srinivasa Ragavan 2008-02-05 04:14:47 UTC
Milan, seems ok to me. I think Ross might be the right person.

Ross, can you approve this small vcard patch?
Comment 24 David Ronis 2008-02-05 18:59:06 UTC
I applied the patch to 2.21.90--it seems to have fixed the problem.

Thanks Milan.

Comment 25 David Ronis 2008-02-06 19:03:19 UTC
The patch may be involved in a new problem.  See Bug 514836.
Comment 26 Srinivasa Ragavan 2008-02-07 04:44:34 UTC
David, I think it is due to this patch? Am I right?
Comment 27 Milan Crha 2008-02-07 12:12:08 UTC
Srag, the patch is correct, you cannot store quotes in parameter values in vCard.
As I had a chat wit David on IRC, you cannot do anything with the above bug without this patch, because it will not allow you to enter contact with quotes. 

To Ross: Maybe better solution for this would be to change the logic itself it the storing attendees in contact-list. I'm talking about "EVC_X_DEST_NAME" and "EVC_X_DEST_EMAIL" because it doesn't make much sense to have there stored same information twice, once as parameter and once as value.
Comment 28 David Ronis 2008-02-07 16:14:31 UTC
Hi Srag, 

I think you're correct, but as Milan says, I can't create the contact list in the first place without the patch.  The only thing that I'm confused about is that the contact list generates a correct list of e-mail addresses in a regular email (e.g., I put the contact list name in the To: field), only in the meeting gui does it seem to mess up.
Comment 29 Srinivasa Ragavan 2008-02-08 06:01:46 UTC
Oh. does mail composer works fine for you with this patch? If so we could just fix meeting editor to handle cases carefully.
Comment 30 David Ronis 2008-02-08 13:12:09 UTC
So it would seem.

Comment 31 Milan Crha 2008-02-22 10:37:21 UTC
EDS patch committed to trunk. Committed revision 8519.

I tried with latest gtk+ the bug when entering +5 attendees, and it has been fixed, so this is done too. David, can you check what left on this bug, please?
Comment 32 Milan Crha 2008-08-27 10:14:51 UTC
David, ping
Comment 33 David Ronis 2008-08-27 15:45:33 UTC
Hi Milan,

I've tried it with 2.23.4 and the problem seems to be fixed.  Haven't tried with svn-trunk because that has other issues that make it risky for me to run (mbox corruption).

David
Comment 34 Milan Crha 2008-08-27 16:39:01 UTC
Thanks for the feedback, closing per last comment then.