Opened 15 years ago
Closed 15 years ago
#1399 closed defect (fixed)
Array slide's create multiple puts a wrong number into batch index field
Reported by: | base | Owned by: | Nicklas Nordborg |
---|---|---|---|
Priority: | minor | Milestone: | BASE 2.13.1 |
Component: | web | Version: | |
Keywords: | create multiple array slide batch index | Cc: |
Description
I noticed that when one uses Create multiple wizard on Array Slide list, created slides get weird values as batchIndex. According to documentation batchIndex should be the index of the array slide in selected array batch. However, the wizard puts there a database ID of an ArrayBatch the slide is connected to. These batch indexes are not given by the user in any place in the wizard, so it has to make up something (batchIndex is 'not null' in db). Maybe it would be better to put there a zero or 1 instead of putting a misleading database index? What was the reason for making batchIndex required attribute? Are there any array technologies that don't have unique enough barcodes?
Change History (4)
comment:1 by , 15 years ago
comment:2 by , 15 years ago
Milestone: | → BASE 2.13.1 |
---|
You are right. The array batch id should not be used as the batch index values. Instead it should use the value from the 'Start at' field in the first dialog in the wizard and then increment this value for each create array slide.
As a workaround, use the batch slide importer to batch update/create the array slides.
comment:3 by , 15 years ago
Owner: | changed from | to
---|---|
Status: | new → assigned |
comment:4 by , 15 years ago
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Problematic assignments are in file BASE_HOME/www/lims/arrayslide/index.jsp on lines 185 and 197 (BASE-2.12.1 sources)