firetweet
firetweet copied to clipboard
getlantern
SubsamplingScaleImage Exception
Reported by Crashlytics
java.lang.NullPointerException: imageSource must not be null
at com.davemorrissey.labs.subscaleview.SubsamplingScaleImageView.setImage(SubsamplingScaleImageView.java:346)
at com.davemorrissey.labs.subscaleview.SubsamplingScaleImageView.setImage(SubsamplingScaleImageView.java:303)
at org.getlantern.firetweet.activity.support.MediaViewerActivity$ImagePageFragment.onLoadFinished(MediaViewerActivity.java:414)
at org.getlantern.firetweet.activity.support.MediaViewerActivity$ImagePageFragment.onLoadFinished(MediaViewerActivity.java:354)
at android.support.v4.app.LoaderManagerImpl$LoaderInfo.callOnLoadFinished(LoaderManager.java:427)
at android.support.v4.app.LoaderManagerImpl$LoaderInfo.onLoadComplete(LoaderManager.java:395)
at android.support.v4.content.Loader.deliverResult(Loader.java:104)
at android.support.v4.content.AsyncTaskLoader.dispatchOnLoadComplete(AsyncTaskLoader.java:223)
at android.support.v4.content.AsyncTaskLoader$LoadTask.onPostExecute(AsyncTaskLoader.java:61)
at android.support.v4.content.ModernAsyncTask.finish(ModernAsyncTask.java:461)
at android.support.v4.content.ModernAsyncTask.access$500(ModernAsyncTask.java:47)
at android.support.v4.content.ModernAsyncTask$InternalHandler.handleMessage(ModernAsyncTask.java:474)
at android.os.Handler.dispatchMessage(Handler.java:102)
at android.os.Looper.loop(Looper.java:136)
at android.app.ActivityThread.main(ActivityThread.java:5032)
at java.lang.reflect.Method.invokeNative(Method.java)
at java.lang.reflect.Method.invoke(Method.java:515)
at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:785)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:601)
at dalvik.system.NativeStart.main(NativeStart.java)
Awesome, thanks for investigating this further. It looks like we're actually setting the image to null in the code in this scenario too (line 414 in MediaViewerActivity.java). This doesn't actually cause the app to crash but just shows an error message, correct? If so, that's probably the right behavior then.
