OnLoadFinished() called twice

Question

I'm trying to figure out if I'm doing something wrong with respect to Loaders. I'm using the support library, and I have a Fragment which in onCreate() calls initLoader() setting itself as the LoaderCallbacks, however on a rotation it is receiving the result twice in onLoadFinished(), once as a result of calling init (and it already having the data), and once as a result of FragmentActivity looping through all Loaders in onStart() and delivering the result since it already has the data.

If I only call init once (on first launch of the Fragment), it doesn't set itself as the callback for the Loader so it doesn't receive a call to onLoadFinished at all. It seems as though onLoadFinished should only be called once since some expensive things may be done in onLoadFinished() (such as clearing list adapters, etc.), so I'm just trying to figure out if this is a bug or if I am just calling init at the wrong time or something else.

Anyone have any insight to this issue?

I'm not sure why this happens, but calling `getLoaderManager().initLoader(0, null, this);` in `onResume()` seems to be the solution for a lot of people (inc. me). See http://stackoverflow.com/a/14524031/1617737 . — ban-geoengineering, Jan 26 '16 at 15:50
This is the best answer I have found on this: http://stackoverflow.com/a/17324776/2578070 — James Wiley, Mar 08 '16 at 14:59

score 3 · Answer 1 · answered May 28 '15 at 05:55

I had a similar problem and the cause was that I had initLoader and restartLoader in my code. Depending on user's action, my query could change, so I needed to restart my loader.

The solution was to use only restartLoader, even in onResume callback method replace initLoader with restartLoader.

score 3 · Answer 2 · answered Nov 29 '15 at 21:41

This is a rather old question, but for future readers I have an alternative solution. Basically what I ended up doing was restart the loader if it existed.

public void onActivityCreated(Bundle savedInstanceState) {
...
       if(getLoaderManager().getLoader(Constants.LOADER_ID) == null) {
            getLoaderManager().initLoader(Constants.LOADER_ID, null, this);
        } else {
            getLoaderManager().restartLoader(Constants.LOADER_ID, null, this);
        }

...
}

This solved my issue with that on screen rotate the loader was triggered twice. One thing too note is that this is only needed for me on Android < 6 that I tested. Android 6 seem to not have this issue at all.

This solution may works, but has the same problem as Alex Lockwood mentioned earlier. — Carlos Hernández Gil, Sep 12 '16 at 08:50

score 2 · Answer 3 · edited May 23 '17 at 11:59

2

I am experiencing same problem my self, with no good solution. It seems as bug in Android framework, here is similar thread in which proposed solution is to place initLoader() in onResume() - I have tried it and it works, on onLoadFinished() gets called only once: Android: LoaderCallbacks.OnLoadFinished called twice

edited May 23 '17 at 11:59

Community

1
1

answered Feb 14 '13 at 10:25

Gordon Freeman

1,117
9
22

score 2 · Answer 4 · edited May 23 '17 at 11:53

2

See my post at Android: LoaderCallbacks.OnLoadFinished called twice

I had a similar problem when restarting Fragments in a ViewPager. My solution is to remove the Loader once I'm finished with it (at the end of onLoadFinished) by calling

getLoaderManager().destroyLoader(LOADER_ID);

Hope it helps!

edited May 23 '17 at 11:53

Community

1
1

answered Mar 04 '14 at 21:22

Matt

3,837
26
29

14

This is a very bad solution... destroying the `Loader` will also destroy all of the data associated with it. One of the main benefits of `Loader`s is that data is retained across configuration changes (such as changes in orientation, etc.)... destroying this data means you will need to requery the data each time a configuration change occurs. – Alex Lockwood Dec 19 '14 at 02:27

Andrij · Answer 5 · 2020-05-10T15:49:22.847

It is looks like framework Loader wrong implementation/bug. 1. look at what I got from Log.v(LOG_TAG, ...) messages from every important method/callback after screen rotation:

...: .initLoader() 100
...: onStartLoading()
...: onLoadFinished()
...: updateUi(); articles size=10
...: loadInBackground()
...: getInputStream() - HttpRequest
...: onLoadFinished()
...: updateUi(); articles size=10

2. As you can see everything after 'updateUi()' is no needed there.

OnLoadFinished() called twice

5 Answers5

Linked