diff options
-rw-r--r-- | addressbook/ChangeLog | 11 | ||||
-rw-r--r-- | addressbook/backend/pas/pas-backend-ldap.c | 112 |
2 files changed, 100 insertions, 23 deletions
diff --git a/addressbook/ChangeLog b/addressbook/ChangeLog index b14f21e210..6381fce3f5 100644 --- a/addressbook/ChangeLog +++ b/addressbook/ChangeLog @@ -1,3 +1,14 @@ +2001-01-11 Chris Toshok <toshok@helixcode.com> + + * backend/pas/pas-backend-ldap.c + (pas_backend_ldap_process_create_card): get a book view from + somewhere so we can give status messages - choose the first one in + the list. + (pas_backend_ldap_process_modify_card): same. + (pas_backend_ldap_process_remove_card): same. + (prop_info): add more properties, like home_phone, mobile, pager, + uri, org_unit, office, title, and manager. + 2001-01-09 Miguel de Icaza <miguel@gnu.org> * gui/widgets/Makefile.am (minicard_widget_test_LDADD): Put GNOME diff --git a/addressbook/backend/pas/pas-backend-ldap.c b/addressbook/backend/pas/pas-backend-ldap.c index f7898708ad..46681af3a6 100644 --- a/addressbook/backend/pas/pas-backend-ldap.c +++ b/addressbook/backend/pas/pas-backend-ldap.c @@ -115,7 +115,7 @@ struct prop_info { ECardSimpleField field_id; char *query_prop; char *ldap_attr; -#define PROP_TYPE_NORMAL 0x01 +#define PROP_TYPE_STRING 0x01 #define PROP_TYPE_LIST 0x02 #define PROP_DN 0x04 int prop_type; @@ -131,21 +131,51 @@ struct prop_info { } prop_info[] = { -#define DN_NORMAL_PROP(fid,q,a) {fid, q, a, PROP_TYPE_NORMAL | PROP_DN, NULL} -#define DN_LIST_PROP(fid,q,a,ctor,ber,cmp) {fid, q, a, PROP_TYPE_LIST | PROP_DN, ctor, ber, cmp} #define LIST_PROP(fid,q,a,ctor,ber,cmp) {fid, q, a, PROP_TYPE_LIST, ctor, ber, cmp} -#define NORMAL_PROP(fid,q,a) {fid, q, a, PROP_TYPE_NORMAL, NULL} - - DN_NORMAL_PROP (E_CARD_SIMPLE_FIELD_FULL_NAME, "full_name", "cn" ), - DN_NORMAL_PROP (E_CARD_SIMPLE_FIELD_FAMILY_NAME, "family_name", "sn" ), - NORMAL_PROP (E_CARD_SIMPLE_FIELD_TITLE, "title", "title"), - DN_NORMAL_PROP (E_CARD_SIMPLE_FIELD_ORG, "org", "o"), - NORMAL_PROP (E_CARD_SIMPLE_FIELD_PHONE_PRIMARY, "phone", "telephonenumber"), - LIST_PROP (E_CARD_SIMPLE_FIELD_EMAIL, "email", "mail", email_populate_func, email_ber_func, email_compare_func) - -#undef DN_NORMAL_PROP -#undef DN_LIST_PROP -#undef NORMAL_PROP +#define STRING_PROP(fid,q,a) {fid, q, a, PROP_TYPE_STRING, NULL} + +/* E_CARD_SIMPLE_FIELD_FILE_AS, */ + STRING_PROP (E_CARD_SIMPLE_FIELD_FULL_NAME, "full_name", "cn" ), + STRING_PROP (E_CARD_SIMPLE_FIELD_FAMILY_NAME, "family_name", "sn" ), + LIST_PROP (E_CARD_SIMPLE_FIELD_EMAIL, "email", "mail", email_populate_func, email_ber_func, email_compare_func), + STRING_PROP (E_CARD_SIMPLE_FIELD_PHONE_PRIMARY, "phone", "telephoneNumber"), +/* E_CARD_SIMPLE_FIELD_PHONE_BUSINESS, */ + STRING_PROP (E_CARD_SIMPLE_FIELD_PHONE_HOME, "home_phone", "homePhone"), + STRING_PROP (E_CARD_SIMPLE_FIELD_ORG, "org", "o"), +/* XXX STRUCT_PROP (E_CARD_SIMPLE_FIELD_ADDRESS_BUSINESS, "business_address", "postalAddress", ....), */ +/* XXX STRUCT_PROP (E_CARD_SIMPLE_FIELD_ADDRESS_HOME, "home_address", "homePostalAddress", ....), */ + STRING_PROP (E_CARD_SIMPLE_FIELD_PHONE_MOBILE, "mobile", "mobile"), +/* E_CARD_SIMPLE_FIELD_PHONE_CAR, */ +/* E_CARD_SIMPLE_FIELD_PHONE_BUSINESS_FAX, */ +/* E_CARD_SIMPLE_FIELD_PHONE_HOME_FAX, */ +/* E_CARD_SIMPLE_FIELD_PHONE_BUSINESS_2, */ +/* E_CARD_SIMPLE_FIELD_PHONE_HOME_2, */ +/* E_CARD_SIMPLE_FIELD_PHONE_ISDN, */ +/* E_CARD_SIMPLE_FIELD_PHONE_OTHER, */ + STRING_PROP (E_CARD_SIMPLE_FIELD_PHONE_PAGER, "pager", "pager"), +/* E_CARD_SIMPLE_FIELD_ADDRESS_OTHER, */ +/* E_CARD_SIMPLE_FIELD_EMAIL_2, */ +/* E_CARD_SIMPLE_FIELD_EMAIL_3, */ + STRING_PROP (E_CARD_SIMPLE_FIELD_URL, "uri", "labeledURI"), + STRING_PROP (E_CARD_SIMPLE_FIELD_ORG_UNIT, "org_unit", "ou"), + STRING_PROP (E_CARD_SIMPLE_FIELD_OFFICE, "office", "roomNumber"), + STRING_PROP (E_CARD_SIMPLE_FIELD_TITLE, "title", "title"), +/* E_CARD_SIMPLE_FIELD_ROLE, */ + STRING_PROP (E_CARD_SIMPLE_FIELD_MANAGER, "manager", "manager"), +/* E_CARD_SIMPLE_FIELD_ASSISTANT, */ +/* E_CARD_SIMPLE_FIELD_NICKNAME, */ +/* E_CARD_SIMPLE_FIELD_SPOUSE, */ +/* E_CARD_SIMPLE_FIELD_NOTE, */ +/* E_CARD_SIMPLE_FIELD_FBURL, */ +/* E_CARD_SIMPLE_FIELD_ANNIVERSARY, */ +/* E_CARD_SIMPLE_FIELD_BIRTH_DATE, */ +/* E_CARD_SIMPLE_FIELD_MAILER, */ +/* E_CARD_SIMPLE_FIELD_NAME_OR_ORG, */ + + +#undef STRING_PROP +#undef LIST_PROP +#undef STRUCT_PROP }; static int num_prop_infos = sizeof(prop_info) / sizeof(prop_info[0]); @@ -446,7 +476,7 @@ build_mods_from_ecards (ECardSimple *current, ECardSimple *new, gboolean *new_dn mod->mod_type = g_strdup (prop_info[i].ldap_attr); - if (prop_info[i].prop_type & PROP_TYPE_NORMAL) { + if (prop_info[i].prop_type & PROP_TYPE_STRING) { mod->mod_values = g_new (char*, 2); mod->mod_values[0] = new_prop; mod->mod_values[1] = NULL; @@ -529,11 +559,17 @@ create_card_handler (PASBackend *backend, LDAPOp *op) ldap_mods = (LDAPMod**)mod_array->pdata; - g_print ("adding card dn: %s\n", dn); + if (op->view) + pas_book_view_notify_status_message (op->view, "Adding card to LDAP server"); + /* actually perform the ldap add */ ldap_error = ldap_add_s (ldap, dn, ldap_mods); - g_print ("ldap_add_s returned 0x%x (%s) status\n", ldap_error, ldap_err2string(ldap_error)); + if (op->view) + pas_book_view_notify_status_message (op->view, ""); + + if (ldap_error != LDAP_SUCCESS) + ldap_perror (ldap, "ldap_add_s"); /* and clean up */ free_mods (mod_array); @@ -566,8 +602,17 @@ pas_backend_ldap_process_create_card (PASBackend *backend, PASRequest *req) { LDAPCreateOp *create_op = g_new (LDAPCreateOp, 1); + PASBackendLDAP *bl = PAS_BACKEND_LDAP (backend); + PASBookView *book_view = NULL; + + if (bl->priv->book_views) { + PASBackendLDAPBookView *v = bl->priv->book_views->data; + book_view = v->book_view; + } - ldap_op_init ((LDAPOp*)create_op, backend, book, NULL, create_card_handler, create_card_dtor); + ldap_op_init ((LDAPOp*)create_op, backend, book, + book_view, + create_card_handler, create_card_dtor); create_op->vcard = req->vcard; @@ -589,6 +634,8 @@ remove_card_handler (PASBackend *backend, LDAPOp *op) int ldap_error; ldap_error = ldap_delete_s (bl->priv->ldap, remove_op->id); + if (ldap_error != LDAP_SUCCESS) + ldap_perror (bl->priv->ldap, "ldap_delete_s"); response = ldap_error_to_response (ldap_error); @@ -614,8 +661,17 @@ pas_backend_ldap_process_remove_card (PASBackend *backend, PASRequest *req) { LDAPRemoveOp *remove_op = g_new (LDAPRemoveOp, 1); + PASBackendLDAP *bl = PAS_BACKEND_LDAP (backend); + PASBookView *book_view = NULL; + + if (bl->priv->book_views) { + PASBackendLDAPBookView *v = bl->priv->book_views->data; + book_view = v->book_view; + } - ldap_op_init ((LDAPOp*)remove_op, backend, book, NULL, remove_card_handler, remove_card_dtor); + ldap_op_init ((LDAPOp*)remove_op, backend, book, + book_view, + remove_card_handler, remove_card_dtor); remove_op->id = req->id; @@ -704,7 +760,8 @@ modify_card_handler (PASBackend *backend, LDAPOp *op) /* actually perform the ldap modify */ ldap_error = ldap_modify_s (ldap, id, ldap_mods); - g_print ("ldap_modify_s returned 0x%x (%s) status\n", ldap_error, ldap_err2string(ldap_error)); + if (ldap_error != LDAP_SUCCESS) + ldap_perror (ldap, "ldap_modify_s"); } else { g_print ("modify list empty. no modification sent\n"); @@ -742,8 +799,17 @@ pas_backend_ldap_process_modify_card (PASBackend *backend, PASRequest *req) { LDAPModifyOp *modify_op = g_new (LDAPModifyOp, 1); + PASBackendLDAP *bl = PAS_BACKEND_LDAP (backend); + PASBookView *book_view = NULL; + + if (bl->priv->book_views) { + PASBackendLDAPBookView *v = bl->priv->book_views->data; + book_view = v->book_view; + } - ldap_op_init ((LDAPOp*)modify_op, backend, book, NULL, modify_card_handler, modify_card_dtor); + ldap_op_init ((LDAPOp*)modify_op, backend, book, + book_view, + modify_card_handler, modify_card_dtor); modify_op->vcard = req->vcard; @@ -1322,7 +1388,7 @@ build_card_from_entry (LDAP *ldap, LDAPMessage *e) values = ldap_get_values (ldap, e, attr); if (values) { - if (info->prop_type & PROP_TYPE_NORMAL) { + if (info->prop_type & PROP_TYPE_STRING) { /* if it's a normal property just set the string */ e_card_simple_set (card, info->field_id, values[0]); |