Print warning if redundant state is set in Asset.Builder

Author: Steven-Chan

skygear/src/main/java/io/skygear/skygear/Asset.java

@@ -21,6 +21,7 @@
 import android.content.Context;
 import android.net.Uri;
 import android.os.ParcelFileDescriptor;
+import android.util.Log;
 
 import org.json.JSONException;
 import org.json.JSONObject;
@@ -33,6 +34,7 @@
  * The Skygear Asset Model.
  */
 public class Asset {
+    private static final String TAG = "Skygear SDK";
 
     /**
      * The Name.
@@ -248,6 +250,7 @@ private Asset buildFromInputStream() {
 
         private Asset buildFromByteArray() {
             ensureStateIsNonNull(this.mimeType, "MimeType");
+            warnRedundantStateIfNeeded(this.size, "Size", "data");
 
             long size = this.data.length;
             InputStream inputStream = new ByteArrayInputStream(this.data);
@@ -257,6 +260,8 @@ private Asset buildFromByteArray() {
 
         private Asset buildFromUri() {
             ensureStateIsNonNull(this.context, "Context");
+            warnRedundantStateIfNeeded(this.mimeType, "MimeType", "uri");
+            warnRedundantStateIfNeeded(this.size, "Size", "uri");
 
             ContentResolver contentResolver = context.getContentResolver();
             InputStream inputStream;
@@ -280,5 +285,11 @@ private static void ensureStateIsNonNull(Object state, String name) {
                 throw new IllegalStateException(name + " must be set first");
             }
         }
+
+        private static void warnRedundantStateIfNeeded(Object state, String name, String context) {
+            if (state != null) {
+                Log.w(TAG, name + " would be ignored when using " + context + " as data source");
+            }
+        }
     }
 }