I have run into a bit of a roadblock. I have a scenario VERY similar to the one described at: DialogFragment - retaining listener after screen rotation
The proposed solution works fine for the author because his dialog is being called from an activity. My case is the exact same, but my Custom Dialog is called from a Fragment instead of an Activity. (IE Activity->Fragment->Dialog)
I implemented the same solution (setting the listener in onResume from the calling Fragment) but it doesn't work in this case.
What seems to be happening is that when the screen is rotated, Android kills the Dialog and Fragment. Then recreates them IN THAT ORDER. So when my onCreateDialog is called on my custom dialog the containing Fragment is yet to be recreated, so it has null for the listener to set for positive and negative buttons.
Does anyone know a way around this?
public class RecipeDetailEditFragment extends SherlockFragment implements DialogInterface.OnClickListener {
private EditStepFragmentDialog stepDialog;
private Recipe newRecipe; //main data object implements parcelable
...
public void onCreate(Bundle savedInstanceState) {
super.onCreate(savedInstanceState);
...
stepDialog = EditStepFragmentDialog.newInstance(newRecipe);
//I've also tried passing 'this' into the newInstance constructor and
//setting the listener there, but that doesn't work either
}
public void onResume() {
stepDialog.setListener(this);
super.onResume();
}
...
}
public class EditStepFragmentDialog extends DialogFragment {
private DialogInterface.OnClickListener ocl;
private static final String ARG_RECIPE = "recipe";
private Recipe recipe;
public EditStepFragmentDialog() {}
public static EditStepFragmentDialog newInstance(Recipe rec) { //(Recipe rec, DialogInterface.OnClickListener oc) as mentioned doesn't work.
EditStepFragmentDialog dia = new EditStepFragmentDialog();
Bundle args = new Bundle();
args.putParcelable(ARG_RECIPE, rec);
//dia.setListener(oc);
return dia;
}
public Dialog onCreateDialog(Bundle savedInstanceState) {
AlertDialog.Builder adb = new AlertDialog.Builder(getActivity());
if (getArguments().containsKey(ARG_RECIPE)) {
recipe = (Recipe) getArguments().getParcelable(ARG_RECIPE);
}
...
adb.setPositiveButton("Done", ocl);
adb.setNegativeButton("Cancel", ocl);
...
return adb.create();
}
public void setListener(DialogInterface.OnClickListener cl) {
ocl = cl;
}
}
I ran through all of the options on the discussed links and none of the solutions ended up working for me. I also tried a number of additional options after further googling like get/setTargetFragment, and FragmentManager.put/getFragment. These didn't work for me either. Then I took another look at:
http://developer.android.com/training/basics/fragments/communicating.html
Where they specifically say "Two Fragments should never communicate directly". I think this is one of the cases where that's really proven true.
I ended up implementing the suggested callback mechanism provided there and ended up with this:
In DialogFragment:
public interface OnEditStepDialogListener {
public void onEditStepDialogPositive(int pos, String description);
}
@Override
public void onAttach(Activity activity) {
super.onAttach(activity);
try {
mCallback = (OnEditStepDialogListener) activity;
} catch (ClassCastException e) {
throw new ClassCastException(activity.toString() + " must implement OnEditStepDialogListener");
}
}
In hosting Activity:
public class MyActivity extends SherlockFragmentActivity implements EditStepFragmentDialog.OnEditStepDialogListener {
...
@Override
public void onEditStepDialogPositive(int pos, String desc) {
FragmentManager fm = getSupportFragmentManager();
RecipeDetailEditFragment ef = (RecipeDetailEditFragment)fm.findFragmentByTag(RecipeDetailEditFragment.TAG);
ef.applyStepEdit(pos, desc);
}
In Fragment that fires off the FragmentDialog:
public static final String TAG = "tag1";
public void applyStepEdit(int pos, String description) {
...
}
This works perfectly, if opened then orientation change and edit is completed, it actually triggers the ultimate function I need run in the calling Fragment instead of either crashing or not doing anything (null listener).
If you love us? You can donate to us via Paypal or buy me a coffee so we can maintain and grow! Thank you!
Donate Us With