March 2012 - Jtimer-commits - mailman.cloud.codelutin.com

r2827 - trunk
by echatellier＠users.chorem.org March 16, 2012

March 16, 2012

Author: echatellier Date: 2012-03-16 10:29:46 +0100 (Fri, 16 Mar 2012) New Revision: 2827 Url: http://chorem.org/repositories/revision/jtimer/2827 Log: Fix openjdk compilation Modified: trunk/pom.xml Modified: trunk/pom.xml =================================================================== --- trunk/pom.xml 2012-03-15 10:46:06 UTC (rev 2826) +++ trunk/pom.xml 2012-03-16 09:29:46 UTC (rev 2827) @@ -230,6 +230,12 @@ <artifactId>bsaf</artifactId> <version>1.9.2</version> <scope>compile</scope> + <exclusions> + <exclusion> + <groupId>javax.jnlp</groupId> + <artifactId>jnlp</artifactId> + </exclusion> + </exclusions> </dependency> <dependency> <groupId>org.swinglabs.swingx</groupId>

1 0

r2826 - trunk/src/main/java/org/chorem/jtimer/ui/systray
by echatellier＠users.chorem.org March 15, 2012

March 15, 2012

Author: echatellier Date: 2012-03-15 11:46:06 +0100 (Thu, 15 Mar 2012) New Revision: 2826 Url: http://chorem.org/repositories/revision/jtimer/2826 Log: #486: Add close to systray option Modified: trunk/src/main/java/org/chorem/jtimer/ui/systray/SystrayManager.java Modified: trunk/src/main/java/org/chorem/jtimer/ui/systray/SystrayManager.java =================================================================== --- trunk/src/main/java/org/chorem/jtimer/ui/systray/SystrayManager.java 2012-03-15 10:41:39 UTC (rev 2825) +++ trunk/src/main/java/org/chorem/jtimer/ui/systray/SystrayManager.java 2012-03-15 10:46:06 UTC (rev 2826) @@ -504,7 +504,7 @@ public void windowClosing(WindowEvent e) { // tray available only if trayIcon != null - if (trayIcon != null) { + if (trayIcon != null && JTimer.config.isCloseToSystray()) { // hide window (without exiting) parent.hide(); showItem.setText(resourceMap.getString("showMenuText"));

1 0

r2825 - in trunk/src/main: java/org/chorem/jtimer java/org/chorem/jtimer/ui/report java/org/chorem/jtimer/ui/treetable resources/org/chorem/jtimer/resources resources/org/chorem/jtimer/ui/report/resources
by echatellier＠users.chorem.org March 15, 2012

March 15, 2012

Author: echatellier Date: 2012-03-15 11:41:39 +0100 (Thu, 15 Mar 2012) New Revision: 2825 Url: http://chorem.org/repositories/revision/jtimer/2825 Log: #281: Ability to save report options #486: Add close to systray option #488: Remember show closed option Added: trunk/src/main/resources/org/chorem/jtimer/ui/report/resources/applications-system.png trunk/src/main/resources/org/chorem/jtimer/ui/report/resources/mail-forward.png Modified: trunk/src/main/java/org/chorem/jtimer/JTimer.java trunk/src/main/java/org/chorem/jtimer/JTimerConfig.java trunk/src/main/java/org/chorem/jtimer/ui/report/ReportView.java trunk/src/main/java/org/chorem/jtimer/ui/treetable/ProjectsAndTasksTable.java trunk/src/main/resources/org/chorem/jtimer/resources/JTimer.properties trunk/src/main/resources/org/chorem/jtimer/resources/JTimer_fr.properties trunk/src/main/resources/org/chorem/jtimer/ui/report/resources/ReportView.properties Modified: trunk/src/main/java/org/chorem/jtimer/JTimer.java =================================================================== --- trunk/src/main/java/org/chorem/jtimer/JTimer.java 2012-03-15 09:56:38 UTC (rev 2824) +++ trunk/src/main/java/org/chorem/jtimer/JTimer.java 2012-03-15 10:41:39 UTC (rev 2825) @@ -38,16 +38,19 @@ import java.util.List; import java.util.Timer; +import javax.swing.ButtonGroup; import javax.swing.InputMap; import javax.swing.JButton; import javax.swing.JCheckBoxMenuItem; import javax.swing.JComponent; +import javax.swing.JFrame; import javax.swing.JMenu; import javax.swing.JMenuBar; import javax.swing.JMenuItem; import javax.swing.JOptionPane; import javax.swing.JPanel; import javax.swing.JPopupMenu; +import javax.swing.JRadioButtonMenuItem; import javax.swing.JScrollPane; import javax.swing.JSeparator; import javax.swing.JToolBar; @@ -75,6 +78,7 @@ import org.chorem.jtimer.ui.tasks.RefreshTreeTask; import org.chorem.jtimer.ui.tasks.RunTaskJob; import org.chorem.jtimer.ui.treetable.ProjectsAndTasksTable; +import org.chorem.jtimer.ui.widget.WindowProperty2; import org.chorem.jtimer.ui.ws.SwingConnectionInformationHandler; import org.chorem.jtimer.ws.ConnectionDataHandler; import org.chorem.jtimer.ws.ProjectManagement; @@ -175,6 +179,9 @@ ApplicationContext ctxt = getContext(); resourceMap = ctxt.getResourceMap(); + // fix start in iconified mode + ctxt.getSessionStorage().putProperty(JFrame.class, new WindowProperty2()); + // init timercore core = new TimerCore(); @@ -289,6 +296,7 @@ projectsAndTasksTable.setName("projectslist"); projectsAndTasksTable.addTreeSelectionListener(this); projectsAndTasksTable.addMouseListener(this); + projectsAndTasksTable.setShowClosed(config.isShowClosed()); // since merge option, selection can be multiple projectsAndTasksTable @@ -360,8 +368,8 @@ String[] reportMenuActionNames = { "makeReport" }; menuBar.add(createMenu("reportMenu", reportMenuActionNames)); - String[] optionMenuActionNames = { "isShowClosed" }; - menuBar.add(createMenu("optionMenu", optionMenuActionNames)); + JMenu optionmMenu = createOptionMenu(); + menuBar.add(optionmMenu); String[] helpMenuActionNames = { "about" }; menuBar.add(createMenu("helpMenu", helpMenuActionNames)); @@ -370,6 +378,49 @@ } /** + * Create option dynamic menu. + * + * @return option menu + */ + protected JMenu createOptionMenu() { + JMenu menu = new JMenu(); + menu.setName("optionMenu"); + + // show closed + JMenuItem showClosedItem = new JCheckBoxMenuItem(); + showClosedItem.setAction(getAction("isShowClosed")); + showClosedItem.setSelected(config.isShowClosed()); + showClosedItem.setIcon(null); + menu.add(showClosedItem); + + // close to systray + JMenuItem closeToSysItem = new JCheckBoxMenuItem(); + closeToSysItem.setAction(getAction("isCloseToSystray")); + closeToSysItem.setSelected(config.isCloseToSystray()); + closeToSysItem.setIcon(null); + menu.add(closeToSysItem); + + // report first day of week + JMenu reportFDoW = new JMenu(); + reportFDoW.setName("optionReportFirstDayMenu"); + Calendar calendar = Calendar.getInstance(); + ButtonGroup bg = new ButtonGroup(); + // affiche la liste des jours dans l'ordre de la locale utilisateur + for (int day = calendar.getFirstDayOfWeek() ; day < calendar.getFirstDayOfWeek() + 7 ; day++) { + int realDay = (day - 1) % 7 + 1; + JRadioButtonMenuItem fdowItem = new JRadioButtonMenuItem(); + fdowItem.setAction(getAction("isReportFirstDayOfWeek" + realDay)); + fdowItem.setSelected(realDay == JTimer.config.getReportFirstDayOfWeek()); + fdowItem.setIcon(null); + reportFDoW.add(fdowItem); + bg.add(fdowItem); + } + menu.add(reportFDoW); + + return menu; + } + + /** * Create single menu. * * @param menuName menu name @@ -786,17 +837,87 @@ } /** - * Increment task time. + * Change show closed option. * * @param event action event */ @Action public void isShowClosed(ActionEvent event) { JCheckBoxMenuItem source = (JCheckBoxMenuItem) event.getSource(); - projectsAndTasksTable.setShowClosed(source.isSelected()); + boolean showClosed = source.isSelected(); + projectsAndTasksTable.setShowClosed(showClosed); + config.setShowClosed(showClosed); } /** + * Change close to systray option. + * + * @param event action event + */ + @Action + public void isCloseToSystray(ActionEvent event) { + JCheckBoxMenuItem source = (JCheckBoxMenuItem) event.getSource(); + boolean closeToSystray = source.isSelected(); + config.setCloseToSystray(closeToSystray); + } + + /** + * Change report first day of week. + */ + @Action + public void isReportFirstDayOfWeek1() { + config.setReportFirstDayOfWeek(1); + } + + /** + * Change report first day of week. + */ + @Action + public void isReportFirstDayOfWeek2() { + config.setReportFirstDayOfWeek(2); + } + + /** + * Change report first day of week. + */ + @Action + public void isReportFirstDayOfWeek3() { + config.setReportFirstDayOfWeek(3); + } + + /** + * Change report first day of week. + */ + @Action + public void isReportFirstDayOfWeek4() { + config.setReportFirstDayOfWeek(4); + } + + /** + * Change report first day of week. + */ + @Action + public void isReportFirstDayOfWeek5() { + config.setReportFirstDayOfWeek(5); + } + + /** + * Change report first day of week. + */ + @Action + public void isReportFirstDayOfWeek6() { + config.setReportFirstDayOfWeek(6); + } + + /** + * Change report first day of week. + */ + @Action + public void isReportFirstDayOfWeek7() { + config.setReportFirstDayOfWeek(7); + } + + /** * Increment task time. */ @Action(enabledProperty = "selectedSingleTask") Modified: trunk/src/main/java/org/chorem/jtimer/JTimerConfig.java =================================================================== --- trunk/src/main/java/org/chorem/jtimer/JTimerConfig.java 2012-03-15 09:56:38 UTC (rev 2824) +++ trunk/src/main/java/org/chorem/jtimer/JTimerConfig.java 2012-03-15 10:41:39 UTC (rev 2825) @@ -109,6 +109,44 @@ } /** + * Show closed project on main ui tree. + * + * @return show closed project + */ + public boolean isShowClosed() { + return appConfig.getOptionAsBoolean(JTimerOption.UI_SHOW_CLOSED.key); + } + + /** + * Set show closed option. + * + * @param showClosed show closed + */ + public void setShowClosed(boolean showClosed) { + appConfig.setOption(JTimerOption.UI_SHOW_CLOSED.key, String.valueOf(showClosed)); + appConfig.saveForUser(); + } + + /** + * Get close to systray option. + * + * @return close to systray + */ + public boolean isCloseToSystray() { + return appConfig.getOptionAsBoolean(JTimerOption.UI_CLOSE_TO_SYSTRAY.key); + } + + /** + * Set close to systray option + * + * @param closeToSystray close to systray + */ + public void setCloseToSystray(boolean closeToSystray) { + appConfig.setOption(JTimerOption.UI_CLOSE_TO_SYSTRAY.key, String.valueOf(closeToSystray)); + appConfig.saveForUser(); + } + + /** * Return first day of week. * Default to {@code -1} (no preference). * @@ -118,6 +156,16 @@ return appConfig.getOptionAsInt(JTimerOption.UI_REPORT_FIRSTDAYOFWEEK.key); } + /** + * Report first day of week. + * + * @param firstDayOfWeek first day of week + */ + public void setReportFirstDayOfWeek(int firstDayOfWeek) { + appConfig.setOption(JTimerOption.UI_REPORT_FIRSTDAYOFWEEK.key, String.valueOf(firstDayOfWeek)); + appConfig.saveForUser(); + } + public enum JTimerOption { CONFIG_FILENAME(ApplicationConfig.CONFIG_FILE_NAME, "jtimer.properties"), SERVICE_CLASS("jtimer.service.class", null), @@ -127,7 +175,9 @@ IO_SAVER_DIRECTORY("jtimer.io.saver.directory", "${user.home}/.gtimer"), IO_SAVER_AUTOSAVEDELAY("jtimer.io.saver.autosavedelay", "300"), UI_IDLE_TIME("jtimer.ui.idletime", "300"), - UI_REPORT_FIRSTDAYOFWEEK("jtimer.ui.report.firstdayofweek", "-1"); + UI_SHOW_CLOSED("jtimer.ui.showclosed", "false"), + UI_CLOSE_TO_SYSTRAY("jtimer.ui.closetosystray", "true"), + UI_REPORT_FIRSTDAYOFWEEK("jtimer.ui.report.firstdayofweek", "0"); protected String key; Modified: trunk/src/main/java/org/chorem/jtimer/ui/report/ReportView.java =================================================================== --- trunk/src/main/java/org/chorem/jtimer/ui/report/ReportView.java 2012-03-15 09:56:38 UTC (rev 2824) +++ trunk/src/main/java/org/chorem/jtimer/ui/report/ReportView.java 2012-03-15 10:41:39 UTC (rev 2825) @@ -54,6 +54,7 @@ import javax.swing.JPanel; import javax.swing.JRadioButton; import javax.swing.JScrollPane; +import javax.swing.JSplitPane; import javax.swing.JTextArea; import javax.swing.JTree; import javax.swing.event.DocumentEvent; @@ -162,7 +163,7 @@ */ protected JComponent getMainComponent() { - JPanel mainComponent = new JPanel(new GridBagLayout()); + JPanel configComponent = new JPanel(new GridBagLayout()); // panel for options JPanel panelOption = new JPanel(new GridBagLayout()); @@ -275,8 +276,16 @@ JScrollPane jspTable = new JScrollPane(projectsTree); panelProjects.add(jspTable, BorderLayout.CENTER); + configComponent.add(panelOption, new GridBagConstraints(0, 0, 1, 1, 0, 0, + GridBagConstraints.CENTER, GridBagConstraints.HORIZONTAL, + new Insets(1, 1, 1, 1), 0, 0)); + + configComponent.add(panelProjects, new GridBagConstraints(0, 1, 1, 1, 1, + 1, GridBagConstraints.CENTER, GridBagConstraints.BOTH, + new Insets(1, 1, 1, 1), 0, 0)); + // panel for report text - JPanel panelReports = new JPanel(new BorderLayout()); + JPanel panelReports = new JPanel(new GridBagLayout()); panelReports.setBorder(BorderFactory.createTitledBorder(getResourceMap() .getString("reportContent"))); @@ -284,42 +293,34 @@ reportArea.setFont(new Font("Courier", Font.PLAIN, 12)); reportArea.getDocument().addDocumentListener(this); JScrollPane jspReport = new JScrollPane(reportArea); - panelReports.add(jspReport, BorderLayout.CENTER); - - mainComponent.add(panelOption, new GridBagConstraints(0, 0, 3, 1, 2, 0, - GridBagConstraints.CENTER, GridBagConstraints.HORIZONTAL, + panelReports.add(jspReport, new GridBagConstraints(0, 0, 3, 1, 1, 1, + GridBagConstraints.CENTER, GridBagConstraints.BOTH, new Insets(1, 1, 1, 1), 0, 0)); - mainComponent.add(panelProjects, new GridBagConstraints(0, 1, 3, 1, 2, - 1, GridBagConstraints.CENTER, GridBagConstraints.BOTH, - new Insets(1, 1, 1, 1), 0, 0)); - // buttons JButton generateButton = new JButton(); generateButton.setAction(getContext().getActionMap(this).get( "generateReport")); - mainComponent.add(generateButton, new GridBagConstraints(0, 2, 1, 1, 1, + panelReports.add(generateButton, new GridBagConstraints(0, 1, 1, 1, 1, 0, GridBagConstraints.CENTER, GridBagConstraints.HORIZONTAL, new Insets(1, 1, 1, 1), 0, 0)); JButton sendMailButton = new JButton(); sendMailButton.setAction(getContext().getActionMap(this) .get("sendMail")); - mainComponent.add(sendMailButton, new GridBagConstraints(1, 2, 1, 1, 1, + panelReports.add(sendMailButton, new GridBagConstraints(1, 1, 1, 1, 1, 0, GridBagConstraints.CENTER, GridBagConstraints.HORIZONTAL, new Insets(1, 1, 1, 1), 0, 0)); JButton closeButton = new JButton(); closeButton.setAction(getContext().getActionMap(this).get("closeView")); - mainComponent.add(closeButton, new GridBagConstraints(2, 2, 1, 1, 1, 0, + panelReports.add(closeButton, new GridBagConstraints(2, 1, 1, 1, 1, 0, GridBagConstraints.CENTER, GridBagConstraints.HORIZONTAL, new Insets(1, 1, 1, 1), 0, 0)); - - mainComponent.add(panelReports, new GridBagConstraints(3, 0, 1, 3, 8, - 1, GridBagConstraints.CENTER, GridBagConstraints.BOTH, - new Insets(1, 1, 1, 1), 250, 0)); - - return mainComponent; + + JSplitPane splitPane = new JSplitPane(JSplitPane.HORIZONTAL_SPLIT, configComponent, panelReports); + splitPane.setOneTouchExpandable(true); + return splitPane; } /** @@ -348,7 +349,7 @@ // init dates Calendar calendarBegin = Calendar.getInstance(); int firstDayOfWeek = JTimer.config.getReportFirstDayOfWeek(); - if (firstDayOfWeek < 0 || firstDayOfWeek > 6) { + if (firstDayOfWeek <= 0 || firstDayOfWeek > 7) { firstDayOfWeek = calendarBegin.getFirstDayOfWeek(); } calendarBegin.set(Calendar.DAY_OF_WEEK, firstDayOfWeek); Modified: trunk/src/main/java/org/chorem/jtimer/ui/treetable/ProjectsAndTasksTable.java =================================================================== --- trunk/src/main/java/org/chorem/jtimer/ui/treetable/ProjectsAndTasksTable.java 2012-03-15 09:56:38 UTC (rev 2824) +++ trunk/src/main/java/org/chorem/jtimer/ui/treetable/ProjectsAndTasksTable.java 2012-03-15 10:41:39 UTC (rev 2825) @@ -86,8 +86,7 @@ columnIdentifiers.add(resourceMap.getString("totalTimeColumnName")); // set model - treeTableModel = new ProjectsAndTasksModel(this, core, - columnIdentifiers); + treeTableModel = new ProjectsAndTasksModel(this, core, columnIdentifiers); // set renderer treeCellRenderer = new ProjectsAndTasksCellRenderer(core); Modified: trunk/src/main/resources/org/chorem/jtimer/resources/JTimer.properties =================================================================== --- trunk/src/main/resources/org/chorem/jtimer/resources/JTimer.properties 2012-03-15 09:56:38 UTC (rev 2824) +++ trunk/src/main/resources/org/chorem/jtimer/resources/JTimer.properties 2012-03-15 10:41:39 UTC (rev 2825) @@ -119,6 +119,17 @@ isShowClosed.Action.text = Show closed isShowClosed.Action.shortDescription = Show closed task and project +isCloseToSystray.Action.text = Close to systray +isCloseToSystray.Action.shortDescription = Reduce application to system tray instead of closing + +isReportFirstDayOfWeek1.Action.text = Sunday +isReportFirstDayOfWeek2.Action.text = Monday +isReportFirstDayOfWeek3.Action.text = Tuesday +isReportFirstDayOfWeek4.Action.text = Wednesday +isReportFirstDayOfWeek5.Action.text = Thursday +isReportFirstDayOfWeek6.Action.text = Friday +isReportFirstDayOfWeek7.Action.text = Saturday + about.Action.text = About... about.Action.shortDescription = About ${Application.title} @@ -136,6 +147,7 @@ taskMenu.text = Task reportMenu.text = Report optionMenu.text = Options +optionReportFirstDayMenu.text = Report - First day of week helpMenu.text = Help # Input @@ -156,11 +168,11 @@ input.addAnnotationTitle=Add annotation input.addAnnotationMessage=Annotation for task "%s" : -#�Error +# Error action.invalidActionTitle=Can't do action action.missingErrorMessage=Error message is missing (%s) -#�listeners errors +# listeners errors vetoable.common.duplicated.project.name=A project already exists with that name at this level ! vetoable.common.duplicated.task.name=A task already exists with that name at this level ! vetoable.common.merge.invalid.types=Can't merge project and task ! @@ -169,6 +181,6 @@ vetoable.saver.invalid.task.characters=Task name contains invalid characters ! vetoable.ws.chorem.cant.modify.synchronized.project=Can't do this action on a synchronized project ! -#�Start fail i18n +# Start fail i18n startFail.title=Error startFail.message=${Application.title} fail to init.\nCheck that ${Application.title} is not already launched. Modified: trunk/src/main/resources/org/chorem/jtimer/resources/JTimer_fr.properties =================================================================== --- trunk/src/main/resources/org/chorem/jtimer/resources/JTimer_fr.properties 2012-03-15 09:56:38 UTC (rev 2824) +++ trunk/src/main/resources/org/chorem/jtimer/resources/JTimer_fr.properties 2012-03-15 10:41:39 UTC (rev 2825) @@ -90,7 +90,7 @@ decrement30Task.Action.shortDescription = Retire 30 minutes au temps de la t\u00E2che setToZero.Action.text = Remettre \u00E0 Zero -setToZero.Action.shortDecription = Remet le temps de la t\u00E2che \u00E0 zero +setToZero.Action.shortDecription = Remet le temps de la t\u00E2che \u00E0 z\u00E9ro mergeTasks.Action.text = Fusionner mergeTasks.Action.shortDecription = Fusionner les t\u00E2ches @@ -98,9 +98,20 @@ makeReport.Action.text = Rapport... makeReport.Action.shortDescription = Cr\u00E9er un rapport -isShowClosed.Action.text = Afficher les cach\u00E9es -isShowClosed.Action.shortDescription = Affiche les projets et t\u00E2ches ferm\u00E9es +isShowClosed.Action.text = Afficher les cach\u00E9s +isShowClosed.Action.shortDescription = Affiche les projets et t\u00E2ches ferm\u00E9s +isCloseToSystray.Action.text = Fermer vers le systray +isCloseToSystray.Action.shortDescription = R\u00E9duit l'application dans le systray au lieu de la fermer + +isReportFirstDayOfWeek1.Action.text = Dimanche +isReportFirstDayOfWeek2.Action.text = Lundi +isReportFirstDayOfWeek3.Action.text = Mardi +isReportFirstDayOfWeek4.Action.text = Mercredi +isReportFirstDayOfWeek5.Action.text = Jeudi +isReportFirstDayOfWeek6.Action.text = Vendredi +isReportFirstDayOfWeek7.Action.text = Samedi + about.Action.text = \u00C0 propos... about.Action.shortDescription = About ${Application.title} @@ -115,6 +126,7 @@ taskMenu.text = T\u00E2che reportMenu.text = Rapport optionMenu.text = Options +optionReportFirstDayMenu.text = Rapport - Premier jour de la semaine helpMenu.text = Aide # Input @@ -135,11 +147,11 @@ input.addAnnotationTitle=Ajouter une annotation input.addAnnotationMessage=Annotation pour la t\u00E2che "%s" : -#�Error +# Error action.invalidActionTitle=Impossible d'effectuer cette action action.missingErrorMessage=Message d'erreur manquant (%s) -#�listeners errors +# listeners errors vetoable.common.duplicated.project.name=Un projet avec ce nom existe d\u00E9j\u00E0 \u00E0 ce niveau ! vetoable.common.duplicated.task.name=Une t\u00E2che avec ce nom existe d\u00E9j\u00E0 \u00E0 ce niveau ! vetoable.common.merge.invalid.types=Impossible de fusionner un projet et une t\u00E2che ! @@ -148,6 +160,6 @@ vetoable.saver.invalid.characters=Le nom contient des caract\u00E8res invalide ! vetoable.ws.chorem.cant.modify.synchronized.project=Impossible d'effectuer cette action sur un projet synchronis\u00E9 ! -#�Start fail i18n +# Start fail i18n startFail.title=Erreur startFail.message=${Application.title} n'a pas r\u00E9ussi \u00E0 s'initiliser.\nV\u00E9rifiez que ${Application.title} n'est pas d\u00E9j\u00E0 lanc\u00E9. \ No newline at end of file Modified: trunk/src/main/resources/org/chorem/jtimer/ui/report/resources/ReportView.properties =================================================================== --- trunk/src/main/resources/org/chorem/jtimer/ui/report/resources/ReportView.properties 2012-03-15 09:56:38 UTC (rev 2824) +++ trunk/src/main/resources/org/chorem/jtimer/ui/report/resources/ReportView.properties 2012-03-15 10:41:39 UTC (rev 2825) @@ -47,9 +47,11 @@ showHiddenProjects.Action.shortDescription = Show hidden projects generateReport.Action.text = Generate +generateReport.Action.icon = applications-system.png generateReport.Action.shortDescription = Generate report sendMail.Action.text = Send by mail +sendMail.Action.icon = mail-forward.png sendMail.Action.shortDescription = Send report by mail closeView.Action.text = Close Added: trunk/src/main/resources/org/chorem/jtimer/ui/report/resources/applications-system.png =================================================================== (Binary files differ) Property changes on: trunk/src/main/resources/org/chorem/jtimer/ui/report/resources/applications-system.png ___________________________________________________________________ Added: svn:mime-type + image/png Added: trunk/src/main/resources/org/chorem/jtimer/ui/report/resources/mail-forward.png =================================================================== (Binary files differ) Property changes on: trunk/src/main/resources/org/chorem/jtimer/ui/report/resources/mail-forward.png ___________________________________________________________________ Added: svn:mime-type + image/png

1 0

r2824 - trunk/src/main/java/org/chorem/jtimer/ui/widget
by echatellier＠users.chorem.org March 15, 2012

March 15, 2012

Author: echatellier Date: 2012-03-15 10:56:38 +0100 (Thu, 15 Mar 2012) New Revision: 2824 Url: http://chorem.org/repositories/revision/jtimer/2824 Log: #487 : Force jtimer to start in non iconified frame mode Added: trunk/src/main/java/org/chorem/jtimer/ui/widget/WindowProperty2.java Added: trunk/src/main/java/org/chorem/jtimer/ui/widget/WindowProperty2.java =================================================================== --- trunk/src/main/java/org/chorem/jtimer/ui/widget/WindowProperty2.java (rev 0) +++ trunk/src/main/java/org/chorem/jtimer/ui/widget/WindowProperty2.java 2012-03-15 09:56:38 UTC (rev 2824) @@ -0,0 +1,53 @@ +/* + * #%L + * + * + * $Id$ + * $HeadURL$ + * %% + * Copyright (C) 2012 Codelutin, Chatellier Eric + * %% + * This program is free software: you can redistribute it and/or modify + * it under the terms of the GNU Lesser General Public License as + * published by the Free Software Foundation, either version 3 of the + * License, or (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Lesser Public License for more details. + * + * You should have received a copy of the GNU General Lesser Public + * License along with this program. If not, see + * <http://www.gnu.org/licenses/lgpl-3.0.html>. + * #L% + */ + +package org.chorem.jtimer.ui.widget; + +import java.awt.Component; +import java.awt.Frame; + +import org.jdesktop.application.session.WindowProperty; + +/** + * Code from http://kenai.com/jira/browse/BSAF-74. + * + * Force jtimer to always start in not iconified mode. + * + * @author chatellier + * @version $Revision$ + * + * Last update : $Date$ + * By : $Author$ + */ +public class WindowProperty2 extends WindowProperty { + + @Override + public void setSessionState(final Component comp, final Object state) { + super.setSessionState(comp, state); + if (comp instanceof Frame && !((Frame) comp).isLocationByPlatform()) { + ((Frame) comp).setExtendedState(((Frame) comp).getExtendedState() & ~Frame.ICONIFIED); + } + } +} Property changes on: trunk/src/main/java/org/chorem/jtimer/ui/widget/WindowProperty2.java ___________________________________________________________________ Added: svn:keywords + Author Date Id Revision HeadURL

1 0

r2823 - trunk/src/main/java/org/chorem/jtimer/ui/treetable
by echatellier＠users.chorem.org March 15, 2012

March 15, 2012

Author: echatellier Date: 2012-03-15 10:10:31 +0100 (Thu, 15 Mar 2012) New Revision: 2823 Url: http://chorem.org/repositories/revision/jtimer/2823 Log: Fix node removal before cache modification. Modified: trunk/src/main/java/org/chorem/jtimer/ui/treetable/ProjectsAndTasksModel.java Modified: trunk/src/main/java/org/chorem/jtimer/ui/treetable/ProjectsAndTasksModel.java =================================================================== --- trunk/src/main/java/org/chorem/jtimer/ui/treetable/ProjectsAndTasksModel.java 2012-03-14 13:12:33 UTC (rev 2822) +++ trunk/src/main/java/org/chorem/jtimer/ui/treetable/ProjectsAndTasksModel.java 2012-03-15 09:10:31 UTC (rev 2823) @@ -331,10 +331,10 @@ break; } case OPERATION_DELETE: { - subTasksCache.remove(pathLastComponent); // invalidate cache - subTasksCache.remove(taskUO); // no in cache anymore - taskNameCache.remove(taskUO); // no in cache anymore + // before cache invalidate (use index of himself) modelSupport.fireChildRemoved(path, childIndex, taskUO); + subTasksCache.get(pathLastComponent).remove(taskUO); // not in cache anymore + taskNameCache.remove(taskUO); // not in cache anymore break; } case OPERATION_MODIFY: {

1 0

r2822 - trunk/src/main/java/org/chorem/jtimer/ui/tasks
by echatellier＠users.chorem.org March 14, 2012

March 14, 2012

Author: echatellier Date: 2012-03-14 14:12:33 +0100 (Wed, 14 Mar 2012) New Revision: 2822 Url: http://chorem.org/repositories/revision/jtimer/2822 Log: #484 : Exception sometime :( Caused by: java.lang.StackOverflowError Modified: trunk/src/main/java/org/chorem/jtimer/ui/tasks/IdleDialog.java Modified: trunk/src/main/java/org/chorem/jtimer/ui/tasks/IdleDialog.java =================================================================== --- trunk/src/main/java/org/chorem/jtimer/ui/tasks/IdleDialog.java 2012-03-12 08:58:08 UTC (rev 2821) +++ trunk/src/main/java/org/chorem/jtimer/ui/tasks/IdleDialog.java 2012-03-14 13:12:33 UTC (rev 2822) @@ -123,7 +123,6 @@ // don't make reference on other parent // windows, cause idle to unlock some task // if parent window is hidden by systray - super(); // init resources map this.application = application; @@ -131,7 +130,12 @@ ResourceManager mgr = ctxt.getResourceManager(); resourceMap = mgr.getResourceMap(IdleDialog.class); - setName("idleFrame"); + // remove dialog name to this dialog, due to bug + // http://chorem.org/issues/484 and + // http://kenai.com/jira/browse/BSAF-107 + // cause infinite loop under openjdk + setName(null); // force nameExplicitlySet to true + setTitle(resourceMap.getString("idleTitle")); setResizable(false); setModal(true);

1 0

r2821 - trunk/src/main/assembly
by echatellier＠users.chorem.org March 12, 2012

March 12, 2012

Author: echatellier Date: 2012-03-12 09:58:08 +0100 (Mon, 12 Mar 2012) New Revision: 2821 Url: http://chorem.org/repositories/revision/jtimer/2821 Log: Don't include project artifact twice (change from assembly plugin 2.2+) Modified: trunk/src/main/assembly/bin.xml Modified: trunk/src/main/assembly/bin.xml =================================================================== --- trunk/src/main/assembly/bin.xml 2012-03-09 17:47:06 UTC (rev 2820) +++ trunk/src/main/assembly/bin.xml 2012-03-12 08:58:08 UTC (rev 2821) @@ -34,6 +34,7 @@ <dependencySet> <outputDirectory>lib</outputDirectory> <scope>runtime</scope> + <useProjectArtifact>false</useProjectArtifact> </dependencySet> </dependencySets> <files>

1 0

r2820 - trunk/src/test/java/org/chorem/jtimer/entities
by echatellier＠users.chorem.org March 9, 2012

March 9, 2012

Author: echatellier Date: 2012-03-09 18:47:06 +0100 (Fri, 09 Mar 2012) New Revision: 2820 Url: http://chorem.org/repositories/revision/jtimer/2820 Log: Remove failing tests (now equality only works with uuid) Modified: trunk/src/test/java/org/chorem/jtimer/entities/TimerProjectTest.java trunk/src/test/java/org/chorem/jtimer/entities/TimerTaskTest.java Modified: trunk/src/test/java/org/chorem/jtimer/entities/TimerProjectTest.java =================================================================== --- trunk/src/test/java/org/chorem/jtimer/entities/TimerProjectTest.java 2012-03-09 17:46:41 UTC (rev 2819) +++ trunk/src/test/java/org/chorem/jtimer/entities/TimerProjectTest.java 2012-03-09 17:47:06 UTC (rev 2820) @@ -80,22 +80,4 @@ Assert.assertEquals("name1", project2.getName()); Assert.assertEquals(project, project2); } - - /** - * Test project clone(). - * - * Test que une fois le clone modifier, - * les objets ne sont plus egaux. - */ - @Test - public void testClone2() { - - TimerProject project = new TimerProject(); - project.setName("name1"); - - TimerProject project2 = project.clone(); - project.setName("name2"); - - Assert.assertFalse(project.equals(project2)); - } } Modified: trunk/src/test/java/org/chorem/jtimer/entities/TimerTaskTest.java =================================================================== --- trunk/src/test/java/org/chorem/jtimer/entities/TimerTaskTest.java 2012-03-09 17:46:41 UTC (rev 2819) +++ trunk/src/test/java/org/chorem/jtimer/entities/TimerTaskTest.java 2012-03-09 17:47:06 UTC (rev 2820) @@ -60,45 +60,6 @@ * Test task clone(); */ @Test - public void testClone() { - - TimerTask task = new TimerTask(); - task.setName("test"); - - TimerTask clonedTask = task.clone(); - Assert.assertEquals("test", clonedTask.getName()); - - clonedTask.setName("name2"); - Assert.assertFalse(clonedTask.equals(task)); - } - - /** - * Test task clone(); - */ - @Test - public void testClone3() { - - Calendar now = Calendar.getInstance(); - - TimerTask task = new TimerTask(); - task.setName("test"); - task.setCreationDate(new Date()); - task.setClosed(true); - task.setTime(now.getTime(), 10l); - task.addAnnotation(now.getTime(), "blah"); - - TimerTask clonedTask = task.clone(); - Assert.assertEquals(clonedTask, task); - - now.add(Calendar.DAY_OF_YEAR, 1); - clonedTask.addAnnotation(now.getTime(), "blah2"); - Assert.assertFalse(clonedTask.equals(task)); - } - - /** - * Test task clone(); - */ - @Test public void testCloneDailyMap() { TimerTask task = new TimerTask();

1 0

r2819 - trunk/src/main/resources/ftl
by echatellier＠users.chorem.org March 9, 2012

March 9, 2012

Author: echatellier Date: 2012-03-09 18:46:41 +0100 (Fri, 09 Mar 2012) New Revision: 2819 Url: http://chorem.org/repositories/revision/jtimer/2819 Log: Fix report format (too much blank line a top of report) Modified: trunk/src/main/resources/ftl/reportByProject.ftl trunk/src/main/resources/ftl/reportByProject_fr.ftl Modified: trunk/src/main/resources/ftl/reportByProject.ftl =================================================================== --- trunk/src/main/resources/ftl/reportByProject.ftl 2012-03-09 17:20:13 UTC (rev 2818) +++ trunk/src/main/resources/ftl/reportByProject.ftl 2012-03-09 17:46:41 UTC (rev 2819) @@ -25,7 +25,6 @@ Report by project ================= -<#assign periods=utils.getDailyDates(begin?date,end?date)/> <#macro displaySubtaskReport task identation=0> <#local ident=identation + 1/> <#list utils.getSubTaskOrdered(task) as subtask> @@ -54,6 +53,7 @@ </#list> </#macro> +<#assign periods=utils.getDailyDates(begin?date,end?date)/> <#list projects as project> <#assign periodTotalTime=0/> <#list periods as period> Modified: trunk/src/main/resources/ftl/reportByProject_fr.ftl =================================================================== --- trunk/src/main/resources/ftl/reportByProject_fr.ftl 2012-03-09 17:20:13 UTC (rev 2818) +++ trunk/src/main/resources/ftl/reportByProject_fr.ftl 2012-03-09 17:46:41 UTC (rev 2819) @@ -25,7 +25,6 @@ Rapport par projet ================== -<#assign periods=utils.getDailyDates(begin?date,end?date)/> <#macro displaySubtaskReport task identation=0> <#local ident=identation + 1/> <#list utils.getSubTaskOrdered(task) as subtask> @@ -54,6 +53,7 @@ </#list> </#macro> +<#assign periods=utils.getDailyDates(begin?date,end?date)/> <#list projects as project> <#assign periodTotalTime=0/> <#list periods as period>

1 0

r2818 - trunk/src/main/java/org/chorem/jtimer/entities
by echatellier＠users.chorem.org March 9, 2012

March 9, 2012

Author: echatellier Date: 2012-03-09 18:20:13 +0100 (Fri, 09 Mar 2012) New Revision: 2818 Url: http://chorem.org/repositories/revision/jtimer/2818 Log: Remove unused deprecated method Modified: trunk/src/main/java/org/chorem/jtimer/entities/TimerTaskHelper.java Modified: trunk/src/main/java/org/chorem/jtimer/entities/TimerTaskHelper.java =================================================================== --- trunk/src/main/java/org/chorem/jtimer/entities/TimerTaskHelper.java 2012-03-09 17:18:53 UTC (rev 2817) +++ trunk/src/main/java/org/chorem/jtimer/entities/TimerTaskHelper.java 2012-03-09 17:20:13 UTC (rev 2818) @@ -73,18 +73,6 @@ }; /** - * Get task's subtasks ordered alphabetically. - * - * @param task task - * @return the subTasks ordered - * @deprecated since 1.4.0 use {@link #sortTask(task.getSubTasks())} - */ - public static List<TimerTask> getSubTasksOrderedByName(TimerTask task) { - List<TimerTask> taksToSort = task.getSubTasks(); - return sortTask(taksToSort); - } - - /** * Get tasks ordered alphabetically. * * @param tasks tasks

1 0

r2817 - in trunk: . src/main/java/org/chorem/jtimer/ui/report src/main/resources/org/chorem/jtimer/ui/report/resources
by echatellier＠users.chorem.org March 9, 2012

March 9, 2012

Author: echatellier Date: 2012-03-09 18:18:53 +0100 (Fri, 09 Mar 2012) New Revision: 2817 Url: http://chorem.org/repositories/revision/jtimer/2817 Log: Add current week and previous week buttons Added: trunk/src/main/resources/org/chorem/jtimer/ui/report/resources/date_current.png trunk/src/main/resources/org/chorem/jtimer/ui/report/resources/date_previous.png Modified: trunk/README.txt trunk/src/main/java/org/chorem/jtimer/ui/report/ReportView.java trunk/src/main/resources/org/chorem/jtimer/ui/report/resources/ReportView.properties trunk/src/main/resources/org/chorem/jtimer/ui/report/resources/ReportView_fr.properties Modified: trunk/README.txt =================================================================== --- trunk/README.txt 2012-03-09 15:31:15 UTC (rev 2816) +++ trunk/README.txt 2012-03-09 17:18:53 UTC (rev 2817) @@ -24,3 +24,12 @@ jtimer (linux) jtimer.exe (windows) + + +Credits +------- + +Images "src/main/resources/org/chorem/jtimer/ui/report/resources/date_current.png" +and "src/main/resources/org/chorem/jtimer/ui/report/resources/date_previous.png" +are licensed under Creative Commons Attribution (by) and available at: +http://www.famfamfam.com/ Modified: trunk/src/main/java/org/chorem/jtimer/ui/report/ReportView.java =================================================================== --- trunk/src/main/java/org/chorem/jtimer/ui/report/ReportView.java 2012-03-09 15:31:15 UTC (rev 2816) +++ trunk/src/main/java/org/chorem/jtimer/ui/report/ReportView.java 2012-03-09 17:18:53 UTC (rev 2817) @@ -123,9 +123,6 @@ /** Report output */ protected JTextArea reportArea; - /** Send by mail button */ - protected JButton sendMailButton; - /** Can send mail property. */ protected boolean canSendMail; @@ -167,29 +164,6 @@ JPanel mainComponent = new JPanel(new GridBagLayout()); - // init dates - Calendar calendarBegin = Calendar.getInstance(); - int firstDayOfWeek = JTimer.config.getReportFirstDayOfWeek(); - if (firstDayOfWeek < 0 || firstDayOfWeek > 6) { - firstDayOfWeek = calendarBegin.getFirstDayOfWeek(); - } - calendarBegin.set(Calendar.DAY_OF_WEEK, firstDayOfWeek); - calendarBegin.set(Calendar.HOUR, 0); - calendarBegin.set(Calendar.MINUTE, 0); - calendarBegin.set(Calendar.SECOND, 0); - calendarBegin.set(Calendar.MILLISECOND, 0); - - // calendar must be in current week by default - if (calendarBegin.getTime().after(new Date())) { - calendarBegin.add(Calendar.WEEK_OF_YEAR, -1); - } - - // get end of week - // take calendarBegin and add a week time - Calendar calendarEnd = (Calendar) calendarBegin.clone(); - calendarEnd.add(Calendar.WEEK_OF_YEAR, 1); - calendarEnd.add(Calendar.DAY_OF_YEAR, -1); // take the day before - // panel for options JPanel panelOption = new JPanel(new GridBagLayout()); panelOption.setBorder(BorderFactory.createTitledBorder(getResourceMap() @@ -202,9 +176,8 @@ new Insets(1, 1, 1, 1), 0, 0)); datePickerFrom = new JXDatePicker(); datePickerFrom.getMonthView().setDayForeground(Calendar.SUNDAY, Color.RED); - datePickerFrom.setDate(calendarBegin.getTime()); datePickerFrom.setFormats(DateFormat.getDateInstance(DateFormat.FULL)); - panelOption.add(datePickerFrom, new GridBagConstraints(1, 0, 3, 1, 1, + panelOption.add(datePickerFrom, new GridBagConstraints(1, 0, 2, 1, 1, 0, GridBagConstraints.CENTER, GridBagConstraints.HORIZONTAL, new Insets(1, 1, 1, 1), 0, 0)); @@ -215,13 +188,29 @@ new Insets(1, 1, 1, 1), 0, 0)); datePickerTo = new JXDatePicker(); datePickerTo.getMonthView().setDayForeground(Calendar.SUNDAY, Color.RED); - datePickerTo.setDate(calendarEnd.getTime()); datePickerTo.setFormats(DateFormat.getDateInstance(DateFormat.FULL)); - panelOption.add(datePickerTo, new GridBagConstraints(1, 1, 3, 1, 0, 0, + panelOption.add(datePickerTo, new GridBagConstraints(1, 1, 2, 1, 0, 0, GridBagConstraints.CENTER, GridBagConstraints.HORIZONTAL, new Insets(1, 1, 1, 1), 0, 0)); - // Option for period regroupment + // set current week by default + currentWeek(); + + // action button to show current or previous week + JButton currentWeekButton = new JButton(); + currentWeekButton.setAction(getContext().getActionMap(this).get( + "currentWeek")); + panelOption.add(currentWeekButton, new GridBagConstraints(3, 0, 1, 1, 0, + 0, GridBagConstraints.CENTER, GridBagConstraints.HORIZONTAL, + new Insets(1, 1, 1, 1), 0, 0)); + JButton previousWeekButton = new JButton(); + previousWeekButton.setAction(getContext().getActionMap(this).get( + "previousWeek")); + panelOption.add(previousWeekButton, new GridBagConstraints(3, 1, 1, 1, 0, + 0, GridBagConstraints.CENTER, GridBagConstraints.HORIZONTAL, + new Insets(1, 1, 1, 1), 0, 0)); + + // Option for period grouping radioByDay = new JRadioButton( getResourceMap().getString("reportDaily"), true); radioByMonth = new JRadioButton(getResourceMap().getString( @@ -246,13 +235,13 @@ panelOption.add(radioByMonth, new GridBagConstraints(2, 2, 2, 1, 1, 0, GridBagConstraints.CENTER, GridBagConstraints.HORIZONTAL, new Insets(1, 1, 1, 1), 0, 0)); - panelOption.add(radioByWeek, new GridBagConstraints(0, 3, 2, 1, 1, 0, + panelOption.add(radioByWeek, new GridBagConstraints(0, 3, 2, 1, 0, 0, GridBagConstraints.CENTER, GridBagConstraints.HORIZONTAL, new Insets(1, 1, 1, 1), 0, 0)); - panelOption.add(radioByYear, new GridBagConstraints(2, 3, 2, 1, 1, 0, + panelOption.add(radioByYear, new GridBagConstraints(2, 3, 2, 1, 0, 0, GridBagConstraints.CENTER, GridBagConstraints.HORIZONTAL, new Insets(1, 1, 1, 1), 0, 0)); - panelOption.add(radioByProject, new GridBagConstraints(0, 4, 4, 1, 1, + panelOption.add(radioByProject, new GridBagConstraints(0, 4, 4, 1, 0, 0, GridBagConstraints.CENTER, GridBagConstraints.HORIZONTAL, new Insets(1, 1, 1, 1), 0, 0)); @@ -260,7 +249,7 @@ checkIncludesAnnotations = new JCheckBox(getResourceMap().getString( "reportAnnotations")); panelOption.add(checkIncludesAnnotations, new GridBagConstraints(0, 5, - 4, 1, 1, 0, GridBagConstraints.CENTER, GridBagConstraints.BOTH, + 4, 1, 0, 0, GridBagConstraints.CENTER, GridBagConstraints.HORIZONTAL, new Insets(1, 1, 1, 1), 0, 0)); // panel form projects @@ -313,7 +302,7 @@ 0, GridBagConstraints.CENTER, GridBagConstraints.HORIZONTAL, new Insets(1, 1, 1, 1), 0, 0)); - sendMailButton = new JButton(); + JButton sendMailButton = new JButton(); sendMailButton.setAction(getContext().getActionMap(this) .get("sendMail")); mainComponent.add(sendMailButton, new GridBagConstraints(1, 2, 1, 1, 1, @@ -334,6 +323,60 @@ } /** + * Select current week in date pickers. + */ + @Action + public void currentWeek() { + fillPickerDates(0); + } + + /** + * Display previous week in date pickers. + */ + @Action + public void previousWeek() { + fillPickerDates(-1); + } + + /** + * Fill picker date with predefined week selection (from current) + * and apply a delay (-1 = previous week). + * + * @param delay delay to add to current week + */ + protected void fillPickerDates(int delay) { + // init dates + Calendar calendarBegin = Calendar.getInstance(); + int firstDayOfWeek = JTimer.config.getReportFirstDayOfWeek(); + if (firstDayOfWeek < 0 || firstDayOfWeek > 6) { + firstDayOfWeek = calendarBegin.getFirstDayOfWeek(); + } + calendarBegin.set(Calendar.DAY_OF_WEEK, firstDayOfWeek); + calendarBegin.set(Calendar.HOUR, 0); + calendarBegin.set(Calendar.MINUTE, 0); + calendarBegin.set(Calendar.SECOND, 0); + calendarBegin.set(Calendar.MILLISECOND, 0); + + // calendar must be in current week by default + if (calendarBegin.getTime().after(new Date())) { + calendarBegin.add(Calendar.WEEK_OF_YEAR, -1); + } + + // get end of week + // take calendarBegin and add a week time + Calendar calendarEnd = (Calendar) calendarBegin.clone(); + calendarEnd.add(Calendar.WEEK_OF_YEAR, 1); + calendarEnd.add(Calendar.DAY_OF_YEAR, -1); // take the day before + + // apply delai + calendarBegin.add(Calendar.WEEK_OF_YEAR, delay); + calendarEnd.add(Calendar.WEEK_OF_YEAR, delay); + + datePickerFrom.setDate(calendarBegin.getTime()); + datePickerTo.setDate(calendarEnd.getTime()); + } + + /** * Close action. */ @Action Modified: trunk/src/main/resources/org/chorem/jtimer/ui/report/resources/ReportView.properties =================================================================== --- trunk/src/main/resources/org/chorem/jtimer/ui/report/resources/ReportView.properties 2012-03-09 15:31:15 UTC (rev 2816) +++ trunk/src/main/resources/org/chorem/jtimer/ui/report/resources/ReportView.properties 2012-03-09 17:18:53 UTC (rev 2817) @@ -5,7 +5,7 @@ # $Id$ # $HeadURL$ # %% -# Copyright (C) 2007 - 2011 CodeLutin, Chatellier Eric +# Copyright (C) 2007 - 2012 CodeLutin, Chatellier Eric # %% # This program is free software: you can redistribute it and/or modify # it under the terms of the GNU General Public License as @@ -37,6 +37,12 @@ reportContent=Report reportProjectsList=Projects list +currentWeek.Action.icon = date_current.png +currentWeek.Action.shortDescription = Current week + +previousWeek.Action.icon = date_previous.png +previousWeek.Action.shortDescription = Previous week + showHiddenProjects.Action.text = Show hidden projects showHiddenProjects.Action.shortDescription = Show hidden projects Modified: trunk/src/main/resources/org/chorem/jtimer/ui/report/resources/ReportView_fr.properties =================================================================== --- trunk/src/main/resources/org/chorem/jtimer/ui/report/resources/ReportView_fr.properties 2012-03-09 15:31:15 UTC (rev 2816) +++ trunk/src/main/resources/org/chorem/jtimer/ui/report/resources/ReportView_fr.properties 2012-03-09 17:18:53 UTC (rev 2817) @@ -5,7 +5,7 @@ # $Id$ # $HeadURL$ # %% -# Copyright (C) 2007 - 2011 CodeLutin, Chatellier Eric +# Copyright (C) 2007 - 2012 CodeLutin, Chatellier Eric # %% # This program is free software: you can redistribute it and/or modify # it under the terms of the GNU General Public License as @@ -37,6 +37,10 @@ reportContent=Rapport reportProjectsList=Liste des projets +currentWeek.Action.shortDescription = Semaine courante + +previousWeek.Action.shortDescription = Semaine pr\u00E9c\u00E9dente + showHiddenProjects.Action.text = Afficher les cach\u00E9s showHiddenProjects.Action.shortDescription = Afficher les cach\u00E9s Added: trunk/src/main/resources/org/chorem/jtimer/ui/report/resources/date_current.png =================================================================== (Binary files differ) Property changes on: trunk/src/main/resources/org/chorem/jtimer/ui/report/resources/date_current.png ___________________________________________________________________ Added: svn:mime-type + image/png Added: trunk/src/main/resources/org/chorem/jtimer/ui/report/resources/date_previous.png =================================================================== (Binary files differ) Property changes on: trunk/src/main/resources/org/chorem/jtimer/ui/report/resources/date_previous.png ___________________________________________________________________ Added: svn:mime-type + image/png

1 0

r2816 - in trunk/src/main/java/org/chorem/jtimer: . ui/report
by echatellier＠users.chorem.org March 9, 2012

March 9, 2012

Author: echatellier Date: 2012-03-09 16:31:15 +0100 (Fri, 09 Mar 2012) New Revision: 2816 Url: http://chorem.org/repositories/revision/jtimer/2816 Log: #281 : Ability to save report options Modified: trunk/src/main/java/org/chorem/jtimer/JTimerConfig.java trunk/src/main/java/org/chorem/jtimer/ui/report/ReportView.java Modified: trunk/src/main/java/org/chorem/jtimer/JTimerConfig.java =================================================================== --- trunk/src/main/java/org/chorem/jtimer/JTimerConfig.java 2012-03-09 15:03:30 UTC (rev 2815) +++ trunk/src/main/java/org/chorem/jtimer/JTimerConfig.java 2012-03-09 15:31:15 UTC (rev 2816) @@ -85,7 +85,7 @@ public Class getIOSaverClass() { return appConfig.getOptionAsClass(JTimerOption.IO_SAVER_CLASS.key); } - + public String getIOSaverDirectory() { return appConfig.getOption(JTimerOption.IO_SAVER_DIRECTORY.key); } @@ -108,6 +108,16 @@ return appConfig.getOptionAsLong(JTimerOption.UI_IDLE_TIME.key); } + /** + * Return first day of week. + * Default to {@code -1} (no preference). + * + * @return first day of week in report + */ + public int getReportFirstDayOfWeek() { + return appConfig.getOptionAsInt(JTimerOption.UI_REPORT_FIRSTDAYOFWEEK.key); + } + public enum JTimerOption { CONFIG_FILENAME(ApplicationConfig.CONFIG_FILE_NAME, "jtimer.properties"), SERVICE_CLASS("jtimer.service.class", null), @@ -116,7 +126,8 @@ IO_SAVER_CLASS("jtimer.io.saver.class", "org.chorem.jtimer.io.GTimerIncrementalSaver"), IO_SAVER_DIRECTORY("jtimer.io.saver.directory", "${user.home}/.gtimer"), IO_SAVER_AUTOSAVEDELAY("jtimer.io.saver.autosavedelay", "300"), - UI_IDLE_TIME("jtimer.ui.idletime", "300"); + UI_IDLE_TIME("jtimer.ui.idletime", "300"), + UI_REPORT_FIRSTDAYOFWEEK("jtimer.ui.report.firstdayofweek", "-1"); protected String key; Modified: trunk/src/main/java/org/chorem/jtimer/ui/report/ReportView.java =================================================================== --- trunk/src/main/java/org/chorem/jtimer/ui/report/ReportView.java 2012-03-09 15:03:30 UTC (rev 2815) +++ trunk/src/main/java/org/chorem/jtimer/ui/report/ReportView.java 2012-03-09 15:31:15 UTC (rev 2816) @@ -5,7 +5,7 @@ * $Id$ * $HeadURL$ * %% - * Copyright (C) 2008 - 2011 CodeLutin, Chatellier Eric + * Copyright (C) 2008 - 2012 CodeLutin, Chatellier Eric * %% * This program is free software: you can redistribute it and/or modify * it under the terms of the GNU General Public License as @@ -40,6 +40,7 @@ import java.util.ArrayList; import java.util.Calendar; import java.util.Collection; +import java.util.Date; import java.util.HashSet; import java.util.List; import java.util.Set; @@ -60,6 +61,7 @@ import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; +import org.chorem.jtimer.JTimer; import org.chorem.jtimer.data.TimerCore; import org.chorem.jtimer.entities.TimerProject; import org.chorem.jtimer.entities.TimerTask; @@ -159,7 +161,7 @@ * Get main view component. * * TODO use less complicated UI (no gbl) - * @return + * @return main component */ protected JComponent getMainComponent() { @@ -167,13 +169,21 @@ // init dates Calendar calendarBegin = Calendar.getInstance(); - calendarBegin.set(Calendar.DAY_OF_WEEK, calendarBegin - .getFirstDayOfWeek()); + int firstDayOfWeek = JTimer.config.getReportFirstDayOfWeek(); + if (firstDayOfWeek < 0 || firstDayOfWeek > 6) { + firstDayOfWeek = calendarBegin.getFirstDayOfWeek(); + } + calendarBegin.set(Calendar.DAY_OF_WEEK, firstDayOfWeek); calendarBegin.set(Calendar.HOUR, 0); calendarBegin.set(Calendar.MINUTE, 0); calendarBegin.set(Calendar.SECOND, 0); calendarBegin.set(Calendar.MILLISECOND, 0); + // calendar must be in current week by default + if (calendarBegin.getTime().after(new Date())) { + calendarBegin.add(Calendar.WEEK_OF_YEAR, -1); + } + // get end of week // take calendarBegin and add a week time Calendar calendarEnd = (Calendar) calendarBegin.clone();

1 0

r2815 - in trunk/src/main: java/org/chorem/jtimer/ui/report resources/ftl
by echatellier＠users.chorem.org March 9, 2012

March 9, 2012

Author: echatellier Date: 2012-03-09 16:03:30 +0100 (Fri, 09 Mar 2012) New Revision: 2815 Url: http://chorem.org/repositories/revision/jtimer/2815 Log: #283 : Sort task on reports Modified: trunk/src/main/java/org/chorem/jtimer/ui/report/ReportUtils.java trunk/src/main/resources/ftl/reportByDay.ftl trunk/src/main/resources/ftl/reportByDay_fr.ftl trunk/src/main/resources/ftl/reportByMonth.ftl trunk/src/main/resources/ftl/reportByMonth_fr.ftl trunk/src/main/resources/ftl/reportByProject.ftl trunk/src/main/resources/ftl/reportByProject_fr.ftl trunk/src/main/resources/ftl/reportByWeek.ftl trunk/src/main/resources/ftl/reportByWeek_fr.ftl trunk/src/main/resources/ftl/reportByYear.ftl trunk/src/main/resources/ftl/reportByYear_fr.ftl Modified: trunk/src/main/java/org/chorem/jtimer/ui/report/ReportUtils.java =================================================================== --- trunk/src/main/java/org/chorem/jtimer/ui/report/ReportUtils.java 2012-03-09 14:52:09 UTC (rev 2814) +++ trunk/src/main/java/org/chorem/jtimer/ui/report/ReportUtils.java 2012-03-09 15:03:30 UTC (rev 2815) @@ -545,4 +545,17 @@ return result; } + + /** + * Return task subtasks ordered by name. + * + * @param task task to get subtasks + * @return subtask ordered by name + * @since 1.4.0 + */ + public List<TimerTask> getSubTaskOrdered(TimerTask task) { + List<TimerTask> subTask = new ArrayList<TimerTask>(task.getSubTasks()); + TimerTaskHelper.sortTask(subTask); + return subTask; + } } Modified: trunk/src/main/resources/ftl/reportByDay.ftl =================================================================== --- trunk/src/main/resources/ftl/reportByDay.ftl 2012-03-09 14:52:09 UTC (rev 2814) +++ trunk/src/main/resources/ftl/reportByDay.ftl 2012-03-09 15:03:30 UTC (rev 2815) @@ -27,7 +27,7 @@ <#macro displaySubtaskReport task periodTime identation=0> <#local ident=identation + 1/> - <#list task.subTasks as subtask> + <#list utils.getSubTaskOrdered(task) as subtask> <#local taskTotalTime=utils.getDailyTotalTaskTime(subtask, periodTime?date)/> <#if taskTotalTime > 0> <#local taskTime=utils.getDailyTaskTime(subtask, periodTime?date)/> Modified: trunk/src/main/resources/ftl/reportByDay_fr.ftl =================================================================== --- trunk/src/main/resources/ftl/reportByDay_fr.ftl 2012-03-09 14:52:09 UTC (rev 2814) +++ trunk/src/main/resources/ftl/reportByDay_fr.ftl 2012-03-09 15:03:30 UTC (rev 2815) @@ -27,7 +27,7 @@ <#macro displaySubtaskReport task periodTime identation=0> <#local ident=identation + 1/> - <#list task.subTasks as subtask> + <#list utils.getSubTaskOrdered(task) as subtask> <#local taskTotalTime=utils.getDailyTotalTaskTime(subtask, periodTime?date)/> <#if taskTotalTime > 0> <#local taskTime=utils.getDailyTaskTime(subtask, periodTime?date)/> Modified: trunk/src/main/resources/ftl/reportByMonth.ftl =================================================================== --- trunk/src/main/resources/ftl/reportByMonth.ftl 2012-03-09 14:52:09 UTC (rev 2814) +++ trunk/src/main/resources/ftl/reportByMonth.ftl 2012-03-09 15:03:30 UTC (rev 2815) @@ -27,7 +27,7 @@ <#macro displaySubtaskReport task periodTime identation=0> <#local ident=identation + 1/> - <#list task.subTasks as subtask> + <#list utils.getSubTaskOrdered(task) as subtask> <#local taskTotalTime=utils.getMonthlyTotalTaskTime(subtask, periodTime?date, begin?date, end?date)/> <#if taskTotalTime > 0> <#local taskTime=utils.getMonthlyTaskTime(subtask, periodTime?date, begin?date, end?date)/> Modified: trunk/src/main/resources/ftl/reportByMonth_fr.ftl =================================================================== --- trunk/src/main/resources/ftl/reportByMonth_fr.ftl 2012-03-09 14:52:09 UTC (rev 2814) +++ trunk/src/main/resources/ftl/reportByMonth_fr.ftl 2012-03-09 15:03:30 UTC (rev 2815) @@ -27,7 +27,7 @@ <#macro displaySubtaskReport task periodTime identation=0> <#local ident=identation + 1/> - <#list task.subTasks as subtask> + <#list utils.getSubTaskOrdered(task) as subtask> <#local taskTotalTime=utils.getMonthlyTotalTaskTime(subtask, periodTime?date, begin?date, end?date)/> <#if taskTotalTime > 0> <#local taskTime=utils.getMonthlyTaskTime(subtask, periodTime?date, begin?date, end?date)/> Modified: trunk/src/main/resources/ftl/reportByProject.ftl =================================================================== --- trunk/src/main/resources/ftl/reportByProject.ftl 2012-03-09 14:52:09 UTC (rev 2814) +++ trunk/src/main/resources/ftl/reportByProject.ftl 2012-03-09 15:03:30 UTC (rev 2815) @@ -28,7 +28,7 @@ <#assign periods=utils.getDailyDates(begin?date,end?date)/> <#macro displaySubtaskReport task identation=0> <#local ident=identation + 1/> - <#list task.subTasks as subtask> + <#list utils.getSubTaskOrdered(task) as subtask> <#local taskTotalTime=0/> <#list periods as period> <#local subtaskPeriodTotalTime=utils.getDailyTotalTaskTime(subtask, period?date)/> Modified: trunk/src/main/resources/ftl/reportByProject_fr.ftl =================================================================== --- trunk/src/main/resources/ftl/reportByProject_fr.ftl 2012-03-09 14:52:09 UTC (rev 2814) +++ trunk/src/main/resources/ftl/reportByProject_fr.ftl 2012-03-09 15:03:30 UTC (rev 2815) @@ -28,7 +28,7 @@ <#assign periods=utils.getDailyDates(begin?date,end?date)/> <#macro displaySubtaskReport task identation=0> <#local ident=identation + 1/> - <#list task.subTasks as subtask> + <#list utils.getSubTaskOrdered(task) as subtask> <#local taskTotalTime=0/> <#list periods as period> <#local subtaskPeriodTotalTime=utils.getDailyTotalTaskTime(subtask, period?date)/> Modified: trunk/src/main/resources/ftl/reportByWeek.ftl =================================================================== --- trunk/src/main/resources/ftl/reportByWeek.ftl 2012-03-09 14:52:09 UTC (rev 2814) +++ trunk/src/main/resources/ftl/reportByWeek.ftl 2012-03-09 15:03:30 UTC (rev 2815) @@ -27,7 +27,7 @@ <#macro displaySubtaskReport task periodTime identation=0> <#local ident=identation + 1/> - <#list task.subTasks as subtask> + <#list utils.getSubTaskOrdered(task) as subtask> <#local taskTotalTime=utils.getWeeklyTotalTaskTime(subtask, periodTime?date, begin?date, end?date)/> <#if taskTotalTime > 0> <#local taskTime=utils.getWeeklyTaskTime(subtask, periodTime?date, begin?date, end?date)/> Modified: trunk/src/main/resources/ftl/reportByWeek_fr.ftl =================================================================== --- trunk/src/main/resources/ftl/reportByWeek_fr.ftl 2012-03-09 14:52:09 UTC (rev 2814) +++ trunk/src/main/resources/ftl/reportByWeek_fr.ftl 2012-03-09 15:03:30 UTC (rev 2815) @@ -27,7 +27,7 @@ <#macro displaySubtaskReport task periodTime identation=0> <#local ident=identation + 1/> - <#list task.subTasks as subtask> + <#list utils.getSubTaskOrdered(task) as subtask> <#local taskTotalTime=utils.getWeeklyTotalTaskTime(subtask, periodTime?date, begin?date, end?date)/> <#if taskTotalTime > 0> <#local taskTime=utils.getWeeklyTaskTime(subtask, periodTime?date, begin?date, end?date)/> Modified: trunk/src/main/resources/ftl/reportByYear.ftl =================================================================== --- trunk/src/main/resources/ftl/reportByYear.ftl 2012-03-09 14:52:09 UTC (rev 2814) +++ trunk/src/main/resources/ftl/reportByYear.ftl 2012-03-09 15:03:30 UTC (rev 2815) @@ -27,7 +27,7 @@ <#macro displaySubtaskReport task periodTime identation=0> <#local ident=identation + 1/> - <#list task.subTasks as subtask> + <#list utils.getSubTaskOrdered(task) as subtask> <#local taskTotalTime=utils.getYearlyTotalTaskTime(subtask, periodTime?date, begin?date, end?date)/> <#if taskTotalTime > 0> <#local taskTime=utils.getYearlyTaskTime(subtask, periodTime?date, begin?date, end?date)/> Modified: trunk/src/main/resources/ftl/reportByYear_fr.ftl =================================================================== --- trunk/src/main/resources/ftl/reportByYear_fr.ftl 2012-03-09 14:52:09 UTC (rev 2814) +++ trunk/src/main/resources/ftl/reportByYear_fr.ftl 2012-03-09 15:03:30 UTC (rev 2815) @@ -27,7 +27,7 @@ <#macro displaySubtaskReport task periodTime identation=0> <#local ident=identation + 1/> - <#list task.subTasks as subtask> + <#list utils.getSubTaskOrdered(task) as subtask> <#local taskTotalTime=utils.getYearlyTotalTaskTime(subtask, periodTime?date, begin?date, end?date)/> <#if taskTotalTime > 0> <#local taskTime=utils.getYearlyTaskTime(subtask, periodTime?date, begin?date, end?date)/>

1 0

r2814 - in trunk/src/main/java/org/chorem/jtimer: data entities io ui ui/report/tree ui/systray ui/treetable ws/xmlrpc
by echatellier＠users.chorem.org March 9, 2012

March 9, 2012

Author: echatellier Date: 2012-03-09 15:52:09 +0100 (Fri, 09 Mar 2012) New Revision: 2814 Url: http://chorem.org/repositories/revision/jtimer/2814 Log: #78 : Renaming a task break tasks tree rendering Modified: trunk/src/main/java/org/chorem/jtimer/data/DataEventListener.java trunk/src/main/java/org/chorem/jtimer/data/TimerDataManager.java trunk/src/main/java/org/chorem/jtimer/entities/TimerObject.java trunk/src/main/java/org/chorem/jtimer/entities/TimerTask.java trunk/src/main/java/org/chorem/jtimer/entities/TimerTaskHelper.java trunk/src/main/java/org/chorem/jtimer/io/GTimerIncrementalSaver.java trunk/src/main/java/org/chorem/jtimer/ui/StatusBar.java trunk/src/main/java/org/chorem/jtimer/ui/report/tree/CheckBoxTreeModel.java trunk/src/main/java/org/chorem/jtimer/ui/systray/SystrayManager.java trunk/src/main/java/org/chorem/jtimer/ui/treetable/ProjectsAndTasksCellRenderer.java trunk/src/main/java/org/chorem/jtimer/ui/treetable/ProjectsAndTasksModel.java trunk/src/main/java/org/chorem/jtimer/ws/xmlrpc/ChoremXMLRPCClient.java Modified: trunk/src/main/java/org/chorem/jtimer/data/DataEventListener.java =================================================================== --- trunk/src/main/java/org/chorem/jtimer/data/DataEventListener.java 2012-03-09 14:46:31 UTC (rev 2813) +++ trunk/src/main/java/org/chorem/jtimer/data/DataEventListener.java 2012-03-09 14:52:09 UTC (rev 2814) @@ -70,26 +70,8 @@ * @param task modified task */ public void modifyTask(TimerTask task); - - /** - * Before delete project. - * - * TODO only usefull for UI (can't known on to delete if already deleted) - * - * @param project deleted project - */ - public void preDeleteProject(TimerProject project); /** - * Before delete task. - * - * TODO only usefull for UI (can't known on to delete if already deleted) - * - * @param task deleted task - */ - public void preDeleteTask(TimerTask task); - - /** * Delete project. * * @param project deleted project Modified: trunk/src/main/java/org/chorem/jtimer/data/TimerDataManager.java =================================================================== --- trunk/src/main/java/org/chorem/jtimer/data/TimerDataManager.java 2012-03-09 14:46:31 UTC (rev 2813) +++ trunk/src/main/java/org/chorem/jtimer/data/TimerDataManager.java 2012-03-09 14:52:09 UTC (rev 2814) @@ -243,18 +243,11 @@ vetoableDataEventListener.checkDeleteTask(task); } - // send notification (before) - Iterator<DataEventListener> itDataEventListener = dataEventListeners - .iterator(); - while (itDataEventListener.hasNext()) { - itDataEventListener.next().preDeleteTask(task); - } - // task deletion task.getParent().getSubTasks().remove(task); // send notification - itDataEventListener = dataEventListeners.iterator(); + Iterator<DataEventListener> itDataEventListener = dataEventListeners.iterator(); while (itDataEventListener.hasNext()) { itDataEventListener.next().deleteTask(task); } @@ -272,17 +265,10 @@ vetoableDataEventListener.checkDeleteProject(project); } - // send notification (before) - Iterator<DataEventListener> itDataEventListener = dataEventListeners - .iterator(); - while (itDataEventListener.hasNext()) { - itDataEventListener.next().preDeleteProject(project); - } - projectList.remove(project); // send notification - itDataEventListener = dataEventListeners.iterator(); + Iterator<DataEventListener> itDataEventListener = dataEventListeners.iterator(); while (itDataEventListener.hasNext()) { itDataEventListener.next().deleteProject(project); } Modified: trunk/src/main/java/org/chorem/jtimer/entities/TimerObject.java =================================================================== --- trunk/src/main/java/org/chorem/jtimer/entities/TimerObject.java 2012-03-09 14:46:31 UTC (rev 2813) +++ trunk/src/main/java/org/chorem/jtimer/entities/TimerObject.java 2012-03-09 14:52:09 UTC (rev 2814) @@ -3,7 +3,7 @@ * $Id$ * $HeadURL$ * %% - * Copyright (C) 2011 Codelutin, Chatellier Eric + * Copyright (C) 2011 - 2012 Codelutin, Chatellier Eric * %% * This program is free software: you can redistribute it and/or modify * it under the terms of the GNU General Public License as Modified: trunk/src/main/java/org/chorem/jtimer/entities/TimerTask.java =================================================================== --- trunk/src/main/java/org/chorem/jtimer/entities/TimerTask.java 2012-03-09 14:46:31 UTC (rev 2813) +++ trunk/src/main/java/org/chorem/jtimer/entities/TimerTask.java 2012-03-09 14:52:09 UTC (rev 2814) @@ -31,6 +31,7 @@ import java.util.List; import java.util.SortedMap; import java.util.TreeMap; +import java.util.UUID; import org.apache.commons.lang3.SerializationUtils; import org.chorem.jtimer.utils.DailySortedMap; @@ -64,6 +65,9 @@ public static final String PROPERTY_SUB_TASKS = "subTasks"; public static final String PROPERTY_ALERTS = "alerts"; + /** Task uuid used to managed task equality. */ + protected String uuid = UUID.randomUUID().toString(); + /** Task number. */ protected int number; @@ -369,23 +373,12 @@ /* * @see java.lang.Object#hashCode() - * - * TODO must be added, but cause some errors on tree cell renderer. - * + */ @Override public int hashCode() { - final int prime = 31; - int result = 1; - result = prime * result + ((alerts == null) ? 0 : alerts.hashCode()); - result = prime * result + ((allDaysAndTimes == null) ? 0 : allDaysAndTimes.hashCode()); - result = prime * result + ((allDaysAnnotations == null) ? 0 : allDaysAnnotations.hashCode()); - result = prime * result + (closed ? 1231 : 1237); - result = prime * result + ((creationDate == null) ? 0 : creationDate.hashCode()); - result = prime * result + ((name == null) ? 0 : name.hashCode()); - result = prime * result + number; - result = prime * result + ((subTasks == null) ? 0 : subTasks.hashCode()); + int result = uuid.hashCode(); return result; - }*/ + } /* * @see java.lang.Object#equals(java.lang.Object) @@ -397,21 +390,7 @@ return false; } - TimerTask otherTask = (TimerTask) o; - - // name equals - boolean equals = name != null && name.equals(otherTask.name); - // creation date equals - equals &= creationDate == null - || creationDate.equals(otherTask.creationDate); - // subtask equals - equals &= subTasks.equals(otherTask.subTasks); - // compare times - equals &= allDaysTimes.equals(otherTask.allDaysTimes); - // compare times - equals &= allDaysAnnotations.equals(otherTask.allDaysAnnotations); - - return equals; + return uuid.equals(((TimerTask)o).uuid); } /** Modified: trunk/src/main/java/org/chorem/jtimer/entities/TimerTaskHelper.java =================================================================== --- trunk/src/main/java/org/chorem/jtimer/entities/TimerTaskHelper.java 2012-03-09 14:46:31 UTC (rev 2813) +++ trunk/src/main/java/org/chorem/jtimer/entities/TimerTaskHelper.java 2012-03-09 14:52:09 UTC (rev 2814) @@ -55,33 +55,47 @@ } /** + * Timer task comparator. + */ + protected static Comparator<TimerTask> timerTaskComparator = new Comparator<TimerTask>() { + public int compare(TimerTask o1, TimerTask o2) { + if (o1 == null && o2 == null) { + return 0; + } + if (o1 == null) { + return 1; + } + if (o2 == null) { + return -1; + } + return o1.getName().compareTo(o2.getName()); + } + }; + + /** * Get task's subtasks ordered alphabetically. * * @param task task * @return the subTasks ordered + * @deprecated since 1.4.0 use {@link #sortTask(task.getSubTasks())} */ public static List<TimerTask> getSubTasksOrderedByName(TimerTask task) { List<TimerTask> taksToSort = task.getSubTasks(); - List<TimerTask> subTasksCopy = new ArrayList<TimerTask>(taksToSort); - Comparator<TimerTask> c = new Comparator<TimerTask>() { - public int compare(TimerTask o1, TimerTask o2) { - if (o1 == null && o2 == null) { - return 0; - } - if (o1 == null) { - return 1; - } - if (o2 == null) { - return -1; - } - return o1.getName().compareTo(o2.getName()); - } - }; - Collections.sort(subTasksCopy, c); - return subTasksCopy; + return sortTask(taksToSort); } /** + * Get tasks ordered alphabetically. + * + * @param tasks tasks + * @return {@code tasks} + */ + public static List<TimerTask> sortTask(List<TimerTask> tasks) { + Collections.sort(tasks, timerTaskComparator); + return tasks; + } + + /** * Get total task time without subtasks time. * * @param task task to get time Modified: trunk/src/main/java/org/chorem/jtimer/io/GTimerIncrementalSaver.java =================================================================== --- trunk/src/main/java/org/chorem/jtimer/io/GTimerIncrementalSaver.java 2012-03-09 14:46:31 UTC (rev 2813) +++ trunk/src/main/java/org/chorem/jtimer/io/GTimerIncrementalSaver.java 2012-03-09 14:52:09 UTC (rev 2814) @@ -1214,22 +1214,6 @@ } /* - * @see org.chorem.jtimer.data.DataEventListener#preDeleteProject(org.chorem.jtimer.entities.TimerProject) - */ - @Override - public void preDeleteProject(TimerProject project) { - - } - - /* - * @see org.chorem.jtimer.data.DataEventListener#preDeleteTask(org.chorem.jtimer.entities.TimerTask) - */ - @Override - public void preDeleteTask(TimerTask task) { - - } - - /* * @see org.chorem.jtimer.data.event.DataEventListener#deleteProject(org.chorem.jtimer.entities.TimerProject) */ @Override Modified: trunk/src/main/java/org/chorem/jtimer/ui/StatusBar.java =================================================================== --- trunk/src/main/java/org/chorem/jtimer/ui/StatusBar.java 2012-03-09 14:46:31 UTC (rev 2813) +++ trunk/src/main/java/org/chorem/jtimer/ui/StatusBar.java 2012-03-09 14:52:09 UTC (rev 2814) @@ -151,22 +151,6 @@ } /* - * @see org.chorem.jtimer.data.DataEventListener#preDeleteProject(org.chorem.jtimer.entities.TimerProject) - */ - @Override - public void preDeleteProject(TimerProject project) { - - } - - /* - * @see org.chorem.jtimer.data.DataEventListener#preDeleteTask(org.chorem.jtimer.entities.TimerTask) - */ - @Override - public void preDeleteTask(TimerTask task) { - - } - - /* * @see org.chorem.jtimer.data.event.DataEventListener#deleteProject(org.chorem.jtimer.entities.TimerProject) */ @Override Modified: trunk/src/main/java/org/chorem/jtimer/ui/report/tree/CheckBoxTreeModel.java =================================================================== --- trunk/src/main/java/org/chorem/jtimer/ui/report/tree/CheckBoxTreeModel.java 2012-03-09 14:46:31 UTC (rev 2813) +++ trunk/src/main/java/org/chorem/jtimer/ui/report/tree/CheckBoxTreeModel.java 2012-03-09 14:52:09 UTC (rev 2814) @@ -5,7 +5,7 @@ * $Id$ * $HeadURL$ * %% - * Copyright (C) 2009 - 2011 CodeLutin, Chatellier Eric + * Copyright (C) 2009 - 2012 CodeLutin, Chatellier Eric * %% * This program is free software: you can redistribute it and/or modify * it under the terms of the GNU General Public License as @@ -126,16 +126,12 @@ result.addAll(core.getData().getProjectsList()); } else { // not root node TimerTask task = (TimerTask) parent; - - // Since sort is not supported by the table, do a manual sorting. - //result.addAll(t.getSubTasks()); - result.addAll(TimerTaskHelper.getSubTasksOrderedByName(task)); + result.addAll(task.getSubTasks()); } // filter list, if only show closed if (!showClosed) { CollectionUtils.filter(result, new Predicate() { - @Override public boolean evaluate(Object object) { @@ -146,10 +142,12 @@ } return result; } - }); } + // Since sort is not supported by the table, do a manual sorting. + result = TimerTaskHelper.sortTask(result); + return result; } Modified: trunk/src/main/java/org/chorem/jtimer/ui/systray/SystrayManager.java =================================================================== --- trunk/src/main/java/org/chorem/jtimer/ui/systray/SystrayManager.java 2012-03-09 14:46:31 UTC (rev 2813) +++ trunk/src/main/java/org/chorem/jtimer/ui/systray/SystrayManager.java 2012-03-09 14:52:09 UTC (rev 2814) @@ -222,22 +222,6 @@ } /* - * @see org.chorem.jtimer.data.DataEventListener#preDeleteProject(org.chorem.jtimer.entities.TimerProject) - */ - @Override - public void preDeleteProject(TimerProject project) { - - } - - /* - * @see org.chorem.jtimer.data.DataEventListener#preDeleteTask(org.chorem.jtimer.entities.TimerTask) - */ - @Override - public void preDeleteTask(TimerTask task) { - - } - - /* * @see org.chorem.jtimer.data.event.DataEventListener#deleteProject(org.chorem.jtimer.entities.TimerProject) */ @Override Modified: trunk/src/main/java/org/chorem/jtimer/ui/treetable/ProjectsAndTasksCellRenderer.java =================================================================== --- trunk/src/main/java/org/chorem/jtimer/ui/treetable/ProjectsAndTasksCellRenderer.java 2012-03-09 14:46:31 UTC (rev 2813) +++ trunk/src/main/java/org/chorem/jtimer/ui/treetable/ProjectsAndTasksCellRenderer.java 2012-03-09 14:52:09 UTC (rev 2814) @@ -202,22 +202,6 @@ } /* - * @see org.chorem.jtimer.data.DataEventListener#preDeleteProject(org.chorem.jtimer.entities.TimerProject) - */ - @Override - public void preDeleteProject(TimerProject project) { - - } - - /* - * @see org.chorem.jtimer.data.DataEventListener#preDeleteTask(org.chorem.jtimer.entities.TimerTask) - */ - @Override - public void preDeleteTask(TimerTask task) { - - } - - /* * @see org.chorem.jtimer.data.event.DataEventListener#deleteProject(org.chorem.jtimer.entities.TimerProject) */ @Override Modified: trunk/src/main/java/org/chorem/jtimer/ui/treetable/ProjectsAndTasksModel.java =================================================================== --- trunk/src/main/java/org/chorem/jtimer/ui/treetable/ProjectsAndTasksModel.java 2012-03-09 14:46:31 UTC (rev 2813) +++ trunk/src/main/java/org/chorem/jtimer/ui/treetable/ProjectsAndTasksModel.java 2012-03-09 14:52:09 UTC (rev 2814) @@ -28,7 +28,9 @@ import java.util.ArrayList; import java.util.Collection; import java.util.Date; +import java.util.HashMap; import java.util.List; +import java.util.Map; import javax.swing.SwingUtilities; import javax.swing.table.TableColumn; @@ -45,7 +47,6 @@ import org.chorem.jtimer.entities.TimerProject; import org.chorem.jtimer.entities.TimerTask; import org.chorem.jtimer.entities.TimerTaskHelper; -import org.jdesktop.swingx.tree.TreeModelSupport; import org.jdesktop.swingx.treetable.AbstractTreeTableModel; /** @@ -54,6 +55,18 @@ * Cela permet au programme principal de manipuler la liste et d'oublier la * gestion de l'arbre... * + * Le modele de cet arbre utilise directement des TimerTask au lieu des TreeTableNode + * qui ne servent a rien et qui complique tout. Par contre, les tâches étant + * les mêmes instances que celle manipulées par le programme, il doit mettre + * en cache certains infos pour un rafraichissement correct: + * <ul> + * <li>Nom : cas de renommage avec changement d'ordre + * <li>Sous tache : pour supprimer une tache qui disparaitrait directement du modèle + * </ul> + * + * Voir cet article pour la non utilisation des TreeNode : + * http://www.javalobby.org/forums/thread.jspa?threadID=16052&tstart=0 + * * @author chatellier * @version $Revision$ * @@ -66,14 +79,14 @@ /** log */ private static Log log = LogFactory.getLog(ProjectsAndTasksModel.class); - /** Add node operation */ + /** Add node operation. */ public static final int OPERATION_ADD = 0; - /** Modify node operation */ + /** Modify node operation. */ public static final int OPERATION_MODIFY = 1; - /** Delete Node operation */ + /** Delete Node operation. */ public static final int OPERATION_DELETE = 2; - /** Tree managed by this model */ + /** Tree managed by this model. */ protected ProjectsAndTasksTable projectsAndTaskTable; /** Data. */ @@ -81,10 +94,13 @@ /** Tree column identifiers. */ protected List<String> columnIdentifiers; - + /** Show closed task property. Default to false. */ protected boolean showClosedTask; + protected Map<TimerTask, String> taskNameCache; + protected Map<Object, List<TimerTask>> subTasksCache; + /** * Constructor. * @@ -94,24 +110,17 @@ */ public ProjectsAndTasksModel(ProjectsAndTasksTable projectsAndTaskTable, TimerCore core, List<String> columnIdentifiers) { - // super with root node super(new TimerProject("root")); // remember this.projectsAndTaskTable = projectsAndTaskTable; this.dataManager = core.getData(); this.columnIdentifiers = columnIdentifiers; + + taskNameCache = new HashMap<TimerTask, String>(); + subTasksCache = new HashMap<Object, List<TimerTask>>(); } - /** - * Accessor to tree model support. - * - * @return tree model support - */ - protected TreeModelSupport getModelSupport() { - return modelSupport; - } - /* * @see org.jdesktop.swingx.treetable.TreeTableModel#getColumnCount() */ @@ -178,8 +187,8 @@ */ @Override public Object getChild(Object parent, int index) { + TimerTask t = getFiteredSubListFor(parent).get(index); - return t; } @@ -190,48 +199,69 @@ public int getChildCount(Object parent) { int childCount = getFiteredSubListFor(parent).size(); - return childCount; } /** * Recupere la sous liste: data.getProjectsList() si parent = root - * getSubTasks() sinon + * getSubTasks() sinon (cache results). * * @param parent parent to task sublist * @return filtered list */ protected List<TimerTask> getFiteredSubListFor(Object parent) { + return getFiteredSubListFor(parent, false); + } - List<TimerTask> result = new ArrayList<TimerTask>(); + /** + * Recupere la sous liste: data.getProjectsList() si parent = root + * getSubTasks() sinon. + * + * @param parent parent to task sublist + * @param noCache disable use of cached result and result caching + * @return filtered list + */ + protected List<TimerTask> getFiteredSubListFor(Object parent, boolean noCache) { - // get correct list - if (parent == root) { // case root node - result.addAll(dataManager.getProjectsList()); - } else { // not root node - TimerTask task = (TimerTask) parent; + List<TimerTask> result = subTasksCache.get(parent); + if (result == null || noCache) { + result = new ArrayList<TimerTask>(); - // Since sort is not supported by the table, do a manual sorting. - //result.addAll(t.getSubTasks()); - result.addAll(TimerTaskHelper.getSubTasksOrderedByName(task)); - } + // get correct list + if (parent == root) { // case root node + List<TimerProject> projects = dataManager.getProjectsList(); + result.addAll(projects); + } else { // not root node + TimerTask task = (TimerTask) parent; + result.addAll(task.getSubTasks()); + } - // filter list, if only show closed - if (!showClosedTask) { - CollectionUtils.filter(result, new Predicate() { + // filter list, if only show closed + if (!showClosedTask) { + CollectionUtils.filter(result, new Predicate() { + @Override + public boolean evaluate(Object object) { + boolean result = false; + if (object instanceof TimerTask) { + TimerTask task = (TimerTask) object; + result = !task.isClosed(); + } + return result; + } + }); + } - @Override - public boolean evaluate(Object object) { - - boolean result = false; - if (object instanceof TimerTask) { - TimerTask task = (TimerTask) object; - result = !task.isClosed(); - } - return result; + // Since sort is not supported by the table, do a manual sorting. + result = TimerTaskHelper.sortTask(result); + + if (!noCache) { + // cache tasks name + for (TimerTask task : result) { + taskNameCache.put(task, task.getName()); } - - }); + + subTasksCache.put(parent, result); + } } return result; @@ -244,7 +274,6 @@ public int getIndexOfChild(Object parent, Object child) { int childIndex = getFiteredSubListFor(parent).indexOf(child); - return childIndex; } @@ -265,7 +294,6 @@ protected void notifyTaskChanged(TimerTask task, int operation) { // this.modelSupport. TreePath path = new TreePath(root); - updateChildren(task, path, operation); } @@ -277,56 +305,66 @@ * @param operation operation (add, modify, delete) * @return <tt>true</tt> if children have been found and updated in current recursion */ - protected boolean updateChildren(TimerTask task, TreePath path, + protected synchronized boolean updateChildren(TimerTask task, TreePath path, int operation) { TimerTask pathLastComponent = (TimerTask) path.getLastPathComponent(); - // pour ses enfants boolean updated = false; - int childCount = getChildCount(pathLastComponent); + // get childreen without cache in case of add operation + // delete operation MUST use cached result + List<TimerTask> subTask = getFiteredSubListFor(pathLastComponent, operation == OPERATION_ADD); + int childCount = subTask.size(); for (int childIndex = 0; !updated && childIndex < childCount; ++childIndex) { - TimerTask taskUO = (TimerTask) getChild(pathLastComponent, - childIndex); + TimerTask taskUO = (TimerTask) subTask.get(childIndex); - TreePath childTreePath = path.pathByAddingChild(taskUO); - if (task.equals(taskUO)) { // this update only node, not all path... switch (operation) { - case OPERATION_ADD: - getModelSupport().fireChildAdded(path, childIndex, taskUO); + case OPERATION_ADD: { + subTasksCache.remove(pathLastComponent); // force child recache + int newIndex = getIndexOfChild(pathLastComponent, taskUO); + modelSupport.fireChildAdded(path, newIndex, taskUO); // expand path projectsAndTaskTable.expandPath(path); break; - - case OPERATION_DELETE: - getModelSupport() - .fireChildRemoved(path, childIndex, taskUO); + } + case OPERATION_DELETE: { + subTasksCache.remove(pathLastComponent); // invalidate cache + subTasksCache.remove(taskUO); // no in cache anymore + taskNameCache.remove(taskUO); // no in cache anymore + modelSupport.fireChildRemoved(path, childIndex, taskUO); break; - - case OPERATION_MODIFY: - getModelSupport() - .fireChildChanged(path, childIndex, taskUO); + } + case OPERATION_MODIFY: { + String cacheName = taskNameCache.get(taskUO); + if (cacheName.equals(task.getName())) { + // not name modification (time...) + modelSupport.fireChildChanged(path, childIndex, taskUO); + } else { + // name modification (order can change) + modelSupport.fireChildRemoved(path, childIndex, taskUO); + subTasksCache.remove(pathLastComponent); // force child recache + int newIndex = getIndexOfChild(pathLastComponent, taskUO); + modelSupport.fireChildAdded(path, newIndex, taskUO); + } break; - + } default: if (log.isErrorEnabled()) { log.error("Unknow operation : " + operation); } } - if (log.isTraceEnabled()) { - log.trace(" updated : " + childTreePath); - } updated = true; } else { + TreePath childTreePath = path.pathByAddingChild(taskUO); updated = updateChildren(task, childTreePath, operation); if (updated) { // ...and by recursion update all path - getModelSupport().firePathChanged(path); + modelSupport.firePathChanged(path); } } } @@ -362,9 +400,12 @@ tcs.add(tc); } + taskNameCache.clear(); + subTasksCache.clear(); + // works but it's not beautiful // and header are resized - getModelSupport().fireNewRoot(); + modelSupport.fireNewRoot(); // hack, restore size after SwingUtilities.invokeLater(new Runnable() { @@ -377,29 +418,12 @@ }); } - /* - * @see org.chorem.jtimer.data.DataEventListener#preDeleteProject(org.chorem.jtimer.entities.TimerProject) - */ - @Override - public void preDeleteProject(TimerProject project) { - notifyTaskChanged(project, OPERATION_DELETE); - } - - /* - * @see org.chorem.jtimer.data.DataEventListener#preDeleteTask(org.chorem.jtimer.entities.TimerTask) - */ - @Override - public void preDeleteTask(TimerTask task) { - notifyTaskChanged(task, OPERATION_DELETE); - } - - /* * @see org.chorem.jtimer.data.event.DataEventListener#deleteProject(org.chorem.jtimer.entities.TimerProject) */ @Override public void deleteProject(TimerProject project) { - + notifyTaskChanged(project, OPERATION_DELETE); } /* @@ -407,7 +431,7 @@ */ @Override public void deleteTask(TimerTask task) { - + notifyTaskChanged(task, OPERATION_DELETE); } /* @@ -501,8 +525,8 @@ */ public void setShowClosed(boolean showClosedTask) { this.showClosedTask = showClosedTask; - // TODO add better diff code - getModelSupport().fireNewRoot(); + // TODO echatellier 20120309 add better diff code + dataLoaded(null); } /* Modified: trunk/src/main/java/org/chorem/jtimer/ws/xmlrpc/ChoremXMLRPCClient.java =================================================================== --- trunk/src/main/java/org/chorem/jtimer/ws/xmlrpc/ChoremXMLRPCClient.java 2012-03-09 14:46:31 UTC (rev 2813) +++ trunk/src/main/java/org/chorem/jtimer/ws/xmlrpc/ChoremXMLRPCClient.java 2012-03-09 14:52:09 UTC (rev 2814) @@ -5,7 +5,7 @@ * $Id$ * $HeadURL$ * %% - * Copyright (C) 2007 - 2011 CodeLutin, Chatellier Eric + * Copyright (C) 2007 - 2012 CodeLutin, Chatellier Eric * %% * This program is free software: you can redistribute it and/or modify * it under the terms of the GNU General Public License as @@ -735,22 +735,6 @@ } /* - * @see org.chorem.jtimer.data.DataEventListener#preDeleteProject(org.chorem.jtimer.entities.TimerProject) - */ - @Override - public void preDeleteProject(TimerProject project) { - - } - - /* - * @see org.chorem.jtimer.data.DataEventListener#preDeleteTask(org.chorem.jtimer.entities.TimerTask) - */ - @Override - public void preDeleteTask(TimerTask task) { - - } - - /* * @see org.chorem.jtimer.event.DataEventListener#deleteProject(org.chorem.jtimer.entities.TimerProject) */ @Override

1 0

r2813 - trunk/src/main/java/org/chorem/jtimer
by echatellier＠users.chorem.org March 9, 2012

March 9, 2012

Author: echatellier Date: 2012-03-09 15:46:31 +0100 (Fri, 09 Mar 2012) New Revision: 2813 Url: http://chorem.org/repositories/revision/jtimer/2813 Log: #483 : Delete running task still tacking time Modified: trunk/src/main/java/org/chorem/jtimer/JTimer.java Modified: trunk/src/main/java/org/chorem/jtimer/JTimer.java =================================================================== --- trunk/src/main/java/org/chorem/jtimer/JTimer.java 2012-03-07 17:24:20 UTC (rev 2812) +++ trunk/src/main/java/org/chorem/jtimer/JTimer.java 2012-03-09 14:46:31 UTC (rev 2813) @@ -680,6 +680,7 @@ * Stop a task. * * Delegate method. Called by idle detect ? + * Called by delete method too. * * @param task task to stop */ @@ -687,13 +688,14 @@ RunTaskJob rtt = getJobForRunningTask(task); // test if task is already running - // should never be call if null !!! - rtt.wantToStop(); - core.getData().stopTask(task); - - // re-enable/disable buttons - setSelectedSingleRunningTask(false); - setSelectedSingleStoppedTask(true); + if (rtt != null) { + rtt.wantToStop(); + core.getData().stopTask(task); + + // re-enable/disable buttons + setSelectedSingleRunningTask(false); + setSelectedSingleStoppedTask(true); + } } /** @@ -774,6 +776,7 @@ if (confirm == JOptionPane.YES_OPTION) { // approved try { + stopTask(task); core.getData().deleteTask(task); } catch (DataViolationException e) { displayErrorMessage(e.getExceptionKey());

1 0

r2812 - in trunk/src: main/java/org/chorem/jtimer/ui main/java/org/chorem/jtimer/ui/tasks test/java/org/chorem/jtimer/io
by echatellier＠users.chorem.org March 7, 2012

March 7, 2012

Author: echatellier Date: 2012-03-07 18:24:20 +0100 (Wed, 07 Mar 2012) New Revision: 2812 Url: http://chorem.org/repositories/revision/jtimer/2812 Log: Fix import/unused code Modified: trunk/src/main/java/org/chorem/jtimer/ui/HelpFrame.java trunk/src/main/java/org/chorem/jtimer/ui/tasks/RunTaskJob.java trunk/src/test/java/org/chorem/jtimer/io/GTimerIncrementalSaverTest.java Modified: trunk/src/main/java/org/chorem/jtimer/ui/HelpFrame.java =================================================================== --- trunk/src/main/java/org/chorem/jtimer/ui/HelpFrame.java 2012-03-07 17:21:15 UTC (rev 2811) +++ trunk/src/main/java/org/chorem/jtimer/ui/HelpFrame.java 2012-03-07 17:24:20 UTC (rev 2812) @@ -5,7 +5,7 @@ * $Id$ * $HeadURL$ * %% - * Copyright (C) 2008 - 2011 CodeLutin, Chatellier Eric + * Copyright (C) 2008 - 2012 CodeLutin, Chatellier Eric * %% * This program is free software: you can redistribute it and/or modify * it under the terms of the GNU General Public License as @@ -62,9 +62,6 @@ */ public class HelpFrame extends FrameView implements HyperlinkListener { - /** serialVersionUID */ - private static final long serialVersionUID = -8693584537185015506L; - /** log */ private static Log log = LogFactory.getLog(HelpFrame.class); Modified: trunk/src/main/java/org/chorem/jtimer/ui/tasks/RunTaskJob.java =================================================================== --- trunk/src/main/java/org/chorem/jtimer/ui/tasks/RunTaskJob.java 2012-03-07 17:21:15 UTC (rev 2811) +++ trunk/src/main/java/org/chorem/jtimer/ui/tasks/RunTaskJob.java 2012-03-07 17:24:20 UTC (rev 2812) @@ -38,7 +38,6 @@ import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; import org.chorem.jtimer.JTimer; -import org.chorem.jtimer.JTimerFactory; import org.chorem.jtimer.data.TimerDataManager; import org.chorem.jtimer.entities.TimerAlert; import org.chorem.jtimer.entities.TimerTask; Modified: trunk/src/test/java/org/chorem/jtimer/io/GTimerIncrementalSaverTest.java =================================================================== --- trunk/src/test/java/org/chorem/jtimer/io/GTimerIncrementalSaverTest.java 2012-03-07 17:21:15 UTC (rev 2811) +++ trunk/src/test/java/org/chorem/jtimer/io/GTimerIncrementalSaverTest.java 2012-03-07 17:24:20 UTC (rev 2812) @@ -5,7 +5,7 @@ * $Id$ * $HeadURL$ * %% - * Copyright (C) 2008 - 2011 CodeLutin, Chatellier Eric + * Copyright (C) 2008 - 2012 CodeLutin, Chatellier Eric * %% * This program is free software: you can redistribute it and/or modify * it under the terms of the GNU General Public License as @@ -35,8 +35,6 @@ import java.util.SortedMap; import java.util.TreeMap; -import org.apache.commons.logging.Log; -import org.apache.commons.logging.LogFactory; import org.chorem.jtimer.AbstractJTimerTest; import org.chorem.jtimer.entities.TimerAlert; import org.chorem.jtimer.entities.TimerProject; @@ -56,9 +54,6 @@ */ public class GTimerIncrementalSaverTest extends AbstractJTimerTest { - /** Class log */ - private static Log log = LogFactory.getLog(GTimerIncrementalSaverTest.class); - /** * Test que saveDirectory a une valeur attendue. */

1 0

r2811 - in trunk/src/main/java/org/chorem/jtimer: . ui/tasks
by echatellier＠users.chorem.org March 7, 2012

March 7, 2012

Author: echatellier Date: 2012-03-07 18:21:15 +0100 (Wed, 07 Mar 2012) New Revision: 2811 Url: http://chorem.org/repositories/revision/jtimer/2811 Log: Fix seconds/ms convertion Modified: trunk/src/main/java/org/chorem/jtimer/JTimerConfig.java trunk/src/main/java/org/chorem/jtimer/ui/tasks/IdleDialog.java trunk/src/main/java/org/chorem/jtimer/ui/tasks/RunTaskJob.java Modified: trunk/src/main/java/org/chorem/jtimer/JTimerConfig.java =================================================================== --- trunk/src/main/java/org/chorem/jtimer/JTimerConfig.java 2012-03-07 17:11:12 UTC (rev 2810) +++ trunk/src/main/java/org/chorem/jtimer/JTimerConfig.java 2012-03-07 17:21:15 UTC (rev 2811) @@ -1,25 +1,23 @@ /* * #%L - * - * * $Id$ * $HeadURL$ * %% * Copyright (C) 2012 Codelutin, Chatellier Eric * %% * This program is free software: you can redistribute it and/or modify - * it under the terms of the GNU Lesser General Public License as + * it under the terms of the GNU General Public License as * published by the Free Software Foundation, either version 3 of the * License, or (at your option) any later version. * * This program is distributed in the hope that it will be useful, * but WITHOUT ANY WARRANTY; without even the implied warranty of * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the - * GNU General Lesser Public License for more details. + * GNU General Public License for more details. * - * You should have received a copy of the GNU General Lesser Public + * You should have received a copy of the GNU General Public * License along with this program. If not, see - * <http://www.gnu.org/licenses/lgpl-3.0.html>. + * <http://www.gnu.org/licenses/gpl-3.0.html>. * #L% */ @@ -91,11 +89,21 @@ public String getIOSaverDirectory() { return appConfig.getOption(JTimerOption.IO_SAVER_DIRECTORY.key); } - + + /** + * Return auto save delay in seconds. + * + * @return auto save delay + */ public long getIOSaverAutoSaveDelay() { return appConfig.getOptionAsLong(JTimerOption.IO_SAVER_AUTOSAVEDELAY.key); } + /** + * Return user idle time threshold in seconds. + * + * @return idle time threshold + */ public long getIdleTime() { return appConfig.getOptionAsLong(JTimerOption.UI_IDLE_TIME.key); } Modified: trunk/src/main/java/org/chorem/jtimer/ui/tasks/IdleDialog.java =================================================================== --- trunk/src/main/java/org/chorem/jtimer/ui/tasks/IdleDialog.java 2012-03-07 17:11:12 UTC (rev 2810) +++ trunk/src/main/java/org/chorem/jtimer/ui/tasks/IdleDialog.java 2012-03-07 17:21:15 UTC (rev 2811) @@ -160,7 +160,7 @@ // label JLabel idleLabel = new JLabel(resourceMap.getString("idleMessage", Long - .valueOf(JTimer.config.getIdleTime() / (60 * 1000)))); + .valueOf(JTimer.config.getIdleTime() / (60)))); mainComponent.add(idleLabel, new GridBagConstraints(1, 0, 3, 1, 1, 0, GridBagConstraints.CENTER, GridBagConstraints.HORIZONTAL, new Insets(5, 0, 0, 3), 0, 0)); Modified: trunk/src/main/java/org/chorem/jtimer/ui/tasks/RunTaskJob.java =================================================================== --- trunk/src/main/java/org/chorem/jtimer/ui/tasks/RunTaskJob.java 2012-03-07 17:11:12 UTC (rev 2810) +++ trunk/src/main/java/org/chorem/jtimer/ui/tasks/RunTaskJob.java 2012-03-07 17:21:15 UTC (rev 2811) @@ -247,7 +247,7 @@ resetTiming(); // get idle time - long configIdleTime = JTimer.config.getIdleTime(); + long configIdleTime = JTimer.config.getIdleTime() * 1000; boolean dontWantToStop = true; while (dontWantToStop) { @@ -279,7 +279,7 @@ checkReset(); // if long idleTime is unavailable, if is always false - if (idleTime >= JTimer.config.getIdleTime()) { + if (idleTime >= configIdleTime) { // to not display negative time near midnight long idleTimeOffset = Math.min(idleTime, currentTime - taskStartCalendar.getTimeInMillis());

1 0

r2810 - trunk/src/main/java/org/chorem/jtimer
by echatellier＠users.chorem.org March 7, 2012

March 7, 2012

Author: echatellier Date: 2012-03-07 18:11:12 +0100 (Wed, 07 Mar 2012) New Revision: 2810 Url: http://chorem.org/repositories/revision/jtimer/2810 Log: Fix config file migration (from user home, not /etc) Modified: trunk/src/main/java/org/chorem/jtimer/JTimer.java Modified: trunk/src/main/java/org/chorem/jtimer/JTimer.java =================================================================== --- trunk/src/main/java/org/chorem/jtimer/JTimer.java 2012-03-07 16:49:27 UTC (rev 2809) +++ trunk/src/main/java/org/chorem/jtimer/JTimer.java 2012-03-07 17:11:12 UTC (rev 2810) @@ -206,7 +206,7 @@ // add file migration for configuration file created before version 1.4 File homeDir = new File(System.getProperty("user.home"), ".jtimer"); File oldFile = new File(homeDir, "JTimer.properties"); - File newFile = new File(config.appConfig.getConfigPath(), config.appConfig.getConfigFileName()); + File newFile = new File(config.appConfig.getUserConfigDirectory(), config.appConfig.getConfigFileName()); if (oldFile.isFile() && !newFile.isFile()) { if (log.isInfoEnabled()) { log.info("Migration configuration file location");

1 0

r2809 - in trunk/src: main/java/org/chorem/jtimer test/java/org/chorem/jtimer
by echatellier＠users.chorem.org March 7, 2012

March 7, 2012

Author: echatellier Date: 2012-03-07 17:49:27 +0100 (Wed, 07 Mar 2012) New Revision: 2809 Url: http://chorem.org/repositories/revision/jtimer/2809 Log: Fix configuration test Modified: trunk/src/main/java/org/chorem/jtimer/JTimerConfig.java trunk/src/test/java/org/chorem/jtimer/JTimerFactoryTest.java Modified: trunk/src/main/java/org/chorem/jtimer/JTimerConfig.java =================================================================== --- trunk/src/main/java/org/chorem/jtimer/JTimerConfig.java 2012-03-07 16:46:58 UTC (rev 2808) +++ trunk/src/main/java/org/chorem/jtimer/JTimerConfig.java 2012-03-07 16:49:27 UTC (rev 2809) @@ -67,7 +67,7 @@ public Class getServiceClass() { Class serviceClass = null; try { - appConfig.getOptionAsClass(JTimerOption.SERVICE_CLASS.key); + serviceClass = appConfig.getOptionAsClass(JTimerOption.SERVICE_CLASS.key); } catch (Exception ex) { if (log.isWarnEnabled()) { log.warn("Can't find service class implementation, sync will not work", ex); Modified: trunk/src/test/java/org/chorem/jtimer/JTimerFactoryTest.java =================================================================== --- trunk/src/test/java/org/chorem/jtimer/JTimerFactoryTest.java 2012-03-07 16:46:58 UTC (rev 2808) +++ trunk/src/test/java/org/chorem/jtimer/JTimerFactoryTest.java 2012-03-07 16:49:27 UTC (rev 2809) @@ -27,7 +27,6 @@ import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; -import org.chorem.jtimer.JTimerConfig.JTimerOption; import org.chorem.jtimer.io.GTimerIncrementalSaver; import org.chorem.jtimer.io.Saver; import org.chorem.jtimer.ws.ProjectManagement;

1 0

r2808 - trunk
by echatellier＠users.chorem.org March 7, 2012

March 7, 2012

Author: echatellier Date: 2012-03-07 17:46:58 +0100 (Wed, 07 Mar 2012) New Revision: 2808 Url: http://chorem.org/repositories/revision/jtimer/2808 Log: #250: Udpate to testng 6.4 #482: Update to freemarker 2.3.19 Modified: trunk/pom.xml Modified: trunk/pom.xml =================================================================== --- trunk/pom.xml 2012-03-07 16:41:22 UTC (rev 2807) +++ trunk/pom.xml 2012-03-07 16:46:58 UTC (rev 2808) @@ -257,7 +257,7 @@ <dependency> <groupId>org.freemarker</groupId> <artifactId>freemarker</artifactId> - <version>2.3.18</version> + <version>2.3.19</version> </dependency> <dependency> <groupId>org.apache.xmlrpc</groupId> @@ -307,7 +307,7 @@ <dependency> <groupId>org.testng</groupId> <artifactId>testng</artifactId> - <version>6.3.1</version> + <version>6.4</version> <scope>test</scope> <exclusions> <exclusion>

1 0

r2807 - trunk
by echatellier＠users.chorem.org March 7, 2012

March 7, 2012

Author: echatellier Date: 2012-03-07 17:41:22 +0100 (Wed, 07 Mar 2012) New Revision: 2807 Url: http://chorem.org/repositories/revision/jtimer/2807 Log: Modify svn:ignore Modified: trunk/ Property changes on: trunk ___________________________________________________________________ Modified: svn:ignore - target .settings .project .classpath JTimer.properties THIRD-PARTY.txt *testng*.xml *.ipr *.iws *.iml + target .settings .project .classpath *.properties *testng*.xml *.ipr *.iws *.iml Deleted: svn:mergeinfo - /branches/1.3.x:2624-2628

1 0

r2806 - in trunk: . src/main/java/org/chorem/jtimer src/main/java/org/chorem/jtimer/entities src/main/java/org/chorem/jtimer/io src/main/java/org/chorem/jtimer/ui src/main/java/org/chorem/jtimer/ui/report src/main/java/org/chorem/jtimer/ui/tasks src/main/java/org/chorem/jtimer/ui/treetable src/main/java/org/chorem/jtimer/ui/treetable/dnd src/main/resources src/test/java/org/chorem/jtimer src/test/resources
by echatellier＠users.chorem.org March 7, 2012

March 7, 2012

Author: echatellier Date: 2012-03-07 17:38:55 +0100 (Wed, 07 Mar 2012) New Revision: 2806 Url: http://chorem.org/repositories/revision/jtimer/2806 Log: #478: Update to commons-lang 3.1 #481: Migration configuration to NuitonUtils's ApplicationConfig Added: trunk/src/main/java/org/chorem/jtimer/JTimerConfig.java trunk/src/test/resources/jtimertest.properties Removed: trunk/src/main/resources/JTimer.properties trunk/src/main/resources/JTimer_default.properties trunk/src/test/resources/JTimerTestRelatif.properties Modified: trunk/pom.xml trunk/src/main/java/org/chorem/jtimer/JTimer.java trunk/src/main/java/org/chorem/jtimer/JTimerFactory.java trunk/src/main/java/org/chorem/jtimer/entities/TimerTask.java trunk/src/main/java/org/chorem/jtimer/io/GTimerIncrementalSaver.java trunk/src/main/java/org/chorem/jtimer/io/Saver.java trunk/src/main/java/org/chorem/jtimer/package-info.java trunk/src/main/java/org/chorem/jtimer/ui/StatusBar.java trunk/src/main/java/org/chorem/jtimer/ui/report/ReportUtils.java trunk/src/main/java/org/chorem/jtimer/ui/tasks/IdleDialog.java trunk/src/main/java/org/chorem/jtimer/ui/tasks/RunTaskJob.java trunk/src/main/java/org/chorem/jtimer/ui/treetable/ProjectsAndTasksModel.java trunk/src/main/java/org/chorem/jtimer/ui/treetable/dnd/TimerTaskTransferHandler.java trunk/src/main/resources/log4j.properties trunk/src/test/java/org/chorem/jtimer/AbstractJTimerTest.java trunk/src/test/java/org/chorem/jtimer/JTimerFactoryTest.java Modified: trunk/pom.xml =================================================================== --- trunk/pom.xml 2012-02-16 14:27:52 UTC (rev 2805) +++ trunk/pom.xml 2012-03-07 16:38:55 UTC (rev 2806) @@ -10,7 +10,7 @@ <groupId>org.chorem</groupId> <artifactId>jtimer</artifactId> - <version>1.3.4-SNAPSHOT</version> + <version>1.4.0-SNAPSHOT</version> <name>jTimer</name> <description> @@ -172,7 +172,7 @@ <plugins> <plugin> <artifactId>maven-checkstyle-plugin</artifactId> - <version>2.8</version> + <version>2.9.1</version> <configuration> <configLocation>${maven.src.dir}/main/config/checkstyle.xml </configLocation> @@ -206,6 +206,26 @@ <dependencies> <dependency> + <groupId>org.nuiton</groupId> + <artifactId>nuiton-utils</artifactId> + <version>2.4.4</version> + <scope>compile</scope> + <exclusions> + <exclusion> + <groupId>commons-primitives</groupId> + <artifactId>commons-primitives</artifactId> + </exclusion> + <exclusion> + <groupId>commons-jxpath</groupId> + <artifactId>commons-jxpath</artifactId> + </exclusion> + <exclusion> + <groupId>org.nuiton.i18n</groupId> + <artifactId>nuiton-i18n</artifactId> + </exclusion> + </exclusions> + </dependency> + <dependency> <groupId>org.jdesktop.bsaf</groupId> <artifactId>bsaf</artifactId> <version>1.9.2</version> @@ -265,8 +285,8 @@ <artifactId>commons-collections</artifactId> </dependency> <dependency> - <groupId>commons-lang</groupId> - <artifactId>commons-lang</artifactId> + <groupId>org.apache.commons</groupId> + <artifactId>commons-lang3</artifactId> </dependency> <dependency> <groupId>commons-logging</groupId> Modified: trunk/src/main/java/org/chorem/jtimer/JTimer.java =================================================================== --- trunk/src/main/java/org/chorem/jtimer/JTimer.java 2012-02-16 14:27:52 UTC (rev 2805) +++ trunk/src/main/java/org/chorem/jtimer/JTimer.java 2012-03-07 16:38:55 UTC (rev 2806) @@ -5,7 +5,7 @@ * $Id$ * $HeadURL$ * %% - * Copyright (C) 2007 - 2011 CodeLutin, Chatellier Eric + * Copyright (C) 2007 - 2012 CodeLutin, Chatellier Eric * %% * This program is free software: you can redistribute it and/or modify * it under the terms of the GNU General Public License as @@ -31,6 +31,8 @@ import java.awt.event.KeyEvent; import java.awt.event.MouseEvent; import java.awt.event.MouseListener; +import java.io.File; +import java.io.IOException; import java.util.Calendar; import java.util.Date; import java.util.List; @@ -55,7 +57,8 @@ import javax.swing.event.TreeSelectionListener; import javax.swing.tree.TreePath; -import org.apache.commons.lang.StringUtils; +import org.apache.commons.io.FileUtils; +import org.apache.commons.lang3.StringUtils; import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; import org.chorem.jtimer.data.DataViolationException; @@ -76,6 +79,7 @@ import org.chorem.jtimer.ws.ConnectionDataHandler; import org.chorem.jtimer.ws.ProjectManagement; import org.jdesktop.application.Action; +import org.jdesktop.application.Application; import org.jdesktop.application.ApplicationContext; import org.jdesktop.application.ResourceMap; import org.jdesktop.application.SingleFrameApplication; @@ -111,6 +115,9 @@ /** I18n resources map. */ protected ResourceMap resourceMap; + /** Jtimer application config. */ + public static JTimerConfig config; + /** Single project selection property. */ protected boolean selectedSingleProject; @@ -161,6 +168,9 @@ // super, but does nothing super.initialize(args); + // load configuration + loadConfiguration(args); + // init resources map ApplicationContext ctxt = getContext(); resourceMap = ctxt.getResourceMap(); @@ -185,6 +195,36 @@ } /** + * Load configuration. + * + * @param args args to parse command line options + */ + protected void loadConfiguration(String[] args) { + + config = new JTimerConfig(); + + // add file migration for configuration file created before version 1.4 + File homeDir = new File(System.getProperty("user.home"), ".jtimer"); + File oldFile = new File(homeDir, "JTimer.properties"); + File newFile = new File(config.appConfig.getConfigPath(), config.appConfig.getConfigFileName()); + if (oldFile.isFile() && !newFile.isFile()) { + if (log.isInfoEnabled()) { + log.info("Migration configuration file location"); + } + try { + FileUtils.copyFile(oldFile, newFile); + } catch (IOException ex) { + if (log.isErrorEnabled()) { + log.error("Can't copy config file to new location", ex); + } + } + } + + // parse after file migration + config.parse(args); + } + + /** * startup. * * Create frame menu bar. Create main component. Added: trunk/src/main/java/org/chorem/jtimer/JTimerConfig.java =================================================================== --- trunk/src/main/java/org/chorem/jtimer/JTimerConfig.java (rev 0) +++ trunk/src/main/java/org/chorem/jtimer/JTimerConfig.java 2012-03-07 16:38:55 UTC (rev 2806) @@ -0,0 +1,122 @@ +/* + * #%L + * + * + * $Id$ + * $HeadURL$ + * %% + * Copyright (C) 2012 Codelutin, Chatellier Eric + * %% + * This program is free software: you can redistribute it and/or modify + * it under the terms of the GNU Lesser General Public License as + * published by the Free Software Foundation, either version 3 of the + * License, or (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Lesser Public License for more details. + * + * You should have received a copy of the GNU General Lesser Public + * License along with this program. If not, see + * <http://www.gnu.org/licenses/lgpl-3.0.html>. + * #L% + */ + +package org.chorem.jtimer; + +import org.apache.commons.logging.Log; +import org.apache.commons.logging.LogFactory; +import org.nuiton.util.ApplicationConfig; +import org.nuiton.util.ArgumentsParserException; + +/** + * JTimer configuration. + * + * @author chatellier + * @version $Revision$ + * + * Last update : $Date$ + * By : $Author$ + */ +public class JTimerConfig { + + private static Log log = LogFactory.getLog(JTimerConfig.class); + + protected ApplicationConfig appConfig; + + public JTimerConfig() { + appConfig = new ApplicationConfig(); + for (JTimerOption o : JTimerOption.values()) { + if (o.defaultValue != null) { + appConfig.setDefaultOption(o.key, o.defaultValue); + } + } + } + + public void parse(String... args) { + try { + appConfig.parse(args); + } catch (ArgumentsParserException ex) { + if (log.isErrorEnabled()) { + log.error("Can't parse configuration", ex); + } + } + } + + public Class getServiceClass() { + Class serviceClass = null; + try { + appConfig.getOptionAsClass(JTimerOption.SERVICE_CLASS.key); + } catch (Exception ex) { + if (log.isWarnEnabled()) { + log.warn("Can't find service class implementation, sync will not work", ex); + } + } + return serviceClass; + } + + public String getServiceEndpoint() { + return appConfig.getOption(JTimerOption.SERVICE_ENDPOINT.key); + } + + public String getServiceResource() { + return appConfig.getOption(JTimerOption.SERVICE_RESOURCE.key); + } + + public Class getIOSaverClass() { + return appConfig.getOptionAsClass(JTimerOption.IO_SAVER_CLASS.key); + } + + public String getIOSaverDirectory() { + return appConfig.getOption(JTimerOption.IO_SAVER_DIRECTORY.key); + } + + public long getIOSaverAutoSaveDelay() { + return appConfig.getOptionAsLong(JTimerOption.IO_SAVER_AUTOSAVEDELAY.key); + } + + public long getIdleTime() { + return appConfig.getOptionAsLong(JTimerOption.UI_IDLE_TIME.key); + } + + public enum JTimerOption { + CONFIG_FILENAME(ApplicationConfig.CONFIG_FILE_NAME, "jtimer.properties"), + SERVICE_CLASS("jtimer.service.class", null), + SERVICE_ENDPOINT("jtimer.service.endpoint", null), + SERVICE_RESOURCE("jtimer.service.resource", "JTimer"), + IO_SAVER_CLASS("jtimer.io.saver.class", "org.chorem.jtimer.io.GTimerIncrementalSaver"), + IO_SAVER_DIRECTORY("jtimer.io.saver.directory", "${user.home}/.gtimer"), + IO_SAVER_AUTOSAVEDELAY("jtimer.io.saver.autosavedelay", "300"), + UI_IDLE_TIME("jtimer.ui.idletime", "300"); + + protected String key; + + protected String defaultValue; + + private JTimerOption(String key, String defaultValue) { + this.key = key; + this.defaultValue = defaultValue; + } + } +} Property changes on: trunk/src/main/java/org/chorem/jtimer/JTimerConfig.java ___________________________________________________________________ Added: svn:keywords + Author Date Id Revision HeadURL Modified: trunk/src/main/java/org/chorem/jtimer/JTimerFactory.java =================================================================== --- trunk/src/main/java/org/chorem/jtimer/JTimerFactory.java 2012-02-16 14:27:52 UTC (rev 2805) +++ trunk/src/main/java/org/chorem/jtimer/JTimerFactory.java 2012-03-07 16:38:55 UTC (rev 2806) @@ -5,7 +5,7 @@ * $Id$ * $HeadURL$ * %% - * Copyright (C) 2007 - 2011 CodeLutin, Chatellier Eric + * Copyright (C) 2007 - 2012 CodeLutin, Chatellier Eric * %% * This program is free software: you can redistribute it and/or modify * it under the terms of the GNU General Public License as @@ -25,16 +25,6 @@ package org.chorem.jtimer; -import java.io.File; -import java.io.FileInputStream; -import java.io.FileOutputStream; -import java.io.IOException; -import java.io.InputStream; -import java.io.OutputStream; -import java.net.URL; -import java.util.Properties; - -import org.apache.commons.io.IOUtils; import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; import org.chorem.jtimer.io.Saver; @@ -54,15 +44,6 @@ /** Logger */ private static Log log = LogFactory.getLog(JTimerFactory.class); - /** Default idle time in milliseconds */ - protected static final long DEFAULT_IDLE_TIME = 5 * 60 * 1000; // 5 min - - /** Properties */ - protected static Properties props; - - /** Configured idle time */ - protected static long idleTime = -1L; - /** ProjectManagement */ protected static ProjectManagement projectManagement; @@ -77,138 +58,6 @@ } /** - * Init. - */ - protected static void init() { - - // init once - if (props == null) { - - // change load order here : - // Take : - // - classpath one - // - user home one - // - ./ one - - try { - props = new Properties(); - - // read configuration file - String fileName = getConfigurationFileName(); - - // use file in classpath - URL url = JTimerFactory.class.getResource("/" + fileName); - props.load(url.openStream()); - - // and override by file in user home - File homeUserFile = new File(getUserHomeConfigurationFilePath()); - if (homeUserFile.exists()) { - if (log.isInfoEnabled()) { - log.info("Use user home configuration in " - + homeUserFile.getAbsolutePath()); - } - props.load(new FileInputStream(homeUserFile)); - } else { - - // homeUserFile is $HOME/JTimer.properties - // on linux, parent dir is created by SAF - // on other, SAF dir is not this one, so - // could not exists - if (homeUserFile.getParentFile() != null - && !homeUserFile.getParentFile().exists()) { - homeUserFile.getParentFile().mkdirs(); - if (log.isInfoEnabled()) { - log.info("Creating directory " - + homeUserFile.getParentFile() - .getAbsolutePath()); - } - } - - if (log.isInfoEnabled()) { - log.info("Creating new default configuration file in " - + homeUserFile.getAbsolutePath()); - } - - URL defautFileURL = JTimerFactory.class.getResource("/" - + getDefaultConfigurationFileName()); - copyConfigurationFile(defautFileURL, homeUserFile); - } - - // read file in current directory - File currentDirectoryFile = new File(fileName); - if (currentDirectoryFile.canRead()) { - props.load(new FileInputStream(currentDirectoryFile)); - } - - } catch (IOException e) { - if (log.isErrorEnabled()) { - log.error("Can't read configuration file", e); - } - } - } - } - - /** - * Configuration filepath. - * - * @return Configuration filename - */ - protected static String getUserHomeConfigurationFilePath() { - // user home - String filePath = System.getProperty("user.home"); - // + .jtimer directory - // TODO use saf config - filePath += File.separator + ".jtimer"; - // add filename - filePath += File.separator + getConfigurationFileName(); - - if (log.isDebugEnabled()) { - log.debug("Look for configuration file in : " + filePath); - } - - return filePath; - } - - /** - * Configuration filename. - * - * @return Configuration filename - */ - protected static String getConfigurationFileName() { - String fileName = JTimer.class.getSimpleName() + ".properties"; - return fileName; - } - - /** - * Default configuration filename. - * - * @return Configuration filename - */ - protected static String getDefaultConfigurationFileName() { - String fileName = JTimer.class.getSimpleName() + "_default.properties"; - return fileName; - } - - /** - * Copy copyFrom content in copyTo file. - * - * @param copyFrom from - * @param copyTo to - * @throws IOException - */ - protected static void copyConfigurationFile(URL copyFrom, File copyTo) - throws IOException { - - InputStream in = copyFrom.openStream(); - OutputStream out = new FileOutputStream(copyTo); - - IOUtils.copy(in, out); - - in.close(); - out.close(); - } - - /** * Return an implementation on a distant web service. * * Use jtimer.service.class property to found class and @@ -220,37 +69,26 @@ if (projectManagement == null) { - // check that init has been done - init(); + Class implementationClass = JTimer.config.getServiceClass(); - // get implementation class - String implementationClass = props - .getProperty("jtimer.service.class"); - // get service endpoint - String serviceEndPoint = props - .getProperty("jtimer.service.endpoint"); - // get service resource name - String serviceResouceName = props - .getProperty("jtimer.service.resource"); - // log if (log.isInfoEnabled()) { log.info("Using service class : " + implementationClass); - log.info(" with service endpoint : " + serviceEndPoint); - log.info(" with service resource : " + serviceResouceName); + log.info(" with service endpoint : " + JTimer.config.getServiceEndpoint()); + log.info(" with service resource : " + JTimer.config.getServiceResource()); } // By default jTimer won't synchronize // so implementationClass can be null if (implementationClass != null) { + try { // get instance - projectManagement = (ProjectManagement) Class.forName( - implementationClass).newInstance(); + projectManagement = (ProjectManagement)implementationClass.newInstance(); // init instance - projectManagement.setEndpoint(serviceEndPoint); - projectManagement.setResourceName(serviceResouceName); + projectManagement.setEndpoint(JTimer.config.getServiceEndpoint()); + projectManagement.setResourceName(JTimer.config.getServiceResource()); } catch (InstantiationException e) { if (log.isErrorEnabled()) { log.error("Can't instanciate class : " @@ -262,11 +100,6 @@ "Can't access class : " + implementationClass, e); } - } catch (ClassNotFoundException e) { - if (log.isErrorEnabled()) { - log.error("Can't found class : " + implementationClass, - e); - } } } else { if (log.isInfoEnabled()) { @@ -286,125 +119,37 @@ public static Saver getFileSaver() { if (saver == null) { - // check that init has been done - init(); - // get implementation class - String implementationClass = props - .getProperty("jtimer.io.saver.class"); - // get saver directory - String saverDirectory = props - .getProperty("jtimer.io.saver.directory"); - // get saver save delay - String autoSaveDelay = props - .getProperty("jtimer.io.saver.autosavedelay"); + Class saverClass = JTimer.config.getIOSaverClass(); - // replace $HOME by user.home - saverDirectory = getHomeReplacement(saverDirectory); - // log if (log.isInfoEnabled()) { - log.info("Using saver class : " + implementationClass); - log.info(" with saver home directory : " + saverDirectory); - log.info(" with auto save delay : " + autoSaveDelay); + log.info("Using saver class : " + saverClass); + log.info(" with saver home directory : " + JTimer.config.getIOSaverDirectory()); + log.info(" with auto save delay : " + JTimer.config.getIOSaverAutoSaveDelay()); } try { // get instance - saver = (Saver) Class.forName(implementationClass) - .newInstance(); + saver = (Saver) saverClass.newInstance(); // init instance - saver.setSaveDirectory(saverDirectory); + saver.setSaveDirectory(JTimer.config.getIOSaverDirectory()); // set delay to saver - try { - long autoSaveDelayInS = Long.parseLong(autoSaveDelay); - saver.setAutoSaveDelay(autoSaveDelayInS * 1000); - } catch (NumberFormatException e) { - if (log.isWarnEnabled()) { - log.warn("jtimer.io.saver.autosavedelay is non numeric value", - e); - } - } + saver.setAutoSaveDelay(JTimer.config.getIOSaverAutoSaveDelay() * 1000); + } catch (InstantiationException e) { if (log.isErrorEnabled()) { - log.error("Can't instanciate class : " - + implementationClass, e); + log.error("Can't instanciate class : " + saverClass, e); } } catch (IllegalAccessException e) { if (log.isErrorEnabled()) { - log.error("Can't access class : " + implementationClass, e); + log.error("Can't access class : " + saverClass, e); } - } catch (ClassNotFoundException e) { - if (log.isErrorEnabled()) { - log.error("Can't found class : " + implementationClass, e); - } } } return saver; } - - /** - * Replace $HOME in path. - * - * @param path path to replace home - * @return path without $HOME - */ - protected static String getHomeReplacement(String path) { - - String localPath = path; - - // replace $HOME by user.home - if (localPath != null) { - // replace / in configuration file by - // default system file.separator - // FIXME find better method in 2.0 - localPath = localPath.replace("/", File.separator); - - // correct bug on windows system for example - // \ in path are interpreted on regex - String escapedUserHome = System.getProperty("user.home") - .replaceAll("\\\\", "\\\\\\\\"); - localPath = localPath.replaceFirst("\\$HOME", escapedUserHome); - } - - return localPath; - } - - /** - * Get configuration idle time in ms. - * - * @return idle time - */ - public static long getIdleTime() { - - if (idleTime <= 0) { - - // check that init has been done - init(); - - String idleTimeInS = props.getProperty("jtimer.ui.idletime"); - if (idleTimeInS == null) { - idleTime = DEFAULT_IDLE_TIME; - } else { - try { - idleTime = Long.parseLong(idleTimeInS) * 1000L; - } catch (NumberFormatException e) { - if (log.isWarnEnabled()) { - log.warn("Can't get idle time as number, using default idle time", - e); - } - idleTime = DEFAULT_IDLE_TIME; - } - } - - if (log.isInfoEnabled()) { - log.info("Idle time set to : " + idleTime + "ms"); - } - } - - return idleTime; - } } Modified: trunk/src/main/java/org/chorem/jtimer/entities/TimerTask.java =================================================================== --- trunk/src/main/java/org/chorem/jtimer/entities/TimerTask.java 2012-02-16 14:27:52 UTC (rev 2805) +++ trunk/src/main/java/org/chorem/jtimer/entities/TimerTask.java 2012-03-07 16:38:55 UTC (rev 2806) @@ -5,7 +5,7 @@ * $Id$ * $HeadURL$ * %% - * Copyright (C) 2007 - 2011 CodeLutin, Chatellier Eric + * Copyright (C) 2007 - 2012 CodeLutin, Chatellier Eric * %% * This program is free software: you can redistribute it and/or modify * it under the terms of the GNU General Public License as @@ -32,7 +32,7 @@ import java.util.SortedMap; import java.util.TreeMap; -import org.apache.commons.lang.SerializationUtils; +import org.apache.commons.lang3.SerializationUtils; import org.chorem.jtimer.utils.DailySortedMap; /** Modified: trunk/src/main/java/org/chorem/jtimer/io/GTimerIncrementalSaver.java =================================================================== --- trunk/src/main/java/org/chorem/jtimer/io/GTimerIncrementalSaver.java 2012-02-16 14:27:52 UTC (rev 2805) +++ trunk/src/main/java/org/chorem/jtimer/io/GTimerIncrementalSaver.java 2012-03-07 16:38:55 UTC (rev 2806) @@ -5,7 +5,7 @@ * $Id$ * $HeadURL$ * %% - * Copyright (C) 2007 - 2011 CodeLutin, Chatellier Eric + * Copyright (C) 2007 - 2012 CodeLutin, Chatellier Eric * %% * This program is free software: you can redistribute it and/or modify * it under the terms of the GNU General Public License as @@ -87,9 +87,6 @@ public class GTimerIncrementalSaver extends AbstractSaver implements Saver, DataEventListener { - /** serialVersionUID. */ - private static final long serialVersionUID = 4999800626942050992L; - /** log. */ private static Log log = LogFactory.getLog(GTimerIncrementalSaver.class); Modified: trunk/src/main/java/org/chorem/jtimer/io/Saver.java =================================================================== --- trunk/src/main/java/org/chorem/jtimer/io/Saver.java 2012-02-16 14:27:52 UTC (rev 2805) +++ trunk/src/main/java/org/chorem/jtimer/io/Saver.java 2012-03-07 16:38:55 UTC (rev 2806) @@ -5,7 +5,7 @@ * $Id$ * $HeadURL$ * %% - * Copyright (C) 2007 - 2011 CodeLutin, Chatellier Eric + * Copyright (C) 2007 - 2012 CodeLutin, Chatellier Eric * %% * This program is free software: you can redistribute it and/or modify * it under the terms of the GNU General Public License as Modified: trunk/src/main/java/org/chorem/jtimer/package-info.java =================================================================== --- trunk/src/main/java/org/chorem/jtimer/package-info.java 2012-02-16 14:27:52 UTC (rev 2805) +++ trunk/src/main/java/org/chorem/jtimer/package-info.java 2012-03-07 16:38:55 UTC (rev 2806) @@ -5,7 +5,7 @@ * $Id$ * $HeadURL$ * %% - * Copyright (C) 2007 - 2011 CodeLutin, Chatellier Eric + * Copyright (C) 2007 - 2012 CodeLutin, Chatellier Eric * %% * This program is free software: you can redistribute it and/or modify * it under the terms of the GNU General Public License as Modified: trunk/src/main/java/org/chorem/jtimer/ui/StatusBar.java =================================================================== --- trunk/src/main/java/org/chorem/jtimer/ui/StatusBar.java 2012-02-16 14:27:52 UTC (rev 2805) +++ trunk/src/main/java/org/chorem/jtimer/ui/StatusBar.java 2012-03-07 16:38:55 UTC (rev 2806) @@ -5,7 +5,7 @@ * $Id$ * $HeadURL$ * %% - * Copyright (C) 2007 - 2011 CodeLutin, Chatellier Eric + * Copyright (C) 2007 - 2012 CodeLutin, Chatellier Eric * %% * This program is free software: you can redistribute it and/or modify * it under the terms of the GNU General Public License as @@ -35,7 +35,7 @@ import javax.swing.JPanel; import javax.swing.SwingConstants; -import org.apache.commons.lang.time.DurationFormatUtils; +import org.apache.commons.lang3.time.DurationFormatUtils; import org.chorem.jtimer.data.DataEventListener; import org.chorem.jtimer.data.TimerDataManager; import org.chorem.jtimer.entities.TimerProject; Modified: trunk/src/main/java/org/chorem/jtimer/ui/report/ReportUtils.java =================================================================== --- trunk/src/main/java/org/chorem/jtimer/ui/report/ReportUtils.java 2012-02-16 14:27:52 UTC (rev 2805) +++ trunk/src/main/java/org/chorem/jtimer/ui/report/ReportUtils.java 2012-03-07 16:38:55 UTC (rev 2806) @@ -5,7 +5,7 @@ * $Id$ * $HeadURL$ * %% - * Copyright (C) 2008 - 2011 CodeLutin, Chatellier Eric + * Copyright (C) 2008 - 2012 CodeLutin, Chatellier Eric * %% * This program is free software: you can redistribute it and/or modify * it under the terms of the GNU General Public License as @@ -30,7 +30,7 @@ import java.util.Date; import java.util.List; -import org.apache.commons.lang.time.DurationFormatUtils; +import org.apache.commons.lang3.time.DurationFormatUtils; import org.chorem.jtimer.entities.TimerTask; import org.chorem.jtimer.entities.TimerTaskHelper; Modified: trunk/src/main/java/org/chorem/jtimer/ui/tasks/IdleDialog.java =================================================================== --- trunk/src/main/java/org/chorem/jtimer/ui/tasks/IdleDialog.java 2012-02-16 14:27:52 UTC (rev 2805) +++ trunk/src/main/java/org/chorem/jtimer/ui/tasks/IdleDialog.java 2012-03-07 16:38:55 UTC (rev 2806) @@ -5,7 +5,7 @@ * $Id$ * $HeadURL$ * %% - * Copyright (C) 2009 - 2011 CodeLutin, Chatellier Eric + * Copyright (C) 2009 - 2012 CodeLutin, Chatellier Eric * %% * This program is free software: you can redistribute it and/or modify * it under the terms of the GNU General Public License as @@ -40,10 +40,10 @@ import javax.swing.SwingConstants; import javax.swing.WindowConstants; -import org.apache.commons.lang.time.DurationFormatUtils; +import org.apache.commons.lang3.time.DurationFormatUtils; import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; -import org.chorem.jtimer.JTimerFactory; +import org.chorem.jtimer.JTimer; import org.jdesktop.application.Action; import org.jdesktop.application.ApplicationContext; import org.jdesktop.application.ResourceManager; @@ -160,7 +160,7 @@ // label JLabel idleLabel = new JLabel(resourceMap.getString("idleMessage", Long - .valueOf(JTimerFactory.getIdleTime() / (60 * 1000)))); + .valueOf(JTimer.config.getIdleTime() / (60 * 1000)))); mainComponent.add(idleLabel, new GridBagConstraints(1, 0, 3, 1, 1, 0, GridBagConstraints.CENTER, GridBagConstraints.HORIZONTAL, new Insets(5, 0, 0, 3), 0, 0)); Modified: trunk/src/main/java/org/chorem/jtimer/ui/tasks/RunTaskJob.java =================================================================== --- trunk/src/main/java/org/chorem/jtimer/ui/tasks/RunTaskJob.java 2012-02-16 14:27:52 UTC (rev 2805) +++ trunk/src/main/java/org/chorem/jtimer/ui/tasks/RunTaskJob.java 2012-03-07 16:38:55 UTC (rev 2806) @@ -5,7 +5,7 @@ * $Id$ * $HeadURL$ * %% - * Copyright (C) 2007 - 2011 CodeLutin, Chatellier Eric + * Copyright (C) 2007 - 2012 CodeLutin, Chatellier Eric * %% * This program is free software: you can redistribute it and/or modify * it under the terms of the GNU General Public License as @@ -34,7 +34,7 @@ import javax.swing.JOptionPane; import javax.swing.SwingUtilities; -import org.apache.commons.lang.time.DurationFormatUtils; +import org.apache.commons.lang3.time.DurationFormatUtils; import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; import org.chorem.jtimer.JTimer; @@ -65,7 +65,7 @@ /** Class logger. */ private static Log log = LogFactory.getLog(RunTaskJob.class); - /** Parent app referenc. */ + /** Parent app reference. */ protected JTimer parentApp; /** The task to manage. */ @@ -77,17 +77,18 @@ /** System information (idle...). */ protected SystemInfo systemInfo; + /** Already thrown alert. */ + protected Collection<TimerAlert> alreadyTrownAlerts; + /** The today total task time at start time. */ protected long taskTimingBeforeStartInMs; /** Calendar instance when timing start. */ protected Calendar taskStartCalendar; - /** Already thrown alert. */ - protected Collection<TimerAlert> alreadyTrownAlerts; - /** - * Offset to adjust time. Include : + * Offset to adjust time for current day. + * Include : * - Task idle time (task was running but time not counted) * - Manual user incremented time */ @@ -206,11 +207,7 @@ taskStartCalendar = Calendar.getInstance(); lastPublishTimestamp = taskStartCalendar.getTimeInMillis(); - // need to be 0 here not managedTask.getTime(new Date()); - // for idle detected when 0hOO goes... - //taskTimingBeforeStartInMs = managedTask.getTime(new Date()); - - // need to be this for hibernation in same day + // need to be this for hibernate in same day taskTimingBeforeStartInMs = managedTask.getTime(new Date()) * 1000; offsetTimeInMs = 0; } @@ -250,14 +247,14 @@ resetTiming(); // get idle time - long configIdleTime = JTimerFactory.getIdleTime(); + long configIdleTime = JTimer.config.getIdleTime(); boolean dontWantToStop = true; while (dontWantToStop) { - long loopTimestamp = Calendar.getInstance().getTimeInMillis(); + long currentTime = Calendar.getInstance().getTimeInMillis(); // try to detect hibernate idle time - if (loopTimestamp - lastPublishTimestamp >= configIdleTime) { + if (currentTime - lastPublishTimestamp >= configIdleTime) { // hibernate detected // update time without idle time for specified day if (log.isInfoEnabled()) { @@ -266,7 +263,7 @@ resetTiming(); } - lastPublishTimestamp = loopTimestamp; + lastPublishTimestamp = currentTime; // check user idle time long idleTime = 0; @@ -282,18 +279,22 @@ checkReset(); // if long idleTime is unavailable, if is always false - if (idleTime >= JTimerFactory.getIdleTime()) { + if (idleTime >= JTimer.config.getIdleTime()) { + // to not display negative time near midnight + long idleTimeOffset = Math.min(idleTime, currentTime - taskStartCalendar.getTimeInMillis()); + // idle detected // update time without idle time - offsetTimeInMs -= configIdleTime; - publish(taskTimingBeforeStartInMs + loopTimestamp - taskStartCalendar.getTimeInMillis() + offsetTimeInMs); + // math.min > for idle detected at 0:00 + offsetTimeInMs -= idleTimeOffset; + publish(taskTimingBeforeStartInMs + currentTime - taskStartCalendar.getTimeInMillis() + offsetTimeInMs); // send idle detect event JTimer parentApplication = (JTimer)getApplication(); parentApplication.preIdleDetect(); // ask user what to do (long blocking call) - int option = IdleDialog.showIdleDialog(loopTimestamp - idleTime); + int option = IdleDialog.showIdleDialog(currentTime - idleTime); // send idle detect event parentApplication.postIdleDetect(); @@ -301,8 +302,8 @@ checkReset(); // get time after user idle - long currentTime = Calendar.getInstance().getTimeInMillis(); - lastPublishTimestamp = currentTime; + long afterIdleTime = Calendar.getInstance().getTimeInMillis(); + lastPublishTimestamp = afterIdleTime; switch (option) { @@ -314,8 +315,8 @@ case IdleDialog.CONTINUE: // refresh time // remove idle time previously added - offsetTimeInMs += configIdleTime; - publish(taskTimingBeforeStartInMs + currentTime + offsetTimeInMs += idleTimeOffset; + publish(taskTimingBeforeStartInMs + afterIdleTime - taskStartCalendar.getTimeInMillis() + offsetTimeInMs); break; @@ -323,9 +324,9 @@ default: // RESUME // resume = increment idle time - offsetTimeInMs -= currentTime - loopTimestamp; + offsetTimeInMs -= afterIdleTime - currentTime; // update time - publish(taskTimingBeforeStartInMs + currentTime + publish(taskTimingBeforeStartInMs + afterIdleTime - taskStartCalendar.getTimeInMillis() + offsetTimeInMs); break; @@ -333,7 +334,7 @@ } } else { // pas de idle, met a jour le temps - publish(taskTimingBeforeStartInMs + loopTimestamp + publish(taskTimingBeforeStartInMs + currentTime - taskStartCalendar.getTimeInMillis() + offsetTimeInMs); } Modified: trunk/src/main/java/org/chorem/jtimer/ui/treetable/ProjectsAndTasksModel.java =================================================================== --- trunk/src/main/java/org/chorem/jtimer/ui/treetable/ProjectsAndTasksModel.java 2012-02-16 14:27:52 UTC (rev 2805) +++ trunk/src/main/java/org/chorem/jtimer/ui/treetable/ProjectsAndTasksModel.java 2012-03-07 16:38:55 UTC (rev 2806) @@ -5,7 +5,7 @@ * $Id$ * $HeadURL$ * %% - * Copyright (C) 2007 - 2011 CodeLutin, Chatellier Eric + * Copyright (C) 2007 - 2012 CodeLutin, Chatellier Eric * %% * This program is free software: you can redistribute it and/or modify * it under the terms of the GNU General Public License as @@ -36,7 +36,7 @@ import org.apache.commons.collections.CollectionUtils; import org.apache.commons.collections.Predicate; -import org.apache.commons.lang.time.DurationFormatUtils; +import org.apache.commons.lang3.time.DurationFormatUtils; import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; import org.chorem.jtimer.data.DataEventListener; Modified: trunk/src/main/java/org/chorem/jtimer/ui/treetable/dnd/TimerTaskTransferHandler.java =================================================================== --- trunk/src/main/java/org/chorem/jtimer/ui/treetable/dnd/TimerTaskTransferHandler.java 2012-02-16 14:27:52 UTC (rev 2805) +++ trunk/src/main/java/org/chorem/jtimer/ui/treetable/dnd/TimerTaskTransferHandler.java 2012-03-07 16:38:55 UTC (rev 2806) @@ -5,7 +5,7 @@ * $Id$ * $HeadURL$ * %% - * Copyright (C) 2008 - 2011 CodeLutin, Chatellier Eric + * Copyright (C) 2008 - 2012 CodeLutin, Chatellier Eric * %% * This program is free software: you can redistribute it and/or modify * it under the terms of the GNU General Public License as @@ -36,7 +36,7 @@ import javax.swing.JOptionPane; import javax.swing.TransferHandler; -import org.apache.commons.lang.StringUtils; +import org.apache.commons.lang3.StringUtils; import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; import org.chorem.jtimer.data.DataViolationException; Deleted: trunk/src/main/resources/JTimer.properties =================================================================== --- trunk/src/main/resources/JTimer.properties 2012-02-16 14:27:52 UTC (rev 2805) +++ trunk/src/main/resources/JTimer.properties 2012-03-07 16:38:55 UTC (rev 2806) @@ -1,51 +0,0 @@ -### -# #%L -# jTimer -# -# $Id$ -# $HeadURL$ -# %% -# Copyright (C) 2007 - 2011 CodeLutin, Chatellier Eric -# %% -# This program is free software: you can redistribute it and/or modify -# it under the terms of the GNU General Public License as -# published by the Free Software Foundation, either version 3 of the -# License, or (at your option) any later version. -# -# This program is distributed in the hope that it will be useful, -# but WITHOUT ANY WARRANTY; without even the implied warranty of -# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the -# GNU General Public License for more details. -# -# You should have received a copy of the GNU General Public -# License along with this program. If not, see -# <http://www.gnu.org/licenses/gpl-3.0.html>. -# #L% -### -# JTimer.properties file - -########################## -# Synchronization Services -########################## -# Service class -#jtimer.service.class=org.chorem.jtimer.ws.xmlrpc.ChoremXMLRPCClient -# Service resource to use for previous class -#jtimer.service.resource=JTimer -# Service endpoint to use for previous class -#jtimer.service.endpoint=http://localhost:8080 - -########################## -# UI Options -########################## -# Idle time (integer in seconds) -jtimer.ui.idletime=300 - -########################## -# Save options -########################## -# jtimer io saver directory -jtimer.io.saver.class=org.chorem.jtimer.io.GTimerIncrementalSaver -# Saver directory -jtimer.io.saver.directory=$HOME/.gtimer -# Save delay -jtimer.io.saver.autosavedelay=300 Deleted: trunk/src/main/resources/JTimer_default.properties =================================================================== --- trunk/src/main/resources/JTimer_default.properties 2012-02-16 14:27:52 UTC (rev 2805) +++ trunk/src/main/resources/JTimer_default.properties 2012-03-07 16:38:55 UTC (rev 2806) @@ -1,51 +0,0 @@ -### -# #%L -# jTimer -# -# $Id$ -# $HeadURL$ -# %% -# Copyright (C) 2007 - 2011 CodeLutin, Chatellier Eric -# %% -# This program is free software: you can redistribute it and/or modify -# it under the terms of the GNU General Public License as -# published by the Free Software Foundation, either version 3 of the -# License, or (at your option) any later version. -# -# This program is distributed in the hope that it will be useful, -# but WITHOUT ANY WARRANTY; without even the implied warranty of -# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the -# GNU General Public License for more details. -# -# You should have received a copy of the GNU General Public -# License along with this program. If not, see -# <http://www.gnu.org/licenses/gpl-3.0.html>. -# #L% -### -# JTimer.properties file - -########################## -# Synchronization Services -########################## -# Service class -#jtimer.service.class=org.chorem.jtimer.ws.xmlrpc.ChoremXMLRPCClient -# Service resource to use for previous class -#jtimer.service.resource=JTimer -# Service endpoint to use for previous class -#jtimer.service.endpoint=http://localhost:8080 - -########################## -# UI Options -########################## -# Idle time (integer in seconds) -#jtimer.ui.idletime=300 - -########################## -# Save options -########################## -# jtimer io saver directory -#jtimer.io.saver.class=org.chorem.jtimer.io.GTimerIncrementalSaver -# Saver directory -#jtimer.io.saver.directory=$HOME/.gtimer -# Save delay -#jtimer.io.saver.autosavedelay=300 Modified: trunk/src/main/resources/log4j.properties =================================================================== --- trunk/src/main/resources/log4j.properties 2012-02-16 14:27:52 UTC (rev 2805) +++ trunk/src/main/resources/log4j.properties 2012-03-07 16:38:55 UTC (rev 2806) @@ -5,7 +5,7 @@ # $Id$ # $HeadURL$ # %% -# Copyright (C) 2007 - 2011 CodeLutin, Chatellier Eric +# Copyright (C) 2007 - 2012 CodeLutin, Chatellier Eric # %% # This program is free software: you can redistribute it and/or modify # it under the terms of the GNU General Public License as Modified: trunk/src/test/java/org/chorem/jtimer/AbstractJTimerTest.java =================================================================== --- trunk/src/test/java/org/chorem/jtimer/AbstractJTimerTest.java 2012-02-16 14:27:52 UTC (rev 2805) +++ trunk/src/test/java/org/chorem/jtimer/AbstractJTimerTest.java 2012-03-07 16:38:55 UTC (rev 2806) @@ -5,7 +5,7 @@ * $Id$ * $HeadURL$ * %% - * Copyright (C) 2008 - 2011 CodeLutin, Chatellier Eric + * Copyright (C) 2008 - 2012 CodeLutin, Chatellier Eric * %% * This program is free software: you can redistribute it and/or modify * it under the terms of the GNU General Public License as @@ -57,10 +57,9 @@ /** Class log */ private static Log log = LogFactory.getLog(AbstractJTimerTest.class); - protected static final String RELATIF_PROPERTIES_RESOURCE = "/JTimerTestRelatif.properties"; - /** Saver instance on specific test directory. */ protected Saver testSaver; + /** Test directory. */ protected File testDataDirectory; @@ -86,16 +85,18 @@ */ protected void initDataDirectory() throws IOException { - URL testConfigFile = AbstractJTimerTest.class - .getResource(RELATIF_PROPERTIES_RESOURCE); + URL testConfigFile = AbstractJTimerTest.class.getResource("/jtimertest.properties"); + Properties props = new Properties(); + props.load(testConfigFile.openStream()); - JTimerFactory.props = new Properties(); - JTimerFactory.props.load(testConfigFile.openStream()); testDataDirectory = new File(getTempDirPath(), String.valueOf(System .currentTimeMillis())); - JTimerFactory.props.put("jtimer.io.saver.directory", testDataDirectory + props.put("jtimer.io.saver.directory", testDataDirectory .getAbsolutePath()); + JTimer.config = new JTimerConfig(); + JTimer.config.appConfig.setOptions(props); // not call parse in test + if (log.isDebugEnabled()) { log.debug("Copy resource test directory to " + testDataDirectory.getAbsolutePath()); Modified: trunk/src/test/java/org/chorem/jtimer/JTimerFactoryTest.java =================================================================== --- trunk/src/test/java/org/chorem/jtimer/JTimerFactoryTest.java 2012-02-16 14:27:52 UTC (rev 2805) +++ trunk/src/test/java/org/chorem/jtimer/JTimerFactoryTest.java 2012-03-07 16:38:55 UTC (rev 2806) @@ -25,12 +25,13 @@ package org.chorem.jtimer; -import java.util.Properties; - import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; +import org.chorem.jtimer.JTimerConfig.JTimerOption; +import org.chorem.jtimer.io.GTimerIncrementalSaver; import org.chorem.jtimer.io.Saver; import org.chorem.jtimer.ws.ProjectManagement; +import org.chorem.jtimer.ws.xmlrpc.ChoremXMLRPCClient; import org.testng.Assert; import org.testng.annotations.Test; @@ -49,61 +50,30 @@ private static Log log = LogFactory.getLog(JTimerFactoryTest.class); /** - * Test que les proprietes ne sont pas - * nulles et que les valeures sont bonnes. + * Verifie que les options par default sont bien chargées. */ @Test - public void initTest() { + public void testInitConfiguration() { if (log.isInfoEnabled()) { log.info("Test property loading " + JTimerFactoryTest.class); } - Properties props = JTimerFactory.props; + Assert.assertEquals(JTimer.config.getServiceClass(), ChoremXMLRPCClient.class); + Assert.assertEquals(JTimer.config.getServiceResource(), "JTimer"); + Assert.assertEquals(JTimer.config.getServiceEndpoint(), "http://localhost:8080"); - Assert.assertEquals( - "org.chorem.jtimer.ws.xmlrpc.ChoremXMLRPCClient", props - .getProperty("jtimer.service.class")); - Assert.assertEquals("JTimer", props - .getProperty("jtimer.service.resource")); - Assert.assertEquals("http://localhost:8080", props - .getProperty("jtimer.service.endpoint")); + Assert.assertEquals(JTimer.config.getIdleTime(), 299); - Assert.assertEquals("299", props.getProperty("jtimer.ui.idletime")); - - Assert.assertEquals("org.chorem.jtimer.io.GTimerIncrementalSaver", - props.getProperty("jtimer.io.saver.class")); + Assert.assertEquals(JTimer.config.getIOSaverClass(), GTimerIncrementalSaver.class); // Temp dir is dynamic //Assert.assertEquals(AbstractJTimerTest.getTempDirPath() + "/testngdata", - // props.getProperty("jtimer.io.saver.directory")); - Assert.assertEquals("199", props - .getProperty("jtimer.io.saver.autosavedelay")); + // JTimer.config.getIOSaverDirectory()); + Assert.assertEquals(JTimer.config.getIOSaverAutoSaveDelay(), 199); } /** - * Test que le fichier de conf s'appelle bien JTimer.properties. - */ - @Test - public void getConfigurationFileName() { - String name = JTimerFactory.getConfigurationFileName(); - Assert.assertNotNull(name); - Assert.assertEquals("JTimer.properties", name); - } - - /** - * Test que le fichier de path du fichier n'est pas null. - * Le chemin complet est $HOME/.jtimer/getConfigurationFileName(). - * Mais ca depend de $HOME - */ - @Test - public void getConfigurationFilePath() { - String path = JTimerFactory.getUserHomeConfigurationFilePath(); - Assert.assertNotNull(path); - Assert.assertTrue(path.endsWith("/.jtimer/JTimer.properties")); - } - - /** * Test que le saver de fichier s'est bien initialise. */ @Test @@ -114,45 +84,14 @@ } /** - * Test la valeur de idletime. - */ - @Test - public void getIdleTime() { - long idleTime = JTimerFactory.getIdleTime(); - - Assert.assertTrue(idleTime > 0L); - Assert.assertEquals(299000, idleTime); - } - - /** - * Test que le $HOME est bien remplacé - */ - @Test - public void getHomeReplacementTest() { - - Assert.assertEquals("/home/test/.gtimer", JTimerFactory - .getHomeReplacement("/home/test/.gtimer")); - - Assert.assertEquals(System.getProperty("user.home") + "/.gtimer", - JTimerFactory.getHomeReplacement("$HOME/.gtimer")); - - Assert.assertEquals( - "C:\\Documents And Settings\\user\\.gtimer", - JTimerFactory - .getHomeReplacement("C:\\Documents And Settings\\user\\.gtimer")); - } - - /** * Test que le composant gerant les webservices s'est bien initialise. */ @Test public void getProjectManagementService() { - ProjectManagement projectManagement = JTimerFactory - .getProjectManagementService(); + ProjectManagement projectManagement = JTimerFactory.getProjectManagementService(); Assert.assertNotNull(projectManagement); - Assert.assertEquals("http://localhost:8080", projectManagement - .getEndpoint()); - Assert.assertEquals("JTimer", projectManagement.getResourceName()); + Assert.assertEquals(projectManagement.getEndpoint(), "http://localhost:8080"); + Assert.assertEquals(projectManagement.getResourceName(), "JTimer"); } } Deleted: trunk/src/test/resources/JTimerTestRelatif.properties =================================================================== --- trunk/src/test/resources/JTimerTestRelatif.properties 2012-02-16 14:27:52 UTC (rev 2805) +++ trunk/src/test/resources/JTimerTestRelatif.properties 2012-03-07 16:38:55 UTC (rev 2806) @@ -1,40 +0,0 @@ -### -# #%L -# jTimer -# -# $Id$ -# $HeadURL$ -# %% -# Copyright (C) 2007 - 2011 CodeLutin, Chatellier Eric -# %% -# This program is free software: you can redistribute it and/or modify -# it under the terms of the GNU General Public License as -# published by the Free Software Foundation, either version 3 of the -# License, or (at your option) any later version. -# -# This program is distributed in the hope that it will be useful, -# but WITHOUT ANY WARRANTY; without even the implied warranty of -# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the -# GNU General Public License for more details. -# -# You should have received a copy of the GNU General Public -# License along with this program. If not, see -# <http://www.gnu.org/licenses/gpl-3.0.html>. -# #L% -### -# Service class -jtimer.service.class=org.chorem.jtimer.ws.xmlrpc.ChoremXMLRPCClient -# Service resource to use for previous class -jtimer.service.resource=JTimer -# Service endpoint to use for previous class -jtimer.service.endpoint=http://localhost:8080 - -# Idle time (integer in seconds) -jtimer.ui.idletime=299 - -# jtimer io saver directory -jtimer.io.saver.class=org.chorem.jtimer.io.GTimerIncrementalSaver -# Saver directory -jtimer.io.saver.directory=$HOME/testngdata -# Save delay -jtimer.io.saver.autosavedelay=199 \ No newline at end of file Copied: trunk/src/test/resources/jtimertest.properties (from rev 2803, trunk/src/test/resources/JTimerTestRelatif.properties) =================================================================== --- trunk/src/test/resources/jtimertest.properties (rev 0) +++ trunk/src/test/resources/jtimertest.properties 2012-03-07 16:38:55 UTC (rev 2806) @@ -0,0 +1,40 @@ +### +# #%L +# jTimer +# +# $Id$ +# $HeadURL$ +# %% +# Copyright (C) 2007 - 2011 CodeLutin, Chatellier Eric +# %% +# This program is free software: you can redistribute it and/or modify +# it under the terms of the GNU General Public License as +# published by the Free Software Foundation, either version 3 of the +# License, or (at your option) any later version. +# +# This program is distributed in the hope that it will be useful, +# but WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +# GNU General Public License for more details. +# +# You should have received a copy of the GNU General Public +# License along with this program. If not, see +# <http://www.gnu.org/licenses/gpl-3.0.html>. +# #L% +### +# Service class +jtimer.service.class=org.chorem.jtimer.ws.xmlrpc.ChoremXMLRPCClient +# Service resource to use for previous class +jtimer.service.resource=JTimer +# Service endpoint to use for previous class +jtimer.service.endpoint=http://localhost:8080 + +# Idle time (integer in seconds) +jtimer.ui.idletime=299 + +# jtimer io saver directory +jtimer.io.saver.class=org.chorem.jtimer.io.GTimerIncrementalSaver +# Saver directory +jtimer.io.saver.directory=${user.home}/testngdata +# Save delay +jtimer.io.saver.autosavedelay=199 \ No newline at end of file

1 0