Fixed crash on assigning shapes second time
PeakCollection::setShapeCollection
transfers ownership of the collection from the GUI element to the PeakCollection
, so the pointer is no longer valid after the "Assign peak shapes" button is clicked once.