Fixed setAdapter bug

lzanita09 · lzanita09 · commit 39d3b05d44de · 2014-04-05T10:59:03.000-04:00
adapter should be mAdapter, otherwise when adding header view, mItemCount will get the original adapter's getCount(), which doesn't count the number of headers and/or footers. Later on in layoutChildren() method it'll throw IllegalStateException, because the number of mItemCount and mAdapter.getCount() will never be equal.
diff --git a/library/src/main/java/com/etsy/android/grid/ExtendableListView.java b/library/src/main/java/com/etsy/android/grid/ExtendableListView.java
@@ -265,7 +265,7 @@ public void setAdapter(final ListAdapter adapter) {
         }
 
         mDataChanged = true;
-        mItemCount = adapter != null ? adapter.getCount() : 0;
+        mItemCount = mAdapter != null ? mAdapter.getCount() : 0;
 
         if (adapter != null) {
             adapter.registerDataSetObserver(mObserver);

Original file line number	Diff line number	Diff line change
`@@ -265,7 +265,7 @@ public void setAdapter(final ListAdapter adapter) {`
`265`	`265`	`}`
`266`	`266`
`267`	`267`	`mDataChanged = true;`
`268`		`- mItemCount = adapter != null ? adapter.getCount() : 0;`
	`268`	`+ mItemCount = mAdapter != null ? mAdapter.getCount() : 0;`
`269`	`269`
`270`	`270`	`if (adapter != null) {`
`271`	`271`	`adapter.registerDataSetObserver(mObserver);`