[#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:
Waiting for Test
Project:
Onyx
Component/s:
None
Affects
Version/s:
None
Fix Version/s:
1.0
Type:
Code Review
Priority:
Major
Reporter:
Dennis Spathis
Assignee:
Dennis Spathis
Resolution:
Fixed
Votes:
0
Labels:
None
Remaining
Estimate:
Not Specified
Time Spent:
Not Specified
Original
Estimate:
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#71013-
sha1:1aa0586d78efb7ed8b3de5c648d4b24912070b66.