78933 adds support for Outlook. We also need to add support for Outlook express.
In order to do this we will have to support the WAB API as well as the
Created attachment 36631 [details] [diff] [review]
This patch to be applied after patch #78933.
Cyrille is modifying code to support WAB so this bug has been reassigned.
This patch must be applied on top of patch #78933. This patch will enable
support for both Outlook Express(WAB) and Outlook(MAPI). To activate Outlook
Express(OE) you simply need to edit the preferences file prefs.js:
Add 4 entries:
user_pref("ldap_2.servers.OE.description", "Outlook Express");
we choose OE to represent Outlook Express. This name must be unique. The dirType
must be set to 3. 'x' is set to the next available position and the 'uri' is set
as above where 'oe' will distinguish Outlook Express from 'op' which will
Thus to enable Outlook you also need to add 4 corresponding entries:
To activate Outlook in Mozilla, Outlook must be set as the default mailer. This
is consistent with the way the Mozilla Import facility works. The WAB or Outlook
Express will be activated regardless and to activate both, Outlook must be set
as the default mailer.
-The WAB interface will return the contents of the OE address book WHATEVER the
default mail client is.
-The MAPI interface will return the contents of the Outlook address book ONLY if
Outlook is defined as the default mail client.
The contents of the Outlook address book are NOT necessarily the same as the OE
address book. They only are, if a specific option has been selected in OE,
instructing it to 'share' (i.e. override) the contents of its address book with
those of Outlook's. The WAB interface displays the correct address book
depending on this setting.
We will provide an experimental build at http://abzilla.mozdev.com
Created attachment 37671 [details] [diff] [review]
update against latest patch for #78933
*** Bug 57658 has been marked as a duplicate of this bug. ***
The last comment marking this as a duplicate of #57658 misses the central point
of this bug. This fix is not about importing Outlook Express into Mozilla but
rather it makes available Outlook Express Address Books to Mozilla dynamically.
Created attachment 42394 [details] [diff] [review]
Keep patch up-to-date against changes to 78933
Created attachment 43219 [details] [diff] [review]
Keep patch up-to-date against #78933 plus address possible MAPI Threading issue
Missed the 0.9.3 train.
Created attachment 44114 [details] [diff] [review]
Keep patch up-to-date against 'sr' changes to 78933
The 'sr' changes requested for #78933 have resulted in this latest patch. This
patch is now up-to-date against #78933 and also any issues raised during the
'sr' of #78933 have been applied to this patch.
r=chuang. I am not familiar with WAB, but it looks good to me.
Created attachment 44362 [details] [diff] [review]
Apply syntactical changes requested by sr of 78933 to this patch
I have just updated the patch against the requested syntactical changes
requested by the sr of #78933. These changes are now located in #93092.
Typically the changes required a code change e.g.
if(aCard == nsnull)....
Created attachment 45071 [details] [diff] [review]
Keep in sync with head and support #83023 Query Totals
sr=mscott. My only comment is that you might want to wrap alot of the printfs
you added with #ifdef DEBUG flags. I know someone (cls?) is creating bugs on
people who are using printfs that will show up in release builds.
Good stuff. The PRINTF() are defined as NS_LOG_PRINTF() which I believe only
gets enabled in DEBUG builds as defined in nslog.h. But I will contact cls to
verify this before applying for a landing. I assume cls is firstname.lastname@example.org.
I have got the foloiwng reply from cls....
> We received the following sr for:
> with the proviso that we clean up the PRINTF()s. Typically we
> have used the following:
> #include "nslog.h"
> #define PRINTF NS_LOG_PRINTF(nsWabLog)
> ....PRINTF("Cannot load library.\n");
> I wanted to clarify that this is OK. I would appreciate a reply.
Looks fine to me.
Code checked in.
Developer, please advise QA how to verify this bug.
To verify the bug, add to the preferences the four lines described in one of the
first comments, only with the uri equal to "moz-aboutlookdirectory://oe/".
Provided Outlook Express is installed on the machine, and regardless of its
configuration (default mailer or not, sharing address book with Outlook or not),
you should see additional address books available which names start with OE.
These address books should support standard address book features
(addition/deletion/edition of cards/mailing lists).
In addition to Cyrille's comments about how to test this code, the code will not
be activated until the corresponding UI patch is landed:
It is hoped that a patch and experimental builds for the 83100 bug will be
produced soon which would allow this code to be tested. In the meantime,
manually editing the preferences file is the only means to activate this code.
Let us know if you need any help with this.
Thanks. I'll wait for bug 83100 to be fixed.