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 546397 - Missused summary::uids in eex?
Missused summary::uids in eex?
Status: RESOLVED FIXED
Product: Evolution Exchange
Classification: Deprecated
Component: Connector
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Connector Maintainer
Ximian Connector QA
evolution[disk-summary]
Depends on:
Blocks: 543389
 
 
Reported: 2008-08-05 11:31 UTC by Milan Crha
Modified: 2008-08-06 09:18 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
proposed eex patch (1.96 KB, patch)
2008-08-05 16:06 UTC, Milan Crha
committed Details | Review
proposed eds patch (4.01 KB, patch)
2008-08-05 16:18 UTC, Milan Crha
rejected Details | Review
proposed eds patch ][ (4.33 KB, patch)
2008-08-06 09:07 UTC, Milan Crha
committed Details | Review

Description Milan Crha 2008-08-05 11:31:47 UTC
See the debug session below, I just selected "personal/Drafts" on the Exchange account and no messages has been shown. I found the summary::uids doesn't contain any UID, but folder names. What did happen there?

Breakpoint 2, search_by_expression (folder=0x7f2384de8980, 
    expression=0x7f2396913050 "(match-all (and (not (system-flag \"deleted\")) (not (system-flag \"junk\"))))", ex=0x7f238c0602c0)
    at camel-exchange-folder.c:470
470		search = camel_exchange_search_new ();
(gdb) c
Continuing.
search_by_expression: matches->len:24

Breakpoint 3, camel_folder_summary_array (s=0x7f238e8896d0) at camel-folder-summary.c:484
484		for (i=0;i<s->uids->len;i++)
(gdb) display s->uids->len
1: s->uids->len = 24
(gdb) n
485			res->pdata[i] = (gpointer) camel_pstring_strdup ((char *)g_ptr_array_index(s->uids, i));
1: s->uids->len = 24
(gdb) s
camel_pstring_strdup (s=0x7f23929d7580 "personal/Drafts") at camel-string-utils.c:249
249		return camel_pstring_add ((char *) s, FALSE);
(gdb) n
250	}
(gdb) 
camel_folder_summary_array (s=0x7f238e8896d0) at camel-folder-summary.c:484
484		for (i=0;i<s->uids->len;i++)
1: s->uids->len = 24
(gdb) 
485			res->pdata[i] = (gpointer) camel_pstring_strdup ((char *)g_ptr_array_index(s->uids, i));
1: s->uids->len = 24
(gdb) s
camel_pstring_strdup (s=0x7f238b974b80 "Drafts") at camel-string-utils.c:249
249		return camel_pstring_add ((char *) s, FALSE);
(gdb) n
250	}
(gdb) 
camel_folder_summary_array (s=0x7f238e8896d0) at camel-folder-summary.c:484
484		for (i=0;i<s->uids->len;i++)
1: s->uids->len = 24
(gdb) s
485			res->pdata[i] = (gpointer) camel_pstring_strdup ((char *)g_ptr_array_index(s->uids, i));
1: s->uids->len = 24
(gdb) 
[New Thread 0x4252d950 (LWP 30003)]
camel_pstring_strdup (s=0x7f239083ebc0 "Inbox/_atest/meetings") at camel-string-utils.c:249
249		return camel_pstring_add ((char *) s, FALSE);
(gdb) c
Continuing.
Comment 1 Milan Crha 2008-08-05 11:34:17 UTC
quite funny thing is the "Total" count in the info-label is correct, even nothing shown in the message list.
Comment 2 Srinivasa Ragavan 2008-08-05 13:02:27 UTC
Dont understand the issue clearyly :( 
Comment 3 Milan Crha 2008-08-05 16:06:19 UTC
Created attachment 115910 [details] [review]
proposed eex patch

for evolution-exchange;

OK, the search_free in the eex was an evel here, get rid of it.
Comment 4 Milan Crha 2008-08-05 16:18:54 UTC
Created attachment 115912 [details] [review]
proposed eds patch

for evolution-data-server;

other things I found during investigation on this bug...
Comment 5 Srinivasa Ragavan 2008-08-05 16:44:11 UTC
This you can commit. Not sure about the other patch. I need to analyse/discuss with you more.
Comment 6 Milan Crha 2008-08-06 08:04:08 UTC
eex patch committed to trunk. Committed revision 1729.
Comment 7 Milan Crha 2008-08-06 09:07:12 UTC
Created attachment 115954 [details] [review]
proposed eds patch ][

for evolution-data-server;

I'm sorry, I was odd with the ...remove_range chunk, but we found other thing there with srag during the discussion, thus benefit came at the end...
Comment 8 Srinivasa Ragavan 2008-08-06 09:09:48 UTC
Awesome. Commit
Comment 9 Milan Crha 2008-08-06 09:18:13 UTC
eds patch committed to trunk. Committed revision 9274.