[#ONYX-156] Code review for ONYX-129

[ONYX-156] Code review for ONYX-129 Created: 09/Dec/08
Updated: 03/Aug/09 Due: 12/Jan/09 Resolved:
10/Dec/08
Status:
Project:
Component/s:
Affects
Version/s:
Fix Version/s:
Waiting for Test
Onyx
None
None
Type:
Reporter:
Resolution:
Labels:
Remaining
Estimate:
Time Spent:
Original
Estimate:
Code Review
Dennis Spathis
Fixed
None
Not Specified
1.0
Priority:
Assignee:
Votes:
Major
Dennis Spathis
0
Not Specified
Not Specified
Description
Code to review:






IParticipantReader.java
DefaultParticipantExcelReader.java
ExcelReaderSupport.java
onyx-core/src/main/resources/META-INF/spring/onyx-core/beans.xml
onyx-core/src/main/resources/META-INF/participant-attributes.xml
cartagene-webapp/src/main/webapp/WEB-INF/config/participant-attributes.xml
Comments
Comment by Alice Carey [ 09/Dec/08 ]
DefaultParticipantExcelReader
methodes processAppointment et setParticipantEssentialAttributes: beaucoup de code répétitif.
On pourrait créer une fonction qui prendrait comme param l'attribute name (constante),
effectuerait les différentes manipulations et retournerait le dataValue
ExcelReaderSupport
méthode getDateValue, getNumericValue, getTextValue et containsWhitespace: switchs avec
un seul case: ce n'est peut-être pas nécessaire de faire un switch dasn ces cas.
Comment by Jenkins Continuous Integration Server [ 10/Dec/08 ]
Integrated in Onyx #1034 (See http://132.204.211.183:8080/hudson/job/Onyx/1034/)
Generated at Fri May 26 13:41:12 EDT 2017 using JIRA 7.1.9#71013sha1:1aa0586d78efb7ed8b3de5c648d4b24912070b66.