Author: sbavencoff Date: 2014-01-31 17:41:50 +0100 (Fri, 31 Jan 2014) New Revision: 3744 Url: http://chorem.org/projects/lima/repository/revisions/3744 Log: refs #768 : change error message, don't create next fiscal period if already exist Modified: trunk/lima-business/src/main/resources/i18n/lima-business_fr_FR.properties trunk/lima-swing/src/main/java/org/chorem/lima/ui/fiscalperiod/FiscalPeriodViewHandler.java Modified: trunk/lima-business/src/main/resources/i18n/lima-business_fr_FR.properties =================================================================== --- trunk/lima-business/src/main/resources/i18n/lima-business_fr_FR.properties 2014-01-31 15:31:55 UTC (rev 3743) +++ trunk/lima-business/src/main/resources/i18n/lima-business_fr_FR.properties 2014-01-31 16:41:50 UTC (rev 3744) @@ -77,7 +77,7 @@ lima-business.franceaccountingrules.accountnumbernotnumeric=Le numéro de compte n'est pas numérique \: %s lima-business.franceaccountingrules.accountstartnumbererror=Le numéro du compte général n'est pas compris entre 1 et 8 \: %s lima-business.franceaccountingrules.antefinancialperiodnotblockedentrybook=Les périodes financières précédentes pour ce journal doivent être bloquées avant -lima-business.franceaccountingrules.antefiscalperiodnotblocked=La période fiscale précédente n'est pas bloquée +lima-business.franceaccountingrules.antefiscalperiodnotblocked=Le nombre maximal d'exercices ouverts est déjà ateint lima-business.franceaccountingrules.fiscalperiodnotempty=La période fiscale à supprimer n'est pas vide \! lima-business.franceaccountingrules.lastFiscalPeriodCantBlocked=Impossible de clore la dernière période fiscale ouverte lima-business.franceaccountingrules.newfiscalperiodadjoiningerror=La nouvelle période fiscale doit suivre la précédente Modified: trunk/lima-swing/src/main/java/org/chorem/lima/ui/fiscalperiod/FiscalPeriodViewHandler.java =================================================================== --- trunk/lima-swing/src/main/java/org/chorem/lima/ui/fiscalperiod/FiscalPeriodViewHandler.java 2014-01-31 15:31:55 UTC (rev 3743) +++ trunk/lima-swing/src/main/java/org/chorem/lima/ui/fiscalperiod/FiscalPeriodViewHandler.java 2014-01-31 16:41:50 UTC (rev 3744) @@ -260,16 +260,25 @@ boolean retainedEarnings = false; EntryBook entryBook = null; - //check if the user wants to create a new fiscal year - int answerCreate = JOptionPane.showConfirmDialog(view, - _("lima.ui.fiscalperiod.block.newyear"), - _("lima.ui.fiscalperiod.block.title"), JOptionPane.YES_NO_OPTION, - JOptionPane.QUESTION_MESSAGE); + // find next fiscal year + FiscalPeriod lastFiscalPeriod = fiscalPeriodService.getLastFiscalPeriod(); + if (selectedFiscalPeriod.equals(lastFiscalPeriod)) { - // create a new fiscal year - if (answerCreate == JOptionPane.YES_OPTION) { - addFiscalPeriod(); + //check if the user wants to create a new fiscal year + int answerCreate = JOptionPane.showConfirmDialog(view, + _("lima.ui.fiscalperiod.block.newyear"), + _("lima.ui.fiscalperiod.block.title"), JOptionPane.YES_NO_OPTION, + JOptionPane.QUESTION_MESSAGE); + // create a new fiscal year + if (answerCreate == JOptionPane.YES_OPTION) { + addFiscalPeriod(); + } + } + + // if next fiscal year exist + if (selectedFiscalPeriod.equals(lastFiscalPeriod)) { + //report if they are at least one transaction to be report if (fiscalPeriodService.isRetainedEarnings(selectedFiscalPeriod)) { @@ -297,7 +306,7 @@ } FiscalPeriod fiscalPeriodBlocked = fiscalPeriodService.retainedEarningsAndBlockFiscalPeriod(selectedFiscalPeriod, entryBook, retainedEarnings); - tableModel.fireTableRowsUpdated(selectedRow, selectedRow); + tableModel.setValue(selectedRow, fiscalPeriodBlocked); view.setBlockEnabled(false); view.setDeleteEnabled(false);