Skip to content
This repository was archived by the owner on Jan 10, 2025. It is now read-only.

Performed fragment injection in onFragmentAttached instead onFragmentCreated #973

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Performed fragment injection in onFragmentAttached instead onFragmentCreated #973

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 3 additions & 2 deletions GithubBrowserSample/app/src/main/java/com/android/example/github/di/AppInjector.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,7 @@ package com.android.example.github.di

import android.app.Activity
import android.app.Application
import android.content.Context
import android.os.Bundle
import androidx.fragment.app.Fragment
import androidx.fragment.app.FragmentActivity
Expand Down Expand Up @@ -74,10 +75,10 @@ object AppInjector {
activity.supportFragmentManager
.registerFragmentLifecycleCallbacks(
object : FragmentManager.FragmentLifecycleCallbacks() {
override fun onFragmentCreated(
override fun onFragmentAttached(
fm: FragmentManager,
f: Fragment,
savedInstanceState: Bundle?
context: Context
) {
if (f is Injectable) {
AndroidSupportInjection.inject(f)
Expand Down