[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.