Final cleanup - remove unnecessary defensive code

Co-authored-by: adbenitez <24558636+adbenitez@users.noreply.github.com>

src/main/java/org/thoughtcrime/securesms/ConversationFragment.java

@@ -891,7 +891,7 @@ public class ConversationFragment extends MessageSelectorFragment
 
       @Override
       public void onReactionClicked(DcMsg messageRecord) {
-        ReactionsDetailsFragment dialog = new ReactionsDetailsFragment(messageRecord.getId());
+        ReactionsDetailsFragment dialog = ReactionsDetailsFragment.newInstance(messageRecord.getId());
         dialog.show(getActivity().getSupportFragmentManager(), null);
       }
     }

src/main/java/org/thoughtcrime/securesms/accounts/AccountSelectionListFragment.java

@@ -47,20 +47,40 @@ import chat.delta.rpc.RpcException;
 public class AccountSelectionListFragment extends DialogFragment implements DcEventCenter.DcEventDelegate
 {
   private static final String TAG = AccountSelectionListFragment.class.getSimpleName();
-  private final ConversationListActivity activity;
+  private static final String ARG_SELECT_ONLY = "select_only";
+  
+  private ConversationListActivity activity;
   private RecyclerView recyclerView;
   private AccountSelectionListAdapter adapter;
-  private final boolean selectOnly;
+  private boolean selectOnly;
 
-  public AccountSelectionListFragment(ConversationListActivity activity, boolean selectOnly) {
+  public AccountSelectionListFragment() {
     super();
-    this.activity = activity;
-    this.selectOnly = selectOnly;
+  }
+
+  public static AccountSelectionListFragment newInstance(boolean selectOnly) {
+    AccountSelectionListFragment fragment = new AccountSelectionListFragment();
+    Bundle args = new Bundle();
+    args.putBoolean(ARG_SELECT_ONLY, selectOnly);
+    fragment.setArguments(args);
+    return fragment;
   }
 
   @NonNull
   @Override
   public Dialog onCreateDialog(Bundle savedInstanceState) {
+    // Retrieve selectOnly from arguments (defaults to false if not present)
+    if (getArguments() != null) {
+      selectOnly = getArguments().getBoolean(ARG_SELECT_ONLY, false);
+    }
+    
+    // Get the activity from the context - must be ConversationListActivity
+    if (getActivity() instanceof ConversationListActivity) {
+      activity = (ConversationListActivity) getActivity();
+    } else {
+      throw new IllegalStateException("AccountSelectionListFragment must be attached to ConversationListActivity");
+    }
+    
     AlertDialog.Builder builder = new AlertDialog.Builder(getActivity())
             .setTitle(R.string.switch_account)
             .setNegativeButton(R.string.cancel, null);

src/main/java/org/thoughtcrime/securesms/connect/AccountManager.java

@@ -138,7 +138,7 @@ public class AccountManager {
     // ui
 
     public void showSwitchAccountMenu(ConversationListActivity activity, boolean selectOnly) {
-        AccountSelectionListFragment dialog = new AccountSelectionListFragment(activity, selectOnly);
+        AccountSelectionListFragment dialog = AccountSelectionListFragment.newInstance(selectOnly);
         dialog.show(((FragmentActivity) activity).getSupportFragmentManager(), null);
     }
 

src/main/java/org/thoughtcrime/securesms/qr/QrActivity.java

@@ -69,7 +69,7 @@ public class QrActivity extends BaseActionBarActivity implements View.OnClickLis
 
         scanRelay = getIntent().getBooleanExtra(EXTRA_SCAN_RELAY, false);
 
-        qrShowFragment = new QrShowFragment(this);
+        qrShowFragment = new QrShowFragment();
         tabLayout = ViewUtil.findById(this, R.id.tab_layout);
         viewPager = ViewUtil.findById(this, R.id.pager);
         ProfilePagerAdapter adapter = new ProfilePagerAdapter(this, getSupportFragmentManager());

src/main/java/org/thoughtcrime/securesms/qr/QrShowFragment.java

@@ -54,15 +54,8 @@ public class QrShowFragment extends Fragment implements DcEventCenter.DcEventDel
 
     private DcContext dcContext;
 
-    private View.OnClickListener scanClicklistener;
-
     public QrShowFragment() {
-        this(null);
-    }
-
-    public QrShowFragment(View.OnClickListener scanClicklistener) {
         super();
-        this.scanClicklistener = scanClicklistener;
     }
 
     @Override
@@ -79,9 +72,14 @@ public class QrShowFragment extends Fragment implements DcEventCenter.DcEventDel
         dcContext = DcHelper.getContext(getActivity());
         dcEventCenter = DcHelper.getEventCenter(getActivity());
 
-        Bundle extras = getActivity().getIntent().getExtras();
-        if (extras != null) {
-            chatId = extras.getInt(CHAT_ID);
+        // First try to get chatId from fragment arguments, then fall back to activity intent
+        if (getArguments() != null && getArguments().containsKey(CHAT_ID)) {
+            chatId = getArguments().getInt(CHAT_ID);
+        } else {
+            Bundle extras = getActivity().getIntent().getExtras();
+            if (extras != null) {
+                chatId = extras.getInt(CHAT_ID);
+            }
         }
 
         dcEventCenter.addObserver(DcContext.DC_EVENT_SECUREJOIN_INVITER_PROGRESS, this);
@@ -108,9 +106,16 @@ public class QrShowFragment extends Fragment implements DcEventCenter.DcEventDel
 
         view.findViewById(R.id.share_link_button).setOnClickListener((v) -> showInviteLinkDialog());
         Button scanBtn = view.findViewById(R.id.scan_qr_button);
-        if (scanClicklistener != null) {
+        
+        // Check if the activity implements View.OnClickListener to provide scan functionality
+        View.OnClickListener scanClickListener = null;
+        if (getActivity() instanceof View.OnClickListener) {
+            scanClickListener = (View.OnClickListener) getActivity();
+        }
+        
+        if (scanClickListener != null) {
             scanBtn.setVisibility(View.VISIBLE);
-            scanBtn.setOnClickListener(scanClicklistener);
+            scanBtn.setOnClickListener(scanClickListener);
         } else {
             scanBtn.setVisibility(View.GONE);
         }

src/main/java/org/thoughtcrime/securesms/reactions/ReactionsDetailsFragment.java

@@ -36,19 +36,32 @@ import chat.delta.rpc.types.Reactions;
 
 public class ReactionsDetailsFragment extends DialogFragment implements DcEventCenter.DcEventDelegate {
   private static final String TAG = ReactionsDetailsFragment.class.getSimpleName();
+  private static final String ARG_MSG_ID = "msg_id";
 
   private RecyclerView recyclerView;
   private ReactionRecipientsAdapter adapter;
-  private final int msgId;
+  private int msgId;
 
-  public ReactionsDetailsFragment(int msgId) {
+  public ReactionsDetailsFragment() {
     super();
-    this.msgId = msgId;
+  }
+
+  public static ReactionsDetailsFragment newInstance(int msgId) {
+    ReactionsDetailsFragment fragment = new ReactionsDetailsFragment();
+    Bundle args = new Bundle();
+    args.putInt(ARG_MSG_ID, msgId);
+    fragment.setArguments(args);
+    return fragment;
   }
 
   @NonNull
   @Override
   public Dialog onCreateDialog(Bundle savedInstanceState) {
+    // Retrieve msgId from arguments
+    if (getArguments() != null) {
+      msgId = getArguments().getInt(ARG_MSG_ID, 0);
+    }
+    
     adapter = new ReactionRecipientsAdapter(requireActivity(), GlideApp.with(requireActivity()), new ListClickListener());
 
     LayoutInflater inflater = requireActivity().getLayoutInflater();