Skip to content

Check ALL addressbooks before offering to save #14

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
Halfwalker opened this issue Jul 17, 2014 · 1 comment
Open

Check ALL addressbooks before offering to save #14

Halfwalker opened this issue Jul 17, 2014 · 1 comment

Comments

@Halfwalker
Copy link

Only sort of an issue. I think R-T-C needs to check all addressbooks of a particular type for the contact info before offering to save as a new contact. For example, in my configuration I have

main.inc.php - autocomplete_addressbooks = array('sql');
config.in.php - recipient_to_contact_addressbooks = array();

I'm also using the google_addressbook plugin, which creates a "Google Addressbook" addressbook, stored in mysql (contacts_google table) synced down from Google.

When sending an email to somone from the google contact list, R-C-T still pops up and offers to save the info. So it's not checking the google addressbook.

While I don't currently use ldap, I can definitely see a situation where some might be using more than one ldap directory. Would R-T-C check all of them ?

@mckaygerhard
Copy link
Contributor

due the get_address_sources function,and the "sql" parameter, only gets the internal, and not all, due the google addressbook are not sql either ldap types.. of course, the google plugin stored in sql, but not in the rc pai structure. i see that

due the contact are not in the build in sql contact list, it offers to save it! so from point of view of plguin and rc his are not a bug, this are a google or rc integration feature

so u must specify in the config file the addressboook and the plugin should work in both config files.. due the google plugin does not use the standars address book that by default uses the rc.. so manually specify should solve.-.-

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants