Changeset 3516

Timestamp:

Oct 1, 2015, 12:55:22 PM (8 years ago)

Author:

olle

Message:

Refs #815. Bug fix: The wizard "Update processed start lists with lot numbers from extracts" in the "Special conversion tools" sub-section of the "Admin" section should update processed start item lists created before MeLuDI v1.3.2 with lot number info taken from the first DNA extract associated with the list. However, no check was performed that the DNA was extracted from a MeLuDI specimen, instead of being an extract source item. The latter items have no extraction and QIAcube lot number information, since they were not extracted in the MeLuDI lab. The routine is therefore updated to use the first DNA extract, that is not an extract source item:

1. Java servlet class/file ExtractionServlet.java in src/net/sf/basedb/meludi/servlet/ updated in protected method void doPost(HttpServletRequest req, HttpServletResponse resp) for command "UpdateStartListsWithLotNumbersFromExtracts" to use the first DNA extract, that is not an extract source item, when copying lot number information to the start item list.

File:

• extensions/net.sf.basedb.meludi/trunk/src/net/sf/basedb/meludi/servlet/ExtractionServlet.java (modified) (1 diff)

extensions/net.sf.basedb.meludi/trunk/src/net/sf/basedb/meludi/servlet/ExtractionServlet.java

@@ -2915 +2915 @@
               if (extractList != null)
               {
-                // Get first extract member item
-                Extract e = (Extract)extractList.get(0);
+                // Get first extract member item that was processed from specimen
+                Extract e = null;
+                for (Extract extract: extractList)
+                {
+                  // Exclude input DNA and RNA, since these lack lot number annotations
+                  if (extract != null && !Meludi.itemIsExtractSourceItem(extract.getName()))
+                  {
+                    e = extract;
+                    break;
+                  }
+                }
                 if (e != null)
                 {