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 base, 15 years ago

Problematic assignments are in file BASE_HOME/www/lims/arrayslide/index.jsp on lines 185 and 197 (BASE-2.12.1 sources)

comment:2 by Nicklas Nordborg, 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 Nicklas Nordborg, 15 years ago

Owner: changed from everyone to Nicklas Nordborg
Status: newassigned

comment:4 by Nicklas Nordborg, 15 years ago

Resolution: fixed
Status: assignedclosed

(In [5125]) Fixes #1399: Array slide's create multiple puts a wrong number into batch index field

Note: See TracTickets for help on using tickets.