Bug #1790
closedget_magnification on existing TEM corrupts the database
0%
Description
It adds a new TEM instrument without Cs value. Most likely the insertion did not query database first.
Updated by Anchi Cheng about 12 years ago
- Status changed from Assigned to In Code Review
fixed with r16600 and r16602. Tested on new database already. This requires update of the installation on the microscope.
Updated by Jim Pulokas about 12 years ago
Anchi,
I made a different approach to your fix in r16620, then realized it was a bad idea and reverted back in r16622. My confusion is whether this nested insert will ever create a new InstrumentData. If that is the case, then my alternate approach would not work, since it queries InstrumentData before the mag list is inserted. The advantage of my approach is that it will continue to work even if we add new attributes just like we added "cs".
Updated by Jim Pulokas about 12 years ago
- Status changed from In Code Review to Closed
works good enough, will open a new issue for some future enhancements.