Skip to content

Commit

Permalink
fix: add sentry reporting if URI in null when saving file in ScanHist…
Browse files Browse the repository at this point in the history
…oryActivity
  • Loading branch information
VaiTon committed Jul 3, 2021
1 parent ca146a4 commit d1d0f81
Showing 1 changed file with 12 additions and 1 deletion.
Expand Up @@ -10,6 +10,7 @@ import android.os.Build
import android.os.Bundle
import android.os.Environment
import android.provider.Settings
import android.util.Log
import android.view.Menu
import android.view.MenuItem
import android.widget.Toast
Expand All @@ -33,6 +34,7 @@ import openfoodfacts.github.scrachx.openfood.AppFlavors.OFF
import openfoodfacts.github.scrachx.openfood.AppFlavors.isFlavors
import openfoodfacts.github.scrachx.openfood.BuildConfig
import openfoodfacts.github.scrachx.openfood.R
import openfoodfacts.github.scrachx.openfood.analytics.SentryAnalytics
import openfoodfacts.github.scrachx.openfood.databinding.ActivityHistoryScanBinding
import openfoodfacts.github.scrachx.openfood.features.listeners.CommonBottomListenerInstaller.installBottomNavigation
import openfoodfacts.github.scrachx.openfood.features.listeners.CommonBottomListenerInstaller.selectNavigationItem
Expand All @@ -43,6 +45,7 @@ import openfoodfacts.github.scrachx.openfood.network.OpenFoodAPIClient
import openfoodfacts.github.scrachx.openfood.utils.*
import openfoodfacts.github.scrachx.openfood.utils.SortType.*
import java.io.File
import java.io.IOException
import java.text.SimpleDateFormat
import java.util.*
import javax.inject.Inject
Expand Down Expand Up @@ -96,9 +99,17 @@ class ScanHistoryActivity : BaseActivity() {
}
}

@Inject
lateinit var sentryAnalytics: SentryAnalytics

@RequiresApi(Build.VERSION_CODES.KITKAT)
val fileWriterLauncher = registerForActivityResult(CreateCSVContract()) {
writeHistoryToFile(this, adapter.products, it)
if (it == null) {
Log.w(LOG_TAG, "Could not write to file.")
sentryAnalytics.record(IOException("Could not write to file."))
} else {
writeHistoryToFile(this, adapter.products, it)
}
}

override fun onCreate(savedInstanceState: Bundle?) {
Expand Down

0 comments on commit d1d0f81

Please sign in to comment.