Skip to content
This repository has been archived by the owner on Oct 25, 2024. It is now read-only.

fix state restoration #144

Merged
merged 1 commit into from
Feb 11, 2016
Merged

fix state restoration #144

merged 1 commit into from
Feb 11, 2016

Conversation

loganj
Copy link
Collaborator

@loganj loganj commented Feb 11, 2016

We were getting a new fragment instance to replace our old fragment
instance, but it needed to be reconfigured.

We were getting a new fragment instance to replace our old fragment
instance, but it needed to be reconfigured.
@@ -44,9 +44,11 @@ static void install(Application app, final Activity activity,
@Override public void onActivityCreated(Activity a, Bundle savedInstanceState) {
if (a == activity) {
InternalLifecycleIntegration fragment = find(activity);
final boolean newFragment = fragment == null;
boolean newFragment = fragment == null;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why not final?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

doesn't need to be. that was an IntelliJ burp.

@rjrjr
Copy link
Collaborator

rjrjr commented Feb 11, 2016

One nit, will merge anyway.

rjrjr added a commit that referenced this pull request Feb 11, 2016
@rjrjr rjrjr merged commit 92d16c2 into master Feb 11, 2016
@loganj loganj deleted the loganj/restore branch February 18, 2016 22:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants