Declare visibility on class properties of Exif

Change-Id: Ia729bd114e1d48864aa0a37f5a5ba13e6fa5a313

diff --git a/includes/media/Exif.php b/includes/media/Exif.php
index bc55000..7593a16 100644 (file)
--- a/includes/media/Exif.php
+++ b/includes/media/Exif.php
@@ -66,65 +66,38 @@ class Exif {
        /** A fake value for things we don't want or don't support. */
        const IGNORE = -1;
 
-       //@{
-       /* @var array
-        * @private
+       /** @var array 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 separated by commas.
         */
+       private $mExifTags;
 
-       /**
-        * 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
-        * separated by commas.
-        */
-       var $mExifTags;
+       /** @var array The raw Exif data returned by exif_read_data() */
+       private $mRawExifData;
 
-       /**
-        * The raw Exif data returned by exif_read_data()
+       /** @var array 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 $mRawExifData;
+       private $mFilteredExifData;
 
-       /**
-        * 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;
+       /** @var array Filtered and formatted Exif data, see FormatMetadata::getFormattedData() */
+       private $mFormattedExifData;
 
-       /**
-        * Filtered and formatted Exif data, see FormatMetadata::getFormattedData()
-        */
-       var $mFormattedExifData;
+       /** @var string The file being processed */
+       private $file;
 
-       //@}
+       /** @var string The basename of the file being processed */
+       private $basename;
 
-       //@{
-       /* @var string
-        * @private
-        */
+       /** @var string The private log to log to, e.g. 'exif' */
+       private $log = false;
 
-       /**
-        * The file being processed
-        */
-       var $file;
-
-       /**
-        * The basename of the file being processed
-        */
-       var $basename;
-
-       /**
-        * The private log to log to, e.g. 'exif'
-        */
-       var $log = false;
-
-       /**
-        * The byte order of the file. Needed because php's
-        * extension doesn't fully process some obscure props.
+       /** @var string The byte order of the file. Needed because php's extension
+        *    doesn't fully process some obscure props.
         */
        private $byteOrder;
 
-       //@}
-
        /**
         * Constructor
         *