[Suiviobsmer-commits] r188 - in trunk: suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/impl suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/pages suiviobsmer-ui/src/main/webapp
Author: fdesbois Date: 2010-01-12 16:58:54 +0000 (Tue, 12 Jan 2010) New Revision: 188 Modified: trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/impl/ServiceSamplingImpl.java trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/pages/SampleRowForm.java trunk/suiviobsmer-ui/src/main/webapp/SampleRowForm.tml Log: Fix some errors on SampleRow creation Modified: trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/impl/ServiceSamplingImpl.java =================================================================== --- trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/impl/ServiceSamplingImpl.java 2010-01-12 16:38:23 UTC (rev 187) +++ trunk/suiviobsmer-business/src/main/java/fr/ifremer/suiviobsmer/impl/ServiceSamplingImpl.java 2010-01-12 16:58:54 UTC (rev 188) @@ -362,9 +362,12 @@ log.debug("Max code found : " + maxCode); } - String[] part = maxCode.split("_"); - int inc = Integer.parseInt(part[1]) + 1; - String str = StringUtils.leftPad("" + inc, 2, "0"); + int num = 1; + if (!StringUtils.isEmpty(maxCode)) { + String[] part = maxCode.split("_"); + num = Integer.parseInt(part[1]) + 1; + } + String str = StringUtils.leftPad("" + num, 2, "0"); result = year + "_" + str; transaction.closeContext(); Modified: trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/pages/SampleRowForm.java =================================================================== --- trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/pages/SampleRowForm.java 2010-01-12 16:38:23 UTC (rev 187) +++ trunk/suiviobsmer-ui/src/main/java/fr/ifremer/suiviobsmer/ui/pages/SampleRowForm.java 2010-01-12 16:58:54 UTC (rev 188) @@ -372,7 +372,7 @@ sampleMonths.add(curr); // If an other period is set } else if (periodChanged) { - if (curr != null) { + if (curr == null) { curr = serviceSampling.getNewSampleMonth(currMonth, getSampleRow()); } sampleMonths.add(curr); @@ -528,13 +528,15 @@ SampleMonth firstMonth = getSampleMonths().get(0); SampleMonth lastMonth = getSampleMonths().get(size-1); - Date firstDate = DateUtils.createDate( - 1, firstMonth.getPeriodMonth(), firstMonth.getPeriodYear()); + if (firstMonth != null) { + Date firstDate = DateUtils.createDate( + 1, firstMonth.getPeriodMonth(), firstMonth.getPeriodYear()); - if (periodBegin.after(firstDate) && firstMonth.getRealTidesValue() != 0) { - sampleRowForm.recordError(programPeriodBegin, - "Vous ne pouvez pas réduire la période du programme. " + - "Il existe des enregistrements de marées réels."); + if (periodBegin.after(firstDate) && firstMonth.getRealTidesValue() != 0) { + sampleRowForm.recordError(programPeriodBegin, + "Vous ne pouvez pas réduire la période du programme. " + + "Il existe des enregistrements de marées réels."); + } } } Modified: trunk/suiviobsmer-ui/src/main/webapp/SampleRowForm.tml =================================================================== --- trunk/suiviobsmer-ui/src/main/webapp/SampleRowForm.tml 2010-01-12 16:38:23 UTC (rev 187) +++ trunk/suiviobsmer-ui/src/main/webapp/SampleRowForm.tml 2010-01-12 16:58:54 UTC (rev 188) @@ -133,7 +133,7 @@ <strong>${sampleRow.code}</strong> <p:else> <span class="code-missing"> - Date de début du programme manquante pour générer le code de la ligne ! + Date de début manquante pour générer le code de la ligne ! </span> </p:else> </t:if>
participants (1)
-
fdesbois@users.labs.libre-entreprise.org