replaced icafe with metadata extractor

Author: vanshikaarora

app/build.gradle

@@ -90,6 +90,8 @@ dependencies {
     //swipe_layout
     implementation 'com.daimajia.swipelayout:library:1.2.0@aar'
     implementation 'com.nineoldandroids:library:2.4.0'
+    //metadata extractor
+    implementation 'com.drewnoakes:metadata-extractor:2.11.0'
     implementation files('libs/icafe-1.1-SNAPSHOT.jar')
 }
 

app/libs/icafe-1.1-SNAPSHOT.jar

@@ -1,5 +1,8 @@
 package fr.free.nrw.commons.upload;
 
+import android.content.Context;
+import android.net.Uri;
+
 import java.io.IOException;
 
 import javax.inject.Inject;
@@ -20,7 +23,10 @@
 
 /**
  * Methods for pre-processing images to be uploaded
- */
+ *//*if (dataInBytes[0] == 70 && dataInBytes[1] == 66 && dataInBytes[2] == 77 && dataInBytes[3] == 68) {
+                Timber.d("Contains FBMD");
+                return Single.just(ImageUtils.FILE_FBMD);
+            }*/
 @Singleton
 public class ImageProcessingService {
     private final FileUtilsWrapper fileUtilsWrapper;
@@ -56,11 +62,13 @@ Single<Integer> validateImage(UploadModel.UploadItem uploadItem, boolean checkTi
         }
         Timber.d("Checking the validity of image");
         String filePath = uploadItem.getMediaUri().getPath();
+        Uri contentUri=uploadItem.getContentUri();
+        Context context=uploadItem.getContext();
         Single<Integer> duplicateImage = checkDuplicateImage(filePath);
         Single<Integer> wrongGeoLocation = checkImageGeoLocation(uploadItem.getPlace(), filePath);
         Single<Integer> darkImage = checkDarkImage(filePath);
         Single<Integer> itemTitle = checkTitle ? validateItemTitle(uploadItem) : Single.just(ImageUtils.IMAGE_OK);
-        Single<Integer> checkFBMD = checkFBMD(filePath);
+        Single<Integer> checkFBMD = checkFBMD(context,contentUri);
 
         Single<Integer> zipResult = Single.zip(duplicateImage, wrongGeoLocation, darkImage, itemTitle,
                 (duplicate, wrongGeo, dark, title) -> {
@@ -84,9 +92,9 @@ Single<Integer> validateImage(UploadModel.UploadItem uploadItem, boolean checkTi
      * Thus we successfully protect common's from Facebook's copyright violation
      */
 
-    public Single<Integer> checkFBMD(String filePath) {
+    public Single<Integer> checkFBMD(Context context,Uri contentUri) {
         try {
-            return readFBMD.processMetadata(filePath);
+            return readFBMD.processMetadata(context,contentUri);
         } catch (IOException e) {
             return Single.just(ImageUtils.FILE_FBMD);
         }

app/src/main/java/fr/free/nrw/commons/upload/ImageProcessingService.java

@@ -1,43 +1,54 @@
 package fr.free.nrw.commons.upload;
 
-import com.icafe4j.image.meta.Metadata;
-import com.icafe4j.image.meta.MetadataType;
-import com.icafe4j.image.meta.iptc.IPTC;
+import android.content.Context;
+import android.net.Uri;
+
+import com.drew.imaging.ImageMetadataReader;
+import com.drew.imaging.ImageProcessingException;
+import com.drew.metadata.Metadata;
+import com.drew.metadata.Tag;
+import com.drew.metadata.iptc.IptcDirectory;
 
 import java.io.IOException;
-import java.util.Map;
 
 import javax.inject.Inject;
 import javax.inject.Singleton;
 
 import fr.free.nrw.commons.utils.ImageUtils;
 import io.reactivex.Single;
-import timber.log.Timber;
 
 @Singleton
 public class ReadFBMD {
 
     @Inject
-    public ReadFBMD(){
+    public ReadFBMD() {
 
     }
-    public Single<Integer> processMetadata(String path) throws IOException {
-        Map<MetadataType, Metadata> metadataMap = Metadata.readMetadata(path);
-        Metadata metadata = metadataMap.get(MetadataType.IPTC);
-        byte[] dataInBytes = new byte[0];
+
+    public Single<Integer> processMetadata(Context context, Uri contentUri) throws IOException {
+        Metadata readMetadata = null;
         try {
-            dataInBytes = ((IPTC) metadata).getDataSets().get("SpecialInstructions").get(0).getData();
-        } catch (NullPointerException e) {
+            readMetadata = ImageMetadataReader.readMetadata(context.getContentResolver().openInputStream(contentUri));
+        } catch (ImageProcessingException e) {
+            e.printStackTrace();
+        } catch (IOException e) {
+            e.printStackTrace();
+        }
+
+        IptcDirectory iptcDirectory = readMetadata != null ? readMetadata.getFirstDirectoryOfType(IptcDirectory.class) : null;
+        if (iptcDirectory == null) {
             return Single.just(ImageUtils.IMAGE_OK);
         }
         /**
-        * The byte array so obtained is used is tested to contain FBMD data
-        * Note: Any image downloaded from Facebook contains the ASCII code of the letters 'FBMD' in this bytecode extracted from it's IPTC metadata
-        * */
-            if (dataInBytes[0] == 70 && dataInBytes[1] == 66 && dataInBytes[2] == 77 && dataInBytes[3] == 68) {
-                Timber.d("Contains FBMD");
+         * We parse through all the tags in the IPTC directory  if the tagname equals "Special Instructions".
+         * And the description string starts with FBMD.
+         * Then the source of image is facebook
+         * */
+        for (Tag tag : iptcDirectory.getTags()) {
+            if (tag.getTagName().equals("Special Instructions") && tag.getDescription().substring(0, 4).equals("FBMD")) {
                 return Single.just(ImageUtils.FILE_FBMD);
             }
+        }
         return Single.just(ImageUtils.IMAGE_OK);
     }
 }

app/src/main/java/fr/free/nrw/commons/upload/ReadFBMD.java

@@ -1,8 +1,11 @@
 package fr.free.nrw.commons.upload;
 
 import android.annotation.SuppressLint;
+import android.content.ContentValues;
 import android.content.Context;
+import android.database.Cursor;
 import android.net.Uri;
+import android.provider.MediaStore;
 import android.support.annotation.NonNull;
 import android.support.annotation.Nullable;
 
@@ -54,7 +57,7 @@ public class UploadModel {
     private boolean bottomCardState = true;
     private boolean rightCardState = true;
     private int currentStepIndex = 0;
-    private Context context;
+    public static Context context;
     private Disposable badImageSubscription;
 
     private SessionManager sessionManager;
@@ -428,6 +431,34 @@ public String getFileName() {
         public Place getPlace() {
             return place;
         }
+
+        public Uri getContentUri() {
+            return generateContentUri(UploadModel.context,getMediaUri().getPath());
+        }
+        private Uri generateContentUri(Context context,String path){
+            Cursor cursor = context.getContentResolver().query(
+                    MediaStore.Images.Media.EXTERNAL_CONTENT_URI
+                    , new String[] { MediaStore.Images.Media._ID }
+                    , MediaStore.Images.Media.DATA + "=? "
+                    , new String[] { path }, null);
+
+            if (cursor != null && cursor.moveToFirst()) {
+                int id = cursor.getInt(cursor.getColumnIndex(MediaStore.MediaColumns._ID));
+                return Uri.withAppendedPath(MediaStore.Images.Media.EXTERNAL_CONTENT_URI , Integer.toString(id));
+
+            } else if (!path.isEmpty()) {
+                ContentValues values = new ContentValues();
+                values.put(MediaStore.Images.Media.DATA, path);
+                return context.getContentResolver().insert(
+                        MediaStore.Images.Media.EXTERNAL_CONTENT_URI, values);
+            } else {
+                return null;
+            }
+        }
+
+        public Context getContext(){
+            return UploadModel.context;
+        }
     }
 
 }