Author: chatellier Date: 2010-11-24 10:06:13 +0000 (Wed, 24 Nov 2010) New Revision: 255 Log: Ajout de plusieurs lignes pour une erreur. Selection des lignes en doublon Modified: trunk/coser-business/src/main/java/fr/ifremer/coser/control/ValidationError.java trunk/coser-business/src/main/java/fr/ifremer/coser/services/ProjectService.java trunk/coser-business/src/main/java/fr/ifremer/coser/services/ValidationService.java trunk/coser-business/src/main/resources/i18n/coser-business_en_GB.properties trunk/coser-business/src/main/resources/i18n/coser-business_fr_FR.properties trunk/coser-ui/src/main/java/fr/ifremer/coser/ui/control/ControlHandler.java trunk/coser-ui/src/main/resources/i18n/coser-ui_en_GB.properties trunk/coser-ui/src/main/resources/i18n/coser-ui_fr_FR.properties Modified: trunk/coser-business/src/main/java/fr/ifremer/coser/control/ValidationError.java =================================================================== --- trunk/coser-business/src/main/java/fr/ifremer/coser/control/ValidationError.java 2010-11-23 13:09:58 UTC (rev 254) +++ trunk/coser-business/src/main/java/fr/ifremer/coser/control/ValidationError.java 2010-11-24 10:06:13 UTC (rev 255) @@ -26,6 +26,8 @@ package fr.ifremer.coser.control; import java.io.Serializable; +import java.util.HashSet; +import java.util.Set; import fr.ifremer.coser.CoserConstants.Category; import fr.ifremer.coser.CoserConstants.ValidationLevel; @@ -59,14 +61,16 @@ protected String detailMessage; /** - * Numero de la ligne dans le fichier CSV. + * Numero des lignes dans le fichier CSV. * - * Peut être {@code null} si l'erreur n'est pas associé à une ligne specifiques. + * Peut être vide si l'erreur n'est pas associé à une ligne specifiques. */ - protected String lineNumber; + protected Set<String> lineNumbers; public ValidationError() { super(); + + lineNumbers = new HashSet<String>(); } /** @@ -150,30 +154,30 @@ } /** - * Numero de la ligne dans le fichier CSV. + * Numero des lignes dans le fichier CSV. * - * Peut être {@code null} si l'erreur n'est pas associé a une ligne specifiques. + * Peut être vide si l'erreur n'est pas associé a une ligne specifiques. * * @return line number */ - public String getLineNumber() { - return lineNumber; + public Set<String> getLineNumbers() { + return lineNumbers; } /** - * Numero de la ligne dans le fichier CSV. + * Numero des lignes dans le fichier CSV. * - * Peut être {@code null} si l'erreur n'est pas associé a une ligne specifiques. + * Peut être vide si l'erreur n'est pas associé a une ligne specifiques. * * @param lineNumber line number */ - public void setLineNumber(String lineNumber) { - this.lineNumber = lineNumber; + public void addLineNumber(String lineNumber) { + lineNumbers.add(lineNumber); } @Override public String toString() { return "ValidationError [level=" + level + ", message=" + message - + ", lineNumber=" + lineNumber + "]"; + + ", lineNumber=" + lineNumbers + "]"; } } Modified: trunk/coser-business/src/main/java/fr/ifremer/coser/services/ProjectService.java =================================================================== --- trunk/coser-business/src/main/java/fr/ifremer/coser/services/ProjectService.java 2010-11-23 13:09:58 UTC (rev 254) +++ trunk/coser-business/src/main/java/fr/ifremer/coser/services/ProjectService.java 2010-11-24 10:06:13 UTC (rev 255) @@ -1482,7 +1482,6 @@ String[] tuple = itTuple.next(); // Campagne;Annee;Trait;Mois;Strate;SurfaceBalayee;Lat;Long;ProfMoy - String annee = tuple[Haul.INDEX_YEAR]; String strataName = tuple[Haul.INDEX_STRATUM]; if (strataNames.contains(strataName)) { String traitsName = tuple[Haul.INDEX_HAUL]; Modified: trunk/coser-business/src/main/java/fr/ifremer/coser/services/ValidationService.java =================================================================== --- trunk/coser-business/src/main/java/fr/ifremer/coser/services/ValidationService.java 2010-11-23 13:09:58 UTC (rev 254) +++ trunk/coser-business/src/main/java/fr/ifremer/coser/services/ValidationService.java 2010-11-24 10:06:13 UTC (rev 255) @@ -148,7 +148,7 @@ error.setLevel(validationLevel); // add error line - error.setLineNumber(bean.getLine()); + error.addLineNumber(bean.getLine()); result.add(error); } @@ -162,7 +162,7 @@ error.setLevel(validationLevel); // add error line - error.setLineNumber(bean.getLine()); + error.addLineNumber(bean.getLine()); result.add(error); } @@ -298,7 +298,7 @@ progress.setTotal(total); } - Set<String> uniqueDataKeys = new HashSet<String>(); + Map<String, String> uniqueDataKeys = new HashMap<String, String>(); List<ValidationError> validationErrors = new ArrayList<ValidationError>(); Iterator<String[]> itDataToCheck = dataToCheck.iterator(); int lineIndex = 1; // 1 = skip header @@ -339,17 +339,20 @@ } // check for duplicated lines + String lineNumber = line[AbstractDataEntity.INDEX_LINE]; String uniqueDataKey = getSignificantData(category, line); - if (uniqueDataKeys.contains(uniqueDataKey)) { + if (uniqueDataKeys.containsKey(uniqueDataKey)) { ValidationError error = new ValidationError(); error.setCategory(category); error.setLevel(ValidationLevel.ERROR); - error.setLineNumber(line[AbstractDataEntity.INDEX_LINE]); - error.setMessage(_("coser.business.control.error.duplicatedLine")); + error.addLineNumber(uniqueDataKeys.get(uniqueDataKey)); + error.addLineNumber(lineNumber); + error.setMessage(_("coser.business.control.error.duplicatedLine", uniqueDataKey)); + error.setMessage(_("coser.business.control.error.duplicatedLineDetails", uniqueDataKey)); validationErrors.add(error); } else { - uniqueDataKeys.add(uniqueDataKey); + uniqueDataKeys.put(uniqueDataKey, lineNumber); } } @@ -529,7 +532,7 @@ ValidationError error = new ValidationError(); error.setCategory(Category.CATCH); error.setLevel(ValidationLevel.WARNING); - error.setLineNumber(lineNumber); + error.addLineNumber(lineNumber); error.setMessage(_("coser.business.control.error.minObservationCount")); error.setDetailMessage(_("coser.business.control.error.minObservationCountDetail", key, value)); validationErrors.add(error); @@ -749,7 +752,7 @@ ValidationError error = new ValidationError(); error.setCategory(Category.LENGTH); error.setLevel(ValidationLevel.WARNING); - error.setLineNumber(lineNumber); + error.addLineNumber(lineNumber); error.setMessage(_("coser.business.control.error.minObservationCount")); error.setDetailMessage(_("coser.business.control.error.minObservationCountDetail", key, value)); validationErrors.add(error); @@ -766,7 +769,6 @@ */ protected List<ValidationError> validateCategorySpecificHaul(Control control, ProgressMonitor progress) { - // TODO Auto-generated method stub return null; } @@ -840,7 +842,7 @@ if (!refTaxSpecies.contains(catchData[Catch.INDEX_SPECIES])) { ValidationError error = new ValidationError(); error.setCategory(Category.CATCH); - error.setLineNumber(catchData[Catch.INDEX_LINE]); + error.addLineNumber(catchData[Catch.INDEX_LINE]); error.setLevel(ValidationLevel.ERROR); error.setMessage(_("coser.business.control.error.nonExistantSpecies")); error.setDetailMessage(_("coser.business.control.error.nonExistantSpeciesDetail", catchData[Catch.INDEX_SPECIES])); @@ -867,7 +869,7 @@ if (!refTaxSpecies.contains(lengthData[Length.INDEX_SPECIES])) { ValidationError error = new ValidationError(); error.setCategory(Category.LENGTH); - error.setLineNumber(lengthData[Length.INDEX_LINE]); + error.addLineNumber(lengthData[Length.INDEX_LINE]); error.setLevel(ValidationLevel.ERROR); error.setMessage(_("coser.business.control.error.nonExistantSpecies")); error.setDetailMessage(_("coser.business.control.error.nonExistantSpeciesDetail", lengthData[Length.INDEX_SPECIES])); Modified: trunk/coser-business/src/main/resources/i18n/coser-business_en_GB.properties =================================================================== --- trunk/coser-business/src/main/resources/i18n/coser-business_en_GB.properties 2010-11-23 13:09:58 UTC (rev 254) +++ trunk/coser-business/src/main/resources/i18n/coser-business_en_GB.properties 2010-11-24 10:06:13 UTC (rev 255) @@ -37,6 +37,7 @@ coser.business.control.error.diffCatchLength= coser.business.control.error.diffCatchLengthDetail= coser.business.control.error.duplicatedLine= +coser.business.control.error.duplicatedLineDetails= coser.business.control.error.incompleteHaulDataCatch= coser.business.control.error.incompleteHaulDataLength= coser.business.control.error.incompleteHaulSpeciesData= @@ -54,7 +55,6 @@ coser.business.control.step.xworks= coser.business.line=Line coser.business.result.rsufiResultAlreadyExists= -coser.business.selection.noControlForSelection= coser.business.selection.notValidatedControl= coser.config.control.diffcatchlength.description= coser.config.database.directory.description= @@ -64,15 +64,11 @@ coser.config.reference.typeEspeces.description= coser.config.validator.directory.description= lat\ attribute\ is\ not\ a\ valid\ double= -lat\ must\ contains\ at\ least\ 3\ decimal= lat\ must\ contains\ at\ least\ 3\ decimals= -lat\ must\ contains\ at\ least\ 3\ decimasl= length\ attribute\ is\ not\ a\ valid\ double= long\ attribute\ is\ not\ a\ valid\ double= -long\ must\ contains\ at\ least\ 3\ decimal= long\ must\ contains\ at\ least\ 3\ decimals= number\ attribute\ is\ not\ a\ valid\ double= sweptSurface\ attribute\ is\ not\ a\ valid\ double= -sweptSurface\ must\ contains\ at\ least\ 3\ decimal= sweptSurface\ must\ contains\ at\ least\ 3\ decimals= year\ is\ not\ valid= Modified: trunk/coser-business/src/main/resources/i18n/coser-business_fr_FR.properties =================================================================== --- trunk/coser-business/src/main/resources/i18n/coser-business_fr_FR.properties 2010-11-23 13:09:58 UTC (rev 254) +++ trunk/coser-business/src/main/resources/i18n/coser-business_fr_FR.properties 2010-11-24 10:06:13 UTC (rev 255) @@ -37,6 +37,7 @@ coser.business.control.error.diffCatchLength=Diff\u00E9rence entre les captures et taille coser.business.control.error.diffCatchLengthDetail=Diff\u00E9rence entre les captures et taille pour l'esp\u00E8ce %s (ann\u00E9e %s) coser.business.control.error.duplicatedLine=Ligne en doublon +coser.business.control.error.duplicatedLineDetails=Ligne en doublon pour la cl\u00E9 \: %s coser.business.control.error.incompleteHaulDataCatch=Certains couples Annee|Trait ne sont pas pr\u00E9sents dans les captures coser.business.control.error.incompleteHaulDataLength=Certains couples Annee|Trait ne sont pas pr\u00E9sents dans les tailles coser.business.control.error.incompleteHaulSpeciesData=Certains couples Annee|Trait|Espece ne sont pas pr\u00E9sents dans les captures @@ -54,7 +55,6 @@ coser.business.control.step.xworks=Validation par lignes (%d%%) coser.business.line=Ligne coser.business.result.rsufiResultAlreadyExists=Le r\u00E9sultat %D existe d\u00E9j\u00E0 \! -coser.business.selection.noControlForSelection=Impossible de creer une s\u00E9lection sans donn\u00E9es control\u00E9es. coser.business.selection.notValidatedControl=Contr\u00F4le non valid\u00E9 \! coser.config.control.diffcatchlength.description=Pourcentage d'\u00E9cart tol\u00E9r\u00E9 entre les captures et les tailles coser.config.database.directory.description=Emplacement de la base de campagnes de coser @@ -64,15 +64,11 @@ coser.config.reference.typeEspeces.description=Emplacement du fichier de code type des esp\u00E8ces coser.config.validator.directory.description=Emplacement des fichiers de validations lat\ attribute\ is\ not\ a\ valid\ double= -lat\ must\ contains\ at\ least\ 3\ decimal= lat\ must\ contains\ at\ least\ 3\ decimals= -lat\ must\ contains\ at\ least\ 3\ decimasl= length\ attribute\ is\ not\ a\ valid\ double= long\ attribute\ is\ not\ a\ valid\ double= -long\ must\ contains\ at\ least\ 3\ decimal= long\ must\ contains\ at\ least\ 3\ decimals= number\ attribute\ is\ not\ a\ valid\ double= sweptSurface\ attribute\ is\ not\ a\ valid\ double= -sweptSurface\ must\ contains\ at\ least\ 3\ decimal= sweptSurface\ must\ contains\ at\ least\ 3\ decimals= year\ is\ not\ valid= Modified: trunk/coser-ui/src/main/java/fr/ifremer/coser/ui/control/ControlHandler.java =================================================================== --- trunk/coser-ui/src/main/java/fr/ifremer/coser/ui/control/ControlHandler.java 2010-11-23 13:09:58 UTC (rev 254) +++ trunk/coser-ui/src/main/java/fr/ifremer/coser/ui/control/ControlHandler.java 2010-11-24 10:06:13 UTC (rev 255) @@ -38,6 +38,7 @@ import java.util.Iterator; import java.util.List; import java.util.Map; +import java.util.Set; import java.util.UUID; import javax.swing.JLabel; @@ -441,15 +442,23 @@ view.getCategoryComboBoxModel().setSelectedItem(category); ValidationError error = (ValidationError)pathWay[3]; - String errorLineNumber = error.getLineNumber(); - // peut être null, si l'erreur ne porte pas sur un bean en particulier - if (errorLineNumber != null) { - int errorLineIndex = view.getControlDataTableModel().getRealIndexOfLine(errorLineNumber); - - // ca peut arriver si la ligne a été supprimée - if (errorLineIndex >= 0) { - view.getControlDataTableSelectionModel().setSelectionInterval(errorLineIndex, errorLineIndex); - scrollToVisible(view.getControlDataTable(), errorLineIndex, 0); + Set<String> errorLineNumbers = error.getLineNumbers(); + // peut être vide, si l'erreur ne porte pas sur un bean en particulier + if (errorLineNumbers != null) { + view.getControlDataTableSelectionModel().clearSelection(); + + boolean first = true; + for (String errorLineNumber : errorLineNumbers) { + int errorLineIndex = view.getControlDataTableModel().getRealIndexOfLine(errorLineNumber); + + // ca peut arriver si la ligne a été supprimée + if (errorLineIndex >= 0) { + view.getControlDataTableSelectionModel().addSelectionInterval(errorLineIndex, errorLineIndex); + if (first) { + scrollToVisible(view.getControlDataTable(), errorLineIndex, 0); + first = false; + } + } } } } @@ -507,17 +516,22 @@ // select all lines GlobalValidationModel model = controlView.getGlobalValidationModel(); controlView.getControlDataTableSelectionModel().clearSelection(); + boolean first = true; int childCount = model.getChildCount(validationGroup); for (int indexChild = 0 ; indexChild < childCount ; ++indexChild) { ValidationError validationError = (ValidationError)model.getChild(validationGroup, indexChild); - String errorLineNumber = validationError.getLineNumber(); - int errorLineIndex = controlView.getControlDataTableModel().getRealIndexOfLine(errorLineNumber); + Set<String> errorLineNumbers = validationError.getLineNumbers(); + + for (String errorLineNumber : errorLineNumbers) { + int errorLineIndex = controlView.getControlDataTableModel().getRealIndexOfLine(errorLineNumber); - // ca peut arriver si la ligne a été supprimée - if (errorLineIndex >= 0) { - controlView.getControlDataTableSelectionModel().addSelectionInterval(errorLineIndex, errorLineIndex); - if (indexChild == 0) { - scrollToVisible(controlView.getControlDataTable(), errorLineIndex, 0); + // ca peut arriver si la ligne a été supprimée + if (errorLineIndex >= 0) { + controlView.getControlDataTableSelectionModel().addSelectionInterval(errorLineIndex, errorLineIndex); + if (first) { + scrollToVisible(controlView.getControlDataTable(), errorLineIndex, 0); + first = false; + } } } } Modified: trunk/coser-ui/src/main/resources/i18n/coser-ui_en_GB.properties =================================================================== --- trunk/coser-ui/src/main/resources/i18n/coser-ui_en_GB.properties 2010-11-23 13:09:58 UTC (rev 254) +++ trunk/coser-ui/src/main/resources/i18n/coser-ui_en_GB.properties 2010-11-24 10:06:13 UTC (rev 255) @@ -96,10 +96,8 @@ coser.ui.result.extractDataButton=Extract coser.ui.result.extractDataLabel=Extraction directory coser.ui.result.extractDataTitle=Extract RSufi data -coser.ui.result.newDataTitle=New result coser.ui.result.newResult= coser.ui.result.otherDataFile=Other files \: -coser.ui.result.requiredauthor= coser.ui.result.requiredname= coser.ui.result.requiredrsufiVersion= coser.ui.result.resultName=Result name \: @@ -113,18 +111,15 @@ coser.ui.selection.allSpecies=L1 \: All species coser.ui.selection.comment= coser.ui.selection.createError=Creation error -coser.ui.selection.details.beginDate=Begin date \: coser.ui.selection.details.comment=Comments coser.ui.selection.details.dates= coser.ui.selection.details.description=Description \: -coser.ui.selection.details.endDate=End date \: coser.ui.selection.details.filteredSpecies= coser.ui.selection.details.name=Selection name \: coser.ui.selection.details.saveSelection=Save selection coser.ui.selection.details.selectedSpecies= coser.ui.selection.details.showMapTip= coser.ui.selection.details.showSpreciesTip= -coser.ui.selection.details.species=Species \: coser.ui.selection.details.strata=Strata \: coser.ui.selection.details.type= coser.ui.selection.details.validDates= Modified: trunk/coser-ui/src/main/resources/i18n/coser-ui_fr_FR.properties =================================================================== --- trunk/coser-ui/src/main/resources/i18n/coser-ui_fr_FR.properties 2010-11-23 13:09:58 UTC (rev 254) +++ trunk/coser-ui/src/main/resources/i18n/coser-ui_fr_FR.properties 2010-11-24 10:06:13 UTC (rev 255) @@ -96,7 +96,6 @@ coser.ui.result.extractDataButton=Extraire coser.ui.result.extractDataLabel=Dossier d'extraction \: coser.ui.result.extractDataTitle=Extraction des donn\u00E9es d'entr\u00E9e de RSufi -coser.ui.result.newDataTitle=Nouveau r\u00E9sultat coser.ui.result.newResult=Nouveau r\u00E9sultat coser.ui.result.otherDataFile=Autre fichiers \: coser.ui.result.requiredname=Le champ 'name' est requis @@ -121,7 +120,6 @@ coser.ui.selection.details.selectedSpecies=Esp\u00E8ces s\u00E9lectionn\u00E9es \: coser.ui.selection.details.showMapTip=Afficher la carte des zones coser.ui.selection.details.showSpreciesTip=Afficher la table des esp\u00E8ces -coser.ui.selection.details.species=Esp\u00E8ces \: coser.ui.selection.details.strata=Strates \: coser.ui.selection.details.type=Filtrer par type \: coser.ui.selection.details.validDates=Valider les ann\u00E9es