Add support for Outlook express to the addressbook

RESOLVED FIXED in mozilla0.9.4

Status

SeaMonkey
MailNews: Address Book & Contacts
RESOLVED FIXED
16 years ago
10 years ago

People

(Reporter: Cuchulainn, Assigned: Cyrille Moureaux)

Tracking

(Blocks: 1 bug)

Trunk
mozilla0.9.4
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [br])

Attachments

(7 attachments)

(Reporter)

Description

16 years ago
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
MAPI interface.
(Reporter)

Updated

16 years ago
Depends on: 78933

Comment 1

16 years ago
Created attachment 36631 [details] [diff] [review]
This patch to be applied after patch #78933.
(Reporter)

Comment 2

16 years ago
Cyrille is modifying code to support WAB so this bug has been reassigned.
Assignee: dmose → Cyrille.Moureaux

Comment 3

16 years ago
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");
user_pref("ldap_2.servers.OE.dirType, 3);
user_pref("ldap_2.servers.OE.position", x);
user_pref("ldap_2.servers.OE.uri", "aboutlookdirectory://oe/");

Note:

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
respresent Outlook.

Thus to enable Outlook you also need to add 4 corresponding entries:

user_pref("ldap_2.servers.OP.description", "Outlook");
user_pref("ldap_2.servers.OP.dirType", 3);
user_pref("ldap_2.servers.OP.position", x);
user_pref("ldap_2.servers.OP.uri", "aboutloookdirectory://op/");

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.

To Sumarise:

-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

Comment 4

16 years ago
Created attachment 37671 [details] [diff] [review]
update against latest patch for #78933

Comment 5

16 years ago
*** Bug 57658 has been marked as a duplicate of this bug. ***

Comment 6

16 years ago
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.
(Reporter)

Updated

16 years ago
Keywords: nsbeta1
Target Milestone: --- → mozilla0.9.3

Comment 7

16 years ago
Created attachment 42394 [details] [diff] [review]
Keep patch up-to-date against changes to 78933

Comment 8

16 years ago
Created attachment 43219 [details] [diff] [review]
Keep patch up-to-date against #78933 plus address possible MAPI Threading issue

Comment 9

16 years ago
Missed the 0.9.3 train.
Whiteboard: [br]
Target Milestone: mozilla0.9.3 → mozilla0.9.4

Comment 10

16 years ago
Created attachment 44114 [details] [diff] [review]
Keep patch up-to-date against 'sr' changes to 78933

Comment 11

16 years ago
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.

Comment 12

16 years ago
r=chuang.  I am not familiar with WAB,  but it looks good to me.

Comment 13

16 years ago
Created attachment 44362 [details] [diff] [review]
Apply syntactical changes requested by sr of 78933 to this patch

Comment 14

16 years ago
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)....
to
if(!aCard)..........

Comment 15

16 years ago
Created attachment 45071 [details] [diff] [review]
Keep in sync with head and support #83023 Query Totals

Updated

16 years ago
Status: NEW → ASSIGNED

Comment 16

16 years ago
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. 

Comment 17

16 years ago
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 cls@seawood.org.

Comment 18

16 years ago
I have got the foloiwng reply from cls....

> Hi,
> 
> We received the following sr for:
> 
> http://bugzilla.mozilla.org/show_bug.cgi?id=83103
> 
> 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.

- cls
Code checked in.
Status: ASSIGNED → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED

Comment 20

16 years ago
Developer, please advise QA how to verify this bug.

Comment 21

16 years ago
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).

Comment 22

16 years ago
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:
http://bugzilla.mozilla.org/show_bug.cgi?id=83100.

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.

Updated

16 years ago
Blocks: 83100

Comment 23

16 years ago
Thanks. I'll wait for bug 83100 to be fixed.

Updated

16 years ago
QA Contact: fenella → nbaca
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.