What does MAPS_DEBUG=1 gnome-maps say?

I am interested in a log in the code that does:
Utils.debug('Adding contact address: ' + p.name);

(In reply to Jonas Danielsson from comment #1)
> What does MAPS_DEBUG=1 gnome-maps say?
> 
> I am interested in a log in the code that does:
> Utils.debug('Adding contact address: ' + p.name);

They contain the "(1)" as well. Contacts or e-d-s bug?

(In reply to Bastien Nocera from comment #2)
> (In reply to Jonas Danielsson from comment #1)
> > What does MAPS_DEBUG=1 gnome-maps say?
> > 
> > I am interested in a log in the code that does:
> > Utils.debug('Adding contact address: ' + p.name);
> 
> They contain the "(1)" as well. Contacts or e-d-s bug?

No, I think this one is all mine. The reason for the parenthesis is to differentiate between addresses. Since a contact can have more than one. For instance work and home.

So the (home) one I can understand. I have never seen the "(1)" tho. That might be new from e-d-s. Hmm.

It should basically be the field name from Contacts. See: https://blogs.gnome.org/alexl/files/2011/08/robohash-contacts.png

So maybe a fix is do not have the parenthesis if there is only one (1) address? Would that make sense to you? And then have like (Home) and (Work) sort of stuff?

But I still need to investigate where that (1) comes from.

(In reply to Jonas Danielsson from comment #5)
> So maybe a fix is do not have the parenthesis if there is only one (1)
> address? Would that make sense to you? And then have like (Home) and (Work)
> sort of stuff?

That would work for me.

> But I still need to investigate where that (1) comes from.

My contacts are in Google Contacts.

Created attachment 313315 [details] [review]
maps-contact-store: Do not add address type if single address

Works for me!

Attachment 313315 [details] pushed as af0e5e3 - maps-contact-store: Do not add address type if single address