September 2012 - Sammoa-commits - mailman.cloud.codelutin.com

r581 - in trunk: . sammoa-application sammoa-persistence sammoa-ui-swing
by maven-release＠users.forge.codelutin.com 12 Sep '12

12 Sep '12

Author: maven-release Date: 2012-09-12 18:23:50 +0200 (Wed, 12 Sep 2012) New Revision: 581 Url: http://forge.codelutin.com/repositories/revision/sammoa/581 Log: [maven-release-plugin] prepare release sammoa-1.0-rc2 Modified: trunk/pom.xml trunk/sammoa-application/pom.xml trunk/sammoa-persistence/pom.xml trunk/sammoa-ui-swing/pom.xml Modified: trunk/pom.xml =================================================================== --- trunk/pom.xml 2012-09-12 16:18:39 UTC (rev 580) +++ trunk/pom.xml 2012-09-12 16:23:50 UTC (rev 581) @@ -15,7 +15,7 @@ <groupId>fr.ulr</groupId> <artifactId>sammoa</artifactId> - <version>1.0-rc2-SNAPSHOT</version> + <version>1.0-rc2</version> <modules> <module>sammoa-persistence</module> @@ -81,12 +81,12 @@  <scm> - <url>http://svn.forge.codelutin.com/svn/sammoa/trunk</url> + <url>http://svn.forge.codelutin.com/svn/sammoa/tags/sammoa-1.0-rc2</url> <connection> - scm:svn:http://svn.forge.codelutin.com/svn/sammoa/trunk + scm:svn:http://svn.forge.codelutin.com/svn/sammoa/tags/sammoa-1.0-rc2 </connection> <developerConnection> - scm:svn:http://svn.forge.codelutin.com/svn/sammoa/trunk + scm:svn:http://svn.forge.codelutin.com/svn/sammoa/tags/sammoa-1.0-rc2 </developerConnection> </scm> Modified: trunk/sammoa-application/pom.xml =================================================================== --- trunk/sammoa-application/pom.xml 2012-09-12 16:18:39 UTC (rev 580) +++ trunk/sammoa-application/pom.xml 2012-09-12 16:23:50 UTC (rev 581) @@ -9,7 +9,7 @@ <parent> <groupId>fr.ulr</groupId> <artifactId>sammoa</artifactId> - <version>1.0-rc2-SNAPSHOT</version> + <version>1.0-rc2</version> </parent> <artifactId>sammoa-application</artifactId> Modified: trunk/sammoa-persistence/pom.xml =================================================================== --- trunk/sammoa-persistence/pom.xml 2012-09-12 16:18:39 UTC (rev 580) +++ trunk/sammoa-persistence/pom.xml 2012-09-12 16:23:50 UTC (rev 581) @@ -9,7 +9,7 @@ <parent> <groupId>fr.ulr</groupId> <artifactId>sammoa</artifactId> - <version>1.0-rc2-SNAPSHOT</version> + <version>1.0-rc2</version> </parent> <artifactId>sammoa-persistence</artifactId> Modified: trunk/sammoa-ui-swing/pom.xml =================================================================== --- trunk/sammoa-ui-swing/pom.xml 2012-09-12 16:18:39 UTC (rev 580) +++ trunk/sammoa-ui-swing/pom.xml 2012-09-12 16:23:50 UTC (rev 581) @@ -9,7 +9,7 @@ <parent> <groupId>fr.ulr</groupId> <artifactId>sammoa</artifactId> - <version>1.0-rc2-SNAPSHOT</version> + <version>1.0-rc2</version> </parent> <artifactId>sammoa-ui-swing</artifactId>

1 0

r580 - trunk/sammoa-application/src/main/java/fr/ulr/sammoa/application
by fdesbois＠users.forge.codelutin.com 12 Sep '12

12 Sep '12

Author: fdesbois Date: 2012-09-12 18:18:39 +0200 (Wed, 12 Sep 2012) New Revision: 580 Url: http://forge.codelutin.com/repositories/revision/sammoa/580 Log: little improvement Modified: trunk/sammoa-application/src/main/java/fr/ulr/sammoa/application/FlightService.java Modified: trunk/sammoa-application/src/main/java/fr/ulr/sammoa/application/FlightService.java =================================================================== --- trunk/sammoa-application/src/main/java/fr/ulr/sammoa/application/FlightService.java 2012-09-12 15:48:57 UTC (rev 579) +++ trunk/sammoa-application/src/main/java/fr/ulr/sammoa/application/FlightService.java 2012-09-12 16:18:39 UTC (rev 580) @@ -66,6 +66,7 @@ import org.apache.commons.lang3.StringUtils; import org.joda.time.DateTime; import org.joda.time.Interval; +import org.joda.time.Seconds; import org.nuiton.topia.TopiaContext; import org.nuiton.topia.TopiaException; import org.nuiton.topia.TopiaRuntimeException; @@ -495,11 +496,8 @@ result.setSpeed(location.getSpeed()); result.setAltitude(location.getAltitude()); - int captureDelay = Dates - .toInterval(locationTime, newTime) - .toDuration() - .toStandardSeconds() - .getSeconds(); + int captureDelay = + Seconds.secondsBetween(locationTime, newTime).getSeconds(); result.setCaptureDelay(captureDelay);

1 0

r579 - trunk/sammoa-ui-swing
by fdesbois＠users.forge.codelutin.com 12 Sep '12

12 Sep '12

Author: fdesbois Date: 2012-09-12 17:48:57 +0200 (Wed, 12 Sep 2012) New Revision: 579 Url: http://forge.codelutin.com/repositories/revision/sammoa/579 Log: missing dependency joda-time Modified: trunk/sammoa-ui-swing/pom.xml Modified: trunk/sammoa-ui-swing/pom.xml =================================================================== --- trunk/sammoa-ui-swing/pom.xml 2012-09-12 15:48:20 UTC (rev 578) +++ trunk/sammoa-ui-swing/pom.xml 2012-09-12 15:48:57 UTC (rev 579) @@ -88,6 +88,11 @@ </dependency> <dependency> + <groupId>joda-time</groupId> + <artifactId>joda-time</artifactId> + </dependency> + + <dependency> <groupId>org.nuiton.i18n</groupId> <artifactId>nuiton-i18n</artifactId> </dependency>

1 0

r578 - in trunk: sammoa-persistence/src/test/java/fr/ulr/sammoa/persistence sammoa-ui-swing/src/main/java/fr/ulr/sammoa/ui/swing/util
by maven-release＠users.forge.codelutin.com 12 Sep '12

12 Sep '12

Author: maven-release Date: 2012-09-12 17:48:20 +0200 (Wed, 12 Sep 2012) New Revision: 578 Url: http://forge.codelutin.com/repositories/revision/sammoa/578 Log: add file header Modified: trunk/sammoa-persistence/src/test/java/fr/ulr/sammoa/persistence/DatesTest.java trunk/sammoa-ui-swing/src/main/java/fr/ulr/sammoa/ui/swing/util/NestedPropertyChangeListener.java Modified: trunk/sammoa-persistence/src/test/java/fr/ulr/sammoa/persistence/DatesTest.java =================================================================== --- trunk/sammoa-persistence/src/test/java/fr/ulr/sammoa/persistence/DatesTest.java 2012-09-12 15:39:59 UTC (rev 577) +++ trunk/sammoa-persistence/src/test/java/fr/ulr/sammoa/persistence/DatesTest.java 2012-09-12 15:48:20 UTC (rev 578) @@ -1,5 +1,29 @@ package fr.ulr.sammoa.persistence; +/* + * #%L + * SAMMOA :: Persistence + * $Id$ + * $HeadURL$ + * %% + * Copyright (C) 2012 UMS 3462, Code Lutin + * %% + * 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% + */ + import com.google.common.collect.ImmutableList; import org.junit.Assert; import org.junit.Test; Modified: trunk/sammoa-ui-swing/src/main/java/fr/ulr/sammoa/ui/swing/util/NestedPropertyChangeListener.java =================================================================== --- trunk/sammoa-ui-swing/src/main/java/fr/ulr/sammoa/ui/swing/util/NestedPropertyChangeListener.java 2012-09-12 15:39:59 UTC (rev 577) +++ trunk/sammoa-ui-swing/src/main/java/fr/ulr/sammoa/ui/swing/util/NestedPropertyChangeListener.java 2012-09-12 15:48:20 UTC (rev 578) @@ -1,5 +1,29 @@ package fr.ulr.sammoa.ui.swing.util; +/* + * #%L + * SAMMOA :: UI Swing + * $Id$ + * $HeadURL$ + * %% + * Copyright (C) 2012 UMS 3462, Code Lutin + * %% + * 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% + */ + import com.google.common.base.Splitter; import org.jdesktop.beans.AbstractBean; import org.nuiton.topia.persistence.TopiaEntity;

1 0

r577 - in trunk: sammoa-application/src/test/java/fr/ulr/sammoa/application/output/map sammoa-ui-swing/src/main/java/fr/ulr/sammoa/ui/swing/flight/effort
by fdesbois＠users.forge.codelutin.com 12 Sep '12

12 Sep '12

Author: fdesbois Date: 2012-09-12 17:39:59 +0200 (Wed, 12 Sep 2012) New Revision: 577 Url: http://forge.codelutin.com/repositories/revision/sammoa/577 Log: Resolve broken test + NPE in TransectFlightcellEditor Modified: trunk/sammoa-application/src/test/java/fr/ulr/sammoa/application/output/map/ExportMapServiceTest.java trunk/sammoa-ui-swing/src/main/java/fr/ulr/sammoa/ui/swing/flight/effort/EffortPanelHandler.java Modified: trunk/sammoa-application/src/test/java/fr/ulr/sammoa/application/output/map/ExportMapServiceTest.java =================================================================== --- trunk/sammoa-application/src/test/java/fr/ulr/sammoa/application/output/map/ExportMapServiceTest.java 2012-09-12 14:45:33 UTC (rev 576) +++ trunk/sammoa-application/src/test/java/fr/ulr/sammoa/application/output/map/ExportMapServiceTest.java 2012-09-12 15:39:59 UTC (rev 577) @@ -222,6 +222,7 @@ builder.newMandatoryColumn("LON", "longitude", Common.DOUBLE_PRIMITIVE); builder.newMandatoryColumn("SPEED", "speed", Common.DOUBLE_PRIMITIVE); builder.newMandatoryColumn("ALTITUDE", "altitude", Common.DOUBLE_PRIMITIVE); + builder.newMandatoryColumn("GPS_DELAY", "captureDelay", new DoubleToIntegerValueParser()); return builder.getColumnsForImport(); } } Modified: trunk/sammoa-ui-swing/src/main/java/fr/ulr/sammoa/ui/swing/flight/effort/EffortPanelHandler.java =================================================================== --- trunk/sammoa-ui-swing/src/main/java/fr/ulr/sammoa/ui/swing/flight/effort/EffortPanelHandler.java 2012-09-12 14:45:33 UTC (rev 576) +++ trunk/sammoa-ui-swing/src/main/java/fr/ulr/sammoa/ui/swing/flight/effort/EffortPanelHandler.java 2012-09-12 15:39:59 UTC (rev 577) @@ -1010,7 +1010,8 @@ boolean result; TransectFlight value = getCellEditorValue(); - if (Objects.equal(value, initValue) + if (value == null + || Objects.equal(value, initValue) || SammoaUtil.askQuestion(editorComponent, _( "sammoa.confirmDialog.changeRouteTransect.message", value.getTransect().getName()))) {

1 0

r576 - trunk/sammoa-ui-swing/src/main/resources/fr/ulr/sammoa/persistence
by fdesbois＠users.forge.codelutin.com 12 Sep '12

12 Sep '12

Author: fdesbois Date: 2012-09-12 16:45:33 +0200 (Wed, 12 Sep 2012) New Revision: 576 Url: http://forge.codelutin.com/repositories/revision/sammoa/576 Log: fixes #1479 : min value for podSize is 1 Modified: trunk/sammoa-ui-swing/src/main/resources/fr/ulr/sammoa/persistence/Observation-onBoard-warning-validation.xml trunk/sammoa-ui-swing/src/main/resources/fr/ulr/sammoa/persistence/Observation-validation-error-validation.xml Modified: trunk/sammoa-ui-swing/src/main/resources/fr/ulr/sammoa/persistence/Observation-onBoard-warning-validation.xml =================================================================== --- trunk/sammoa-ui-swing/src/main/resources/fr/ulr/sammoa/persistence/Observation-onBoard-warning-validation.xml 2012-09-12 14:38:09 UTC (rev 575) +++ trunk/sammoa-ui-swing/src/main/resources/fr/ulr/sammoa/persistence/Observation-onBoard-warning-validation.xml 2012-09-12 14:45:33 UTC (rev 576) @@ -32,7 +32,7 @@ <field name="podSize"> <field-validator type="int" short-circuit="true"> - <param name="min">0</param> + <param name="min">1</param> <message>sammoa.validator.observation.podSize.invalidMin##${min}</message> </field-validator> Modified: trunk/sammoa-ui-swing/src/main/resources/fr/ulr/sammoa/persistence/Observation-validation-error-validation.xml =================================================================== --- trunk/sammoa-ui-swing/src/main/resources/fr/ulr/sammoa/persistence/Observation-validation-error-validation.xml 2012-09-12 14:38:09 UTC (rev 575) +++ trunk/sammoa-ui-swing/src/main/resources/fr/ulr/sammoa/persistence/Observation-validation-error-validation.xml 2012-09-12 14:45:33 UTC (rev 576) @@ -32,7 +32,7 @@ <field name="podSize"> <field-validator type="int" short-circuit="true"> - <param name="min">0</param> + <param name="min">1</param> <message>sammoa.validator.observation.podSize.invalidMin##${min}</message> </field-validator>

1 0

r575 - in trunk: sammoa-application/src/main/java/fr/ulr/sammoa/application/io/output/map sammoa-ui-swing/src/main/java/fr/ulr/sammoa/ui/swing/flight/effort src/site/rst
by fdesbois＠users.forge.codelutin.com 12 Sep '12

12 Sep '12

Author: fdesbois Date: 2012-09-12 16:38:09 +0200 (Wed, 12 Sep 2012) New Revision: 575 Url: http://forge.codelutin.com/repositories/revision/sammoa/575 Log: refs #1426 : - add listener for observation species even during onBoard - add GPS_DELAY instead of GPS_TIME in export maps Modified: trunk/sammoa-application/src/main/java/fr/ulr/sammoa/application/io/output/map/ExportMapService.java trunk/sammoa-ui-swing/src/main/java/fr/ulr/sammoa/ui/swing/flight/effort/EffortPanelHandler.java trunk/src/site/rst/import-export.rst Modified: trunk/sammoa-application/src/main/java/fr/ulr/sammoa/application/io/output/map/ExportMapService.java =================================================================== --- trunk/sammoa-application/src/main/java/fr/ulr/sammoa/application/io/output/map/ExportMapService.java 2012-09-12 14:37:52 UTC (rev 574) +++ trunk/sammoa-application/src/main/java/fr/ulr/sammoa/application/io/output/map/ExportMapService.java 2012-09-12 14:38:09 UTC (rev 575) @@ -131,7 +131,7 @@ .doubleColumn("LON", 19, 11) // (geoPoint.longitude) .doubleColumn("SPEED", 19, 11) // (geoPoint.speed) .doubleColumn("ALTITUDE", 19, 11) // (geoPoint.altitude) - .stringColumn("GPS_TIME") // (format(geoPoint.recordTime, "HHmmss")) + .integerColumn("GPS_DELAY") // (geoPoint.captureDelay) .stringColumn("AIRCRAFT") // (route.flight.immatriculation) .stringColumn("COMMENT") // (route.comment) .build(); @@ -357,7 +357,7 @@ .doubleColumn("LON", 19, 11) // (geoPoint.longitude) .doubleColumn("SPEED", 19, 11) // (geoPoint.speed) .doubleColumn("ALTITUDE", 19, 11) // (geoPoint.altitude) - .stringColumn("GPS_TIME") // (format(geoPoint.recordTime, "HHmmss")) + .integerColumn("GPS_DELAY") // (geoPoint.captureDelay) .stringColumn("AIRCRAFT") // (flight.immatriculation) .stringColumn("COMMENT") // (observation.comment) .build(); @@ -594,6 +594,7 @@ .doubleColumn("LON", 19, 11) .doubleColumn("SPEED", 19, 11) .doubleColumn("ALTITUDE", 19, 11) + .integerColumn("GPS_DELAY") // (geoPoint.captureDelay) .build(); Campaign campaign = dataModel.getCampaign(); @@ -678,6 +679,7 @@ record.put("LON", geoPoint.getLongitude()); record.put("SPEED", geoPoint.getSpeed()); record.put("ALTITUDE", geoPoint.getAltitude()); + record.put("GPS_DELAY", geoPoint.getCaptureDelay()); builder.addValues(record); Modified: trunk/sammoa-ui-swing/src/main/java/fr/ulr/sammoa/ui/swing/flight/effort/EffortPanelHandler.java =================================================================== --- trunk/sammoa-ui-swing/src/main/java/fr/ulr/sammoa/ui/swing/flight/effort/EffortPanelHandler.java 2012-09-12 14:37:52 UTC (rev 574) +++ trunk/sammoa-ui-swing/src/main/java/fr/ulr/sammoa/ui/swing/flight/effort/EffortPanelHandler.java 2012-09-12 14:38:09 UTC (rev 575) @@ -557,6 +557,18 @@ } }); + // Used to update map for each Species change, Note that this could be + // a heavy coast for onBoard screen + getModel().addPropertyChangeListener( + FlightUIModel.PROPERTY_OBSERVATION_EDIT_BEAN, new PropertyChangeListener() { + + @Override + public void propertyChange(PropertyChangeEvent evt) { + onObservationChanged((Observation) evt.getOldValue(), + (Observation) evt.getNewValue()); + } + }); + if (getModel().isValidationMode()) { getModel().addPropertyChangeListener( @@ -824,6 +836,20 @@ } } + private void onObservationChanged(Observation oldValue, + Observation newValue) { + + if (oldValue != null) { + oldValue.removePropertyChangeListener( + Observation.PROPERTY_SPECIES, observationSpeciesChangeListener); + } + + if (newValue != null) { + newValue.addPropertyChangeListener( + Observation.PROPERTY_SPECIES, observationSpeciesChangeListener); + } + } + private void onTransectFlightChangedForValidation(TransectFlightModel oldValue, TransectFlightModel newValue) { if (oldValue != null) { @@ -873,15 +899,11 @@ if (oldValue != null) { oldValue.removePropertyChangeListener( Observation.PROPERTY_OBSERVATION_TIME, observationTimeChangeListener); - oldValue.removePropertyChangeListener( - Observation.PROPERTY_SPECIES, observationSpeciesChangeListener); } if (newValue != null) { newValue.addPropertyChangeListener( Observation.PROPERTY_OBSERVATION_TIME, observationTimeChangeListener); - newValue.addPropertyChangeListener( - Observation.PROPERTY_SPECIES, observationSpeciesChangeListener); selectRouteByObservation(newValue); Modified: trunk/src/site/rst/import-export.rst =================================================================== --- trunk/src/site/rst/import-export.rst 2012-09-12 14:37:52 UTC (rev 574) +++ trunk/src/site/rst/import-export.rst 2012-09-12 14:38:09 UTC (rev 575) @@ -201,6 +201,8 @@ geoPoint.speed - ALTITUDE : Altitude de l'avion à ce point GPS geoPoint.altitude +- GPS_DELAY : Délai en secondes depuis la capture GPS + geoPoint.captureDelay Effort ~~~~~~ @@ -361,9 +363,9 @@ * geoPoint.altitude -- GPS_TIME : Heure d'enregistrement du point (rarement différente de celle du parcours) +- GPS_DELAY : Délai en secondes depuis la capture GPS (N) - * format(geoPoint.recordTime, "HHmmss") + * geoPoint.captureDelay - AIRCRAFT : Immatriculation de l'avion @@ -540,9 +542,9 @@ * geoPoint.altitude -- GPS_TIME : Heure d'enregistrement du point (rarement différente de celle du parcours) +- GPS_DELAY : Délai en secondes depuis la capture GPS (N) - * format(geoPoint.recordTime, "HHmmss") + * geoPoint.captureDelay - AIRCRAFT : Immatriculation de l'avion

1 0

r574 - in trunk: sammoa-application/src/main/java/fr/ulr/sammoa/application sammoa-application/src/main/java/fr/ulr/sammoa/application/io/output/map sammoa-application/src/main/resources/i18n sammoa-ui-swing/src/main/resources/i18n
by fdesbois＠users.forge.codelutin.com 12 Sep '12

12 Sep '12

Author: fdesbois Date: 2012-09-12 16:37:52 +0200 (Wed, 12 Sep 2012) New Revision: 574 Url: http://forge.codelutin.com/repositories/revision/sammoa/574 Log: some labels corrections Modified: trunk/sammoa-application/src/main/java/fr/ulr/sammoa/application/SammoaConfig.java trunk/sammoa-application/src/main/java/fr/ulr/sammoa/application/io/output/map/ExportMapService.java trunk/sammoa-application/src/main/resources/i18n/sammoa-application_en_GB.properties trunk/sammoa-ui-swing/src/main/resources/i18n/sammoa-ui-swing_en_GB.properties Modified: trunk/sammoa-application/src/main/java/fr/ulr/sammoa/application/SammoaConfig.java =================================================================== --- trunk/sammoa-application/src/main/java/fr/ulr/sammoa/application/SammoaConfig.java 2012-09-12 14:37:34 UTC (rev 573) +++ trunk/sammoa-application/src/main/java/fr/ulr/sammoa/application/SammoaConfig.java 2012-09-12 14:37:52 UTC (rev 574) @@ -384,7 +384,7 @@ ), /** The auto commit delay for {@link AutoSaveListener} */ AUTO_COMMIT_DELAY("auto.commit.delay", - n_("sammoa.config.flight.number"), + n_("sammoa.config.autoCommitDelay"), "30000", Long.class ), Modified: trunk/sammoa-application/src/main/java/fr/ulr/sammoa/application/io/output/map/ExportMapService.java =================================================================== --- trunk/sammoa-application/src/main/java/fr/ulr/sammoa/application/io/output/map/ExportMapService.java 2012-09-12 14:37:34 UTC (rev 573) +++ trunk/sammoa-application/src/main/java/fr/ulr/sammoa/application/io/output/map/ExportMapService.java 2012-09-12 14:37:52 UTC (rev 574) @@ -808,7 +808,7 @@ } protected String computeObservationId(Flight flight, Observation observation) { - return "O" + observation.getObservationNumber() + "-" + + return "S" + observation.getObservationNumber() + "-" + flight.getFlightNumber() + "-" + flight.getSystemId(); } Modified: trunk/sammoa-application/src/main/resources/i18n/sammoa-application_en_GB.properties =================================================================== --- trunk/sammoa-application/src/main/resources/i18n/sammoa-application_en_GB.properties 2012-09-12 14:37:34 UTC (rev 573) +++ trunk/sammoa-application/src/main/resources/i18n/sammoa-application_en_GB.properties 2012-09-12 14:37:52 UTC (rev 574) @@ -1,12 +1,13 @@ sammoa.config.admin.email=Administrator's email sammoa.config.application.site.url=Website URL of the application sammoa.config.application.version=Version of the application -sammoa.config.audio.compression= +sammoa.config.audio.compression=Audio compression (possible values \: ULAW) sammoa.config.audio.recordDelayInSeconds=Delay in seconds between each audio record. This will record multiple files at the same time to avoid too sudden stops. (Warning \: this is not supported by all sound systems) sammoa.config.audio.sampleRate=Rate for the audio sampling (possible values \: 8000, 11025, 16000, 22050, 44100) sammoa.config.audio.sampleSizeInBits=Size of bits for the audio sampling (possible values \: 8, 16) +sammoa.config.autoCommitDelay=Time in milliseconds between each auto save in database sammoa.config.background.shape.file=Location of the Background shape file -sammoa.config.backup.directory=BAckup directory where to sotre all data backup +sammoa.config.backup.directory=Backup directory where to store all data backup sammoa.config.campaign.directory=Directory where are stored campaign data (maps, flight audio files,...) sammoa.config.campaign.id=Id of the default campaign to use sammoa.config.csv.separator=Csv separator caracter to use @@ -19,23 +20,23 @@ sammoa.config.gps.handler=GpsHandler class \: fr.ulr.sammoa.application.device.gps.GpsHandlerGpylon or fr.ulr.sammoa.application.device.gps.FakeGpsHandler sammoa.config.gps.speed=GPS speed for capture sammoa.config.gps.timeout=Time in seconds before GPS going to timeout (always the same data is received for this laps of time) and return error -sammoa.config.license.organizationName= +sammoa.config.license.organizationName=Name of the organization for license sammoa.config.log.file=Path for application log file sammoa.config.system.id=Id of the default system to use sammoa.config.tmp.directory=Temporary directory used for different task (import, export,...) -sammoa.config.ui.add=Add -sammoa.config.ui.begin=Begin -sammoa.config.ui.centerObservation=Observation from the center place +sammoa.config.ui.add=Add a new LEG (conditions have changed) +sammoa.config.ui.begin=Begin the effort on the next transect or restart effort from previous Circle-Back +sammoa.config.ui.centerObservation=Sighting from the center observer sammoa.config.ui.circleBack=Start a circle-back sammoa.config.ui.config=Path for application windows configuration file -sammoa.config.ui.end=End -sammoa.config.ui.leftObservation=Observation from the left place +sammoa.config.ui.end=End the effort +sammoa.config.ui.leftObservation=Sighting from the left observer sammoa.config.ui.locale=Application language -sammoa.config.ui.next=Next -sammoa.config.ui.rightObservation=Observation from the right place -sammoa.config.ui.start=Start -sammoa.config.ui.stop=Stop -sammoa.config.ui.validObservation=Validate the selected observation +sammoa.config.ui.next=Next effort will end the current one and begin on the next transect +sammoa.config.ui.rightObservation=Sighting from the right observer +sammoa.config.ui.start=Start the flight +sammoa.config.ui.stop=Stop the flight +sammoa.config.ui.validObservation=Validate the selected sighting sammoa.config.ui.validRoute=Validate the selected route sammoa.config.ui.validTransect=Validate the selected transect sammoa.dateTimePattern=dd/MM/yyyy HH\:mm\:ss Modified: trunk/sammoa-ui-swing/src/main/resources/i18n/sammoa-ui-swing_en_GB.properties =================================================================== --- trunk/sammoa-ui-swing/src/main/resources/i18n/sammoa-ui-swing_en_GB.properties 2012-09-12 14:37:34 UTC (rev 573) +++ trunk/sammoa-ui-swing/src/main/resources/i18n/sammoa-ui-swing_en_GB.properties 2012-09-12 14:37:52 UTC (rev 574) @@ -4,16 +4,16 @@ sammoa.action.about=About sammoa.action.about.tip=About sammoa.action.add=Add -sammoa.action.add.tip=ADD \: create a new LEG route for the current Transect (the observation conditions have changed) +sammoa.action.add.tip=ADD \: create a new LEG route for the current Transect (the sighting conditions have changed) sammoa.action.audioCheck=Audio test sammoa.action.audioCheck.tip=Audio test sammoa.action.begin=Begin sammoa.action.begin.tip=BEGIN \: begin effort to create a new LEG route for the next selected Transect sammoa.action.cancel=Cancel sammoa.action.cancelExport=Cancel -sammoa.action.center.tip=CENTER \: create a new Observation for the navigator's observer +sammoa.action.center.tip=CENTER \: create a new sighting for the navigator's observer sammoa.action.choose.export.directory=Accept -sammoa.action.circleBack.tip=CIRCLE-BACK \: create a CIRCLE_BACK route for this observation as cause +sammoa.action.circleBack.tip=CIRCLE-BACK \: create a CIRCLE_BACK route for this sighting as cause sammoa.action.close=Close sammoa.action.configuration=Configuration sammoa.action.configuration.tip=Configuration @@ -26,11 +26,11 @@ sammoa.action.exportEffortsMap=Export Efforts sammoa.action.exportGeoPointsMap=Export GPS Points sammoa.action.exportMap=Export Maps -sammoa.action.exportObservationsMap=Export Observations +sammoa.action.exportObservationsMap=Export Sightings sammoa.action.home=Home sammoa.action.home.tip=Home sammoa.action.importApplication=Import -sammoa.action.left.tip=LEFT \: create a new Observation for the observer on the left side +sammoa.action.left.tip=LEFT \: create a new sighting for the observer on the left side sammoa.action.loadImportApplicationFile=Load selected import file sammoa.action.new=New sammoa.action.next=Next @@ -40,11 +40,11 @@ sammoa.action.quitExportMaps=Quit sammoa.action.quitImportApplication=Quit sammoa.action.reload.actions=Reload actions -sammoa.action.reload.audio=Relad Audio device +sammoa.action.reload.audio=Reload Audio device sammoa.action.reload.gps=Reload GPS device sammoa.action.reload.home=Reload home screen sammoa.action.reload.ui=Reload sammoa ui -sammoa.action.right.tip=RIGHT \: create a new Observation for the observer on the right side +sammoa.action.right.tip=RIGHT \: create a new sighting for the observer on the right side sammoa.action.save=Save sammoa.action.site=Site sammoa.action.site.tip=Display project site @@ -53,14 +53,14 @@ sammoa.action.startAudio.tip=Start audio sammoa.action.startExport=Start sammoa.action.stop=Stop flight -sammoa.action.stop.tip=STOP \: stop the flight. No other actions on routes and observations could be done after stop. +sammoa.action.stop.tip=STOP \: stop the flight. No other actions on routes and sightings could be done after stop. sammoa.action.stopAudio.tip=Stop audio -sammoa.action.validObservation=Observation -sammoa.action.validObservation.tip=Validate the selected observation +sammoa.action.validObservation=Sighting +sammoa.action.validObservation.tip=Validate the selected sighting sammoa.action.validRoute=Route -sammoa.action.validRoute.tip=Validate the selected route and all its observations +sammoa.action.validRoute.tip=Validate the selected route and all its sightings sammoa.action.validTransect=Transect -sammoa.action.validTransect.tip=Validate the selected transect and all its routes and observations +sammoa.action.validTransect.tip=Validate the selected transect and all its routes and sightings sammoa.action.validation=Validation sammoa.config.category.applications=Application sammoa.config.category.applications.description=Application @@ -110,7 +110,7 @@ sammoa.importApplication.flightTable.column.toTreat.tip=Select the flight to import it sammoa.info.importApplication.importfile.loaded.on.existingCampaign=Import an existing campaign %s, please choose which flights of the following list to import sammoa.info.importApplication.importfile.loaded.on.newCampaign=Import a new campaign %s, all the flights of the following list will be imported -sammoa.info.importApplication.no.importfile.loaded=No import file loaded, choose an file to import then press the 'load import file' button +sammoa.info.importApplication.no.importfile.loaded=No import file loaded, choose a file to import then press the 'load import file' button sammoa.inputDialog.flightNumber.error.lessThanNextFlightNumber=The flight number can't be less than %d sammoa.inputDialog.flightNumber.error.notANumber=The flight number must be a valid integer sammoa.inputDialog.flightNumber.message=Choose the flight number @@ -181,7 +181,7 @@ sammoa.observation.observations.center=Center sammoa.observation.observations.left=Left sammoa.observation.observations.right=Right -sammoa.observation.title=<html>Observation</html> +sammoa.observation.title=<html>Sighting</html> sammoa.observationCondition.title=<html>Conditions</html> sammoa.observations.observationTable.column.age=Age sammoa.observations.observationTable.column.age.tip=Age @@ -263,7 +263,7 @@ sammoa.title.choose.export.directory=Choose directory where to export maps sammoa.title.export.exportEfforts=Efforts Export sammoa.title.export.exportGeoPoints=GPS Points ExportGPS -sammoa.title.export.exportObservations=Observations Export +sammoa.title.export.exportObservations=Sightings Export sammoa.title.exportApplication=Application Export sammoa.title.exportshape=Export Maps sammoa.title.flight=System %1$s - Campaign %2$s - Region %3$s - Flight n°%4$d @@ -278,7 +278,7 @@ sammoa.validator.campaign.endDate.required=The end date must be defined sammoa.validator.campaign.region.required=The region must be set sammoa.validator.campaign.uniqueKey=The campaign code '%1$s' already exists for the region '%2$s' -sammoa.validator.observation.age.unkownValue=The age must be \: J for "juvénile", I for "immature", A for "adulte" and M for mixed groups +sammoa.validator.observation.age.unkownValue=The age must be \: J for "juvenile", I for "immature", A for "adult" and M for mixed groups sammoa.validator.observation.behaviour.unkownValue=The behaviour must be one of this value \: SW (normal), MI (milling), BR (breaching), LO (logging), FE (feeding), FA (feeding aggregation), SB (following boat), OT (other) sammoa.validator.observation.cue.unkownValue=The cue must be between 2 and 9 or U for underwater and A for surface sammoa.validator.observation.decAngle.invalidRange=The angle must be between %1$s and %2$s degrees or for the band \: 1 (in the band), 2 (outside the band), 3 (put in the band)

1 0

r573 - trunk/sammoa-application/src/main/java/fr/ulr/sammoa/application/device/gps
by fdesbois＠users.forge.codelutin.com 12 Sep '12

12 Sep '12

Author: fdesbois Date: 2012-09-12 16:37:34 +0200 (Wed, 12 Sep 2012) New Revision: 573 Url: http://forge.codelutin.com/repositories/revision/sammoa/573 Log: little improvement, change state on GPS stop Modified: trunk/sammoa-application/src/main/java/fr/ulr/sammoa/application/device/gps/BaseGpsHandler.java Modified: trunk/sammoa-application/src/main/java/fr/ulr/sammoa/application/device/gps/BaseGpsHandler.java =================================================================== --- trunk/sammoa-application/src/main/java/fr/ulr/sammoa/application/device/gps/BaseGpsHandler.java 2012-09-12 11:52:18 UTC (rev 572) +++ trunk/sammoa-application/src/main/java/fr/ulr/sammoa/application/device/gps/BaseGpsHandler.java 2012-09-12 14:37:34 UTC (rev 573) @@ -127,6 +127,12 @@ timer.cancel(); timer = null; } + if (state == DeviceState.RUNNING) { + setState(DeviceState.READY, null); + + } else if (state == DeviceState.ERROR) { + setState(DeviceState.UNAVAILABLE, null); + } } @Override

1 0

r572 - in trunk: sammoa-application/src/main/java/fr/ulr/sammoa/application sammoa-application/src/main/java/fr/ulr/sammoa/application/device/gps sammoa-application/src/main/java/fr/ulr/sammoa/application/flightController sammoa-persistence/src/main/java/fr/ulr/sammoa/persistence sammoa-ui-swing/src/main/java/fr/ulr/sammoa/ui/swing/flight sammoa-ui-swing/src/main/java/fr/ulr/sammoa/ui/swing/flight/effort sammoa-ui-swing/src/main/java/fr/ulr/sammoa/ui/swing/flight/layer
by fdesbois＠users.forge.codelutin.com 12 Sep '12

12 Sep '12

Author: fdesbois Date: 2012-09-12 13:52:18 +0200 (Wed, 12 Sep 2012) New Revision: 572 Url: http://forge.codelutin.com/repositories/revision/sammoa/572 Log: fixes #1426 : - manage update on map for observation changed (time and species) - add label support - improve GPS currentLocation behavior with captureDelay - improve FlightController, some doubloons were created on currentLocation retrieval Modified: trunk/sammoa-application/src/main/java/fr/ulr/sammoa/application/FlightService.java trunk/sammoa-application/src/main/java/fr/ulr/sammoa/application/device/gps/BaseGpsHandler.java trunk/sammoa-application/src/main/java/fr/ulr/sammoa/application/device/gps/FakeGpsHandler.java trunk/sammoa-application/src/main/java/fr/ulr/sammoa/application/device/gps/GpsHandlerGpsylon.java trunk/sammoa-application/src/main/java/fr/ulr/sammoa/application/flightController/FlightControllerOnBoard.java trunk/sammoa-application/src/main/java/fr/ulr/sammoa/application/flightController/FlightControllerValidation.java trunk/sammoa-persistence/src/main/java/fr/ulr/sammoa/persistence/GeoPoints.java trunk/sammoa-persistence/src/main/java/fr/ulr/sammoa/persistence/Observations.java trunk/sammoa-ui-swing/src/main/java/fr/ulr/sammoa/ui/swing/flight/FlightUIHandler.java trunk/sammoa-ui-swing/src/main/java/fr/ulr/sammoa/ui/swing/flight/effort/EffortPanelHandler.java trunk/sammoa-ui-swing/src/main/java/fr/ulr/sammoa/ui/swing/flight/layer/BaseGeoPointLayer.java trunk/sammoa-ui-swing/src/main/java/fr/ulr/sammoa/ui/swing/flight/layer/LineGeoPointLayer.java Modified: trunk/sammoa-application/src/main/java/fr/ulr/sammoa/application/FlightService.java =================================================================== --- trunk/sammoa-application/src/main/java/fr/ulr/sammoa/application/FlightService.java 2012-09-11 14:17:14 UTC (rev 571) +++ trunk/sammoa-application/src/main/java/fr/ulr/sammoa/application/FlightService.java 2012-09-12 11:52:18 UTC (rev 572) @@ -40,6 +40,8 @@ import fr.ulr.sammoa.persistence.Flights; import fr.ulr.sammoa.persistence.GeoPoint; import fr.ulr.sammoa.persistence.GeoPointDAO; +import fr.ulr.sammoa.persistence.GeoPointImpl; +import fr.ulr.sammoa.persistence.GeoPoints; import fr.ulr.sammoa.persistence.Observation; import fr.ulr.sammoa.persistence.ObservationDAO; import fr.ulr.sammoa.persistence.ObservationStatus; @@ -62,6 +64,7 @@ import fr.ulr.sammoa.persistence.TransectFlights; import org.apache.commons.collections.CollectionUtils; import org.apache.commons.lang3.StringUtils; +import org.joda.time.DateTime; import org.joda.time.Interval; import org.nuiton.topia.TopiaContext; import org.nuiton.topia.TopiaException; @@ -448,6 +451,73 @@ return result; } + public GeoPoint getGeoPoint(Flight flight, + Date date) { + + TopiaContext tx = beginTransaction(); + try { + + List<GeoPoint> geoPoints = getFlightGeoPoints(tx, flight); + + GeoPoint result = getGeoPoint(tx, flight, geoPoints, date); + return result; + + } catch (TopiaException ex) { + throw new TopiaRuntimeException(ex); + } finally { + endTransaction(tx); + } + } + + public GeoPoint getGeoPoint(TopiaContext tx, + Flight flight, + List<GeoPoint> geoPoints, + Date date) throws TopiaException { + + DateTime newTime = Dates.toDateTime(date); + + // Retrieve the appropriate location + GeoPoint location = GeoPoints.getClosestPoint(geoPoints, date); + DateTime locationTime = Dates.toDateTime(location.getRecordTime()); + + if (logger.isDebugEnabled()) { + logger.debug(String.format("Find locationTime %1$tH:%1$tM:%1$tS.%1$tL", + locationTime.toDate()) + ); + } + + GeoPoint result; + + // Create a new location if no one is available for the newTime + if (!locationTime.isEqual(newTime)) { + + result = new GeoPointImpl(location.getLatitude(), location.getLongitude()); + result.setSpeed(location.getSpeed()); + result.setAltitude(location.getAltitude()); + + int captureDelay = Dates + .toInterval(locationTime, newTime) + .toDuration() + .toStandardSeconds() + .getSeconds(); + + result.setCaptureDelay(captureDelay); + + result.setFlight(flight); + result.setRecordTime(newTime.toDate()); + SammoaDAOHelper.getGeoPointDAO(tx).create(result); + geoPoints.add(result); + + if (logger.isDebugEnabled()) { + logger.debug("Create a new GeoPoint : {}", result); + } + + } else { + result = location; + } + return result; + } + public List<GeoPoint> getFlightGeoPoints(Flight flight) { TopiaContext transaction = beginTransaction(); Modified: trunk/sammoa-application/src/main/java/fr/ulr/sammoa/application/device/gps/BaseGpsHandler.java =================================================================== --- trunk/sammoa-application/src/main/java/fr/ulr/sammoa/application/device/gps/BaseGpsHandler.java 2012-09-11 14:17:14 UTC (rev 571) +++ trunk/sammoa-application/src/main/java/fr/ulr/sammoa/application/device/gps/BaseGpsHandler.java 2012-09-12 11:52:18 UTC (rev 572) @@ -28,11 +28,14 @@ import fr.ulr.sammoa.application.device.DeviceStateEvent; import fr.ulr.sammoa.application.device.DeviceStateListener; import fr.ulr.sammoa.application.device.DeviceTechnicalException; +import fr.ulr.sammoa.persistence.Dates; import fr.ulr.sammoa.persistence.GeoPoint; +import fr.ulr.sammoa.persistence.GeoPointImpl; import fr.ulr.sammoa.persistence.GeoPoints; import org.slf4j.Logger; import org.slf4j.LoggerFactory; +import java.util.Date; import java.util.Set; import java.util.Timer; import java.util.TimerTask; @@ -156,6 +159,44 @@ return gpsLocationListeners; } + @Override + public GeoPoint getCurrentLocation() { + + GeoPoint lastLocation = getLastLocation(); + + Date currentDate = getDate(); + + GeoPoint result = new GeoPointImpl(lastLocation.getLatitude(), lastLocation.getLongitude()); + result.setAltitude(lastLocation.getAltitude()); + result.setSpeed(lastLocation.getSpeed()); + result.setRecordTime(currentDate); + + if (lastLocation.getRecordTime() != null) { + long captureDelay = Dates + .toInterval(lastLocation.getRecordTime(), currentDate) + .toDuration() + .getStandardSeconds(); + result.setCaptureDelay((int) captureDelay); + + if (logger.isTraceEnabled()) { + logger.trace("CaptureDelay = {} for GeoPoint {}", captureDelay, result); + } + } + + if (logger.isWarnEnabled() && GeoPoints.isCoordinatesEmpty(result)) { + logger.warn("Retrieve a location without any coordinates : {}", result); + } + return result; + } + + protected Date getDate() { + // FIXME-fdesbois-2012-07-02 : ensure time with GPS value and not system timestamp + Date result = Dates.newDateWithoutMillis(); + return result; + } + + protected abstract GeoPoint getLastLocation(); + public void setState(DeviceState state, DeviceTechnicalException error) { DeviceState oldValue = getState(); this.state = state; @@ -220,7 +261,7 @@ nbFailures = 0; } - location.setCaptureDelay(nbFailures * config.getCheckPeriod()); +// location.setCaptureDelay(nbFailures * config.getCheckPeriod()); // Data is unavailable if (GeoPoints.isCoordinatesEmpty(location) || sameLocation && nbFailures >= nbFailuresMax) { Modified: trunk/sammoa-application/src/main/java/fr/ulr/sammoa/application/device/gps/FakeGpsHandler.java =================================================================== --- trunk/sammoa-application/src/main/java/fr/ulr/sammoa/application/device/gps/FakeGpsHandler.java 2012-09-11 14:17:14 UTC (rev 571) +++ trunk/sammoa-application/src/main/java/fr/ulr/sammoa/application/device/gps/FakeGpsHandler.java 2012-09-12 11:52:18 UTC (rev 572) @@ -26,14 +26,12 @@ import fr.ulr.sammoa.application.device.DeviceState; import fr.ulr.sammoa.application.device.DeviceTechnicalException; -import fr.ulr.sammoa.persistence.Dates; import fr.ulr.sammoa.persistence.GeoPoint; import fr.ulr.sammoa.persistence.GeoPointImpl; import fr.ulr.sammoa.persistence.GeoPoints; import org.slf4j.Logger; import org.slf4j.LoggerFactory; -import java.util.Date; import java.util.Random; /** @@ -130,24 +128,37 @@ } @Override - public GeoPoint getCurrentLocation() { + protected GeoPoint getLastLocation() { GeoPoint result; if (currentLocation == null) { result = new GeoPointImpl(GeoPoints.EMPTY_COORDINATE, GeoPoints.EMPTY_COORDINATE); result.setRecordTime(getDate()); - - } else if (currentLocation.getTopiaId() != null) { - result = new GeoPointImpl(currentLocation.getLatitude(), currentLocation.getLongitude()); - result.setAltitude(currentLocation.getAltitude()); - result.setSpeed(currentLocation.getSpeed()); - result.setRecordTime(getDate()); - } else { result = currentLocation; } return result; } + // +// @Override +// public GeoPoint getCurrentLocation() { +// GeoPoint result; +// if (currentLocation == null) { +// result = new GeoPointImpl(GeoPoints.EMPTY_COORDINATE, GeoPoints.EMPTY_COORDINATE); +// result.setRecordTime(getDate()); +// +// } else if (currentLocation.getTopiaId() != null) { +// result = new GeoPointImpl(currentLocation.getLatitude(), currentLocation.getLongitude()); +// result.setAltitude(currentLocation.getAltitude()); +// result.setSpeed(currentLocation.getSpeed()); +// result.setRecordTime(getDate()); +// +// } else { +// result = currentLocation; +// } +// return result; +// } + protected class GpsPointGenerator implements Runnable { @Override @@ -179,9 +190,4 @@ } } - protected Date getDate() { - Date result = Dates.newDateWithoutMillis(); - return result; - } - } Modified: trunk/sammoa-application/src/main/java/fr/ulr/sammoa/application/device/gps/GpsHandlerGpsylon.java =================================================================== --- trunk/sammoa-application/src/main/java/fr/ulr/sammoa/application/device/gps/GpsHandlerGpsylon.java 2012-09-11 14:17:14 UTC (rev 571) +++ trunk/sammoa-application/src/main/java/fr/ulr/sammoa/application/device/gps/GpsHandlerGpsylon.java 2012-09-12 11:52:18 UTC (rev 572) @@ -27,7 +27,6 @@ import fr.ulr.sammoa.application.device.DeviceState; import fr.ulr.sammoa.application.device.DeviceTechnicalException; -import fr.ulr.sammoa.persistence.Dates; import fr.ulr.sammoa.persistence.GeoPoint; import fr.ulr.sammoa.persistence.GeoPointImpl; import fr.ulr.sammoa.persistence.GeoPoints; @@ -62,6 +61,8 @@ protected GPSPosition lastPosition; + protected Date lastPositionDate; + protected float lastAltitude; protected float lastSpeed; @@ -156,23 +157,12 @@ } @Override - public GeoPoint getCurrentLocation() { + protected GeoPoint getLastLocation() { if (logger.isTraceEnabled()) { logger.trace("Ask lastPosition = {}", lastPosition); } -// if (!Objects.equal(lastPosition, lastSendPosition) && lastPosition != null) { -// -// result = new GeoPointImpl(lastPosition.getLatitude(), lastPosition.getLongitude()); -// result.setAltitude(lastAltitude); -// result.setSpeed(lastSpeed); -// // FIXME-fdesbois-2012-07-02 : ensure time with GPS value and not system timestamp -// result.setRecordTime(new Date()); -// -// lastSendPosition = lastPosition; -// } - double latitude = lastPosition != null ? lastPosition.getLatitude() : GeoPoints.EMPTY_COORDINATE; @@ -184,14 +174,49 @@ GeoPoint result = new GeoPointImpl(latitude, longitude); result.setAltitude(lastAltitude); result.setSpeed(lastSpeed); - result.setRecordTime(getDate()); + result.setRecordTime(lastPositionDate); - if (logger.isWarnEnabled() && lastPosition == null) { - logger.warn("Retrieve a location without any coordinates : {}", result); - } return result; } + // +// @Override +// public GeoPoint getCurrentLocation() { +// +// if (logger.isTraceEnabled()) { +// logger.trace("Ask lastPosition = {}", lastPosition); +// } +// +//// if (!Objects.equal(lastPosition, lastSendPosition) && lastPosition != null) { +//// +//// result = new GeoPointImpl(lastPosition.getLatitude(), lastPosition.getLongitude()); +//// result.setAltitude(lastAltitude); +//// result.setSpeed(lastSpeed); +//// // FIXME-fdesbois-2012-07-02 : ensure time with GPS value and not system timestamp +//// result.setRecordTime(new Date()); +//// +//// lastSendPosition = lastPosition; +//// } +// +// double latitude = lastPosition != null +// ? lastPosition.getLatitude() +// : GeoPoints.EMPTY_COORDINATE; +// +// double longitude = lastPosition != null +// ? lastPosition.getLongitude() +// : GeoPoints.EMPTY_COORDINATE; +// +// GeoPoint result = new GeoPointImpl(latitude, longitude); +// result.setAltitude(lastAltitude); +// result.setSpeed(lastSpeed); +// result.setRecordTime(getDate()); +// +// if (logger.isWarnEnabled() && lastPosition == null) { +// logger.warn("Retrieve a location without any coordinates : {}", result); +// } +// return result; +// } + @Override protected DeviceTechnicalException onError(GeoPoint location) { @@ -217,18 +242,18 @@ if (GeoPoints.isCoordinatesEmpty(location)) { - result = new DeviceTechnicalException(this, - "GPS is not ready, turn it on or wait for it to find satellites"); + result = new DeviceTechnicalException( + this, "GPS is not ready, turn it on or wait for it to find satellites"); } else { if (lastNumberSatellites == 0) { - result = new DeviceTechnicalException(this, - "GPS signal lost, there is no available satellite found"); + result = new DeviceTechnicalException( + this, "GPS signal lost, there is no available satellite found"); } else { - result = new DeviceTechnicalException(this, - "GPS signal lost, please check the connection port"); + result = new DeviceTechnicalException( + this, "GPS signal lost, please check the connection port"); } } } @@ -257,13 +282,13 @@ close(); super.finalize(); } +// +// protected Date getDate() { +// // FIXME-fdesbois-2012-07-02 : ensure time with GPS value and not system timestamp +// Date result = Dates.newDateWithoutMillis(); +// return result; +// } - protected Date getDate() { - // FIXME-fdesbois-2012-07-02 : ensure time with GPS value and not system timestamp - Date result = Dates.newDateWithoutMillis(); - return result; - } - /** * Si l'on recoit des evenements, c'est que le gps fonctionne. * Il sont envoyé par le traitement des flux NMEA et propagé par le @@ -297,6 +322,7 @@ } if (GPSDataProcessor.LOCATION.equals(property)) { + lastPositionDate = getDate(); lastPosition = (GPSPosition) evt.getNewValue(); } else if (GPSDataProcessor.ALTITUDE.equals(property)) { Modified: trunk/sammoa-application/src/main/java/fr/ulr/sammoa/application/flightController/FlightControllerOnBoard.java =================================================================== --- trunk/sammoa-application/src/main/java/fr/ulr/sammoa/application/flightController/FlightControllerOnBoard.java 2012-09-11 14:17:14 UTC (rev 571) +++ trunk/sammoa-application/src/main/java/fr/ulr/sammoa/application/flightController/FlightControllerOnBoard.java 2012-09-12 11:52:18 UTC (rev 572) @@ -25,6 +25,7 @@ import com.google.common.base.Objects; import com.google.common.base.Preconditions; +import com.google.common.collect.Maps; import fr.ulr.sammoa.application.device.DeviceManager; import fr.ulr.sammoa.application.device.audio.AudioRecorder; import fr.ulr.sammoa.application.device.gps.GpsHandler; @@ -38,8 +39,12 @@ import fr.ulr.sammoa.persistence.TransectFlight; import org.nuiton.topia.TopiaContext; import org.nuiton.topia.TopiaException; +import org.slf4j.Logger; +import org.slf4j.LoggerFactory; import java.io.File; +import java.util.Date; +import java.util.Map; /** * Created: 21/08/12 @@ -48,6 +53,11 @@ */ public class FlightControllerOnBoard extends BaseFlightController implements GpsLocationListener { + /** Logger. */ + private static final Logger logger = LoggerFactory.getLogger(FlightControllerOnBoard.class); + + protected Map<Date, GeoPoint> geoPoints = Maps.newHashMap(); + @Override public <T extends DeviceManager> void openDeviceManager(Class<T> deviceManager) { boolean autoStart = initialized && isRunning(); @@ -73,6 +83,10 @@ super.init(flight); + geoPoints = Maps.newHashMap(Maps.uniqueIndex( + service.getFlightGeoPoints(flight), GeoPoints.toDate()) + ); + initCurrentRoute(service.getLastUnfinishedRoute(flight)); if (isRunning()) { @@ -108,18 +122,25 @@ @Override public void locationChanged(GpsLocationEvent event) { GeoPoint newLocation = event.getNewValue(); - if (!GeoPoints.isCoordinatesEmpty(newLocation) && newLocation.getTopiaId() == null) { + GeoPoint existLocation = geoPoints.get(newLocation.getRecordTime()); + if (existLocation == null) { newLocation.setFlight(flight); persistence.delayEntityCreation(newLocation); + geoPoints.put(newLocation.getRecordTime(), newLocation); } } @Override protected GeoPoint getLocation(TopiaContext tx) throws TopiaException { - GeoPoint result = getGpsHandler().getCurrentLocation(); - result.setFlight(flight); - if (result.getTopiaId() == null) { - SammoaDAOHelper.getGeoPointDAO(tx).create(result); + GeoPoint gpsLocation = getGpsHandler().getCurrentLocation(); + GeoPoint result = geoPoints.get(gpsLocation.getRecordTime()); + if (result == null) { + gpsLocation.setFlight(flight); + if (logger.isDebugEnabled()) { + logger.debug("Create a GeoPoint {}", gpsLocation); + } + result = SammoaDAOHelper.getGeoPointDAO(tx).create(gpsLocation); + geoPoints.put(result.getRecordTime(), result); } return result; } Modified: trunk/sammoa-application/src/main/java/fr/ulr/sammoa/application/flightController/FlightControllerValidation.java =================================================================== --- trunk/sammoa-application/src/main/java/fr/ulr/sammoa/application/flightController/FlightControllerValidation.java 2012-09-11 14:17:14 UTC (rev 571) +++ trunk/sammoa-application/src/main/java/fr/ulr/sammoa/application/flightController/FlightControllerValidation.java 2012-09-12 11:52:18 UTC (rev 572) @@ -30,11 +30,8 @@ import fr.ulr.sammoa.persistence.Dates; import fr.ulr.sammoa.persistence.Flight; import fr.ulr.sammoa.persistence.GeoPoint; -import fr.ulr.sammoa.persistence.GeoPointImpl; -import fr.ulr.sammoa.persistence.GeoPoints; import fr.ulr.sammoa.persistence.Route; import fr.ulr.sammoa.persistence.Routes; -import fr.ulr.sammoa.persistence.SammoaDAOHelper; import fr.ulr.sammoa.persistence.TransectFlight; import fr.ulr.sammoa.persistence.TransectFlights; import org.joda.time.DateTime; @@ -85,8 +82,6 @@ Preconditions.checkNotNull(currentRoute, "You must set the current route to retrieve location"); Preconditions.checkState(!geoPoints.isEmpty(), "No geoPoints available"); - GeoPoint result; - if (logger.isDebugEnabled()) { logger.debug(String.format("Get location after startTime %1$tH:%1$tM:%1$tS.%1$tL (audio position = %2$d)", getAudioReader().getStartDate(), @@ -111,42 +106,44 @@ } // Retrieve the appropriate location - GeoPoint location = GeoPoints.getClosestPoint(geoPoints, newTime.toDate()); - DateTime locationTime = Dates.toDateTime(location.getRecordTime()); + GeoPoint result = service.getGeoPoint(tx, flight, geoPoints, newTime.toDate()); - if (logger.isDebugEnabled()) { - logger.debug(String.format("Find locationTime %1$tH:%1$tM:%1$tS.%1$tL", - locationTime.toDate()) - ); - } - - // Create a new location if no one is available for the newTime - if (!locationTime.isEqual(newTime)) { - - result = new GeoPointImpl(location.getLatitude(), location.getLongitude()); - result.setSpeed(location.getSpeed()); - result.setAltitude(location.getAltitude()); - - int captureDelay = Dates - .toInterval(locationTime, newTime) - .toDuration() - .toStandardSeconds() - .getSeconds(); - - result.setCaptureDelay(captureDelay); - - result.setFlight(flight); - result.setRecordTime(newTime.toDate()); - SammoaDAOHelper.getGeoPointDAO(tx).create(result); - geoPoints.add(result); - - if (logger.isDebugEnabled()) { - logger.debug("Create a new GeoPoint : {}", result); - } - - } else { - result = location; - } +// GeoPoint location = GeoPoints.getClosestPoint(geoPoints, newTime.toDate()); +// DateTime locationTime = Dates.toDateTime(location.getRecordTime()); +// +// if (logger.isDebugEnabled()) { +// logger.debug(String.format("Find locationTime %1$tH:%1$tM:%1$tS.%1$tL", +// locationTime.toDate()) +// ); +// } +// +// // Create a new location if no one is available for the newTime +// if (!locationTime.isEqual(newTime)) { +// +// result = new GeoPointImpl(location.getLatitude(), location.getLongitude()); +// result.setSpeed(location.getSpeed()); +// result.setAltitude(location.getAltitude()); +// +// int captureDelay = Dates +// .toInterval(locationTime, newTime) +// .toDuration() +// .toStandardSeconds() +// .getSeconds(); +// +// result.setCaptureDelay(captureDelay); +// +// result.setFlight(flight); +// result.setRecordTime(newTime.toDate()); +// SammoaDAOHelper.getGeoPointDAO(tx).create(result); +// geoPoints.add(result); +// +// if (logger.isDebugEnabled()) { +// logger.debug("Create a new GeoPoint : {}", result); +// } +// +// } else { +// result = location; +// } return result; } Modified: trunk/sammoa-persistence/src/main/java/fr/ulr/sammoa/persistence/GeoPoints.java =================================================================== --- trunk/sammoa-persistence/src/main/java/fr/ulr/sammoa/persistence/GeoPoints.java 2012-09-11 14:17:14 UTC (rev 571) +++ trunk/sammoa-persistence/src/main/java/fr/ulr/sammoa/persistence/GeoPoints.java 2012-09-12 11:52:18 UTC (rev 572) @@ -26,7 +26,6 @@ import com.google.common.base.Function; import com.google.common.base.Objects; import com.google.common.base.Predicate; -import com.google.common.base.Predicates; import com.google.common.collect.FluentIterable; import com.google.common.collect.Iterables; import com.google.common.collect.Ordering; @@ -67,21 +66,14 @@ return result; } - public static List<GeoPoint> retainsValidCoordinates(Iterable<GeoPoint> input) { - List<GeoPoint> result = FluentIterable.from(input) - .filter(Predicates.not(withCoordinatesEmpty())) - .toImmutableList(); - return result; - } - public static boolean isCoordinatesEmpty(GeoPoint geoPoint) { return geoPoint.getLatitude() == GeoPoints.EMPTY_COORDINATE && geoPoint.getLongitude() == GeoPoints.EMPTY_COORDINATE; } - public static Predicate<GeoPoint> withCoordinatesEmpty() { - return IS_COORDINATES_EMPTY_PREDICATE; - } +// public static Predicate<GeoPoint> withCoordinatesEmpty() { +// return IS_COORDINATES_EMPTY_PREDICATE; +// } public static Function<GeoPoint, Date> toDate() { return TO_DATE_FUNCTION; @@ -117,14 +109,14 @@ } }; - private static Predicate<GeoPoint> IS_COORDINATES_EMPTY_PREDICATE = new Predicate<GeoPoint>() { +// private static Predicate<GeoPoint> IS_COORDINATES_EMPTY_PREDICATE = new Predicate<GeoPoint>() { +// +// @Override +// public boolean apply(GeoPoint input) { +// return isCoordinatesEmpty(input); +// } +// }; - @Override - public boolean apply(GeoPoint input) { - return isCoordinatesEmpty(input); - } - }; - private static class WithDatePredicate implements Predicate<GeoPoint> { protected Date date; Modified: trunk/sammoa-persistence/src/main/java/fr/ulr/sammoa/persistence/Observations.java =================================================================== --- trunk/sammoa-persistence/src/main/java/fr/ulr/sammoa/persistence/Observations.java 2012-09-11 14:17:14 UTC (rev 571) +++ trunk/sammoa-persistence/src/main/java/fr/ulr/sammoa/persistence/Observations.java 2012-09-12 11:52:18 UTC (rev 572) @@ -53,9 +53,20 @@ } public static Iterable<Date> toDates(Iterable<Observation> observations) { - return Iterables.transform(observations, TO_DATE_FUNCTION); + return Iterables.transform(observations, toDate()); } + public static String toLabel(Observation observation) { + Species species = observation.getSpecies(); + return (species != null ? species.getCode() : "") + " (S" + observation.getObservationNumber() + ")"; + } + + public static GeoPoint toGeoPoint(Observation observation, + Iterable<GeoPoint> geoPoints) { + Date date = observation.getObservationTime(); + return Iterables.find(geoPoints, GeoPoints.withDate(date), GeoPoints.newEmptyGeoPoint(date)); + } + public static Iterable<Observation> filterInRoute(Iterable<Observation> observations, Route route, Iterable<Route> routes, @@ -172,6 +183,18 @@ return result; } + public static Function<Observation, GeoPoint> toGeoPoint(final Iterable<GeoPoint> geoPoints) { + return new ToGeoPointFunction(geoPoints); + } + + public static Function<Observation, Date> toDate() { + return TO_DATE_FUNCTION; + } + + public static Function<Observation, String> toLabel() { + return TO_LABEL_FUNCTION; + } + public static Predicate<Observation> inDateInterval(Interval interval) { return new InDateIntervalPredicate(interval); } @@ -188,6 +211,28 @@ } }; + private static Function<Observation, String> TO_LABEL_FUNCTION = new Function<Observation, String>() { + + @Override + public String apply(Observation input) { + return toLabel(input); + } + }; + + private static class ToGeoPointFunction implements Function<Observation, GeoPoint> { + + protected Iterable<GeoPoint> geoPoints; + + public ToGeoPointFunction(Iterable<GeoPoint> geoPoints) { + this.geoPoints = geoPoints; + } + + @Override + public GeoPoint apply(Observation input) { + return toGeoPoint(input, geoPoints); + } + } + private static class InRoutePredicate implements Predicate<Observation> { protected Route route; Modified: trunk/sammoa-ui-swing/src/main/java/fr/ulr/sammoa/ui/swing/flight/FlightUIHandler.java =================================================================== --- trunk/sammoa-ui-swing/src/main/java/fr/ulr/sammoa/ui/swing/flight/FlightUIHandler.java 2012-09-11 14:17:14 UTC (rev 571) +++ trunk/sammoa-ui-swing/src/main/java/fr/ulr/sammoa/ui/swing/flight/FlightUIHandler.java 2012-09-12 11:52:18 UTC (rev 572) @@ -233,8 +233,7 @@ List<Observer> observers = flightService.getFlightObserverForPositions(flight); List<Observation> observations = flightService.getObservations(flight); List<Route> routes = flightService.getRoutes(flight); - List<GeoPoint> geoPoints = - GeoPoints.retainsValidCoordinates(flightService.getFlightGeoPoints(flight)); + List<GeoPoint> geoPoints = flightService.getFlightGeoPoints(flight); if (logger.isTraceEnabled()) { for (GeoPoint geoPoint : geoPoints) { @@ -739,13 +738,17 @@ Observation observation = event.getObservation(); getModel().addObservation(observation); - // Add the GeoPoint to the layer - GeoPoint geoPoint = event.getGeoPoint(); - if (!GeoPoints.isCoordinatesEmpty(geoPoint)) { - observationLayer.addGeoPoint(geoPoint); - } + setObservationGeoPointInMap(observation, event.getGeoPoint()); } + public void setObservationGeoPointInMap(Observation observation, GeoPoint geoPoint) { + observationLayer.putGeoPoint(observation, geoPoint, Observations.toLabel(observation)); + } + + public void setObservationLabelInMap(Observation observation) { + observationLayer.setLabel(observation, Observations.toLabel(observation)); + } + public void selectStrate(StrateModel strate) { if (logger.isDebugEnabled()) { @@ -1092,12 +1095,11 @@ observationLayer = new SimpleGeoPointLayer(); observationLayer.setProperties(properties); - - List<GeoPoint> geoPoints = GeoPoints.getClosestPoints( - getModel().getGeoPoints(), - Observations.toDates(getModel().getObservations()) + observationLayer.setGeoPoints( + getModel().getObservations(), + Observations.toGeoPoint(getModel().getGeoPoints()), + Observations.toLabel() ); - observationLayer.setGeoPoints(geoPoints); mapHandler.add(observationLayer); } Modified: trunk/sammoa-ui-swing/src/main/java/fr/ulr/sammoa/ui/swing/flight/effort/EffortPanelHandler.java =================================================================== --- trunk/sammoa-ui-swing/src/main/java/fr/ulr/sammoa/ui/swing/flight/effort/EffortPanelHandler.java 2012-09-11 14:17:14 UTC (rev 571) +++ trunk/sammoa-ui-swing/src/main/java/fr/ulr/sammoa/ui/swing/flight/effort/EffortPanelHandler.java 2012-09-12 11:52:18 UTC (rev 572) @@ -36,6 +36,8 @@ import fr.ulr.sammoa.application.device.audio.AudioRecorder; import fr.ulr.sammoa.application.device.gps.GpsHandler; import fr.ulr.sammoa.application.flightController.FlightController; +import fr.ulr.sammoa.persistence.GeoPoint; +import fr.ulr.sammoa.persistence.GeoPoints; import fr.ulr.sammoa.persistence.Observation; import fr.ulr.sammoa.persistence.ObservationStatus; import fr.ulr.sammoa.persistence.Observations; @@ -837,12 +839,20 @@ getParentUI().getHandler().getFlightController().setCurrentRoute(newValue); + if (oldValue != null) { + oldValue.removePropertyChangeListener( + Route.PROPERTY_BEGIN_TIME, routeTimeChangeListener); + } + if (newValue == null) { // no unselect but ensure valid button (a select will cause a loop) getModel().setObservationEditBean(null); } else { + newValue.addPropertyChangeListener( + Route.PROPERTY_BEGIN_TIME, routeTimeChangeListener); + // Set audio position setAudioReaderPositionDate(newValue.getBeginTime()); @@ -862,14 +872,16 @@ if (oldValue != null) { oldValue.removePropertyChangeListener( - Observation.PROPERTY_OBSERVATION_TIME, - observationTimeChangeListener); + Observation.PROPERTY_OBSERVATION_TIME, observationTimeChangeListener); + oldValue.removePropertyChangeListener( + Observation.PROPERTY_SPECIES, observationSpeciesChangeListener); } if (newValue != null) { newValue.addPropertyChangeListener( - Observation.PROPERTY_OBSERVATION_TIME, - observationTimeChangeListener); + Observation.PROPERTY_OBSERVATION_TIME, observationTimeChangeListener); + newValue.addPropertyChangeListener( + Observation.PROPERTY_SPECIES, observationSpeciesChangeListener); selectRouteByObservation(newValue); @@ -878,14 +890,53 @@ } } + protected GeoPoint ensureGeoPoint(Date date) { + + List<GeoPoint> modelPoints = getModel().getGeoPoints(); + GeoPoint result = Iterables.find(modelPoints, GeoPoints.withDate(date), null); + + if (result == null) { + FlightService service = context.getService(FlightService.class); + result = service.getGeoPoint(getModel().getFlight(), date); + modelPoints.add(result); + } + return result; + } + + private PropertyChangeListener routeTimeChangeListener = new PropertyChangeListener() { + + @Override + public void propertyChange(PropertyChangeEvent evt) { + ensureGeoPoint((Date) evt.getNewValue()); + } + }; + private PropertyChangeListener observationTimeChangeListener = new PropertyChangeListener() { @Override public void propertyChange(PropertyChangeEvent evt) { - selectRouteByObservation((Observation) evt.getSource()); + + Observation observation = (Observation) evt.getSource(); + + GeoPoint geoPoint = ensureGeoPoint((Date) evt.getNewValue()); + + getParentUI().getHandler().setObservationGeoPointInMap(observation, geoPoint); + + selectRouteByObservation(observation); } }; + private PropertyChangeListener observationSpeciesChangeListener = new PropertyChangeListener() { + + @Override + public void propertyChange(PropertyChangeEvent evt) { + + Observation observation = (Observation) evt.getSource(); + + getParentUI().getHandler().setObservationLabelInMap(observation); + } + }; + /** * Editor for type {@link TransectFlight}. This will use the * {@link DefaultCellEditor} with {@link JComboBox} as editor component. Modified: trunk/sammoa-ui-swing/src/main/java/fr/ulr/sammoa/ui/swing/flight/layer/BaseGeoPointLayer.java =================================================================== --- trunk/sammoa-ui-swing/src/main/java/fr/ulr/sammoa/ui/swing/flight/layer/BaseGeoPointLayer.java 2012-09-11 14:17:14 UTC (rev 571) +++ trunk/sammoa-ui-swing/src/main/java/fr/ulr/sammoa/ui/swing/flight/layer/BaseGeoPointLayer.java 2012-09-12 11:52:18 UTC (rev 572) @@ -26,10 +26,19 @@ import com.bbn.openmap.layer.OMGraphicHandlerLayer; import com.bbn.openmap.omGraphics.DrawingAttributes; import com.bbn.openmap.omGraphics.OMGraphic; +import com.bbn.openmap.omGraphics.OMGraphicConstants; import com.bbn.openmap.omGraphics.OMGraphicList; +import com.bbn.openmap.omGraphics.OMText; +import com.bbn.openmap.omGraphics.OMTextLabeler; +import com.google.common.base.Function; +import com.google.common.base.Objects; +import com.google.common.base.Predicate; +import com.google.common.collect.Iterables; import fr.ulr.sammoa.persistence.GeoPoint; +import fr.ulr.sammoa.persistence.GeoPoints; +import org.slf4j.Logger; +import org.slf4j.LoggerFactory; -import java.util.List; import java.util.Properties; /** @@ -41,25 +50,96 @@ private static final long serialVersionUID = 1L; + /** Logger. */ + private static final Logger logger = LoggerFactory.getLogger(BaseGeoPointLayer.class); + protected DrawingAttributes drawingAttributes = DrawingAttributes.getDefaultClone(); - public void setGeoPoints(List<GeoPoint> geoPoints) { + public void setGeoPoints(Iterable<GeoPoint> geoPoints) { + setList(new OMGraphicList()); - for (GeoPoint geoPoint : geoPoints) { - addGraphic(geoPoint); + for (GeoPoint reference : geoPoints) { + if (GeoPoints.isCoordinatesEmpty(reference)) { + if (logger.isWarnEnabled()) { + logger.warn("Can't add geoPoint, no coordinates available " + + "at {}", reference.getRecordTime()); + } + } else { + getList().add(createGraphic(reference, reference, null)); + } } } + public <T> void setGeoPoints(Iterable<T> references, + Function<T, GeoPoint> toGeoPoint, + Function<T, String> toLabel) { + + setList(new OMGraphicList()); + + for (T reference : references) { + GeoPoint geoPoint = toGeoPoint.apply(reference); + String label = toLabel.apply(reference); + + if (GeoPoints.isCoordinatesEmpty(geoPoint)) { + if (logger.isWarnEnabled()) { + logger.warn("Can't add geoPoint, no coordinates available " + + "at {} (label={})", geoPoint.getRecordTime(), label); + } + } else { + getList().add(createGraphic(reference, geoPoint, label)); + } + } + } + public void addGeoPoint(GeoPoint geoPoint) { + putGeoPoint(geoPoint, geoPoint, null); + } - addGraphic(geoPoint); + public void setLabel(Object reference, String label) { + OMGraphic graphic = Iterables.find(getList(), withReference(reference), null); + if (graphic != null) { + setLabel(graphic, label); + applyChange(); + } + } - // Need to generate the projection to apply change - getList().generate(getProjection()); + public void putGeoPoint(Object reference, GeoPoint geoPoint, String label) { - // Refresh the view - repaint(); + int index = Iterables.indexOf(getList(), withReference(reference)); + + if (GeoPoints.isCoordinatesEmpty(geoPoint)) { + if (logger.isWarnEnabled()) { + logger.warn("Can't add geoPoint, no coordinates available " + + "(label={})", label); + } + + if (index != -1) { + if (logger.isDebugEnabled()) { + logger.debug("Remove geoPoint at index {} (label={})", index, label); + } + getList().remove(index); + } + + } else { + + OMGraphic graphic = createGraphic(reference, geoPoint, label); + + if (index != -1) { + if (logger.isDebugEnabled()) { + logger.debug("Update geoPoint at index {} (label={})", index, label); + } + getList().set(index, graphic); + + } else { + if (logger.isDebugEnabled() && label != null) { + logger.debug("Add geoPoint (label={})", label); + } + getList().add(graphic); + } + } + + applyChange(); } @Override @@ -82,12 +162,47 @@ return result; } - protected void addGraphic(GeoPoint geoPoint) { - OMGraphic graphic = newGraphic(geoPoint); - drawingAttributes.setTo(graphic); - graphic.setAppObject(geoPoint); - getList().add(graphic); + protected OMGraphic createGraphic(Object reference, GeoPoint geoPoint, String label) { + OMGraphic result = newGraphic(geoPoint); + if (label != null) { + setLabel(result, label); + } + drawingAttributes.setTo(result); + result.setAppObject(reference); + return result; } + protected void setLabel(OMGraphic graphic, String label) { + graphic.putAttribute(OMGraphicConstants.LABEL, + new OMTextLabeler(label, OMText.JUSTIFY_CENTER)); + } + + protected void applyChange() { + + // Need to generate the projection to apply change + getList().generate(getProjection()); + + // Refresh the view + repaint(); + } + protected abstract OMGraphic newGraphic(GeoPoint geoPoint); + + public static Predicate<OMGraphic> withReference(Object reference) { + return new WithReferencePredicate(reference); + } + + private static class WithReferencePredicate implements Predicate<OMGraphic> { + + protected Object reference; + + private WithReferencePredicate(Object reference) { + this.reference = reference; + } + + @Override + public boolean apply(OMGraphic input) { + return Objects.equal(input.getAppObject(), reference); + } + } } Modified: trunk/sammoa-ui-swing/src/main/java/fr/ulr/sammoa/ui/swing/flight/layer/LineGeoPointLayer.java =================================================================== --- trunk/sammoa-ui-swing/src/main/java/fr/ulr/sammoa/ui/swing/flight/layer/LineGeoPointLayer.java 2012-09-11 14:17:14 UTC (rev 571) +++ trunk/sammoa-ui-swing/src/main/java/fr/ulr/sammoa/ui/swing/flight/layer/LineGeoPointLayer.java 2012-09-12 11:52:18 UTC (rev 572) @@ -47,11 +47,20 @@ OMGraphic lastGraphic = Iterables.getLast(getList()); - GeoPoint previousPoint = (GeoPoint) lastGraphic.getAppObject(); + double lat, lon; + if (lastGraphic instanceof OMPoint) { + lat = ((OMPoint) lastGraphic).getLat(); + lon = ((OMPoint) lastGraphic).getLon(); + } else { + lat = ((OMLine) lastGraphic).getLL()[2]; + lon = ((OMLine) lastGraphic).getLL()[3]; + } +// GeoPoint previousPoint = (GeoPoint) lastGraphic.getAppObject(); + result = new OMLine( - previousPoint.getLatitude(), - previousPoint.getLongitude(), + lat, + lon, geoPoint.getLatitude(), geoPoint.getLongitude(), OMGraphicConstants.LINETYPE_GREATCIRCLE

1 0