Issue 705: Too much audit logging for Refine > Accept Peptides

issues
Status:open
Assigned To:Rita Chupalov
Type:Defect
Area:Skyline
Priority:3
Milestone:4.3
Opened:2019-12-14 20:30 by Brendan MacLean
Changed:2020-08-10 14:19 by Nick Shulman
Resolved:
Resolution:
Closed:
2019-12-14 20:30 Brendan MacLean
Title»Too much audit logging for Refine > Accept Peptides
Assigned ToGuest»Nick Shulman
Type»Defect
Area»Skyline
Priority»3
When you use Refine > Accept Peptides you get an audit log detail record for every peptide and when you click the extra info link for the operation, you get every peptide listed twice, once as item in a list and once in a plain text block of additional information.

This operation should yield just one audit log record (or maybe extra records for the checkboxes in the form) and when you click the extra info link you should get just the paint text block, like you get for a paste operation.

2020-08-10 14:19 Nick Shulman
Assigned ToNick Shulman»ritach
Do you want to take this one?
I think the right fix is probably to remove the call to "entry.Merge(base.CreateEntry(docPair))" in RefineListSettings.CreateEntry.