Opened 8 years ago

Closed 8 years ago

#910 closed defect (fixed)

The Mark extracts for retraction wizard doesn't find all items in some cases

Reported by: Nicklas Nordborg Owned by: Nicklas Nordborg
Priority: major Milestone: Reggie v4.7
Component: net.sf.basedb.reggie Keywords:
Cc:

Description

If case has multiple child specimen items with different SCAN-B ID, only the items with the same SCAN-B ID as the case are listed in the wizard. The generated protocols for destruction of items in the freezer are also missing the items.

Change History (18)

comment:1 by Nicklas Nordborg, 8 years ago

For example, try the cases linked to 'PAT000056' in the test database.

Last edited 8 years ago by Nicklas Nordborg (previous) (diff)

comment:2 by Nicklas Nordborg, 8 years ago

Status: newassigned

comment:3 by Nicklas Nordborg, 8 years ago

(In [4102]) References #910: The Mark extracts for retraction wizard doesn't find all items in some cases

Cleaned up some code:

  • Site information is loaded per case
  • List with case names is generated from the list with case items instead of having special JSON properties just for this (which only worked for 2 cases).

comment:4 by Nicklas Nordborg, 8 years ago

(In [4103]) References #910: The Mark extracts for retraction wizard doesn't find all items in some cases

More cleanup:

  • Display error immediately if no patient is found, since the registration will not work otherwise
  • Simplified loading of cases and blood now that we are sure to have a patient
  • Found another list with case names (patient.allCases). Use this list instead of re-creating a new one for displaying the "All cases" value.

comment:5 by Nicklas Nordborg, 8 years ago

(In [4104]) References #910: The Mark extracts for retraction wizard doesn't find all items in some cases

Load all child items by following links (uses the existing functionality for doing so). Case and Blood items in the returned JSON data is now populated from this information. Other information is still using the old way to load child items.

comment:6 by Nicklas Nordborg, 8 years ago

(In [4105]) References #910: The Mark extracts for retraction wizard doesn't find all items in some cases

Specimen and NoSpecimen item information updated to use the new child loading.

comment:7 by Nicklas Nordborg, 8 years ago

(In [4106]) References #910: The Mark extracts for retraction wizard doesn't find all items in some cases

Same change for Lysate, RNA, DNA and FlowThrough.

comment:8 by Nicklas Nordborg, 8 years ago

(In [4107]) References #910: The Mark extracts for retraction wizard doesn't find all items in some cases

Same change for RNAQC. Needed some changes in the javascript as well since the RNAQC items are now top-level in the JSON data instead of a separate array below each RNA item.

comment:9 by Nicklas Nordborg, 8 years ago

(In [4108]) References #910: The Mark extracts for retraction wizard doesn't find all items in some cases

Same change for Blood DNA, mRNA and cDNA.

comment:10 by Nicklas Nordborg, 8 years ago

(In [4109]) References #910: The Mark extracts for retraction wizard doesn't find all items in some cases

Same change for Histology and Stained items.

comment:11 by Nicklas Nordborg, 8 years ago

(In [4110]) References #910: The Mark extracts for retraction wizard doesn't find all items in some cases

Same change for Library items.

comment:12 by Nicklas Nordborg, 8 years ago

(In [4111]) References #910: The Mark extracts for retraction wizard doesn't find all items in some cases

Removed some of the unused code.

comment:13 by Nicklas Nordborg, 8 years ago

(In [4112]) References #910: The Mark extracts for retraction wizard doesn't find all items in some cases

Pooled libraries and sequencing runs are now loaded from the actual libraries instead of matching against case name. All items displayed in the table should now be those that descend from the top-level patient record.

comment:14 by Nicklas Nordborg, 8 years ago

(In [4113]) References #910: The Mark extracts for retraction wizard doesn't find all items in some cases

Removing the "Preview" and "Download" buttons and all code that is associated with this functionality.

comment:15 by Nicklas Nordborg, 8 years ago

(In [4114]) References #910: The Mark extracts for retraction wizard doesn't find all items in some cases

The "Cases" field in the lab protocol now uses the correct list with cases names. Storage box coordinate is displayed in a separate column instead of mixed with the freezer location.

comment:16 by Nicklas Nordborg, 8 years ago

(In [4115]) References #910: The Mark extracts for retraction wizard doesn't find all items in some cases

Turned out that pooled libraries are loaded as child items so there is no need for special handling.

comment:17 by Nicklas Nordborg, 8 years ago

(In [4116]) References #910: The Mark extracts for retraction wizard doesn't find all items in some cases

Fixed the display of "No case information has been registered" (and other similar messages) so that they use the whole table width.

Removed some unused style declarations.

comment:18 by Nicklas Nordborg, 8 years ago

Resolution: fixed
Status: assignedclosed
Note: See TracTickets for help on using tickets.