Conflicting documentation for dagger.android and Hilt optional inject about dagger HOT 3 CLOSED

Sorry I'm not entirely following. Aren't the dagger.android docs talking about when you should inject the Activity while the Hilt docs are talking about when the Fragment is injected?

from dagger.

Hilt docs have a code snippet with Fragment code, yes, but discuss any @AndroidEntryPoint that has a parent, which means it could be an Activity too:

If you mark an @AndroidEntryPoint class with @OptionalInject (...) This gives you the chance to provide dependencies in a different way

dagger.android docs talk about both too. Part about fragments is this:

before calling super (...) DaggerFragment does the same in onAttach(), which also prevents inconsistencies if the Fragment is reattached.

Furthermore, even if it was like you say, let's say we want to follow dagger.android advice for an Activity: If I try checking OptionalInjectCheck.wasInjectedByHilt(this) before super.onCreate(savedInstanceState), then it will simply not work, because wasInjectedByHilt will return correct value only if it is called after super

I edited OP to try and make this more clear

from dagger.

Ooh I got it now, thanks. This is a good question!

This is actually fairly tricky, but I think to do it you for an Activity you would want to register an OnContextAvailableListener and do the injection logic in there. As long as your OnContextAvailableListener is registered after the Hilt one (and this should happen because we register our listener in the Hilt base class's constructor, so even doing it in your constructor should put yours after ours since super constructors would be called first), then it should work. These listeners get called within the onCreate of the Android Activity base class and so should also satisfy the dagger.android docs where these are called before fragments are attached.

I can update the documentation, thanks for pointing this out!

from dagger.