You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@weex.apache.org by mi...@apache.org on 2018/02/08 10:57:07 UTC

[2/2] incubator-weex git commit: [WEEX-212][android] When PlaceHolder is empty, should not use rewrite url, it will return bundle url

[WEEX-212][android] When PlaceHolder is empty, should not use rewrite url, it will return bundle url


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

Branch: refs/heads/master
Commit: 0da5171cd681cdf96d9445a577b3a8c966cfa3ee
Parents: 283b81a
Author: jianbai.gbj <ji...@alibaba-inc.com>
Authored: Thu Feb 8 18:02:59 2018 +0800
Committer: jianbai.gbj <ji...@alibaba-inc.com>
Committed: Thu Feb 8 18:02:59 2018 +0800

----------------------------------------------------------------------
 .../main/java/com/taobao/weex/adapter/DefaultUriAdapter.java   | 6 ++++--
 .../java/com/taobao/weex/adapter/DefaultUriAdapterTest.java    | 2 +-
 2 files changed, 5 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-weex/blob/0da5171c/android/sdk/src/main/java/com/taobao/weex/adapter/DefaultUriAdapter.java
----------------------------------------------------------------------
diff --git a/android/sdk/src/main/java/com/taobao/weex/adapter/DefaultUriAdapter.java b/android/sdk/src/main/java/com/taobao/weex/adapter/DefaultUriAdapter.java
index cd1510c..c5cc31a 100644
--- a/android/sdk/src/main/java/com/taobao/weex/adapter/DefaultUriAdapter.java
+++ b/android/sdk/src/main/java/com/taobao/weex/adapter/DefaultUriAdapter.java
@@ -47,8 +47,10 @@ public class DefaultUriAdapter implements URIAdapter {
    if (uri.isRelative()) {
       //When uri is empty, means use the base url instead. Web broswer behave this way.
       if(uri.getEncodedPath().length() == 0){
-        if(TextUtils.isEmpty(uri.toString())){
-          return uri;
+        if(URIAdapter.IMAGE.equals(type)){
+          if(TextUtils.isEmpty(uri.toString())){
+            return uri;
+          }
         }
         return base;
       } else {

http://git-wip-us.apache.org/repos/asf/incubator-weex/blob/0da5171c/android/sdk/src/test/java/com/taobao/weex/adapter/DefaultUriAdapterTest.java
----------------------------------------------------------------------
diff --git a/android/sdk/src/test/java/com/taobao/weex/adapter/DefaultUriAdapterTest.java b/android/sdk/src/test/java/com/taobao/weex/adapter/DefaultUriAdapterTest.java
index 3c4d84b..967199e 100644
--- a/android/sdk/src/test/java/com/taobao/weex/adapter/DefaultUriAdapterTest.java
+++ b/android/sdk/src/test/java/com/taobao/weex/adapter/DefaultUriAdapterTest.java
@@ -110,7 +110,7 @@ public class DefaultUriAdapterTest {
     assertEquals(Uri.parse(host + "/test2"), uri);
 
     uri = adapter.rewrite(instance, URIAdapter.IMAGE, Uri.parse(""));
-    assertEquals(Uri.parse(bundleUrl), uri);
+    assertEquals(Uri.parse(""), uri);
   }
 
 }