142845 - mozilla should have a column for screen name

Status: VERIFIED FIXED

(SeaMonkey :: MailNews: Address Book & Contacts, defect)

Description

[(not reading, please use [e-mail] instead)]

I think AIM, ICQ, YIM, IRC nicks are common enough where the mozilla address 
book should have columns for them.  just because mozilla doesn't have an built 
in IM client (it has built in IRC), doesn't mean the data isn't useful in the 
ab.

this will help simplify the code in the ns tree, and remove some cryptic hooks 
in the mozilla tree.

for a bonus feature, to come later:  like a url entries for web pages, it would 
be cool if the the entry could be linkified.  if we have url integration, a 
user could click on the AIM name in the preview pane to aim, icq, yim, jabber, 
irc? to the person.

Comment 1

[Dan Mosedale (:dmosedale, :dmose)]

I agree that this would be a fine thing.  We'd want to support some sort of
nickname/server tuple notation perhaps.

Comment 2

[mental]

Thought this request was the only meat left to bug 85344...

Comment 3

[(not reading, please use [e-mail] instead)]

accepting.

Comment 4

[(not reading, please use [e-mail] instead)]

for this pass, it will just be a single column (Screen Name) and a text field 
for Screen Name. 

The screenname will be linkified, but the format will be in prefs.

so if you use aim, the pref would be: "aim:goim?screenname=%S"

initial patch coming...

Comment 5

[(not reading, please use [e-mail] instead)]

Attachment: initial patch, not cleaned up or completed yet. thanks to cavin for all the BE changes.

Comment 6

[Navin Gupta]

if this is the patch I sent you, it should also include my changes to make
screenname work for our phonebook. 

Comment 7

[(not reading, please use [e-mail] instead)]

>if this is the patch I sent you, 
>it should also include my changes to make
>screenname work for our phonebook. 

It does, thanks navin.

Index: addrbook/src/nsAbLDAPProperties.cpp
+    // ?
+    {MozillaProperty_String, "_AimScreenName",        "nscpaimscreenname"},

note, that's specific to how we have our LDAP servers set up.  I think the
override will come when we allow user defined schemas.

Comment 8

[(not reading, please use [e-mail] instead)]

Attachment: more changes

more changes...

Comment 9

[(not reading, please use [e-mail] instead)]

Attachment: add support for LDIF (and tab,txt,cvs) export of the new fields.

Comment 10

[(not reading, please use [e-mail] instead)]

Attachment: adding support for import

Comment 11

[(not reading, please use [e-mail] instead)]

Attachment: add screenname (and other fields) support to advanced directory search

Comment 12

[(not reading, please use [e-mail] instead)]

Attachment: more work on search, more code cleanup. (includes fix for #161002)

Comment 13

[(not reading, please use [e-mail] instead)]

Attachment: more cleanup, for this bug and #161002

Comment 14

[(not reading, please use [e-mail] instead)]

Attachment: just the fix for this bug, more work still to come...

Comment 15

[(not reading, please use [e-mail] instead)]

Attachment: update

todo list:


move "Card(s)" to properties file
import / export
search (get right fields in there, right order)
log bug on edit card accesskeys?
fix <Screen Name:saspitzer> -> ScreenName: <saspitzer>
add aim presence to preview pane using webmail trick
LDAP mapping of screenname to prefs
goaim: url mapping to prefs
log bug about NS 7.x handling goaim:
use "screenname" instead of "aimscreenname"?
ACString in the idl

Comment 16

[(not reading, please use [e-mail] instead)]

Attachment: updated patch

1) fix search logic for "Any name" and "Any number"
2) add New | IM, New Chat, make them work using goim: urls

more to come.

Comment 17

[(not reading, please use [e-mail] instead)]

Attachment: patch

remove New | IM and New | Chat support for now, but they'll be coming.

Comment 18

[(not reading, please use [e-mail] instead)]

I'm going to spin off making this usable by ICQ, Yahoo, MSN, Jabber, IRC into 
another bug.

Comment 19

[(not reading, please use [e-mail] instead)]

fixed.  this has r/sr=bienvenu.  I've reviewed the changes from cavin and 
naving.

there's a bunch of spin off bugs, I'll log them and link to them here.

Comment 20

[Ninoschka Baca]

Trunk build 2002-11-21: WinMe, Linux 8 , Mac 10.1.3
Verified Fixed.