StickyListHeadersListView has build error with AA 4.1.0 #1845

Activity

1. dodgex commented on Oct 10, 2016

   

   dodgex

   on Oct 10, 2016

   Member

   More actions

   first of all sorry for the late response.

   yeah this is indeed a side effect of #1785. unfortunately I currently have no idea for a good fix.

   one possible way is to allow a class parameter to the annotation to specify the class to be used for the cast.

   e.g.

   @ItemClick(value = R.id.lvStickyList, castTo = StickyListHeadersListView.class)
   void itemClick(int position) {
   }
   

   and then generate the cast like

   ((StickyListHeadersListView) this.lvStickyList).setOnItemClickListener(new OnItemClickListener() {
      //...
   }
   

   but this does not feel clean... @WonderCsabo do you have an idea how to fix this issue?
2. WonderCsabo commented on Oct 10, 2016

   

   WonderCsabo

   on Oct 10, 2016

   Member

   More actions

   The castTo parameter is horrible. However i am not sure why this error is happening. Why the class cannot be cast to its parent class? 😕
3. dodgex commented on Oct 10, 2016

   

   dodgex

   on Oct 10, 2016

   Member

   More actions

   StickyListHeadersListView is not a child of AdapterView and I'm pretty sure this is also true for PullToRefreshListView mentioned in #1871
4. WonderCsabo commented on Oct 10, 2016

   

   WonderCsabo

   on Oct 10, 2016

   Member

   More actions

   I am not sure we want to support these badly designed fake ListView classes.
5. TomWangTW commented on Oct 11, 2016

   

   TomWangTW

   on Oct 11, 2016

   Author

   More actions

   Well, after i traced the source code of AA, i think it is very hard to support customize view gracefully...
   But so many projects used third-party customize view, so if you make sure AA doesn't support it,
   maybe should be written in 4.1 documents or use instanceof to check these views could cast to AdapterView or not.
6. WonderCsabo commented on Nov 23, 2016

   

   WonderCsabo

   on Nov 23, 2016

   Member

   More actions

   I am closing this one, as i think we should put effort into supporting some ListView-ish classes, as development moved to the much robust RecyclerView class.
7. WonderCsabo

   closed this as completedon Nov 23, 2016