* Special:Import/importDump fixes: report XML parse errors, accept <minor/>
[lhc/web/wiklou.git] / includes / Exif.php
index e475771..cf79423 100644 (file)
@@ -1,5 +1,5 @@
 <?php
-if (defined('MEDIAWIKI')) {
+if ( !defined( 'MEDIAWIKI' ) ) die();
 /**
  * @package MediaWiki
  * @subpackage Metadata
@@ -27,6 +27,20 @@ if (defined('MEDIAWIKI')) {
  * @bug 1555, 1947
  */
 
+/**#@+
+ * Exif tag type definition
+ */
+define('MW_EXIF_BYTE', 1);             # An 8-bit (1-byte) unsigned integer.
+define('MW_EXIF_ASCII', 2);            # An 8-bit byte containing one 7-bit ASCII code. The final byte is terminated with NULL.
+define('MW_EXIF_SHORT', 3);            # A 16-bit (2-byte) unsigned integer.
+define('MW_EXIF_LONG', 4);             # A 32-bit (4-byte) unsigned integer.
+define('MW_EXIF_RATIONAL', 5);         # Two LONGs. The first LONG is the numerator and the second LONG expresses the denominator
+define('MW_EXIF_UNDEFINED', 7);                # An 8-bit byte that can take any value depending on the field definition
+define('MW_EXIF_SLONG', 9);            # A 32-bit (4-byte) signed integer (2's complement notation),
+define('MW_EXIF_SRATIONAL', 10);       # Two SLONGs. The first SLONG is the numerator and the second SLONG is the denominator.
+/**#@-*/
+
+
 /**
  * @package MediaWiki
  * @subpackage Metadata
@@ -34,101 +48,108 @@ if (defined('MEDIAWIKI')) {
 class Exif {
        /**#@+
         * @var array
+        * @access private
         */
-        
+       
        /**
         * Exif tags grouped by category, the tagname itself is the key and the type
         * is the value, in the case of more than one possible value type they are 
         * seperated by commas.
-        *
-        * @access private
         */
-       var $mExif;
-       
+       var $mExifTags;
+
        /**
         * A one dimentional array of all Exif tags
         */
-       var $mFlatExif;
-       
+       var $mFlatExifTags;
+
        /**
-        * A one dimentional array of all Exif tags that we'd want to save in
-        * the database or present on a page.
+        * The raw Exif data returned by exif_read_data()
         */
-       var $mValidExif;
+       var $mRawExifData;
 
+       /**
+        * A Filtered version of $mRawExifData that has been pruned of invalid
+        * tags and tags that contain content they shouldn't contain according
+        * to the Exif specification
+        */
+       var $mFilteredExifData;
+
+       /**
+        * Filtered and formatted Exif data, see FormatExif::getFormattedData()
+        */
+       var $mFormattedExifData;
+       
        /**#@-*/
 
        /**
-        * Constructor
+        * The private log to log to
+        *
+        * @var string
+        * @access private
         */
-       function Exif() {
-               /**#@+
-                * Exif tag type definition
-                */
-               define('BYTE', 1);              # An 8-bit unsigned integer.
-               define('ASCII', 2);             # An 8-bit byte containing one 7-bit ASCII code. The final byte is terminated with NULL.
-               define('SHORT', 3);             # A 16-bit (2-byte) unsigned integer.
-               define('LONG', 4);              # A 32-bit (4-byte) unsigned integer.
-               define('RATIONAL', 5);          # Two LONGs. The first LONG is the numerator and the second LONG expresses the denominator
-               define('UNDEFINED', 7);         # An 8-bit byte that can take any value depending on the field definition
-               define('SLONG', 9);             # A 32-bit (4-byte) signed integer (2's complement notation),
-               define('SRATIONAL', 10);        # Two SLONGs. The first SLONG is the numerator and the second SLONG is the denominator.
-               /**#@-*/
+       var $log = 'exif';
 
+       /**
+        * Constructor
+        *
+        * @param string $file
+        */
+       function Exif( $file ) {
                /**
                 * Page numbers here refer to pages in the EXIF 2.2 standard
                 *
                 * @link http://exif.org/Exif2-2.PDF The Exif 2.2 specification
                 */
-               $this->mExif = array(
+               $this->mExifTags = array(
                        # TIFF Rev. 6.0 Attribute Information (p22)
                        'tiff' => array(
                                # Tags relating to image structure
                                'structure' => array(
-                                       'ImageWidth' => SHORT.','.LONG,         # Image width
-                                       'ImageLength' => SHORT.','.LONG,        # Image height
-                                       'BitsPerSample' => SHORT,               # Number of bits per component
+                                       'ImageWidth' => MW_EXIF_SHORT.','.MW_EXIF_LONG,         # Image width
+                                       'ImageLength' => MW_EXIF_SHORT.','.MW_EXIF_LONG,        # Image height
+                                       'BitsPerSample' => MW_EXIF_SHORT,                       # Number of bits per component
                                        # "When a primary image is JPEG compressed, this designation is not"
                                        # "necessary and is omitted." (p23)
-                                       'Compression' => SHORT,                 # Compression scheme #p23
-                                       'PhotometricInterpretation' => SHORT,   # Pixel composition #p23
-                                       'Orientation' => SHORT,                 # Orientation of image #p24
-                                       'SamplesPerPixel' => SHORT,             # Number of components
-                                       'PlanarConfiguration' => SHORT,         # Image data arrangement #p24
-                                       'YCbCrSubSampling' => SHORT,            # Subsampling ratio of Y to C #p24
-                                       'YCbCrPositioning' => SHORT,            # Y and C positioning #p24-25
-                                       'XResolution' => RATIONAL,              # Image resolution in width direction
-                                       'YResolution' => RATIONAL,              # Image resolution in height direction
-                                       'ResolutionUnit' => SHORT,              # Unit of X and Y resolution #(p26)
+                                       'Compression' => MW_EXIF_SHORT,                         # Compression scheme #p23
+                                       'PhotometricInterpretation' => MW_EXIF_SHORT,           # Pixel composition #p23
+                                       'Orientation' => MW_EXIF_SHORT,                         # Orientation of image #p24
+                                       'SamplesPerPixel' => MW_EXIF_SHORT,                     # Number of components
+                                       'PlanarConfiguration' => MW_EXIF_SHORT,                 # Image data arrangement #p24
+                                       'YCbCrSubSampling' => MW_EXIF_SHORT,                    # Subsampling ratio of Y to C #p24
+                                       'YCbCrPositioning' => MW_EXIF_SHORT,                    # Y and C positioning #p24-25
+                                       'XResolution' => MW_EXIF_RATIONAL,                      # Image resolution in width direction
+                                       'YResolution' => MW_EXIF_RATIONAL,                      # Image resolution in height direction
+                                       'ResolutionUnit' => MW_EXIF_SHORT,                      # Unit of X and Y resolution #(p26)
                                ),
                                
                                # Tags relating to recording offset
                                'offset' => array(
-                                       'StripOffsets' => SHORT.','.LONG,                       # Image data location
-                                       'RowsPerStrip' => SHORT.','.LONG,                       # Number of rows per strip
-                                       'StripByteCounts' => SHORT.','.LONG,                    # Bytes per compressed strip
-                                       'JPEGInterchangeFormat' => SHORT.','.LONG,              # Offset to JPEG SOI
-                                       'JPEGInterchangeFormatLength' => SHORT.','.LONG,        # Bytes of JPEG data
+                                       'StripOffsets' => MW_EXIF_SHORT.','.MW_EXIF_LONG,                       # Image data location
+                                       'RowsPerStrip' => MW_EXIF_SHORT.','.MW_EXIF_LONG,                       # Number of rows per strip
+                                       'StripByteCounts' => MW_EXIF_SHORT.','.MW_EXIF_LONG,                    # Bytes per compressed strip
+                                       'JPEGInterchangeFormat' => MW_EXIF_SHORT.','.MW_EXIF_LONG,              # Offset to JPEG SOI
+                                       'JPEGInterchangeFormatLength' => MW_EXIF_SHORT.','.MW_EXIF_LONG,        # Bytes of JPEG data
                                ),
                        
                                # Tags relating to image data characteristics
                                'characteristics' => array(
-                                       'TransferFunction' => SHORT,            # Transfer function
-                                       'WhitePoint' => RATIONAL,               # White point chromaticity
-                                       'PrimaryChromaticities' => RATIONAL,    # Chromaticities of primarities
-                                       'YCbCrCoefficients' => RATIONAL,        # Color space transformation matrix coefficients #p27
-                                       'ReferenceBlackWhite' => RATIONAL       # Pair of black and white reference values
+                                       'TransferFunction' => MW_EXIF_SHORT,            # Transfer function
+                                       'WhitePoint' => MW_EXIF_RATIONAL,               # White point chromaticity
+                                       'PrimaryChromaticities' => MW_EXIF_RATIONAL,    # Chromaticities of primarities
+                                       'YCbCrCoefficients' => MW_EXIF_RATIONAL,        # Color space transformation matrix coefficients #p27
+                                       'ReferenceBlackWhite' => MW_EXIF_RATIONAL       # Pair of black and white reference values
                                ),
                        
                                # Other tags
                                'other' => array(
-                                       'DateTime' => ASCII,            # File change date and time
-                                       'ImageDescription' => ASCII,    # Image title
-                                       'Make' => ASCII,                # Image input equipment manufacturer
-                                       'Model' => ASCII,               # Image input equipment model
-                                       'Software' => ASCII,            # Software used
-                                       'Artist' => ASCII,              # Person who created the image
-                                       'Copyright' => ASCII,           # Copyright holder
+                                       'DateTime' => MW_EXIF_ASCII,            # File change date and time
+                                       'ImageDescription' => MW_EXIF_ASCII,    # Image title
+                                       'Make' => MW_EXIF_ASCII,                # Image input equipment manufacturer
+                                       'Model' => MW_EXIF_ASCII,               # Image input equipment model
+                                       'Software' => MW_EXIF_ASCII,            # Software used
+                                       'Artist' => MW_EXIF_ASCII,              # Person who created the image
+                                       'Copyright' => MW_EXIF_ASCII,           # Copyright holder
                                ),
                        ),
                
@@ -138,490 +159,858 @@ class Exif {
                                'version' => array(
                                        # TODO: NOTE: Nonexistence of this field is taken to mean nonconformance
                                        # to the EXIF 2.1 AND 2.2 standards
-                                       'ExifVersion' => UNDEFINED,     # Exif version
-                                       'FlashpixVersion' => UNDEFINED, # Supported Flashpix version #p32
+                                       'ExifVersion' => MW_EXIF_UNDEFINED,     # Exif version
+                                       'FlashpixVersion' => MW_EXIF_UNDEFINED, # Supported Flashpix version #p32
                                ),
                                
                                # Tags relating to Image Data Characteristics
                                'characteristics' => array(
-                                       'ColorSpace' => SHORT,          # Color space information #p32
+                                       'ColorSpace' => MW_EXIF_SHORT,          # Color space information #p32
                                ),
                
                                # Tags relating to image configuration
                                'configuration' => array(
-                                       'ComponentsConfiguration' => UNDEFINED,         # Meaning of each component #p33
-                                       'CompressedBitsPerPixel' => RATIONAL,           # Image compression mode
-                                       'PixelYDimension' => SHORT.','.LONG,            # Valid image width
-                                       'PixelXDimension' => SHORT.','.LONG,            # Valind image height
+                                       'ComponentsConfiguration' => MW_EXIF_UNDEFINED,         # Meaning of each component #p33
+                                       'CompressedBitsPerPixel' => MW_EXIF_RATIONAL,           # Image compression mode
+                                       'PixelYDimension' => MW_EXIF_SHORT.','.MW_EXIF_LONG,    # Valid image width
+                                       'PixelXDimension' => MW_EXIF_SHORT.','.MW_EXIF_LONG,    # Valind image height
                                ),
                                
                                # Tags relating to related user information
                                'user' => array(
-                                       'MakerNote' => UNDEFINED,                       # Manufacturer notes
-                                       'UserComment' => UNDEFINED,                     # User comments #p34
+                                       'MakerNote' => MW_EXIF_UNDEFINED,                       # Manufacturer notes
+                                       'UserComment' => MW_EXIF_UNDEFINED,                     # User comments #p34
                                ),
                
                                # Tags relating to related file information
                                'related' => array(
-                                       'RelatedSoundFile' => ASCII,                    # Related audio file
+                                       'RelatedSoundFile' => MW_EXIF_ASCII,                    # Related audio file
                                ),
                
                                # Tags relating to date and time
                                'dateandtime' => array(
-                                       'DateTimeOriginal' => ASCII,                    # Date and time of original data generation #p36
-                                       'DateTimeDigitized' => ASCII,                   # Date and time of original data generation
-                                       'SubSecTime' => ASCII,                          # DateTime subseconds 
-                                       'SubSecTimeOriginal' => ASCII,                  # DateTimeOriginal subseconds
-                                       'SubSecTimeDigitized' => ASCII,                 # DateTimeDigitized subseconds
+                                       'DateTimeOriginal' => MW_EXIF_ASCII,                    # Date and time of original data generation #p36
+                                       'DateTimeDigitized' => MW_EXIF_ASCII,                   # Date and time of original data generation
+                                       'SubSecTime' => MW_EXIF_ASCII,                          # DateTime subseconds 
+                                       'SubSecTimeOriginal' => MW_EXIF_ASCII,                  # DateTimeOriginal subseconds
+                                       'SubSecTimeDigitized' => MW_EXIF_ASCII,                 # DateTimeDigitized subseconds
                                ),
                                
                                # Tags relating to picture-taking conditions (p31)
                                'conditions' => array(
-                                       'ExposureTime' => RATIONAL,                     # Exposure time
-                                       'FNumber' => RATIONAL,                          # F Number
-                                       'ExposureProgram' => SHORT,                     # Exposure Program #p38
-                                       'SpectralSensitivity' => ASCII,                 # Spectral sensitivity
-                                       'ISOSpeedRatings' => SHORT,                     # ISO speed rating
-                                       'OECF' => UNDEFINED,                            # Optoelectronic conversion factor
-                                       'ShutterSpeedValue' => SRATIONAL,               # Shutter speed
-                                       'ApertureValue' => RATIONAL,                    # Aperture
-                                       'BrightnessValue' => SRATIONAL,                 # Brightness
-                                       'ExposureBiasValue' => SRATIONAL,               # Exposure bias
-                                       'MaxApertureValue' => RATIONAL,                 # Maximum land aperture
-                                       'SubjectDistance' => RATIONAL,                  # Subject distance
-                                       'MeteringMode' => SHORT,                        # Metering mode #p40
-                                       'LightSource' => SHORT,                         # Light source #p40-41
-                                       'Flash' => SHORT,                               # Flash #p41-42
-                                       'FocalLength' => RATIONAL,                      # Lens focal length
-                                       'SubjectArea' => SHORT,                         # Subject area
-                                       'FlashEnergy' => RATIONAL,                      # Flash energy
-                                       'SpatialFrequencyResponse' => UNDEFINED,        # Spatial frequency response
-                                       'FocalPlaneXResolution' => RATIONAL,            # Focal plane X resolution
-                                       'FocalPlaneYResolution' => RATIONAL,            # Focal plane Y resolution
-                                       'FocalPlaneResolutionUnit' => SHORT,            # Focal plane resolution unit
-                                       'SubjectLocation' => SHORT,                     # Subject location
-                                       'ExposureIndex' => RATIONAL,                    # Exposure index
-                                       'SensingMethod' => SHORT,                       # Sensing method #p46
-                                       'FileSource' => UNDEFINED,                      # File source #p47
-                                       'SceneType' => UNDEFINED,                       # Scene type #p47
-                                       'CFAPattern' => UNDEFINED,                      # CFA pattern
-                                       'CustomRendered' => SHORT,                      # Custom image processing #p48
-                                       'ExposureMode' => SHORT,                        # Exposure mode #p48
-                                       'WhiteBalance' => SHORT,                        # White Balance #p49
-                                       'DigitalZoomRatio' => RATIONAL,                 # Digital zoom ration
-                                       'FocalLengthIn35mmFilm' => SHORT,               # Focal length in 35 mm film
-                                       'SceneCaptureType' => SHORT,                    # Scene capture type #p49
-                                       'GainControl' => RATIONAL,                      # Scene control #p49-50
-                                       'Contrast' => SHORT,                            # Contrast #p50
-                                       'Saturation' => SHORT,                          # Saturation #p50
-                                       'Sharpness' => SHORT,                           # Sharpness #p50
-                                       'DeviceSettingDescription' => UNDEFINED,        # Desice settings description
-                                       'SubjectDistanceRange' => SHORT,                # Subject distance range #p51
+                                       'ExposureTime' => MW_EXIF_RATIONAL,                     # Exposure time
+                                       'FNumber' => MW_EXIF_RATIONAL,                          # F Number
+                                       'ExposureProgram' => MW_EXIF_SHORT,                     # Exposure Program #p38
+                                       'SpectralSensitivity' => MW_EXIF_ASCII,                 # Spectral sensitivity
+                                       'ISOSpeedRatings' => MW_EXIF_SHORT,                     # ISO speed rating
+                                       'OECF' => MW_EXIF_UNDEFINED,                            # Optoelectronic conversion factor
+                                       'ShutterSpeedValue' => MW_EXIF_SRATIONAL,               # Shutter speed
+                                       'ApertureValue' => MW_EXIF_RATIONAL,                    # Aperture
+                                       'BrightnessValue' => MW_EXIF_SRATIONAL,                 # Brightness
+                                       'ExposureBiasValue' => MW_EXIF_SRATIONAL,               # Exposure bias
+                                       'MaxApertureValue' => MW_EXIF_RATIONAL,                 # Maximum land aperture
+                                       'SubjectDistance' => MW_EXIF_RATIONAL,                  # Subject distance
+                                       'MeteringMode' => MW_EXIF_SHORT,                        # Metering mode #p40
+                                       'LightSource' => MW_EXIF_SHORT,                         # Light source #p40-41
+                                       'Flash' => MW_EXIF_SHORT,                               # Flash #p41-42
+                                       'FocalLength' => MW_EXIF_RATIONAL,                      # Lens focal length
+                                       'SubjectArea' => MW_EXIF_SHORT,                         # Subject area
+                                       'FlashEnergy' => MW_EXIF_RATIONAL,                      # Flash energy
+                                       'SpatialFrequencyResponse' => MW_EXIF_UNDEFINED,        # Spatial frequency response
+                                       'FocalPlaneXResolution' => MW_EXIF_RATIONAL,            # Focal plane X resolution
+                                       'FocalPlaneYResolution' => MW_EXIF_RATIONAL,            # Focal plane Y resolution
+                                       'FocalPlaneResolutionUnit' => MW_EXIF_SHORT,            # Focal plane resolution unit #p46
+                                       'SubjectLocation' => MW_EXIF_SHORT,                     # Subject location
+                                       'ExposureIndex' => MW_EXIF_RATIONAL,                    # Exposure index
+                                       'SensingMethod' => MW_EXIF_SHORT,                       # Sensing method #p46
+                                       'FileSource' => MW_EXIF_UNDEFINED,                      # File source #p47
+                                       'SceneType' => MW_EXIF_UNDEFINED,                       # Scene type #p47
+                                       'CFAPattern' => MW_EXIF_UNDEFINED,                      # CFA pattern
+                                       'CustomRendered' => MW_EXIF_SHORT,                      # Custom image processing #p48
+                                       'ExposureMode' => MW_EXIF_SHORT,                        # Exposure mode #p48
+                                       'WhiteBalance' => MW_EXIF_SHORT,                        # White Balance #p49
+                                       'DigitalZoomRatio' => MW_EXIF_RATIONAL,                 # Digital zoom ration
+                                       'FocalLengthIn35mmFilm' => MW_EXIF_SHORT,               # Focal length in 35 mm film
+                                       'SceneCaptureType' => MW_EXIF_SHORT,                    # Scene capture type #p49
+                                       'GainControl' => MW_EXIF_RATIONAL,                      # Scene control #p49-50
+                                       'Contrast' => MW_EXIF_SHORT,                            # Contrast #p50
+                                       'Saturation' => MW_EXIF_SHORT,                          # Saturation #p50
+                                       'Sharpness' => MW_EXIF_SHORT,                           # Sharpness #p50
+                                       'DeviceSettingDescription' => MW_EXIF_UNDEFINED,        # Desice settings description
+                                       'SubjectDistanceRange' => MW_EXIF_SHORT,                # Subject distance range #p51
                                ),
                                
                                'other' => array(
-                                       'ImageUniqueID' => ASCII,       # Unique image ID
+                                       'ImageUniqueID' => MW_EXIF_ASCII,       # Unique image ID
                                ),
                        ),
                
                        # GPS Attribute Information (p52)
                        'gps' => array(
-                               'GPSVersionID' => BYTE,                 # GPS tag version
-                               'GPSLatitudeRef' => ASCII,              # North or South Latitude #p52-53
-                               'GPSLatitude' => RATIONAL,              # Latitude
-                               'GPSLongitudeRef' => ASCII,             # East or West Longitude #p53
-                               'GPSLongitude' => RATIONAL,             # Longitude
-                               'GPSAltitudeRef' => BYTE,               # Altitude reference
-                               'GPSAltitude' => RATIONAL,              # Altitude
-                               'GPSTimeStamp' => RATIONAL,             # GPS time (atomic clock)
-                               'GPSSatellites' => ASCII,               # Satellites used for measurement
-                               'GPSStatus' => ASCII,                   # Receiver status #p54
-                               'GPSMeasureMode' => ASCII,              # Measurement mode #p54-55
-                               'GPSDOP' => RATIONAL,                   # Measurement precision
-                               'GPSSpeedRef' => ASCII,                 # Speed unit #p55
-                               'GPSSpeed' => RATIONAL,                 # Speed of GPS receiver
-                               'GPSTrackRef' => ASCII,                 # Reference for direction of movement #p55
-                               'GPSTrack' => RATIONAL,                 # Direction of movement
-                               'GPSImgDirectionRef' => ASCII,          # Reference for direction of image #p56
-                               'GPSImgDirection' => RATIONAL,          # Direction of image
-                               'GPSMapDatum' => ASCII,                 # Geodetic survey data used
-                               'GPSDestLatitudeRef' => ASCII,          # Reference for latitude of destination #p56
-                               'GPSDestLatitude' => RATIONAL,          # Latitude destination
-                               'GPSDestLongitudeRef' => ASCII,         # Reference for longitude of destination #p57
-                               'GPSDestLongitude' => RATIONAL,         # Longitude of destination
-                               'GPSDestBearingRef' => ASCII,           # Reference for bearing of destination #p57
-                               'GPSDestBearing' => RATIONAL,           # Bearing of destination
-                               'GPSDestDistanceRef' => ASCII,          # Reference for distance to destination #p57-58
-                               'GPSDestDistance' => RATIONAL,          # Distance to destination
-                               'GPSProcessingMethod' => UNDEFINED,     # Name of GPS processing method
-                               'GPSAreaInformation' => UNDEFINED,      # Name of GPS area
-                               'GPSDateStamp' => ASCII,                # GPS date
-                               'GPSDifferential' => SHORT,             # GPS differential correction
+                               'GPSVersionID' => MW_EXIF_BYTE,                 # GPS tag version
+                               'GPSLatitudeRef' => MW_EXIF_ASCII,              # North or South Latitude #p52-53
+                               'GPSLatitude' => MW_EXIF_RATIONAL,              # Latitude
+                               'GPSLongitudeRef' => MW_EXIF_ASCII,             # East or West Longitude #p53
+                               'GPSLongitude' => MW_EXIF_RATIONAL,             # Longitude
+                               'GPSAltitudeRef' => MW_EXIF_BYTE,               # Altitude reference
+                               'GPSAltitude' => MW_EXIF_RATIONAL,              # Altitude
+                               'GPSTimeStamp' => MW_EXIF_RATIONAL,             # GPS time (atomic clock)
+                               'GPSSatellites' => MW_EXIF_ASCII,               # Satellites used for measurement
+                               'GPSStatus' => MW_EXIF_ASCII,                   # Receiver status #p54
+                               'GPSMeasureMode' => MW_EXIF_ASCII,              # Measurement mode #p54-55
+                               'GPSDOP' => MW_EXIF_RATIONAL,                   # Measurement precision
+                               'GPSSpeedRef' => MW_EXIF_ASCII,                 # Speed unit #p55
+                               'GPSSpeed' => MW_EXIF_RATIONAL,                 # Speed of GPS receiver
+                               'GPSTrackRef' => MW_EXIF_ASCII,                 # Reference for direction of movement #p55
+                               'GPSTrack' => MW_EXIF_RATIONAL,                 # Direction of movement
+                               'GPSImgDirectionRef' => MW_EXIF_ASCII,          # Reference for direction of image #p56
+                               'GPSImgDirection' => MW_EXIF_RATIONAL,          # Direction of image
+                               'GPSMapDatum' => MW_EXIF_ASCII,                 # Geodetic survey data used
+                               'GPSDestLatitudeRef' => MW_EXIF_ASCII,          # Reference for latitude of destination #p56
+                               'GPSDestLatitude' => MW_EXIF_RATIONAL,          # Latitude destination
+                               'GPSDestLongitudeRef' => MW_EXIF_ASCII,         # Reference for longitude of destination #p57
+                               'GPSDestLongitude' => MW_EXIF_RATIONAL,         # Longitude of destination
+                               'GPSDestBearingRef' => MW_EXIF_ASCII,           # Reference for bearing of destination #p57
+                               'GPSDestBearing' => MW_EXIF_RATIONAL,           # Bearing of destination
+                               'GPSDestDistanceRef' => MW_EXIF_ASCII,          # Reference for distance to destination #p57-58
+                               'GPSDestDistance' => MW_EXIF_RATIONAL,          # Distance to destination
+                               'GPSProcessingMethod' => MW_EXIF_UNDEFINED,     # Name of GPS processing method
+                               'GPSAreaInformation' => MW_EXIF_UNDEFINED,      # Name of GPS area
+                               'GPSDateStamp' => MW_EXIF_ASCII,                # GPS date
+                               'GPSDifferential' => MW_EXIF_SHORT,             # GPS differential correction
                        ),
                );
 
+               $basename = basename( $file );
+               
                $this->makeFlatExifTags();
-               $this->makeValidExifTags();
+               
+               $this->debugFile( $basename, __FUNCTION__, true );
+               wfSuppressWarnings();
+               $this->mRawExifData = exif_read_data( $file );
+               wfRestoreWarnings();
+               
+               $this->makeFilteredData();
+               $this->makeFormattedData();
+               
+               $this->debugFile( $basename, __FUNCTION__, false );
        }
-
-       /**
-        * Get the raw list of exiftags
-        *
+       
+       /**#@+
         * @access private
-        * @return array
-       */
-       function getExif() {
-               return $this->mExif;
-       }
-
+        */
        /**
         * Generate a flat list of the exif tags
         */
        function makeFlatExifTags() {
-               $exif = $this->getExif();
-               array_walk($exif, array(&$this, 'callback')); // note the reference
+               $this->extractTags( $this->mExifTags );
        }
        
        /**
-        * A callback function used by makeFlatExifTags()
+        * A recursing extractor function used by makeFlatExifTags()
+        *
+        * Note: This used to use an array_walk function, but it made PHP5
+        * segfault, see `cvs diff -u -r 1.4 -r 1.5 Exif.php`
         */
-       function callback($val, $key) {
-               if (gettype($val) === 'array')
-                       array_walk($val, array(&$this, 'callback'));
-               else
-                       $this->mFlatExif[$key] = $val;
+       function extractTags( &$tagset ) {
+               foreach( $tagset as $key => $val ) {
+                       if( is_array( $val ) ) {
+                               $this->extractTags( $val );
+                       } else {
+                               $this->mFlatExifTags[$key] = $val;
+                       }
+               }
        }
-
+       
        /**
-        * Produce a list of all Exif tags appropriate for user output
-        *
-        * Produce a list of all tags that we want to show in output, in order not to 
-        * output binary gibberish such as raw thumbnails we strip all tags
-        * with the datatype of UNDEFINED.
-        *
-        * @todo We might actually want to display some of the UNDEFINED
-        *       stuff, but we strip it for now.
+        * Make $this->mFilteredExifData
         */
-       function makeValidExifTags() {
-               foreach($this->mFlatExif as $key => $val) {
-                       if (strpos( $val, (string)UNDEFINED ) !== false)
-                               continue;
-                       $this->mValidExif[] = $key;
+       function makeFilteredData() {
+               $this->mFilteredExifData = $this->mRawExifData;
+               
+               foreach( $this->mFilteredExifData as $k => $v ) {
+                       if ( !in_array( $k, array_keys( $this->mFlatExifTags ) ) ) {
+                               $this->debug( $v, __FUNCTION__, "'$k' is not a valid Exif tag" );
+                               unset( $this->mFilteredExifData[$k] );
+                       }
+               }
+
+               foreach( $this->mFilteredExifData as $k => $v ) {
+                       if ( !$this->validate($k, $v) ) {
+                               $this->debug( $v, __FUNCTION__, "'$k' contained invalid data" );
+                               unset( $this->mFilteredExifData[$k] );
+                       }
                }
        }
 
-        /**
-         * The version of the output format
-         *
-         * Before the actual metadata information is saved in the database we
-         * strip some of it since we don't want to save things like thumbnails
-         * which usually accompany Exif data. This value gets saved in the
-         * database along with the actual Exif data, and if the version in the
-         * database doesn't equal the value returned by this function the Exif
-         * data is regenerated.
-         *
-         * @return int
-         */
+       function makeFormattedData( $data = null ) {
+               $format = new FormatExif( $this->getFilteredData() );
+               $this->mFormattedExifData = $format->getFormattedData();
+       }
+       /**#@-*/
+
+       /**#@+
+        * @return array
+        */
+       /**
+        * Get $this->mRawExifData
+        */
+       function getData() {
+               return $this->mRawExifData;
+       }
+
+       /**
+        * Get $this->mFilteredExifData
+        */
+       function getFilteredData() {
+               return $this->mFilteredExifData;
+       }
+
+       /**
+        * Get $this->mFormattedExifData
+        */
+       function getFormattedData() {
+               return $this->mFormattedExifData;
+       }
+       /**#@-*/
+       
+       /**
+        * The version of the output format
+        *
+        * Before the actual metadata information is saved in the database we
+        * strip some of it since we don't want to save things like thumbnails
+        * which usually accompany Exif data. This value gets saved in the
+        * database along with the actual Exif data, and if the version in the
+        * database doesn't equal the value returned by this function the Exif
+        * data is regenerated.
+        *
+        * @return int
+        */
        function version() {
                return 1; // We don't need no bloddy constants!
        }
 
        /**#@+
         * Validates if a tag value is of the type it should be according to the Exif spec
-        * 
+        *
+        * @access private
+        *
         * @param mixed $in The input value to check
         * @return bool
         */
        function isByte( $in ) {
-               return is_numeric($in) && $in >= 0 && $in <= 255;
+               if ( sprintf('%d', $in) == $in && $in >= 0 && $in <= 255 ) {
+                       $this->debug( $in, __FUNCTION__, true );
+                       return true;
+               } else {
+                       $this->debug( $in, __FUNCTION__, false );
+                       return false;
+               }
        }
        
        function isASCII( $in ) {
-               return true; // TODO: FIXME
+               if ( preg_match( "/[^\x0a\x20-\x7e]/", $in ) ) {
+                       $this->debug( $in, __FUNCTION__, 'found a character not in our whitelist' );
+                       return false;
+               }
+               
+               if ( preg_match( "/^\s*$/", $in ) ) {
+                       $this->debug( $in, __FUNCTION__, 'input consisted solely of whitespace' );
+                       return false;
+               }
+               
+               return true;
        }
 
        function isShort( $in ) {
-               return is_numeric($in) && $in >= 0 && $in <= 65536;
+               if ( sprintf('%d', $in) == $in && $in >= 0 && $in <= 65536 ) {
+                       $this->debug( $in, __FUNCTION__, true );
+                       return true;
+               } else {
+                       $this->debug( $in, __FUNCTION__, false );
+                       return false;
+               }
        }
 
        function isLong( $in ) {
-               return is_numeric($in) && $in >= 0 && $in <= 4294967296;
-
+               if ( sprintf('%d', $in) == $in && $in >= 0 && $in <= 4294967296 ) {
+                       $this->debug( $in, __FUNCTION__, true );
+                       return true;
+               } else {
+                       $this->debug( $in, __FUNCTION__, false );
+                       return false;
+               }
        }
        
        function isRational( $in ) {
-               $in = explode( '/', $in, 2 );
-               return $this->isLong($in[0]) && $this->isLong($in[1]);
+               if ( @preg_match( "/^(\d+)\/(\d+[1-9]|[1-9]\d*)$/", $in, $m ) ) { # Avoid division by zero
+                       return $this->isLong( $m[1] ) && $this->isLong( $m[2] );
+               } else {
+                       $this->debug( $in, __FUNCTION__, 'fed a non-fraction value' );
+                       return false;
+               }
        }
 
        function isUndefined( $in ) {
-               return true;
+               if ( preg_match( "/^\d{4}$/", $in ) ) { // Allow ExifVersion and FlashpixVersion
+                       $this->debug( $in, __FUNCTION__, true );
+                       return true;
+               } else {
+                       $this->debug( $in, __FUNCTION__, false );
+                       return false;
+               }
        }
 
        function isSlong( $in ) {
-               return $this->isLong( abs( $in ) );
+               if ( $this->isLong( abs( $in ) ) ) {
+                       $this->debug( $in, __FUNCTION__, true );
+                       return true;
+               } else {
+                       $this->debug( $in, __FUNCTION__, false );
+                       return false;
+               }
        }
 
        function isSrational( $in ) {
-               $in = explode( '/', $in, 2 );
-               return $this->isSlong($in[0]) && $this->isSlong($in[1]);
+               if ( preg_match( "/^(\d+)\/(\d+[1-9]|[1-9]\d*)$/", $in, $m ) ) { # Avoid division by zero
+                       return $this->isSlong( $m[0] ) && $this->isSlong( $m[1] );
+               } else {
+                       $this->debug( $in, __FUNCTION__, 'fed a non-fraction value' );
+                       return false;
+               }
        }
        /**#@-*/
 
        /**
-        * Validates if a tag has a legal value according to the Exif spec, presumes
-        * that the given tag is valid ( has been checked in advance with
-        * $this->mValidExif )
+        * Validates if a tag has a legal value according to the Exif spec
+        *
+        * @access private
         *
         * @param string $tag The tag to check
         * @param mixed  $val The value of the tag
         * @return bool
         */
        function validate( $tag, $val ) {
+               $debug = "tag is '$tag'";
                // Fucks up if not typecast 
-               switch((string)$this->mFlatExif[$tag]) {
-                       case (string)BYTE:
+               switch( (string)$this->mFlatExifTags[$tag] ) {
+                       case (string)MW_EXIF_BYTE:
+                               $this->debug( $val, __FUNCTION__, $debug );
                                return $this->isByte( $val );
-                       case (string)ASCII:
+                       case (string)MW_EXIF_ASCII:
+                               $this->debug( $val, __FUNCTION__, $debug );
                                return $this->isASCII( $val );
-                       case (string)SHORT:
+                       case (string)MW_EXIF_SHORT:
+                               $this->debug( $val, __FUNCTION__, $debug );
                                return $this->isShort( $val );
-                       case (string)LONG:
+                       case (string)MW_EXIF_LONG:
+                               $this->debug( $val, __FUNCTION__, $debug );
                                return $this->isLong( $val );
-                       case (string)RATIONAL:
+                       case (string)MW_EXIF_RATIONAL:
+                               $this->debug( $val, __FUNCTION__, $debug );
                                return $this->isRational( $val );
-                       case (string)UNDEFINED:
+                       case (string)MW_EXIF_UNDEFINED:
+                               $this->debug( $val, __FUNCTION__, $debug );
                                return $this->isUndefined( $val );
-                       case (string)SLONG:
+                       case (string)MW_EXIF_SLONG:
+                               $this->debug( $val, __FUNCTION__, $debug );
                                return $this->isSlong( $val );
-                       case (string)SRATIONAL:
+                       case (string)MW_EXIF_SRATIONAL:
+                               $this->debug( $val, __FUNCTION__, $debug );
                                return $this->isSrational( $val );
-                       case (string)SHORT.','.LONG:
+                       case (string)MW_EXIF_SHORT.','.MW_EXIF_LONG:
+                               $this->debug( $val, __FUNCTION__, $debug );
                                return $this->isShort( $val ) || $this->isLong( $val );
                        default:
-                               wfDebug( "Exif: The tag \"$tag\" had an invalid value: \"$val\"\n" );
+                               $this->debug( $val, __FUNCTION__, "The tag '$tag' is unknown" );
                                return false;
                }
        }
 
+       /**
+        * Conviniance function for debugging output
+        *
+        * @access private
+        *
+        * @param mixed $in 
+        * @param string $fname
+        * @param mixed $action
+        */
+        function debug( $in, $fname, $action = null ) {
+               $type = gettype( $in );
+               $class = ucfirst( __CLASS__ );
+               if ( $type === 'array' )
+                       $in = print_r( $in, true ); 
+        
+               if ( $action === true )
+                       wfDebugLog( $this->log, "$class::$fname: accepted: '$in' (type: $type)\n");
+               elseif ( $action === false ) 
+                       wfDebugLog( $this->log, "$class::$fname: rejected: '$in' (type: $type)\n");
+               elseif ( $action === null )
+                       wfDebugLog( $this->log, "$class::$fname: input was: '$in' (type: $type)\n");
+               else
+                       wfDebugLog( $this->log, "$class::$fname: $action (type: $type; content: '$in')\n");
+       }
+
+       /**
+        * Conviniance function for debugging output
+        *
+        * @access private
+        *
+        * @param string $basename The name of the file being processed
+        * @paran string $fname The name of the function calling this function
+        * @param bool $bool $io Specify whether we're beginning or ending
+        */
+       function debugFile( $basename, $fname, $io ) {
+               $class = ucfirst( __CLASS__ );
+               if ( $io )
+                       wfDebugLog( $this->log, "$class::$fname: begin processing: '$basename'\n" );
+               else
+                       wfDebugLog( $this->log, "$class::$fname: end processing: '$basename'\n" );
+       }
+
+}
+
+/**
+ * @package MediaWiki
+ * @subpackage Metadata
+ */
+class FormatExif {
+       /**
+        * The Exif data to format
+        *
+        * @var array
+        * @access private
+        */
+       var $mExif;
+       
+       /**
+        * Constructor
+        *
+        * @param array $exif The Exif data to format ( as returned by
+        *                    Exif::getFilteredData() )
+        */
+       function FormatExif( $exif ) {
+               $this->mExif = $exif;
+       }
+       
        /**
         * Numbers given by Exif user agents are often magical, that is they
         * should be replaced by a detailed explanation depending on their
         * value which most of the time are plain integers. This function
         * formats Exif values into human readable form.
         *
-        * @param string $tag The tag to be formatted
-        * @param mixed  $val The value of the tag
-        * @return string
+        * @return array
         */
-       function format( $tag, $val ) {
+       function getFormattedData() {
                global $wgLang;
                
-               switch ($tag) {
+               $tags =& $this->mExif;
+
+               $resolutionunit = !isset( $tags['ResolutionUnit'] ) || $tags['ResolutionUnit'] == 2 ? 2 : 3;
+               unset( $tags['ResolutionUnit'] );
+               
+               foreach( $tags as $tag => $val ) {
+                       switch( $tag ) {
                        case 'Compression':
-                               switch ($val) {
-                                       case 1: case 6:
-                                               return $this->msg( $tag, $val );
-                               }
-                       
+                               switch( $val ) {
+                               case 1: case 6:
+                                       $tags[$tag] = $this->msg( $tag, $val );
+                                       break;
+                               default:
+                                       $tags[$tag] = $val;
+                                       break;
+                               }
+                               break;
+       
                        case 'PhotometricInterpretation':
-                               switch ($val) {
-                                       case 2: case 6:
-                                               return $this->msg( $tag, $val );
-                               }
+                               switch( $val ) {
+                               case 2: case 6:
+                                       $tags[$tag] = $this->msg( $tag, $val );
+                                       break;
+                               default:
+                                       $tags[$tag] = $val;
+                                       break;
+                               }
+                               break;
                        
                        case 'Orientation':
-                               switch ($val) {
-                                       case 1: case 2: case 3: case 4: case 5: case 6: case 7: case 8:
-                                               return $this->msg( $tag, $val );
-                               }
+                               switch( $val ) {
+                               case 1: case 2: case 3: case 4: case 5: case 6: case 7: case 8:
+                                       $tags[$tag] = $this->msg( $tag, $val );
+                                       break;
+                               default:
+                                       $tags[$tag] = $val;
+                                       break;
+                               }
+                               break;
                        
-                       // TODO: If this field does not exist use 1
                        case 'PlanarConfiguration':
-                               switch ($val) {
-                                       case 1: case 2:
-                                               return $this->msg( $tag, $val );
-                               }
+                               switch( $val ) {
+                               case 1: case 2:
+                                       $tags[$tag] = $this->msg( $tag, $val );
+                                       break;
+                               default:
+                                       $tags[$tag] = $val;
+                                       break;
+                               }
+                               break;
                        
                        // TODO: YCbCrSubSampling
                        // TODO: YCbCrPositioning
-                       // TODO: If this field does not exists use 2
-                       case 'ResolutionUnit': #p26
-                               switch ($val) {
-                                       case 2: case 3:
-                                               return $this->msg( $tag, $val );
-                               }
                        
+                       case 'XResolution':
+                       case 'YResolution':
+                               switch( $resolutionunit ) {
+                                       case 2:
+                                               $tags[$tag] = $this->msg( 'XYResolution', 'i', $this->formatNum( $val ) );
+                                               break;
+                                       case 3:
+                                               $this->msg( 'XYResolution', 'c', $this->formatNum( $val ) );
+                                               break;
+                                       default:
+                                               $tags[$tag] = $val;
+                                               break;
+                               }
+                               break;
+                               
                        // TODO: YCbCrCoefficients  #p27 (see annex E)
                        case 'ExifVersion': case 'FlashpixVersion':
-                               return "$val"/100;
+                               $tags[$tag] = "$val"/100;
+                               break;
                        
                        case 'ColorSpace':
-                               switch ($val) {
-                                       case 1: case 'FFFF.H':
-                                               return $this->msg( $tag, $val );
-                               }
+                               switch( $val ) {
+                               case 1: case 'FFFF.H':
+                                       $tags[$tag] = $this->msg( $tag, $val );
+                                       break;
+                               default:
+                                       $tags[$tag] = $val;
+                                       break;
+                               }
+                               break;
                        
                        case 'ComponentsConfiguration':
-                               switch ($val) {
-                                       case 0: case 1: case 2: case 3: case 4: case 5: case 6:
-                                               return $this->msg( $tag, $val );
-                               }
+                               switch( $val ) {
+                               case 0: case 1: case 2: case 3: case 4: case 5: case 6:
+                                       $tags[$tag] = $this->msg( $tag, $val );
+                                       break;
+                               default:
+                                       $tags[$tag] = $val;
+                                       break;
+                               }
+                               break;
                        
                        case 'DateTime':
                        case 'DateTimeOriginal':
                        case 'DateTimeDigitized':
-                               return $wgLang->timeanddate( wfTimestamp(TS_MW, $val) );
+                               $tags[$tag] = $wgLang->timeanddate( wfTimestamp(TS_MW, $val) );
+                               break;
+                       
                        case 'ExposureProgram':
-                               switch ($val) {
-                                       case 0: case 1: case 2: case 3: case 4: case 5: case 6: case 7: case 8:
-                                               return $this->msg( $tag, $val );
-                               }
+                               switch( $val ) {
+                               case 0: case 1: case 2: case 3: case 4: case 5: case 6: case 7: case 8:
+                                       $tags[$tag] = $this->msg( $tag, $val );
+                                       break;
+                               default:
+                                       $tags[$tag] = $val;
+                                       break;
+                               }
+                               break;
+
+                       case 'SubjectDistance':
+                               $tags[$tag] = $this->msg( $tag, '', $this->formatNum( $val ) );
+                               break;
+       
                        case 'MeteringMode':
-                               switch ($val) {
-                                       case 0: case 1: case 2: case 3: case 4: case 5: case 6: case 7: case 255:
-                                               return $this->msg( $tag, $val );
-                               }
+                               switch( $val ) {
+                               case 0: case 1: case 2: case 3: case 4: case 5: case 6: case 7: case 255:
+                                       $tags[$tag] = $this->msg( $tag, $val );
+                                       break;
+                               default:
+                                       $tags[$tag] = $val;
+                                       break;
+                               }
+                               break;
+       
                        case 'LightSource':
-                               switch ($val) {
-                                       case 0: case 1: case 2: case 3: case 4: case 9: case 10: case 11:
-                                       case 12: case 13: case 14: case 15: case 17: case 18: case 19: case 20:
-                                       case 21: case 22: case 23: case 24: case 255:
-                                               return $this->msg( $tag, $val );
-                               }
+                               switch( $val ) {
+                               case 0: case 1: case 2: case 3: case 4: case 9: case 10: case 11:
+                               case 12: case 13: case 14: case 15: case 17: case 18: case 19: case 20:
+                               case 21: case 22: case 23: case 24: case 255:
+                                       $tags[$tag] = $this->msg( $tag, $val );
+                                       break;
+                               default:
+                                       $tags[$tag] = $val;
+                                       break;
+                               }
+                               break;
+       
                        // TODO: Flash
+                       case 'FocalPlaneResolutionUnit':
+                               switch( $val ) {
+                               case 2:
+                                       $tags[$tag] = $this->msg( $tag, $val );
+                                       break;
+                               default:
+                                       $tags[$tag] = $val;
+                                       break;
+                               }
+                               break;
+                                               
                        case 'SensingMethod':
-                               switch ($val) {
-                                       case 1: case 2: case 3: case 4: case 5: case 7: case 8:
-                                               return $this->msg( $tag, $val );
-                               }
+                               switch( $val ) {
+                               case 1: case 2: case 3: case 4: case 5: case 7: case 8:
+                                       $tags[$tag] = $this->msg( $tag, $val );
+                                       break;
+                               default:
+                                       $tags[$tag] = $val;
+                                       break;
+                               }
+                               break;
+       
                        case 'FileSource':
-                               switch ($val) {
-                                       case 3:
-                                               return $this->msg( $tag, $val );
-                               }
+                               switch( $val ) {
+                               case 3:
+                                       $tags[$tag] = $this->msg( $tag, $val );
+                                       break;
+                               default:
+                                       $tags[$tag] = $val;
+                                       break;
+                               }
+                               break;
+       
                        case 'SceneType':
-                               switch ($val) {
-                                       case 1:
-                                               return $this->msg( $tag, $val );
-                               }
+                               switch( $val ) {
+                               case 1:
+                                       $tags[$tag] = $this->msg( $tag, $val );
+                                       break;
+                               default:
+                                       $tags[$tag] = $val;
+                                       break;
+                               }
+                               break;
+       
                        case 'CustomRendered':
-                               switch ($val) {
-                                       case 0: case 1:
-                                               return $this->msg( $tag, $val );
-                               }
+                               switch( $val ) {
+                               case 0: case 1:
+                                       $tags[$tag] = $this->msg( $tag, $val );
+                                       break;
+                               default:
+                                       $tags[$tag] = $val;
+                                       break;
+                               }
+                               break;
+       
                        case 'ExposureMode':
-                               switch ($val) {
-                                       case 0: case 1: case 2:
-                                               return $this->msg( $tag, $val );
-                               }
+                               switch( $val ) {
+                               case 0: case 1: case 2:
+                                       $tags[$tag] = $this->msg( $tag, $val );
+                                       break;
+                               default:
+                                       $tags[$tag] = $val;
+                                       break;
+                               }
+                               break;
+       
                        case 'WhiteBalance':
-                               switch ($val) {
-                                       case 0: case 1:
-                                               return $this->msg( $tag, $val );
-                               }
+                               switch( $val ) {
+                               case 0: case 1:
+                                       $tags[$tag] = $this->msg( $tag, $val );
+                                       break;
+                               default:
+                                       $tags[$tag] = $val;
+                                       break;
+                               }
+                               break;
+       
                        case 'SceneCaptureType':
-                               switch ($val) {
-                                       case 0: case 1: case 2: case 3:
-                                               return $this->msg( $tag, $val );
-                               }
+                               switch( $val ) {
+                               case 0: case 1: case 2: case 3:
+                                       $tags[$tag] = $this->msg( $tag, $val );
+                                       break;
+                               default:
+                                       $tags[$tag] = $val;
+                                       break;
+                               }
+                               break;
+       
                        case 'GainControl':
-                               switch ($val) {
-                                       case 0: case 1: case 2: case 3: case 4:
-                                               return $this->msg( $tag, $val );
-                               }
+                               switch( $val ) {
+                               case 0: case 1: case 2: case 3: case 4:
+                                       $tags[$tag] = $this->msg( $tag, $val );
+                                       break;
+                               default:
+                                       $tags[$tag] = $val;
+                                       break;
+                               }
+                               break;
+                               
                        case 'Contrast':
-                               switch ($val) {
-                                       case 0: case 1: case 2:
-                                               return $this->msg( $tag, $val );
-                               }
+                               switch( $val ) {
+                               case 0: case 1: case 2:
+                                       $tags[$tag] = $this->msg( $tag, $val );
+                                       break;
+                               default:
+                                       $tags[$tag] = $val;
+                                       break;
+                               }
+                               break;
+                               
                        case 'Saturation':
-                               switch ($val) {
-                                       case 0: case 1: case 2:
-                                               return $this->msg( $tag, $val );
-                               }
+                               switch( $val ) {
+                               case 0: case 1: case 2:
+                                       $tags[$tag] = $this->msg( $tag, $val );
+                                       break;
+                               default:
+                                       $tags[$tag] = $val;
+                                       break;
+                               }
+                               break;
+                               
                        case 'Sharpness':
-                               switch ($val) {
-                                       case 0: case 1: case 2:
-                                               return $this->msg( $tag, $val );
-                               }
+                               switch( $val ) {
+                               case 0: case 1: case 2:
+                                       $tags[$tag] = $this->msg( $tag, $val );
+                                       break;
+                               default:
+                                       $tags[$tag] = $val;
+                                       break;
+                               }
+                               break;
+                               
                        case 'SubjectDistanceRange':
-                               switch ($val) {
-                                       case 0: case 1: case 2: case 3:
-                                               return $this->msg( $tag, $val );
-                               }
+                               switch( $val ) {
+                               case 0: case 1: case 2: case 3:
+                                       $tags[$tag] = $this->msg( $tag, $val );
+                                       break;
+                               default:
+                                       $tags[$tag] = $val;
+                                       break;
+                               }
+                               break;
+                               
                        case 'GPSLatitudeRef':
-                               switch ($val) {
-                                       case 'N': case 'S':
-                                               return $this->msg( $tag, $val );
-                               }
+                       case 'GPSDestLatitudeRef':
+                               switch( $val ) {
+                               case 'N': case 'S':
+                                       $tags[$tag] = $this->msg( 'GPSLatitude', $val );
+                                       break;
+                               default:
+                                       $tags[$tag] = $val;
+                                       break;
+                               }
+                               break;
+                               
                        case 'GPSLongitudeRef':
-                               switch ($val) {
-                                       case 'E': case 'W':
-                                               return $this->msg( $tag, $val );
-                               }
+                       case 'GPSDestLongitudeRef':
+                               switch( $val ) {
+                               case 'E': case 'W':
+                                       $tags[$tag] = $this->msg( 'GPSLongitude', $val );
+                                       break;
+                               default:
+                                       $tags[$tag] = $val;
+                                       break;
+                               }
+                               break;
+                               
                        case 'GPSStatus':
-                               switch ($val) {
-                                       case 'A': case 'V':
-                                               return $this->msg( $tag, $val );
-                               }
+                               switch( $val ) {
+                               case 'A': case 'V':
+                                       $tags[$tag] = $this->msg( $tag, $val );
+                                       break;
+                               default:
+                                       $tags[$tag] = $val;
+                                       break;
+                               }
+                               break;
+                               
                        case 'GPSMeasureMode':
-                               switch ($val) {
-                                       case 2: case 3:
-                                               return $this->msg( $tag, $val );
-                               }
+                               switch( $val ) {
+                               case 2: case 3:
+                                       $tags[$tag] = $this->msg( $tag, $val );
+                                       break;
+                               default:
+                                       $tags[$tag] = $val;
+                                       break;
+                               }
+                               break;
+                               
                        case 'GPSSpeedRef':
-                               switch ($val) {
-                                       case 'K': case 'M': case 'N':
-                                               return $this->msg( $tag, $val );
-                               }
+                       case 'GPSDestDistanceRef':
+                               switch( $val ) {
+                               case 'K': case 'M': case 'N':
+                                       $tags[$tag] = $this->msg( 'GPSSpeed', $val );
+                                       break;
+                               default:
+                                       $tags[$tag] = $val;
+                                       break;
+                               }
+                               break;
+                               
                        case 'GPSTrackRef':
-                               switch ($val) {
-                                       case 'T': case 'M':
-                                               return $this->msg( $tag, $val );
-                               }
                        case 'GPSImgDirectionRef':
-                               switch ($val) {
-                                       case 'T': case 'M':
-                                               return $this->msg( $tag, $val );
-                               }
-                       case 'GPSDestLatitudeRef':
-                               switch ($val) {
-                                       case 'N': case 'S':
-                                               return $this->msg( $tag, $val );
-                               }
-                       case 'GPSDestLongitudeRef':
-                               switch ($val) {
-                                       case 'E': case 'W':
-                                               return $this->msg( $tag, $val );
-                               }
                        case 'GPSDestBearingRef':
-                               switch ($val) {
-                                       case 'T': case 'M':
-                                               return $this->msg( $tag, $val );                                        
-                               }
+                               switch( $val ) {
+                               case 'T': case 'M':
+                                       $tags[$tag] = $this->msg( 'GPSDirection', $val );
+                                       break;
+                               default:
+                                       $tags[$tag] = $val;
+                                       break;
+                               }
+                               break;
+                               
                        case 'GPSDateStamp':
-                               return $wgLang->date( substr($val, 0, 4) . substr($val, 5, 2) . substr($val, 8, 2) . 000000 );
-
+                               $tags[$tag] = $wgLang->date( substr( $val, 0, 4 ) . substr( $val, 5, 2 ) . substr( $val, 8, 2 ) . '000000' );
+                               break;
+       
                        // This is not in the Exif standard, just a special
                        // case for our purposes which enables wikis to wikify
-                       // the make and model to write articles about them.
-                       case 'Make': case 'Model':
-                               return wfMsg( strtolower( "exif-$tag-value" ), $val );
+                       // the make, model and software name to link to their articles.
+                       case 'Make':
+                       case 'Model':
+                       case 'Software':
+                               $tags[$tag] = $this->msg( $tag, '', $val );
+                               break;
                        default:
-                               return $val;
+                               $tags[$tag] = $this->formatNum( $val );
+                               break;
+                       }
                }
+
+               return $tags;
        }
 
        /**
-        * Conviniance function for format()
+        * Conviniance function for getFormattedData()
+        *
+        * @access private
         *
         * @param string $tag The tag name to pass on
         * @param string $val The value of the tag
+        * @param string $arg An argument to pass ($1)
         * @return string A wfMsg of "exif-$tag-$val" in lower case
         */
-       function msg( $tag, $val ) {
-               return wfMsg( strtolower("exif-$tag-$val") );
+       function msg( $tag, $val, $arg = null ) {
+               if ($val === '')
+                       $val = 'value';
+               return wfMsg( strtolower( "exif-$tag-$val" ), $arg );
+       }
+
+       /**
+        * Format a number, convert numbers from fractions into floating point
+        * numbers
+        *
+        * @access private
+        *
+        * @param mixed $num The value to format
+        * @return mixed A floating point number or whatever we were fed
+        */
+       function formatNum( $num ) {
+               if ( preg_match( '/^(\d+)\/(\d+)$/', $num, $m ) )
+                       return $m[2] != 0 ? $m[1] / $m[2] : $num;
+               else
+                       return $num;
        }
 }
-} // MEDIAWIKI