Bug #1025
closedappion params['session'] should be replaced by ['sessionname']
0%
r15037 allows either session and sessionname to be assigned in appionScript.py
Description
The latter is more specific. We have a mixture of these. It seems all upload scripts use 'session' as derived from appionScript. Those derived from appionLoop use 'sessionname'.
Updated by Dmitry Lyumkis over 13 years ago
- Status changed from Assigned to In Code Review
- Assignee changed from Dmitry Lyumkis to Anchi Cheng
there are a ton of revisions associated with this bug:
I went through and manually changed self.params['session'] to self.params['sessionname'], along with minor associated revisions (e.g. documentation, or "is" to "in" error check in uploadImages.py)
Updated by Anchi Cheng over 13 years ago
r15204, r15206, r15207 removed all cases of params['session'] usage in appionlib.
r15207 removed the depreciated getAllImages since it is not used any more and contains the params['session'] assignment not used anywhere.
Updated by Anchi Cheng about 12 years ago
- Status changed from In Code Review to In Test
- Assignee changed from Anchi Cheng to Amber Herold
This can be considered tested if all AppionScripts are tested because it affects every one of them. You can close this when we are done with the general testing.
Updated by Amber Herold about 10 years ago
- Status changed from In Test to Closed