Re: BBDB failed to handle what it considers duplicates

2012-10-13 Thread Leo
On 2012-10-14 00:46 +0800, Roland Winkler wrote:
> See bbdb-allow-duplicates.
>
> I guess it is fair to say that this variable was introduced long ago
> not as a bug, but as a feature. Depending on circumstances, it could
> be more likely that duplicate entries inadvertently refer to the
> same person.
>
> Also, the docstring of this variable expresses the fact...

Sure. So bbdb-no-duplicates was renamed to bbdb-allow-duplicates while
keeping the same value which amounts to disallow duplicates by default
and that is exactly what caused me all the trouble.

[snipped 12 lines]
> ...that so far in BBDB's history nobody considered the case of
> records with duplicate names important enough to extend the code of
> bbdb-complete-mail to handle such records properly. If this is
> urgent for you, you might want to look into this function yourself.
>
> Actually, I am not sure this requires a UID field. Instead, it may
> be sufficient that bbdb-complete-mail does not immediately exit one
> of its loops if it found one record matching a name. Instead it
> needs to continue looking for matching records.
>
> Note that BBDB's hash table associates elements in, say, the
> organization field with multiple records. In the same way it *can*
> associate a name with multiple records. Merely, details are not yet
> spelled out in the code. See the warning in bbdb-allow-duplicates.
>
> Roland

In my view, bbdb-no-duplicates defaults to nil says a lot about people
caring duplicates.

Leo


--
Don't let slow site performance ruin your business. Deploy New Relic APM
Deploy New Relic app performance management and know exactly
what is happening inside your Ruby, Python, PHP, Java, and .NET app
Try New Relic at no cost today and get our sweet Data Nerd shirt too!
http://p.sf.net/sfu/newrelic-dev2dev
___
bbdb-info@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/bbdb-info
BBDB Home Page: http://bbdb.sourceforge.net/


Re: Address completion question

2012-10-13 Thread Roland Winkler
On Sat Oct 13 2012 Alan Schmitt wrote:
> "Roland Winkler"  writes:
> 
> > Which version of emacs are you using? bbdb-complete-mail uses some
> > code for popping up a completion window that only works with GNU
> > Emacs 24. I want to replace it with something that works also with
> > GNU Emacs 23. But this is still on my to-do list.
> 
> I'm on 24.2 (homebrew compiled, os x).

Then I am not sure what your problem is. I do not have 24.2. But
both 24.1 and the trunk work fine for me.

> When I try to complete and address and several match, after "tab" I get
> a window with all the possible matches. If I hit "tab" again I get the
> following message
> 
> # is not a live window
> 
> where ... is a number, and the window is closed. The only time I can
> complete an address is when there is a single match.

What is tab bound to for you? By default bbdb-complete-mail gets
bound to M-tab. Can you send a backtrace of when things go wrong
(plus a description of what you did to get it)?

> I have "(setq bbdb-complete-mail-allow-cycling t)" in my configuration.

That should not be relevant here.

Roland

--
Don't let slow site performance ruin your business. Deploy New Relic APM
Deploy New Relic app performance management and know exactly
what is happening inside your Ruby, Python, PHP, Java, and .NET app
Try New Relic at no cost today and get our sweet Data Nerd shirt too!
http://p.sf.net/sfu/newrelic-dev2dev
___
bbdb-info@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/bbdb-info
BBDB Home Page: http://bbdb.sourceforge.net/


Re: Address completion question

2012-10-13 Thread Alan Schmitt
"Roland Winkler"  writes:

> Which version of emacs are you using? bbdb-complete-mail uses some
> code for popping up a completion window that only works with GNU
> Emacs 24. I want to replace it with something that works also with
> GNU Emacs 23. But this is still on my to-do list.

I'm on 24.2 (homebrew compiled, os x).

Alan

--
Don't let slow site performance ruin your business. Deploy New Relic APM
Deploy New Relic app performance management and know exactly
what is happening inside your Ruby, Python, PHP, Java, and .NET app
Try New Relic at no cost today and get our sweet Data Nerd shirt too!
http://p.sf.net/sfu/newrelic-dev2dev
___
bbdb-info@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/bbdb-info
BBDB Home Page: http://bbdb.sourceforge.net/


Re: Address completion question

2012-10-13 Thread Roland Winkler
On Sat Oct 13 2012 Alan Schmitt wrote:
> When I try to complete and address and several match, after "tab" I get
> a window with all the possible matches. If I hit "tab" again I get the
> following message
> 
> # is not a live window
> 
> where ... is a number, and the window is closed. The only time I can
> complete an address is when there is a single match.
> 
> I have "(setq bbdb-complete-mail-allow-cycling t)" in my configuration.
> Am I doing something wrong or is this a bug?

Which version of emacs are you using? bbdb-complete-mail uses some
code for popping up a completion window that only works with GNU
Emacs 24. I want to replace it with something that works also with
GNU Emacs 23. But this is still on my to-do list.

Roland

--
Don't let slow site performance ruin your business. Deploy New Relic APM
Deploy New Relic app performance management and know exactly
what is happening inside your Ruby, Python, PHP, Java, and .NET app
Try New Relic at no cost today and get our sweet Data Nerd shirt too!
http://p.sf.net/sfu/newrelic-dev2dev
___
bbdb-info@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/bbdb-info
BBDB Home Page: http://bbdb.sourceforge.net/


Address completion question

2012-10-13 Thread Alan Schmitt
Hello,

When I try to complete and address and several match, after "tab" I get
a window with all the possible matches. If I hit "tab" again I get the
following message

# is not a live window

where ... is a number, and the window is closed. The only time I can
complete an address is when there is a single match.

I have "(setq bbdb-complete-mail-allow-cycling t)" in my configuration.
Am I doing something wrong or is this a bug?

Thanks,

Alan

--
Don't let slow site performance ruin your business. Deploy New Relic APM
Deploy New Relic app performance management and know exactly
what is happening inside your Ruby, Python, PHP, Java, and .NET app
Try New Relic at no cost today and get our sweet Data Nerd shirt too!
http://p.sf.net/sfu/newrelic-dev2dev
___
bbdb-info@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/bbdb-info
BBDB Home Page: http://bbdb.sourceforge.net/


Re: BBDB failed to handle what it considers duplicates

2012-10-13 Thread Roland Winkler
On Sun Oct 14 2012 Leo wrote:
> Yes, it is related. The way BBDB3 handles duplicates is pretty bad. Try
> to create two records with the same name. It throws an error in emacs
> -q. 

See bbdb-allow-duplicates.

I guess it is fair to say that this variable was introduced long ago
not as a bug, but as a feature. Depending on circumstances, it could
be more likely that duplicate entries inadvertently refer to the
same person.

Also, the docstring of this variable expresses the fact...

> On 2012-10-13 19:57 +0800, Roland Winkler wrote:
> > Can you please be more specific? Under what circumstances did BBDB
> > what, which now it does not do anymore?
> 
> Create a bbdb db with two duplicate records and each has an email.
> 
> Try completing emails in gnus or whatever mailer.
> 
> Type a substring that matches both emails and complete. Only one email
> address is completed. The other is ignored.

...that so far in BBDB's history nobody considered the case of
records with duplicate names important enough to extend the code of
bbdb-complete-mail to handle such records properly. If this is
urgent for you, you might want to look into this function yourself.

Actually, I am not sure this requires a UID field. Instead, it may
be sufficient that bbdb-complete-mail does not immediately exit one
of its loops if it found one record matching a name. Instead it
needs to continue looking for matching records.

Note that BBDB's hash table associates elements in, say, the
organization field with multiple records. In the same way it *can*
associate a name with multiple records. Merely, details are not yet
spelled out in the code. See the warning in bbdb-allow-duplicates.

Roland

--
Don't let slow site performance ruin your business. Deploy New Relic APM
Deploy New Relic app performance management and know exactly
what is happening inside your Ruby, Python, PHP, Java, and .NET app
Try New Relic at no cost today and get our sweet Data Nerd shirt too!
http://p.sf.net/sfu/newrelic-dev2dev
___
bbdb-info@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/bbdb-info
BBDB Home Page: http://bbdb.sourceforge.net/


Re: BBDB failed to handle what it considers duplicates

2012-10-13 Thread Leo
On 2012-10-13 19:57 +0800, Roland Winkler wrote:
> Can you please be more specific? Under what circumstances did BBDB
> what, which now it does not do anymore?

Create a bbdb db with two duplicate records and each has an email.

Try completing emails in gnus or whatever mailer.

Type a substring that matches both emails and complete. Only one email
address is completed. The other is ignored.

>> BTW, I personally think not having a UID field is a bad design from the
>> beginning. There is nothing in a record that can uniquely identify a
>> record.
>
> When does this matter? Please explain. BBDB has the features it
> needed for what people expected from it. Revising the "design
> from the beginning" is not so easy after 20 years.
>
> Also, is this related to BBDB's handling of duplicates? If not,
> please open a new thread. Otherwise things tend to get lost.

Yes, it is related. The way BBDB3 handles duplicates is pretty bad. Try
to create two records with the same name. It throws an error in emacs
-q. In reality, people with same name are not uncommon.

Leo


--
Don't let slow site performance ruin your business. Deploy New Relic APM
Deploy New Relic app performance management and know exactly
what is happening inside your Ruby, Python, PHP, Java, and .NET app
Try New Relic at no cost today and get our sweet Data Nerd shirt too!
http://p.sf.net/sfu/newrelic-dev2dev
___
bbdb-info@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/bbdb-info
BBDB Home Page: http://bbdb.sourceforge.net/


Re: Some small improvements

2012-10-13 Thread Roland Winkler
On Sat Oct 13 2012 Leo wrote:
> On 2012-09-11 09:21 +0800, Roland Winkler wrote:
> > Thanks. I need to check these more carefully
> 
> Any update on this?

I am sorry, I am rather busy with other things till end of the
month. I cannot do anything big with BBDB till then.

Roland

--
Don't let slow site performance ruin your business. Deploy New Relic APM
Deploy New Relic app performance management and know exactly
what is happening inside your Ruby, Python, PHP, Java, and .NET app
Try New Relic at no cost today and get our sweet Data Nerd shirt too!
http://p.sf.net/sfu/newrelic-dev2dev
___
bbdb-info@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/bbdb-info
BBDB Home Page: http://bbdb.sourceforge.net/


Re: BBDB failed to handle what it considers duplicates

2012-10-13 Thread Roland Winkler
On Sat Oct 13 2012 Leo wrote:
> I just noticed BBDB no longer completes some emails because they are
> part of a record that BBDB considers a duplicate.
> 
> This change is annoying and has caused me a lot of trouble sending
> emails to wrong recipients.

Can you please be more specific? Under what circumstances did BBDB
what, which now it does not do anymore?

> BTW, I personally think not having a UID field is a bad design from the
> beginning. There is nothing in a record that can uniquely identify a
> record.

When does this matter? Please explain. BBDB has the features it
needed for what people expected from it. Revising the "design
from the beginning" is not so easy after 20 years.

Also, is this related to BBDB's handling of duplicates? If not,
please open a new thread. Otherwise things tend to get lost.

Roland

--
Don't let slow site performance ruin your business. Deploy New Relic APM
Deploy New Relic app performance management and know exactly
what is happening inside your Ruby, Python, PHP, Java, and .NET app
Try New Relic at no cost today and get our sweet Data Nerd shirt too!
http://p.sf.net/sfu/newrelic-dev2dev
___
bbdb-info@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/bbdb-info
BBDB Home Page: http://bbdb.sourceforge.net/


Re: Some small improvements

2012-10-13 Thread Leo
On 2012-09-11 09:21 +0800, Roland Winkler wrote:
> Thanks. I need to check these more carefully

Any update on this?

Leo


--
Don't let slow site performance ruin your business. Deploy New Relic APM
Deploy New Relic app performance management and know exactly
what is happening inside your Ruby, Python, PHP, Java, and .NET app
Try New Relic at no cost today and get our sweet Data Nerd shirt too!
http://p.sf.net/sfu/newrelic-dev2dev
___
bbdb-info@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/bbdb-info
BBDB Home Page: http://bbdb.sourceforge.net/


BBDB failed to handle what it considers duplicates

2012-10-13 Thread Leo
Hi there,

I just noticed BBDB no longer completes some emails because they are
part of a record that BBDB considers a duplicate.

This change is annoying and has caused me a lot of trouble sending
emails to wrong recipients.

BTW, I personally think not having a UID field is a bad design from the
beginning. There is nothing in a record that can uniquely identify a
record.

Leo


--
Don't let slow site performance ruin your business. Deploy New Relic APM
Deploy New Relic app performance management and know exactly
what is happening inside your Ruby, Python, PHP, Java, and .NET app
Try New Relic at no cost today and get our sweet Data Nerd shirt too!
http://p.sf.net/sfu/newrelic-dev2dev
___
bbdb-info@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/bbdb-info
BBDB Home Page: http://bbdb.sourceforge.net/