From 041c46284b016eedbd730ef9dc1da2e44ae66ae5 Mon Sep 17 00:00:00 2001
From: Caesar2011 <mail@sebse.de>
Date: Thu, 7 Feb 2019 22:47:51 +0100
Subject: [PATCH] NullPointerException and NumberFormatException

---
 .../java/de/sebse/fuplanner/MainActivity.java | 20 +++++++++++--------
 .../fuplanner/fragments/PrefsFragment.java    | 18 +++++++++--------
 2 files changed, 22 insertions(+), 16 deletions(-)

diff --git a/app/src/main/java/de/sebse/fuplanner/MainActivity.java b/app/src/main/java/de/sebse/fuplanner/MainActivity.java
index 4f23c5d..6f34135 100644
--- a/app/src/main/java/de/sebse/fuplanner/MainActivity.java
+++ b/app/src/main/java/de/sebse/fuplanner/MainActivity.java
@@ -500,15 +500,19 @@ public class MainActivity extends AppCompatActivity
             case FRAGMENT_CANTEENS_DETAILS:
                 getCanteenBrowser().getCanteens(success -> {
                     int size = mNavigationView.getMenu().size();
-                    Canteen canteen = success.getCanteen(Integer.parseInt(mFragmentData));
-                    //noinspection ConstantConditions
-                    String title = canteen == null ? null : canteen.getName();
-                    for (int k = 0; k < size; k++) {
-                        MenuItem menuItem = mNavigationView.getMenu().getItem(k);
-                        if (menuItem.getTitle().equals(title)) {
-                            menuItem.setChecked(true);
-                            break;
+                    try {
+                        Canteen canteen = success.getCanteen(Integer.parseInt(mFragmentData));
+                        //noinspection ConstantConditions
+                        String title = canteen == null ? null : canteen.getName();
+                        for (int k = 0; k < size; k++) {
+                            MenuItem menuItem = mNavigationView.getMenu().getItem(k);
+                            if (menuItem.getTitle().equals(title)) {
+                                menuItem.setChecked(true);
+                                break;
+                            }
                         }
+                    } catch (NumberFormatException e) {
+                        e.printStackTrace();
                     }
                 }, log::e);
                 return;
diff --git a/app/src/main/java/de/sebse/fuplanner/fragments/PrefsFragment.java b/app/src/main/java/de/sebse/fuplanner/fragments/PrefsFragment.java
index c4c0b78..fc32565 100644
--- a/app/src/main/java/de/sebse/fuplanner/fragments/PrefsFragment.java
+++ b/app/src/main/java/de/sebse/fuplanner/fragments/PrefsFragment.java
@@ -53,14 +53,16 @@ public class PrefsFragment extends PreferenceFragmentCompat implements SharedPre
 
         if (getActivity() != null && getActivity() instanceof MainActivity) {
             CustomAccountManager accountManager = ((MainActivity) getActivity()).getAccountManager();
-            Account accountByType = accountManager.getAccountByType(AccountGeneral.ACCOUNT_TYPE);
-            if (accountByType != null) {
-                ContentResolver.setSyncAutomatically(accountByType, KVVContentProvider.PROVIDER_NAME, true);
-                ContentResolver.addPeriodicSync(
-                    accountByType,
-                    KVVContentProvider.PROVIDER_NAME,
-                    Bundle.EMPTY,
-                    Long.parseLong(Preferences.getStringArray(getActivity(), R.array.pref_sync_frequency)));
+            if (accountManager != null) {
+                Account accountByType = accountManager.getAccountByType(AccountGeneral.ACCOUNT_TYPE);
+                if (accountByType != null) {
+                    ContentResolver.setSyncAutomatically(accountByType, KVVContentProvider.PROVIDER_NAME, true);
+                    ContentResolver.addPeriodicSync(
+                            accountByType,
+                            KVVContentProvider.PROVIDER_NAME,
+                            Bundle.EMPTY,
+                            Long.parseLong(Preferences.getStringArray(getActivity(), R.array.pref_sync_frequency)));
+                }
             }
         }
     }