Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 370277 - ATS Commit Manager shows two entries for some (or all) commits.
Summary: ATS Commit Manager shows two entries for some (or all) commits.
Status: CLOSED FIXED
Alias: None
Product: OSEE
Classification: Technology
Component: Action Tracking System (ATS) (show other bugs)
Version: 0.9.9   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 0.9.9   Edit
Assignee: S. Cook CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-01-31 16:00 EST by S. Cook CLA
Modified: 2012-02-04 21:51 EST (History)
1 user (show)

See Also:

Attachments
fixup! ATS needs to show all branches committed to regardless of parallel configuration (6.71 KB, patch)
2012-01-31 16:01 EST, S. Cook CLA 		ryan.d.brooks: iplog+
Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description S. Cook CLA 2012-01-31 16:00:10 EST 
This action was completed with errors: bug[ats_A2QNG]: ATS needs to show all branches committed to regardless of parallel configuration

Now the Commit Manager shows parallel branch commits, but shows too many.
Comment 1 S. Cook CLA 2012-01-31 16:01:36 EST 
Created attachment 210336 [details]
fixup! ATS needs to show all branches committed to regardless of parallel configuration

For-loop in AtsBranchManagerCore was not properly looping through and comparing branches.  (see commit).

AtsBranchManagerCore and AtsBranchManagerCoreTest were both updated to properly test for this problem.

This was committed as a fixup! for the previous commit which had a bug in it.
Comment 2 S. Cook CLA 2012-01-31 16:01:55 EST 
For-loop in AtsBranchManagerCore was not properly looping through and comparing branches.  (see commit).

AtsBranchManagerCore and AtsBranchManagerCoreTest were both updated to properly test for this problem.

This was committed as a fixup! for the previous commit which had a bug in it.
Comment 3 S. Cook CLA 2012-02-02 14:26:40 EST 
K. Wilk pushed this patch 02/02/2012.