Repository: incubator-weex
Updated Branches:
  refs/heads/0.15-dev 571b735ac -> e4ba34646


* [android] Add RAW and None for imageQuality


Project: http://git-wip-us.apache.org/repos/asf/incubator-weex/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-weex/commit/2c248325
Tree: http://git-wip-us.apache.org/repos/asf/incubator-weex/tree/2c248325
Diff: http://git-wip-us.apache.org/repos/asf/incubator-weex/diff/2c248325

Branch: refs/heads/0.15-dev
Commit: 2c248325d808cf3780c96a2d1dafbe2cd4a86038
Parents: b549a22
Author: YorkShen <shenyua...@gmail.com>
Authored: Wed Jun 14 17:39:42 2017 +0800
Committer: YorkShen <shenyua...@gmail.com>
Committed: Mon Jun 26 17:14:46 2017 +0800

----------------------------------------------------------------------
 .../java/com/taobao/weex/common/Constants.java  |  1 +
 .../com/taobao/weex/common/WXImageStrategy.java |  9 ++++-
 .../main/java/com/taobao/weex/dom/WXAttr.java   | 38 ++++++++------------
 .../com/taobao/weex/dom/WXImageQuality.java     |  6 +++-
 4 files changed, 28 insertions(+), 26 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-weex/blob/2c248325/android/sdk/src/main/java/com/taobao/weex/common/Constants.java
----------------------------------------------------------------------
diff --git a/android/sdk/src/main/java/com/taobao/weex/common/Constants.java 
b/android/sdk/src/main/java/com/taobao/weex/common/Constants.java
index f683c99..3da084c 100644
--- a/android/sdk/src/main/java/com/taobao/weex/common/Constants.java
+++ b/android/sdk/src/main/java/com/taobao/weex/common/Constants.java
@@ -199,6 +199,7 @@ public class Constants {
     String LOW = "low";
     String NORMAL = "normal";
     String HIGH = "high";
+    String RAW = "raw";
     String VISIBLE = "visible";
     String HIDDEN = "hidden";
     String TEXT = "text";

http://git-wip-us.apache.org/repos/asf/incubator-weex/blob/2c248325/android/sdk/src/main/java/com/taobao/weex/common/WXImageStrategy.java
----------------------------------------------------------------------
diff --git 
a/android/sdk/src/main/java/com/taobao/weex/common/WXImageStrategy.java 
b/android/sdk/src/main/java/com/taobao/weex/common/WXImageStrategy.java
index 74d06b0..994c99b 100644
--- a/android/sdk/src/main/java/com/taobao/weex/common/WXImageStrategy.java
+++ b/android/sdk/src/main/java/com/taobao/weex/common/WXImageStrategy.java
@@ -24,9 +24,16 @@ import java.util.Map;
 
 public class WXImageStrategy {
 
+
   /**
-   * Whether to clip image. The default value is false.
+   * <strong>Never!</strong>
+   * <strong>Never!</strong>
+   * <strong>Never!</strong>
+   * Never use this flag, ImageView has done all the job of clipping!
+   * There is no method to read this flag any more.
+   * This field will be removed when it's appropriate.
    */
+  @Deprecated
   public boolean isClipping;
 
   /**

http://git-wip-us.apache.org/repos/asf/incubator-weex/blob/2c248325/android/sdk/src/main/java/com/taobao/weex/dom/WXAttr.java
----------------------------------------------------------------------
diff --git a/android/sdk/src/main/java/com/taobao/weex/dom/WXAttr.java 
b/android/sdk/src/main/java/com/taobao/weex/dom/WXAttr.java
index 05e1626..d143dac 100644
--- a/android/sdk/src/main/java/com/taobao/weex/dom/WXAttr.java
+++ b/android/sdk/src/main/java/com/taobao/weex/dom/WXAttr.java
@@ -18,23 +18,23 @@
  */
 package com.taobao.weex.dom;
 
+import static java.lang.Boolean.parseBoolean;
+
 import android.support.annotation.NonNull;
 import android.support.v4.util.ArrayMap;
 import android.text.TextUtils;
-
 import com.taobao.weex.common.Constants;
+import com.taobao.weex.common.Constants.Name;
 import com.taobao.weex.common.WXImageSharpen;
 import com.taobao.weex.ui.view.listview.WXRecyclerView;
 import com.taobao.weex.utils.WXLogUtils;
 import com.taobao.weex.utils.WXUtils;
 import com.taobao.weex.utils.WXViewUtils;
-
 import java.util.Collection;
+import java.util.Locale;
 import java.util.Map;
 import java.util.Set;
 
-import static java.lang.Boolean.parseBoolean;
-
 /**
  * store value of component attribute
  *
@@ -95,27 +95,17 @@ public class WXAttr implements Map<String, 
Object>,Cloneable {
   }
 
   public WXImageQuality getImageQuality() {
-
-    Object obj = get(Constants.Name.QUALITY);
-    if (obj == null) {
-      obj = get(Constants.Name.IMAGE_QUALITY);
-    }
-    if (obj == null) {
-      return WXImageQuality.LOW;
-    }
-    WXImageQuality waImageQuality = WXImageQuality.LOW;
-    String imageQuality = obj.toString();
-    if (imageQuality.equals(Constants.Value.ORIGINAL)) {
-      waImageQuality = WXImageQuality.ORIGINAL;
-    } else if (imageQuality.equals(Constants.Value.LOW)) {
-      waImageQuality = WXImageQuality.LOW;
-    } else if (imageQuality.equals(Constants.Value.NORMAL)) {
-      waImageQuality = WXImageQuality.NORMAL;
-    } else if (imageQuality.equals(Constants.Value.HIGH)) {
-      waImageQuality = WXImageQuality.HIGH;
+    Object obj = containsKey(Name.QUALITY) ? get(Name.QUALITY) : 
get(Name.IMAGE_QUALITY);
+    WXImageQuality imageQuality = WXImageQuality.NONE;
+    String value;
+    if (obj != null && !TextUtils.isEmpty(value = obj.toString())) {
+      try {
+        imageQuality = WXImageQuality.valueOf(value.toUpperCase(Locale.US));
+      }catch (IllegalArgumentException e){
+        WXLogUtils.e("Image", "Invalid value image quality. Only low, normal, 
high, original are valid");
+      }
     }
-
-    return waImageQuality;
+    return imageQuality;
   }
 
   public WXImageSharpen getImageSharpen() {

http://git-wip-us.apache.org/repos/asf/incubator-weex/blob/2c248325/android/sdk/src/main/java/com/taobao/weex/dom/WXImageQuality.java
----------------------------------------------------------------------
diff --git a/android/sdk/src/main/java/com/taobao/weex/dom/WXImageQuality.java 
b/android/sdk/src/main/java/com/taobao/weex/dom/WXImageQuality.java
index 20771d9..488f84a 100644
--- a/android/sdk/src/main/java/com/taobao/weex/dom/WXImageQuality.java
+++ b/android/sdk/src/main/java/com/taobao/weex/dom/WXImageQuality.java
@@ -26,5 +26,9 @@ public enum WXImageQuality {
 
   NORMAL,
 
-  HIGH
+  HIGH,
+
+  RAW,
+
+  NONE
 }

Reply via email to