解决意见反馈Android7.0机型点击图片崩溃问题

raee · raee · commit ff1c26be1ce1 · 2017-09-12T00:33:13.000+08:00
diff --git a/app/src/main/java/com/rae/cnblogs/AppRoute.java b/app/src/main/java/com/rae/cnblogs/AppRoute.java
@@ -192,6 +192,18 @@ public static void jumpToImagePreview(Context context, @NonNull ArrayList<String
         startActivity(context, intent);
     }
 
+
+    /**
+     * 图片查看
+     *
+     * @param imgUrl 图片路径
+     */
+    public static void jumpToImagePreview(Context context, @NonNull String imgUrl) {
+        ArrayList<String> data = new ArrayList<>();
+        data.add(imgUrl);
+        jumpToImagePreview(context, data, 0);
+    }
+
     /**
      * 博主界面
      *
diff --git a/app/src/main/java/com/rae/cnblogs/activity/FeedbackActivity.java b/app/src/main/java/com/rae/cnblogs/activity/FeedbackActivity.java
@@ -47,6 +47,8 @@
 import com.avos.avoscloud.feedback.FeedbackThread;
 import com.avos.avoscloud.feedback.FeedbackThread.SyncCallback;
 import com.avos.avoscloud.feedback.Resources;
+import com.rae.cnblogs.AppRoute;
+import com.rae.cnblogs.AppUI;
 import com.rae.cnblogs.R;
 import com.rae.cnblogs.sdk.UserProvider;
 import com.rae.cnblogs.sdk.bean.UserInfoBean;
@@ -594,11 +596,14 @@ public View getView(int position, View convertView, ViewGroup parent) {
 
                     @Override
                     public void onClick(View v) {
-                        Intent intent = new Intent();
-                        intent.setAction(Intent.ACTION_VIEW);
-                        intent.setDataAndType(
-                                Uri.fromFile(ImageCache.getCacheFile(comment.getAttachment().getUrl())), "image/*");
-                        startActivity(intent);
+                        // fix bugly #432 #422 #412 #382
+                        File file = ImageCache.getCacheFile(comment.getAttachment().getUrl());
+                        if (!file.exists()) {
+                            AppUI.failed(getContext(), "图片不存在！");
+                            return;
+                        }
+
+                        AppRoute.jumpToImagePreview(getContext(), file.getPath());
                     }
                 };
                 Bitmap attachmentCache = cache.getImage(comment.getAttachment().getUrl());
diff --git a/app/src/main/java/com/rae/cnblogs/presenter/impl/blog/BlogContentPresenterImpl.java b/app/src/main/java/com/rae/cnblogs/presenter/impl/blog/BlogContentPresenterImpl.java
@@ -81,11 +81,14 @@ public ObservableSource<String> apply(UserBlogInfo blogInfo) throws Exception {
                 .subscribe(new ApiDefaultObserver<String>() {
                     @Override
                     protected void onError(String message) {
+                        // fix bugly #374
+                        if (mView == null) return;
                         mView.onLoadContentFailed(message);
                     }
 
                     @Override
                     protected void accept(String content) {
+                        if (mView == null) return;
                         mView.getBlog().setContent(content);
                         mView.onLoadContentSuccess(content);
 
