Profile merge overwrites company affiliation to the old one

Bug #1484180 reported by Ilya Shakhat
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Stackalytics
Fix Released
High
Ilya Shakhat

Bug Description

The case is following:
1. There is an existing user profile:
   user_id -> company: Uno
   email -> company: Uno
2. At some moment user does contribution from another_email, which is mapped to the profile automatically, becoming a record in the storage:
   another_email -> company: Uno
3. User updates default_data and sets a new company:
  user_id -> company: Uno, Duo
  email -> company: Uno, Duo
(note that another_email is not listed)

The issue occurs when record attributed with another_email is processed. Stackalytics merges profiles, old (see #2) and new (see #3). List of companies is taken from the first, thus overwriting the good profile.

Ilya Shakhat (shakhat)
description: updated
Changed in stackalytics:
status: New → Triaged
assignee: nobody → Ilya Shakhat (shakhat)
importance: Undecided → High
milestone: none → 0.9
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to stackalytics (master)

Fix proposed to branch: master
Review: https://review.openstack.org/213754

Changed in stackalytics:
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to stackalytics (master)

Reviewed: https://review.openstack.org/213754
Committed: https://git.openstack.org/cgit/stackforge/stackalytics/commit/?id=09e359b643906692fca5de0c1726d8cf5436ad92
Submitter: Jenkins
Branch: master

commit 09e359b643906692fca5de0c1726d8cf5436ad92
Author: Ilya Shakhat <email address hidden>
Date: Mon Aug 17 18:24:19 2015 +0300

    Fix merge between existing and user-defined user profiles

    Change-Id: I0cfe2421cb8b2a0e87cdb0f3b3b3873edcbd8eb5
    Closes-Bug: #1484180

Changed in stackalytics:
status: In Progress → Fix Committed
Ilya Shakhat (shakhat)
Changed in stackalytics:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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