Ticket #10853: photo_geotagging_10853.patch

File photo_geotagging_10853.patch, 10.7 KB (added by holgermappt, 11 years ago)

Patch file for this ticket

  • src/org/openstreetmap/josm/plugins/photo_geotagging/ExifGPSTagger.java

     
    99import java.io.IOException;
    1010import java.text.DecimalFormat;
    1111import java.util.Calendar;
     12import java.util.Date;
    1213import java.util.GregorianCalendar;
    1314import java.util.TimeZone;
    1415
    … …  
    3334     * @param dst The output file.
    3435     * @param lat latitude
    3536     * @param lon longitude
    36      * @param gpsTime time in milliseconds
     37     * @param gpsTime time - can be null if not available
     38     * @param speed speed - can be null if not available
    3739     * @param ele elevation - can be null if not available
     40     * @param imgDir image direction - can be null if not available
    3841     */
    39     public static void setExifGPSTag(File jpegImageFile, File dst, double lat, double lon, long gpsTime, Double ele) throws IOException {
     42    public static void setExifGPSTag(File jpegImageFile, File dst, double lat, double lon, Date gpsTime, Double speed, Double ele, Double imgDir) throws IOException {
    4043        try {
    41             setExifGPSTagWorker(jpegImageFile, dst, lat, lon, gpsTime, ele);
     44            setExifGPSTagWorker(jpegImageFile, dst, lat, lon, gpsTime, speed, ele, imgDir);
    4245        } catch (ImageReadException ire) {
    4346            throw new IOException(tr("Read error: "+ire), ire);
    4447        } catch (ImageWriteException ire2) {
    … …  
    4649        }
    4750    }
    4851
    49     public static void setExifGPSTagWorker(File jpegImageFile, File dst, double lat, double lon, long gpsTime, Double ele)
     52    public static void setExifGPSTagWorker(File jpegImageFile, File dst, double lat, double lon, Date gpsTime, Double speed, Double ele, Double imgDir)
    5053            throws IOException, ImageReadException, ImageWriteException {
    5154        TiffOutputSet outputSet = null;
    5255
    … …  
    6871        gpsDirectory.removeField(GpsTagConstants.GPS_TAG_GPS_VERSION_ID);
    6972        gpsDirectory.add(GpsTagConstants.GPS_TAG_GPS_VERSION_ID, (byte)2, (byte)3, (byte)0, (byte)0);
    7073
    71         Calendar calendar = new GregorianCalendar(TimeZone.getTimeZone("UTC"));
    72         calendar.setTimeInMillis(gpsTime);
     74        if (gpsTime != null) {
     75            Calendar calendar = new GregorianCalendar(TimeZone.getTimeZone("UTC"));
     76            calendar.setTime(gpsTime);
    7377
    74         final int year =   calendar.get(Calendar.YEAR);
    75         final int month =  calendar.get(Calendar.MONTH) + 1;
    76         final int day =    calendar.get(Calendar.DAY_OF_MONTH);
    77         final int hour =   calendar.get(Calendar.HOUR_OF_DAY);
    78         final int minute = calendar.get(Calendar.MINUTE);
    79         final int second = calendar.get(Calendar.SECOND);
     78            final int year =   calendar.get(Calendar.YEAR);
     79            final int month =  calendar.get(Calendar.MONTH) + 1;
     80            final int day =    calendar.get(Calendar.DAY_OF_MONTH);
     81            final int hour =   calendar.get(Calendar.HOUR_OF_DAY);
     82            final int minute = calendar.get(Calendar.MINUTE);
     83            final int second = calendar.get(Calendar.SECOND);
    8084
    81         DecimalFormat yearFormatter = new DecimalFormat("0000");
    82         DecimalFormat monthFormatter = new DecimalFormat("00");
    83         DecimalFormat dayFormatter = new DecimalFormat("00");
     85            DecimalFormat yearFormatter = new DecimalFormat("0000");
     86            DecimalFormat monthFormatter = new DecimalFormat("00");
     87            DecimalFormat dayFormatter = new DecimalFormat("00");
    8488
    85         final String yearStr = yearFormatter.format(year);
    86         final String monthStr = monthFormatter.format(month);
    87         final String dayStr = dayFormatter.format(day);
    88         final String dateStamp = yearStr+":"+monthStr+":"+dayStr;
    89         //System.err.println("date: "+dateStamp+"  h/m/s: "+hour+"/"+minute+"/"+second);
     89            final String yearStr = yearFormatter.format(year);
     90            final String monthStr = monthFormatter.format(month);
     91            final String dayStr = dayFormatter.format(day);
     92            final String dateStamp = yearStr+":"+monthStr+":"+dayStr;
     93            //System.err.println("date: "+dateStamp+"  h/m/s: "+hour+"/"+minute+"/"+second);
    9094
    91         // make sure to remove old value if present (this method will
    92         // not fail if the tag does not exist).
    93         gpsDirectory.removeField(GpsTagConstants.GPS_TAG_GPS_TIME_STAMP);
    94         gpsDirectory.add(GpsTagConstants.GPS_TAG_GPS_TIME_STAMP,
    95                 RationalNumber.valueOf(hour),
    96                 RationalNumber.valueOf(minute),
    97                 RationalNumber.valueOf(second));
     95            // make sure to remove old value if present (this method will
     96            // not fail if the tag does not exist).
     97            gpsDirectory.removeField(GpsTagConstants.GPS_TAG_GPS_TIME_STAMP);
     98            gpsDirectory.add(GpsTagConstants.GPS_TAG_GPS_TIME_STAMP,
     99                    RationalNumber.valueOf(hour),
     100                    RationalNumber.valueOf(minute),
     101                    RationalNumber.valueOf(second));
    98102
    99         // make sure to remove old value if present (this method will
    100         // not fail if the tag does not exist).
    101         gpsDirectory.removeField(GpsTagConstants.GPS_TAG_GPS_DATE_STAMP);
    102         gpsDirectory.add(GpsTagConstants.GPS_TAG_GPS_DATE_STAMP, dateStamp);
     103            // make sure to remove old value if present (this method will
     104            // not fail if the tag does not exist).
     105            gpsDirectory.removeField(GpsTagConstants.GPS_TAG_GPS_DATE_STAMP);
     106            gpsDirectory.add(GpsTagConstants.GPS_TAG_GPS_DATE_STAMP, dateStamp);
     107        }
    103108
    104109        outputSet.setGPSInDegrees(lon, lat);
    105110
     111        if (speed != null) {
     112            gpsDirectory.removeField(GpsTagConstants.GPS_TAG_GPS_SPEED_REF);
     113            gpsDirectory.add(GpsTagConstants.GPS_TAG_GPS_SPEED_REF,
     114                             GpsTagConstants.GPS_TAG_GPS_SPEED_REF_VALUE_KMPH);
     115
     116            gpsDirectory.removeField(GpsTagConstants.GPS_TAG_GPS_SPEED);
     117            gpsDirectory.add(GpsTagConstants.GPS_TAG_GPS_SPEED, RationalNumber.valueOf(speed));
     118        }
     119
    106120        if (ele != null) {
    107121            byte eleRef =  ele >= 0 ? (byte) 0 : (byte) 1;
    108122            gpsDirectory.removeField(GpsTagConstants.GPS_TAG_GPS_ALTITUDE_REF);
    … …  
    111125            gpsDirectory.removeField(GpsTagConstants.GPS_TAG_GPS_ALTITUDE);
    112126            gpsDirectory.add(GpsTagConstants.GPS_TAG_GPS_ALTITUDE, RationalNumber.valueOf(Math.abs(ele)));
    113127        }
     128
     129        if (imgDir != null) {
     130            gpsDirectory.removeField(GpsTagConstants.GPS_TAG_GPS_IMG_DIRECTION_REF);
     131            gpsDirectory.add(GpsTagConstants.GPS_TAG_GPS_IMG_DIRECTION_REF,
     132                             GpsTagConstants.GPS_TAG_GPS_IMG_DIRECTION_REF_VALUE_TRUE_NORTH);
     133            // make sure the value is in the range 0.0...<360.0
     134            if (imgDir < 0.0) {
     135                imgDir %= 360.0; // >-360.0...-0.0
     136                imgDir += 360.0; // >0.0...360.0
     137            }
     138            if (imgDir >= 360.0) {
     139                imgDir %= 360.0;
     140            }
     141            gpsDirectory.removeField(GpsTagConstants.GPS_TAG_GPS_IMG_DIRECTION);
     142            gpsDirectory.add(GpsTagConstants.GPS_TAG_GPS_IMG_DIRECTION, RationalNumber.valueOf(imgDir));
     143        }
     144
    114145        try (BufferedOutputStream os = new BufferedOutputStream(new FileOutputStream(dst))) {
    115146            new ExifRewriter().updateExifMetadataLossless(jpegImageFile, os, outputSet);
    116147        }

  • src/org/openstreetmap/josm/plugins/photo_geotagging/GeotaggingAction.java

     
    1313import java.nio.file.Files;
    1414import java.text.DecimalFormat;
    1515import java.util.ArrayList;
     16import java.util.Date;
    1617import java.util.List;
    1718import java.util.UUID;
    1819
    … …  
    6566        final List<ImageEntry> images = new ArrayList<>();
    6667        for (ImageEntry e : layer.getImages()) {
    6768             /* Only write lat/lon to the file, if the position is known and
    68                 we have a time from the correlation to the gpx track. */
    69             if (e.getPos() != null && e.hasGpsTime()) {
     69                the GPS data changed. */
     70            if (e.getPos() != null && (e.hasNewGpsData() || e.hasGpsTime())) {
    7071                images.add(e);
    7172            }
    7273        }
    … …  
    204205                    }
    205206
    206207                    Long mTime = null;
    207                     if (mTimeMode == MTIME_MODE_PREVIOUS_VALUE) {
     208                    if (mTimeMode == MTIME_MODE_GPS) {
     209                        // check GPS time fields, do nothing if all fails
     210                        Date time;
     211                        if (e.hasGpsTime()) {
     212                            time = e.getGpsTime();
     213                        } else {
     214                            time = e.getExifGpsTime();
     215                        }
     216                        if (time != null) {
     217                            mTime = time.getTime();
     218                        }
     219                    }
     220                    if ( mTimeMode == MTIME_MODE_PREVIOUS_VALUE
     221                         // this is also the fallback if one of the other
     222                         // modes failed to determine the modification time
     223                         || (mTimeMode != 0 && mTime == null)) {
    208224                        mTime = e.getFile().lastModified();
    209225                        if (mTime.equals(0L))
    210226                            throw new IOException(tr("Could not read mtime."));
    … …  
    212228
    213229                    chooseFiles(e.getFile());
    214230                    if (canceled) return;
    215                     ExifGPSTagger.setExifGPSTag(fileFrom, fileTo, e.getPos().lat(), e.getPos().lon(), e.getGpsTime().getTime(), e.getElevation());
     231                    ExifGPSTagger.setExifGPSTag(fileFrom, fileTo, e.getPos().lat(), e.getPos().lon(),
     232                            e.getGpsTime(), e.getSpeed(), e.getElevation(), e.getExifImgDir());
    216233
    217                     if (mTimeMode == MTIME_MODE_GPS) {
    218                         mTime = e.getGpsTime().getTime();
    219                     }
    220 
    221234                    if (mTime != null) {
    222235                        if (!fileTo.setLastModified(mTime))
    223236                            throw new IOException(tr("Could not write mtime."));
    … …  
    377390     */
    378391    private boolean enabled(GeoImageLayer layer) {
    379392        for (ImageEntry e : layer.getImages()) {
    380             if (e.getPos() != null && e.hasGpsTime())
     393            if (e.getPos() != null && (e.hasNewGpsData() || e.hasGpsTime()))
    381394                return true;
    382395        }
    383396        return false;