Bug #599
closedneed to avoid conflict real Leginon sessions and instruments from appion image uploading ones
100%
Description
It will write to people's running session and change the microscope-camera pixel calibration for all Leginon user if it is input accidentally. This is too serious for me to put off for next release.
Updated by Anchi Cheng over 14 years ago
fix it by
1. Searching for unused session name from z to a instead of Leginon's default a to z.
2. Create and force Appion users to upload images from hostname appion.
Committed to branch myami-2.0 as r14216, r14218 and to trunk as r14209 by Neil on part one and r14218 on part two
I have added the same host and TEM, camera names in dbemdata on cronus4.
Test Case 1-New installation:
1. Go through installation and create a project.
2. Select in the project to upload images to a new session.
The uploadimage.php page should show session name starting from z and only have the host as appion, TEM default is AppionTEM and Camrea default is AppionCamera
Test Case 2-Upgrade:
1. Go through upgrade wizard
2. Select an existing project to upload images to a new session.
Should see the same as case one.
Updated by Anchi Cheng over 14 years ago
- Status changed from Assigned to In Code Review
- Assignee changed from Anchi Cheng to Eric Hou
- % Done changed from 0 to 100
Updated by Eric Hou over 14 years ago
code review is good.
I also test it on brand new database.
Updated by Eric Hou over 14 years ago
- Status changed from In Code Review to In Test
Updated by Eric Hou over 14 years ago
- Status changed from In Test to Closed
Also tested on upgrade.
Thanks.
Eric