From f788f0d9b00ef305d2a7455df234bcfe8bce4ae7 Mon Sep 17 00:00:00 2001
From: Miguel Beltran <m@beltran.work>
Date: Thu, 16 Jan 2025 10:42:13 +0100
Subject: [PATCH 1/3] use parenthesis

---
 app/build.gradle | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/build.gradle b/app/build.gradle
index f40eaae..d850dee 100644
--- a/app/build.gradle
+++ b/app/build.gradle
@@ -142,7 +142,7 @@ dependencies {
     implementation(libs.androidx.constraintlayout.constraintlayout)
     implementation(libs.android.material.material)
     implementation(libs.androidx.multidex.multidex)
-    implementation libs.androidx.activity
+    implementation(libs.androidx.activity)
 
     testImplementation(libs.junit)
     testImplementation(libs.mockito.core)

From ada6582ac3c82131a49a1fa60b598a76c4c5f6cb Mon Sep 17 00:00:00 2001
From: Miguel Beltran <m@beltran.work>
Date: Thu, 16 Jan 2025 11:38:21 +0100
Subject: [PATCH 2/3] format

---
 app/src/main/AndroidManifest.xml              |  4 +-
 .../raygun4android/sample/MainActivity.kt     |  7 +++
 .../sample/fragments/FragmentSwap1.kt         | 26 ++++++++
 .../sample/fragments/FragmentSwap2.kt         | 26 ++++++++
 .../NavigationFragmentSwapActivity.kt         | 28 +++++++++
 app/src/main/res/layout/activity_main.xml     | 11 ++++
 .../activity_navigation_fragment_swap.xml     | 20 ++++++
 app/src/main/res/layout/fragment_swap1.xml    | 30 +++++++++
 app/src/main/res/layout/fragment_swap2.xml    | 30 +++++++++
 app/src/main/res/values/strings.xml           |  8 +++
 .../raygun4android/rum/RUMFragment.java       | 62 ++++++++++++++-----
 11 files changed, 234 insertions(+), 18 deletions(-)
 create mode 100644 app/src/main/java/com/raygun/raygun4android/sample/fragments/FragmentSwap1.kt
 create mode 100644 app/src/main/java/com/raygun/raygun4android/sample/fragments/FragmentSwap2.kt
 create mode 100644 app/src/main/java/com/raygun/raygun4android/sample/fragments/NavigationFragmentSwapActivity.kt
 create mode 100644 app/src/main/res/layout/activity_navigation_fragment_swap.xml
 create mode 100644 app/src/main/res/layout/fragment_swap1.xml
 create mode 100644 app/src/main/res/layout/fragment_swap2.xml

diff --git a/app/src/main/AndroidManifest.xml b/app/src/main/AndroidManifest.xml
index c9367fb..646f3ea 100644
--- a/app/src/main/AndroidManifest.xml
+++ b/app/src/main/AndroidManifest.xml
@@ -25,7 +25,9 @@
         <activity
             android:name=".fragments.NavigationActivity"
             android:parentActivityName=".MainActivity" />
-
+        <activity
+            android:name=".fragments.NavigationFragmentSwapActivity"
+            android:parentActivityName=".MainActivity" />
         <meta-data
             android:name="com.raygun.raygun4android.apikey"
             android:value="YOUR_API_KEY_HERE" />
diff --git a/app/src/main/java/com/raygun/raygun4android/sample/MainActivity.kt b/app/src/main/java/com/raygun/raygun4android/sample/MainActivity.kt
index cd70343..9fae5af 100644
--- a/app/src/main/java/com/raygun/raygun4android/sample/MainActivity.kt
+++ b/app/src/main/java/com/raygun/raygun4android/sample/MainActivity.kt
@@ -12,6 +12,7 @@ import com.google.android.material.snackbar.Snackbar
 import com.raygun.raygun4android.RaygunClient
 import com.raygun.raygun4android.messages.shared.RaygunUserInfo
 import com.raygun.raygun4android.sample.fragments.NavigationActivity
+import com.raygun.raygun4android.sample.fragments.NavigationFragmentSwapActivity
 
 class MainActivity : AppCompatActivity() {
     override fun onCreate(savedInstanceState: Bundle?) {
@@ -41,6 +42,7 @@ class MainActivity : AppCompatActivity() {
         val textViewProviderVersion = findViewById<TextView>(R.id.textView_providerVersion)
         val buttonSecondActivity = findViewById<Button>(R.id.button_secondActivity)
         val buttonNavigationActivity = findViewById<Button>(R.id.button_navigationActivity)
+        val buttonFragmentSwapActivity = findViewById<Button>(R.id.button_navigationFragmentSwapActivity)
 
         buttonSend.setOnClickListener {
             val tw = HashMap<String, String>()
@@ -149,6 +151,11 @@ class MainActivity : AppCompatActivity() {
             this.startActivity(NavigationActivity.getIntent(this))
         }
 
+        buttonFragmentSwapActivity.setOnClickListener {
+            // Launches an Activity containing a Fragment
+            this.startActivity(NavigationFragmentSwapActivity.getIntent(this))
+        }
+
         textViewAppVersion.text =
             getString(
                 R.string.app_version_text,
diff --git a/app/src/main/java/com/raygun/raygun4android/sample/fragments/FragmentSwap1.kt b/app/src/main/java/com/raygun/raygun4android/sample/fragments/FragmentSwap1.kt
new file mode 100644
index 0000000..f225608
--- /dev/null
+++ b/app/src/main/java/com/raygun/raygun4android/sample/fragments/FragmentSwap1.kt
@@ -0,0 +1,26 @@
+package com.raygun.raygun4android.sample.fragments
+
+import android.os.Bundle
+import android.view.LayoutInflater
+import android.view.View
+import android.view.ViewGroup
+import android.widget.Button
+import androidx.fragment.app.Fragment
+import com.raygun.raygun4android.sample.R
+
+class FragmentSwap1 : Fragment() {
+    override fun onCreateView(
+        inflater: LayoutInflater,
+        container: ViewGroup?,
+        savedInstanceState: Bundle?,
+    ): View? {
+        val view = inflater.inflate(R.layout.fragment_swap1, container, false)
+        view.findViewById<Button>(R.id.button).setOnClickListener {
+            parentFragmentManager.beginTransaction().apply {
+                replace(R.id.fragment_container_view, FragmentSwap2())
+                commit()
+            }
+        }
+        return view
+    }
+}
diff --git a/app/src/main/java/com/raygun/raygun4android/sample/fragments/FragmentSwap2.kt b/app/src/main/java/com/raygun/raygun4android/sample/fragments/FragmentSwap2.kt
new file mode 100644
index 0000000..9bb150d
--- /dev/null
+++ b/app/src/main/java/com/raygun/raygun4android/sample/fragments/FragmentSwap2.kt
@@ -0,0 +1,26 @@
+package com.raygun.raygun4android.sample.fragments
+
+import android.os.Bundle
+import android.view.LayoutInflater
+import android.view.View
+import android.view.ViewGroup
+import android.widget.Button
+import androidx.fragment.app.Fragment
+import com.raygun.raygun4android.sample.R
+
+class FragmentSwap2 : Fragment() {
+    override fun onCreateView(
+        inflater: LayoutInflater,
+        container: ViewGroup?,
+        savedInstanceState: Bundle?,
+    ): View? {
+        val view = inflater.inflate(R.layout.fragment_swap2, container, false)
+        view.findViewById<Button>(R.id.button).setOnClickListener {
+            parentFragmentManager.beginTransaction().apply {
+                replace(R.id.fragment_container_view, FragmentSwap1())
+                commit()
+            }
+        }
+        return view
+    }
+}
diff --git a/app/src/main/java/com/raygun/raygun4android/sample/fragments/NavigationFragmentSwapActivity.kt b/app/src/main/java/com/raygun/raygun4android/sample/fragments/NavigationFragmentSwapActivity.kt
new file mode 100644
index 0000000..ce9faee
--- /dev/null
+++ b/app/src/main/java/com/raygun/raygun4android/sample/fragments/NavigationFragmentSwapActivity.kt
@@ -0,0 +1,28 @@
+package com.raygun.raygun4android.sample.fragments
+
+import android.content.Context
+import android.content.Intent
+import android.os.Bundle
+import androidx.activity.enableEdgeToEdge
+import androidx.appcompat.app.AppCompatActivity
+import androidx.core.view.ViewCompat
+import androidx.core.view.WindowInsetsCompat
+import com.raygun.raygun4android.sample.R
+
+class NavigationFragmentSwapActivity : AppCompatActivity() {
+    override fun onCreate(savedInstanceState: Bundle?) {
+        super.onCreate(savedInstanceState)
+        enableEdgeToEdge()
+        setContentView(R.layout.activity_navigation_fragment_swap)
+        setTitle(R.string.fragment_swap_text)
+        ViewCompat.setOnApplyWindowInsetsListener(findViewById(R.id.main)) { v, insets ->
+            val systemBars = insets.getInsets(WindowInsetsCompat.Type.systemBars())
+            v.setPadding(systemBars.left, systemBars.top, systemBars.right, systemBars.bottom)
+            insets
+        }
+    }
+
+    companion object {
+        fun getIntent(context: Context): Intent = Intent(context, NavigationFragmentSwapActivity::class.java)
+    }
+}
diff --git a/app/src/main/res/layout/activity_main.xml b/app/src/main/res/layout/activity_main.xml
index 647a190..896d556 100644
--- a/app/src/main/res/layout/activity_main.xml
+++ b/app/src/main/res/layout/activity_main.xml
@@ -111,4 +111,15 @@
         app:layout_constraintRight_toRightOf="parent"
         app:layout_constraintTop_toBottomOf="@+id/button_secondActivity" />
 
+    <Button
+        android:id="@+id/button_navigationFragmentSwapActivity"
+        android:layout_width="wrap_content"
+        android:layout_height="wrap_content"
+        android:layout_marginTop="8dp"
+        android:text="@string/go_to_a_fragment_swap_activity"
+        app:layout_constraintHorizontal_bias="0.497"
+        app:layout_constraintLeft_toLeftOf="parent"
+        app:layout_constraintRight_toRightOf="parent"
+        app:layout_constraintTop_toBottomOf="@+id/button_navigationActivity" />
+
 </androidx.constraintlayout.widget.ConstraintLayout>
diff --git a/app/src/main/res/layout/activity_navigation_fragment_swap.xml b/app/src/main/res/layout/activity_navigation_fragment_swap.xml
new file mode 100644
index 0000000..9afeb1e
--- /dev/null
+++ b/app/src/main/res/layout/activity_navigation_fragment_swap.xml
@@ -0,0 +1,20 @@
+<?xml version="1.0" encoding="utf-8"?>
+<androidx.constraintlayout.widget.ConstraintLayout xmlns:android="http://schemas.android.com/apk/res/android"
+    xmlns:app="http://schemas.android.com/apk/res-auto"
+    xmlns:tools="http://schemas.android.com/tools"
+    android:id="@+id/main"
+    android:layout_width="match_parent"
+    android:layout_height="match_parent"
+    tools:context=".fragments.NavigationFragmentSwapActivity">
+
+    <androidx.fragment.app.FragmentContainerView
+        android:id="@+id/fragment_container_view"
+        android:name="com.raygun.raygun4android.sample.fragments.FragmentSwap1"
+        android:layout_width="wrap_content"
+        android:layout_height="wrap_content"
+        app:layout_constraintBottom_toBottomOf="parent"
+        app:layout_constraintEnd_toEndOf="parent"
+        app:layout_constraintStart_toStartOf="parent"
+        app:layout_constraintTop_toTopOf="parent" />
+
+</androidx.constraintlayout.widget.ConstraintLayout>
diff --git a/app/src/main/res/layout/fragment_swap1.xml b/app/src/main/res/layout/fragment_swap1.xml
new file mode 100644
index 0000000..f641f02
--- /dev/null
+++ b/app/src/main/res/layout/fragment_swap1.xml
@@ -0,0 +1,30 @@
+<?xml version="1.0" encoding="utf-8"?>
+<androidx.constraintlayout.widget.ConstraintLayout xmlns:android="http://schemas.android.com/apk/res/android"
+    xmlns:app="http://schemas.android.com/apk/res-auto"
+    xmlns:tools="http://schemas.android.com/tools"
+    android:layout_width="match_parent"
+    android:layout_height="match_parent"
+    tools:context=".fragments.FragmentSwap1">
+
+    <TextView
+        android:id="@+id/textView2"
+        android:layout_width="wrap_content"
+        android:layout_height="wrap_content"
+        android:layout_marginTop="32dp"
+        android:text="@string/fragmentswap1_title"
+        android:textAppearance="@style/TextAppearance.AppCompat.Display1"
+        app:layout_constraintEnd_toEndOf="parent"
+        app:layout_constraintStart_toStartOf="parent"
+        app:layout_constraintTop_toTopOf="parent" />
+
+    <Button
+        android:id="@+id/button"
+        android:layout_width="wrap_content"
+        android:layout_height="wrap_content"
+        android:layout_marginTop="32dp"
+        android:text="@string/go_to_fragmentswap2"
+        app:layout_constraintEnd_toEndOf="parent"
+        app:layout_constraintStart_toStartOf="parent"
+        app:layout_constraintTop_toBottomOf="@+id/textView2" />
+
+</androidx.constraintlayout.widget.ConstraintLayout>
diff --git a/app/src/main/res/layout/fragment_swap2.xml b/app/src/main/res/layout/fragment_swap2.xml
new file mode 100644
index 0000000..110469b
--- /dev/null
+++ b/app/src/main/res/layout/fragment_swap2.xml
@@ -0,0 +1,30 @@
+<?xml version="1.0" encoding="utf-8"?>
+<androidx.constraintlayout.widget.ConstraintLayout xmlns:android="http://schemas.android.com/apk/res/android"
+    xmlns:app="http://schemas.android.com/apk/res-auto"
+    xmlns:tools="http://schemas.android.com/tools"
+    android:layout_width="match_parent"
+    android:layout_height="match_parent"
+    tools:context=".fragments.FragmentSwap2">
+
+    <TextView
+        android:id="@+id/textView3"
+        android:layout_width="wrap_content"
+        android:layout_height="wrap_content"
+        android:layout_marginTop="32dp"
+        android:text="@string/fragmentswap2_title"
+        android:textAppearance="@style/TextAppearance.AppCompat.Display1"
+        app:layout_constraintEnd_toEndOf="parent"
+        app:layout_constraintStart_toStartOf="parent"
+        app:layout_constraintTop_toTopOf="parent" />
+
+    <Button
+        android:id="@+id/button"
+        android:layout_width="wrap_content"
+        android:layout_height="wrap_content"
+        android:layout_marginTop="32dp"
+        android:text="@string/go_to_fragmentswap1"
+        app:layout_constraintEnd_toEndOf="parent"
+        app:layout_constraintStart_toStartOf="parent"
+        app:layout_constraintTop_toBottomOf="@+id/textView3" />
+
+</androidx.constraintlayout.widget.ConstraintLayout>
diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml
index 8be44a6..b6a51dc 100644
--- a/app/src/main/res/values/strings.xml
+++ b/app/src/main/res/values/strings.xml
@@ -20,4 +20,12 @@
     <string name="we_returned_to_first_activity">We\'re now back from second activity.</string>
     <string name="fragment_text">Fragment Screen</string>
     <string name="go_to_a_navigation_activity">Go to a navigation Activity</string>
+    <!-- TODO: Remove or change this placeholder text -->
+    <string name="hello_blank_fragment">Hello blank fragment</string>
+    <string name="fragmentswap1_title">FragmentSwap1</string>
+    <string name="go_to_fragmentswap2">Go to FragmentSwap2</string>
+    <string name="fragmentswap2_title">FragmentSwap2</string>
+    <string name="go_to_fragmentswap1">Go to FragmentSwap1</string>
+    <string name="fragment_swap_text">Fragment Swap Screen</string>
+    <string name="go_to_a_fragment_swap_activity">Go to a Fragment swap Activity</string>
 </resources>
diff --git a/provider/src/main/java/com/raygun/raygun4android/rum/RUMFragment.java b/provider/src/main/java/com/raygun/raygun4android/rum/RUMFragment.java
index d3294ab..c5bd804 100644
--- a/provider/src/main/java/com/raygun/raygun4android/rum/RUMFragment.java
+++ b/provider/src/main/java/com/raygun/raygun4android/rum/RUMFragment.java
@@ -42,9 +42,13 @@ public void onFragmentCreated(
             @NonNull Fragment fragment,
             @Nullable Bundle savedInstanceState) {
         super.onFragmentCreated(fm, fragment, savedInstanceState);
-        RaygunLogger.v("RUM - Fragment created: " + getFragmentName(fragment));
-        if (!fragmentStartTime.containsKey(fragment.getId())) {
-            fragmentStartTime.put(fragment.getId(), System.nanoTime());
+        RaygunLogger.v(
+                "RUM - Fragment created: "
+                        + getFragmentName(fragment)
+                        + " id: "
+                        + getUniqueId(fragment));
+        if (!fragmentStartTime.containsKey(getUniqueId(fragment))) {
+            fragmentStartTime.put(getUniqueId(fragment), System.nanoTime());
         }
         rum.seen();
     }
@@ -52,9 +56,13 @@ public void onFragmentCreated(
     @Override
     public void onFragmentStarted(@NonNull FragmentManager fm, @NonNull Fragment fragment) {
         super.onFragmentStarted(fm, fragment);
-        RaygunLogger.v("RUM - Fragment started: " + getFragmentName(fragment));
-        if (!fragmentStartTime.containsKey(fragment.getId())) {
-            fragmentStartTime.put(fragment.getId(), System.nanoTime());
+        RaygunLogger.v(
+                "RUM - Fragment started: "
+                        + getFragmentName(fragment)
+                        + " id: "
+                        + getUniqueId(fragment));
+        if (!fragmentStartTime.containsKey(getUniqueId(fragment))) {
+            fragmentStartTime.put(getUniqueId(fragment), System.nanoTime());
         }
         rum.seen();
     }
@@ -62,12 +70,16 @@ public void onFragmentStarted(@NonNull FragmentManager fm, @NonNull Fragment fra
     @Override
     public void onFragmentResumed(@NonNull FragmentManager fm, @NonNull Fragment fragment) {
         super.onFragmentResumed(fm, fragment);
-        RaygunLogger.v("RUM - Fragment resumed: " + getFragmentName(fragment));
+        RaygunLogger.v(
+                "RUM - Fragment resumed: "
+                        + getFragmentName(fragment)
+                        + " id: "
+                        + getUniqueId(fragment));
         long duration = 0;
-        if (!fragmentStartTime.containsKey(fragment.getId())) {
-            fragmentStartTime.put(fragment.getId(), System.nanoTime());
+        if (!fragmentStartTime.containsKey(getUniqueId(fragment))) {
+            fragmentStartTime.put(getUniqueId(fragment), System.nanoTime());
         } else {
-            Long startTime = fragmentStartTime.get(fragment.getId());
+            Long startTime = fragmentStartTime.get(getUniqueId(fragment));
             if (startTime != null) {
                 long diff = System.nanoTime() - startTime;
                 duration = TimeUnit.NANOSECONDS.toMillis(diff);
@@ -79,27 +91,39 @@ public void onFragmentResumed(@NonNull FragmentManager fm, @NonNull Fragment fra
     }
 
     @Override
-    public void onFragmentPaused(@NonNull FragmentManager fm, @NonNull Fragment f) {
-        super.onFragmentPaused(fm, f);
-        RaygunLogger.v(" RUM - Fragment paused: " + getFragmentName(f));
+    public void onFragmentPaused(@NonNull FragmentManager fm, @NonNull Fragment fragment) {
+        super.onFragmentPaused(fm, fragment);
+        RaygunLogger.v(
+                "RUM - Fragment paused: "
+                        + getFragmentName(fragment)
+                        + " id: "
+                        + getUniqueId(fragment));
         rum.seen();
     }
 
     @Override
     public void onFragmentStopped(@NonNull FragmentManager fm, @NonNull Fragment fragment) {
         super.onFragmentStopped(fm, fragment);
-        RaygunLogger.v("RUM - Fragment stopped: " + getFragmentName(fragment));
+        RaygunLogger.v(
+                "RUM - Fragment stopped: "
+                        + getFragmentName(fragment)
+                        + " id: "
+                        + getUniqueId(fragment));
         // Remove the start time for the fragment
-        fragmentStartTime.remove(fragment.getId());
+        fragmentStartTime.remove(getUniqueId(fragment));
         rum.seen();
     }
 
     @Override
     public void onFragmentDestroyed(@NonNull FragmentManager fm, @NonNull Fragment fragment) {
         super.onFragmentDestroyed(fm, fragment);
-        RaygunLogger.v("RUM - Fragment destroyed: " + getFragmentName(fragment));
+        RaygunLogger.v(
+                "RUM - Fragment destroyed: "
+                        + getFragmentName(fragment)
+                        + " id: "
+                        + getUniqueId(fragment));
         // Remove the start time for the fragment
-        fragmentStartTime.remove(fragment.getId());
+        fragmentStartTime.remove(getUniqueId(fragment));
         rum.seen();
     }
 
@@ -112,4 +136,8 @@ public String getFragmentName(Fragment fragment) {
             return simpleName;
         }
     }
+
+    public int getUniqueId(Fragment fragment) {
+        return fragment.hashCode();
+    }
 }

From b85796d36c1ead0c3869ae561ce97a5d37f1e3e1 Mon Sep 17 00:00:00 2001
From: Miguel Beltran <m@beltran.work>
Date: Thu, 16 Jan 2025 12:00:16 +0100
Subject: [PATCH 3/3] remove template string

---
 app/src/main/res/values/strings.xml | 2 --
 1 file changed, 2 deletions(-)

diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml
index b6a51dc..750d045 100644
--- a/app/src/main/res/values/strings.xml
+++ b/app/src/main/res/values/strings.xml
@@ -20,8 +20,6 @@
     <string name="we_returned_to_first_activity">We\'re now back from second activity.</string>
     <string name="fragment_text">Fragment Screen</string>
     <string name="go_to_a_navigation_activity">Go to a navigation Activity</string>
-    <!-- TODO: Remove or change this placeholder text -->
-    <string name="hello_blank_fragment">Hello blank fragment</string>
     <string name="fragmentswap1_title">FragmentSwap1</string>
     <string name="go_to_fragmentswap2">Go to FragmentSwap2</string>
     <string name="fragmentswap2_title">FragmentSwap2</string>