Merge pull request #198 from misaochan/on-activity-result-npe

Hotfix for recent crash
This commit is contained in:
Nicolas Raoul 2016-07-20 14:29:23 +09:00 committed by GitHub
commit 2f95a79abd
3 changed files with 13 additions and 4 deletions

View file

@ -1,5 +1,8 @@
# Wikimedia Commons for Android
##v1.20
- Hotfix for data=null crash
##v1.19
- Fixed adapter crash
- Attempt at fixing Google Photos crash

View file

@ -1,7 +1,7 @@
<manifest xmlns:android="http://schemas.android.com/apk/res/android"
package="fr.free.nrw.commons"
android:versionCode="37"
android:versionName="1.19" >
android:versionCode="38"
android:versionName="1.20" >
<uses-permission android:name="android.permission.INTERNET" />
<uses-permission android:name="android.permission.READ_EXTERNAL_STORAGE"/>

View file

@ -92,8 +92,14 @@ public class ContributionsListFragment extends Fragment {
public void onActivityResult(int requestCode, int resultCode, Intent data) {
//FIXME: must get the file data for Google Photos when receive the intent answer, in the onActivityResult method
super.onActivityResult(requestCode, resultCode, data);
Uri imageData = data.getData();
controller.handleImagePicked(requestCode, imageData);
if (data != null) {
Log.d("Contributions", "OnActivityResult() parameters: Result code: " + resultCode + " Data: " + data.toString());
Uri imageData = data.getData();
controller.handleImagePicked(requestCode, imageData);
} else {
Log.d("Contributions", "OnActivityResult() parameters: Result code: " + resultCode + " Data: null");
}
}