Ubuntu One Servers

Contacts with empty mergeable lists errors in different places

Bug #546833 reported by James Tait
16
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ubuntu One Servers
Triaged
Low
Ubuntu One Client Engineering team

Bug Description

I have four hastily-filled dummy contacts set up:

  fff has a work phone number and a work e-mail
  gbhdfg has a work phone number and a work e-mail
  ggg has a home address, a work address, a work e-mail, an "other" IM address, a home URL and a work URL (no phone numbers)
  sfsgfds has just a work e-mail

The following merge scenarios produced these results:

  fff + gbhdfg = successful preview
  fff + ggg = OOPS-ID-1545appserver53782
  fff + sfsgfds = successful preview
  gbhdfg + ggg = OOPS-ID-1545appserver53819
  gbhdfg + sfsgfds = successful preview
  ggg + sfsgfds = successful preview

The phone numbers are not necessarily valid phone numbers, but are numeric.

James Tait (jamestait)
visibility: private → public
Revision history for this message
Martin Albisetti (beuno) wrote :

For reference: https://pastebin.canonical.com/29646/

I'll fix it!

Changed in ubuntuone-servers:
status: New → Triaged
importance: Undecided → Medium
assignee: nobody → Martin Albisetti (beuno)
milestone: none → funambol-public-alpha
status: Triaged → In Progress
Revision history for this message
Martin Albisetti (beuno) wrote :

And: http://paste.ubuntu.com/400868/

Martin Albisetti (beuno)
tags: added: desktop+ phone-sync u1-lucid
Revision history for this message
Rodrigo Moya (rodrigo-moya) wrote :

Where did you create those contacts?

Revision history for this message
Martin Albisetti (beuno) wrote :

12:09 < thisfred> aquarius: we do allow empty mergeable lists, from now on, and we change the code
                  that infers the type from the json to check for uuid-like keys *OR* an "_order"
                  key (so we will recognize all current instances, and don't need data migration)
12:10 < thisfred> aquarius: for an empty mergeable list (a feature wanted by many client
                  developers, and by beuno for the merge) we have the following dict: {"_order":
                  []}

summary: - OOPS when merging contacts
+ Contacts with empty mergeable lists errors in different places
Changed in ubuntuone-servers:
importance: Medium → Low
status: In Progress → Triaged
tags: removed: u1-lucid
Revision history for this message
Martin Albisetti (beuno) wrote :

Since these contacts where generated outside of our code or evolution, bumping it down to low.

Changed in ubuntuone-servers:
assignee: Martin Albisetti (beuno) → Ubuntu One Desktop+ team (ubuntuone-desktop+)
Revision history for this message
Martin Albisetti (beuno) wrote :

This was a manually generated contact. We should be more resilient to this data, but it's not a bug we produced.

Changed in ubuntuone-servers:
milestone: funambol-public-alpha → none
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.