Opened 10 years ago
Closed 10 years ago
#718 closed task (fixed)
MeLuDi v.1.0 bug fixes and improvements
Reported by: | olle | Owned by: | olle |
---|---|---|---|
Priority: | major | Milestone: | MeLuDi v1.1 |
Component: | net.sf.basedb.meludi | Keywords: | |
Cc: |
Description
This ticket covers bug fixes and improvements to MeLuDi v.1.0.
Change History (15)
comment:1 by , 10 years ago
Status: | new → assigned |
---|
comment:2 by , 10 years ago
Traceability note:
- MeLuDi v.1.0 was introduced in Ticket #690 (MeLuDi - Registration wizard for melanoma-lung cancer projects).
comment:3 by , 10 years ago
comment:4 by , 10 years ago
(In [3041]) Refs #718. Case summary input field hint and notification message updated to "Please enter a case id ("ML" + 5 digits)
", where double quotes have been escaped:
- JSP file
index.jsp
inresources/
updated in case summary input field hint. - Javascript file
index.js
inresources/
updated in functionindex.caseSummary()
to set new case summary input field notification message (displayed if field is empty when submitted). - JSP file
case_summary.jsp
inresources/reports/
updated in case summary input field hint. - Javascript file
case_summary.js
inresources/reports/
updated in functionindex.caseSummary()
to set new case summary input field notification message (displayed if field is empty when submitted).
comment:5 by , 10 years ago
Functional specification update:
- When searching for a free well (position) in a bioplate (storage box), the first empty well starting from the first column in the first row should be used, even if non-empty wells are found after it, e.g. gaps should be filled. Previously, it was required that all wells after the first free well was empty.
comment:6 by , 10 years ago
Design update to allow gaps in the pattern of used wells in a bioplate to be filled:
- Java class/file
SpecimenTubeServlet.java
insrc/net/sf/basedb/meludi/servlet/
updated:
a. Private methodBioWell getFirstFreeWell(BioPlate plate, Integer numberOfWells)
updated to only use argumentBioPlate plate
, and to return the first empty well starting from the first column in the first row, even if non-empty wells are found after it, e.g. gaps should be filled.
b. Private methodBioWell getNextFreeWell(BioWell well)
updated to return the next empty well after the input one, even if non-empty wells are found after it, e.g. gaps should be filled.
c. Private methodJSONArray findFreeWells(DbControl dc, Integer nofTubes, String storageBoxSuffix)
updated in call of private methodBioWell getFirstFreeWell(...)
to only use a single argumentBioPlate plate
.
comment:7 by , 10 years ago
(In [3042]) Refs #718. MeLuDi updated to allow gaps in the pattern of used wells in a bioplate to be filled:
- Java class/file
SpecimenTubeServlet.java
insrc/net/sf/basedb/meludi/servlet/
updated:
a. Private methodBioWell getFirstFreeWell(BioPlate plate, Integer numberOfWells)
updated to only use argumentBioPlate plate
, and to return the first empty well starting from the first column in the first row, even if non-empty wells are found after it, e.g. gaps should be filled.
b. Private methodBioWell getNextFreeWell(BioWell well)
updated to return the next empty well after the input one, even if non-empty wells are found after it, e.g. gaps should be filled.
c. Private methodJSONArray findFreeWells(DbControl dc, Integer nofTubes, String storageBoxSuffix)
updated in call of private methodBioWell getFirstFreeWell(...)
to only use a single argumentBioPlate plate
.
comment:8 by , 10 years ago
comment:9 by , 10 years ago
(In [3044]) Refs #718. Currently unused subtype items have been removed or commented out:
- Java data access object classes/files
BioplateType.java
,Datafiletype.java
,Rna.java
, andSubtype.java
insrc/net/sf/basedb/meludi/dao/
updated by removal or commenting out of unused items. - Java servlet class/file
InstallServlet.java
insrc/net/sf/basedb/meludi/servlet/
updated in protected methodvoid doGet(HttpServletRequest req, HttpServletResponse resp)
to exclude unused items in subtype checks forSubtype.DNA
andSubtype.RNA
.
comment:10 by , 10 years ago
(In [3046]) Refs #718. Java class/file ReportTableUtil.java
and Java servlet class/file SampleReportServlet.java
in src/net/sf/basedb/meludi/servlet/
updated in private convenience method Site fetchSite(DbControl dc, SnapshotManager manager, ItemSubtype subtypeCase, ItemSubtype subtypeSpecimen, Sample s)
to check for null
values for sample and case items before using them. If either item is null
, Site.UNKNOWN
is returned.
comment:11 by , 10 years ago
(In [3047]) Refs #718. Java servlet classes/files CaseSummaryServlet.java
, PersonalRegistrationServlet.java
, and SampleReportServlet.java
in src/net/sf/basedb/meludi/servlet/
updated by removal of debug output, that couldn't be configured through configuration files. Also some removal of unused items.
comment:12 by , 10 years ago
Design update:
- Annotation type
AnnotationType.FLAG
should be removed, since it is currently not used. RNA items are not prepared for the annotation at installation, and the program therefore crashes when loading RNA annotations. - Annotation type
AnnotationType.BIOPSY_TYPE
should be removed, since it is currently not used.
comment:13 by , 10 years ago
(In [3050]) Refs #718. MeLuDi updated by removal of annotation types AnnotationType.BIOPSY_TYPE
and AnnotationType.FLAG
:
- Java data access object class/file
Annotationtype.java
insrc/net/sf/basedb/meludi/dao/
updated by commenting out definitions of annotation typesBIOPSY_TYPE
andFLAG
. - Java servlet
CaseSummaryServlet.java
insrc/net/sf/basedb/meludi/servlet/
updated in private methodvoid loadRnaInfo(DbControl dc, Rna rna)
by commenting out command loading value forAnnotationType.FLAG
. - Java servlet
InstallServlet.java
insrc/net/sf/basedb/meludi/servlet/
updated in protected methodvoid doGet(HttpServletRequest req, HttpServletResponse resp)
to not includingAnnotationType.BIOPSY_TYPE
in list of used annotation types for specimen items.
comment:14 by , 10 years ago
(In [3053]) Refs #718. Updates due to version number change to "1.1
".
- Version number change to "
1.1
":
a. Outermost Ant XML build filebuild.xml
in/
updated.
b. Extensions XML configuration fileextensions.xml
inMETA-INF/
updated.
c. Javascript filemeludi-2.js
inresources/
updated.
d. Javascript filesamplereportgenerator.js
inresources/reports/
updated.
e. Java class/fileMeludi.java
insrc/net/sf/basedb/meludi/
updated.
comment:15 by , 10 years ago
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Ticket closed as MeLuDi version 1.1 has been created. Its released was prompted by bug fix needed for cases containing RNA extracts.
Ticket accepted.