Fixed Parceable Selection States

 issue working for both retained fragments and non retained fragment

Author: geniusgeek

criminalintent-sample/src/main/java/com/bignerdranch/android/criminalintent/CrimeListFragment.java

@@ -42,7 +42,7 @@ public void onCreate(Bundle savedInstanceState) {
  super.onCreate(savedInstanceState);
  setHasOptionsMenu(true);
  getActivity().setTitle(R.string.crimes_title);
- setRetainInstance(true);
+ //setRetainInstance(true);
  mSubtitleVisible = false;
  }
 
@@ -55,17 +55,31 @@ public void onCreate(Bundle savedInstanceState) {
  */
  @Override
  public void onActivityCreated(Bundle savedInstanceState) {
- super.onActivityCreated(savedInstanceState);
+
+ if( mMultiSelector==null)return;
+
+ Bundle bundle=savedInstanceState;
+ if(bundle!=null) mMultiSelector=bundle.getParcelable(MultiSelector.TAG);
+
  if(mMultiSelector.isSelectable()){
- if(mDeleteMode !=null){
- mDeleteMode.setClearOnPrepare(false);
- beginActionMode();
+ if(mDeleteMode !=null){
+ mDeleteMode.setClearOnPrepare(false);
+ beginActionMode();
+
+ }
 
  }
 
- }
+ super.onActivityCreated(savedInstanceState);
  }
 
+ @Override
+ public void onSaveInstanceState(Bundle outState) {
+ outState.putParcelable(MultiSelector.TAG,mMultiSelector);
+ super.onSaveInstanceState(outState);
+ }
+
+
  @TargetApi(11)
  @Override
  public View onCreateView(LayoutInflater inflater, ViewGroup parent, Bundle savedInstanceState) {
@@ -211,6 +225,8 @@ public CrimeHolder(View itemView) {
  mDateTextView = (TextView) itemView.findViewById(R.id.crime_list_item_dateTextView);
  mSolvedCheckBox = (CheckBox) itemView.findViewById(R.id.crime_list_item_solvedCheckBox);
  itemView.setOnClickListener(this);
+ itemView.setClickable(true);
+ itemView.setOnLongClickListener(this);
 
  }
 

recyclerview-multiselect/src/main/java/com/bignerdranch/android/multiselector/MultiSelector.java

@@ -1,5 +1,7 @@
 package com.bignerdranch.android.multiselector;
 
+import android.os.Parcel;
+import android.os.Parcelable;
 import android.util.SparseBooleanArray;
 
 import java.util.ArrayList;
@@ -32,10 +34,11 @@
  * }
  * </pre>
  */
-public class MultiSelector {
+public class MultiSelector implements Parcelable {
  private SparseBooleanArray mSelections = new SparseBooleanArray();
  private WeakHolderTracker mTracker = new WeakHolderTracker();
  private boolean mIsSelectable;
+ public static final String TAG="multiselector";
  /**
  * <p>Toggle whether this MultiSelector is in selection mode or not.
  * {@link com.bignerdranch.android.multiselector.SelectableHolder#setSelectable(boolean)}
@@ -196,4 +199,34 @@ private void refreshHolder(SelectableHolder holder) {
  }
 
 
+ @Override
+ public int describeContents() {
+ return 0;
+ }
+
+ @Override
+ public void writeToParcel(Parcel dest, int flags) {
+ dest.writeSparseBooleanArray(this.mSelections);
+ dest.writeParcelable(this.mTracker,flags);
+ dest.writeByte(mIsSelectable ? (byte) 1 : (byte) 0);//write the boolean in form of logical 0 or 1
+ }
+
+ public MultiSelector() {
+ }
+
+ private MultiSelector(Parcel in) {
+ this.mSelections = in.readSparseBooleanArray();
+ this.mTracker = in.readParcelable(WeakHolderTracker.class.getClassLoader());
+ this.mIsSelectable = in.readByte() != 0;//retrieve the boolean information
+ }
+
+ public static final Parcelable.Creator<MultiSelector> CREATOR = new Parcelable.Creator<MultiSelector>() {
+ public MultiSelector createFromParcel(Parcel source) {
+ return new MultiSelector(source);
+ }
+
+ public MultiSelector[] newArray(int size) {
+ return new MultiSelector[size];
+ }
+ };
 }

recyclerview-multiselect/src/main/java/com/bignerdranch/android/multiselector/WeakHolderTracker.java

@@ -1,14 +1,16 @@
 package com.bignerdranch.android.multiselector;
 
+import android.os.Parcel;
+import android.os.Parcelable;
 import android.util.SparseArray;
 
 import java.lang.ref.WeakReference;
 import java.util.ArrayList;
 import java.util.List;
 
-class WeakHolderTracker {
- private SparseArray<WeakReference<SelectableHolder>> mHoldersByPosition =
- new SparseArray<WeakReference<SelectableHolder>>();
+class WeakHolderTracker implements Parcelable {
+ private SparseArray<WeakReferenceSelectableHolder<SelectableHolder>> mHoldersByPosition =
+ new SparseArray<>();
 
  /**
  * Returns the holder with a given position. If non-null, the returned
@@ -32,7 +34,7 @@ public SelectableHolder getHolder(int position) {
  }
 
  public void bindHolder(SelectableHolder holder, int position) {
- mHoldersByPosition.put(position, new WeakReference<SelectableHolder>(holder));
+ mHoldersByPosition.put(position, new WeakReferenceSelectableHolder<SelectableHolder>(holder));
  }
 
  public List<SelectableHolder> getTrackedHolders() {
@@ -49,4 +51,32 @@ public List<SelectableHolder> getTrackedHolders() {
 
  return holders;
  }
+
+ @Override
+ public int describeContents() {
+ return 0;
+ }
+
+ @Override
+ public void writeToParcel(Parcel dest, int flags) {
+ dest.writeSparseArray((SparseArray)this.mHoldersByPosition);
+ }
+
+ public WeakHolderTracker() {
+ }
+
+ private WeakHolderTracker(Parcel in) {
+
+ this.mHoldersByPosition = in.readSparseArray(SparseArray.class.getClassLoader());
+ }
+
+ public static final Parcelable.Creator<WeakHolderTracker> CREATOR = new Parcelable.Creator<WeakHolderTracker>() {
+ public WeakHolderTracker createFromParcel(Parcel source) {
+ return new WeakHolderTracker(source);
+ }
+
+ public WeakHolderTracker[] newArray(int size) {
+ return new WeakHolderTracker[size];
+ }
+ };
 }