Callback method called multiply times

public void loadTeasers(int skip, boolean refresh, Category category) {
        getViewState().showProgress();

        Subscription s2 = ActiveAndroidHelper.getSources()
                .subscribeOn(Schedulers.io())
                .observeOn(AndroidSchedulers.mainThread())
                .subscribe(teasers -> {
                    getViewState().onTeasersLoaded(teasers);
                    getViewState().hideProgress();
                }, this::onError);
        unsubscribeOnDestroy(s2);
    }

Suppose we have simple method which loads some data from network. I want to refresh my network data every onResume method. And, in this case, on each call of loadTeasers(true, new Category()), the callback method onTeasersLoaded(teasers) is called N times, where N is the number of onResume methods (and loadTeasers) executed. I think I need to unsubscribe and null the previous subsribition, but which method to use here? Thanks

[A-Zaiats]

What strategy do you use for onTeaserLoaded(teasers) method? Was your view stopped before resume? Are you sure that onTeaserLoaded(teasers) method is called N times from subsrciber#onNext callback?

Looks like you use default strategy. So it can be following behavior:

• the View was stopped
• the View was detached from presenter
• the View was attached to the Presenter
• the ViewState restored state (called all onTeaserLoaded(teasers) that was called before)
• one more onTeaserLoaded(teasers) was called by loadTeasers callback

@senneco can you correct me if I am wrong with Moxy lifecycle?

[lectricas]

@A-Zaiats woohoo works as desired. Thanks.