Version Bump

pennlabs · Sep 9, 2024 · b0b1ab9 · b0b1ab9
1 parent 8469628
commit b0b1ab9
Show file tree

Hide file tree

Showing 2 changed files with 30 additions and 22 deletions.
diff --git a/PennMobile/src/main/java/com/pennapps/labs/pennmobile/GsrFragment.kt b/PennMobile/src/main/java/com/pennapps/labs/pennmobile/GsrFragment.kt
@@ -2,6 +2,7 @@ package com.pennapps.labs.pennmobile
 
 import android.app.DatePickerDialog
 import android.app.TimePickerDialog
+import android.content.Context
 import android.content.SharedPreferences
 import android.os.Bundle
 import android.util.Log
@@ -242,27 +243,29 @@ class GsrFragment : Fragment() {
 
     private fun updateStatus() {
         mActivity.mNetworkManager.getAccessToken {
-            val bearerToken =
-                sharedPreferences.getString(getString(R.string.access_token), "").toString()
+            checkIfFragmentAttached {
+                val bearerToken =
+                    sharedPreferences.getString(getString(R.string.access_token), "").toString()
 
-            if (bearerToken.isEmpty()) {
-                Toast.makeText(activity, "You are not logged in!", Toast.LENGTH_LONG).show()
-            } else {
-                try {
-                    mStudentLife
-                        .isWharton(
-                            "Bearer $bearerToken",
-                        )?.subscribe(
-                            { status ->
-                                isWharton = status.isWharton
-                            },
-                            {
-                                Log.e("GsrFragment", "Error getting Wharton status", it)
-                                isWharton = false
-                            },
-                        )
-                } catch (e: Exception) {
-                    e.printStackTrace()
+                if (bearerToken.isEmpty()) {
+                    Toast.makeText(activity, "You are not logged in!", Toast.LENGTH_LONG).show()
+                } else {
+                    try {
+                        mStudentLife
+                            .isWharton(
+                                "Bearer $bearerToken",
+                            )?.subscribe(
+                                { status ->
+                                    isWharton = status.isWharton
+                                },
+                                {
+                                    Log.e("GsrFragment", "Error getting Wharton status", it)
+                                    isWharton = false
+                                },
+                            )
+                    } catch (e: Exception) {
+                        e.printStackTrace()
+                    }
                 }
             }
         }
@@ -681,4 +684,9 @@ class GsrFragment : Fragment() {
             mGSRS.add(newGSRObject)
         }
     }
+    fun checkIfFragmentAttached(operation: Context.() -> Unit) {
+        if (isAdded && context != null) {
+            operation(requireContext())
+        }
+    }
 }
diff --git a/PennMobile/src/main/java/com/pennapps/labs/pennmobile/MenuFragment.kt b/PennMobile/src/main/java/com/pennapps/labs/pennmobile/MenuFragment.kt
@@ -88,9 +88,7 @@ class MenuFragment : Fragment() {
         super.onCreate(savedInstanceState)
         mDiningHall = arguments?.getParcelable("DiningHall")
         mActivity = activity as MainActivity
-        mActivity.toolbar.visibility = View.VISIBLE
         setHasOptionsMenu(true)
-        mActivity.hideBottomBar()
     }
 
     override fun onCreateView(
@@ -105,6 +103,8 @@ class MenuFragment : Fragment() {
         pager.adapter = pageAdapter
         v.setBackgroundColor(Color.WHITE)
         mActivity.addTabs(pageAdapter as TabAdapter, pager, true)
+        mActivity.toolbar.visibility = View.VISIBLE
+        mActivity.hideBottomBar()
         return v
     }