Repository navigation
Allow any kind of constructor for abstract @EBean #746
Description
As i'm unhappy with the spring rest template stuff, i decided to go with another solution and therfore i created an abstract @ebean with this public AbstractRest(String server, Class<T> api) constructor.
in my extending classes i have something like this
public RestService() {
super("http://example.org/api", RestInterface.class);
}the problem is, that @ebean does only allow classes with 1 constructor that has no parameters or a Context parameter.
while i can understand this for regular @ebean classes, i think for abstract classes it is ok to have any constructors as there will be no AA generated class directly for that bean. And the extending beans are still forced to meet the @ebean constructor rules.
i made a pretty simple patch for this, but i have no idea how to get a PR for that as the branch i created also contains the commits for #736 and i don't know how to remove them...
here is the commit https://github.com/dodgex/androidannotations/commit/48e1b730baefed77c2cbe1a30b57fa94dd2d1fb9
Activity
DayS commentedon Oct 4, 2013
Good point.
I'll take a look at your other PR. Please make a new PR for this feature after I merged #736
dodgex commentedon Oct 4, 2013
I'll do that on sunday.
Damien notifications@github.com schrieb:
Diese Nachricht wurde von meinem Mobiltelefon mit Kaiten Mail gesendet.
dodgex commentedon Oct 6, 2013
added the pull request #755
dodgex commentedon Nov 25, 2013
any news on this one? :)
DayS commentedon Nov 25, 2013
Nop. I think I'll take a look on this one after 3.0 release :)
dodgex commentedon Nov 25, 2013
although it would be nice to see this one in 3.0 its your choice to merge it or not :)
i'm looking forward to get rid of the -SNAPSHOT or in my case the self build -SNAPSHOT, as the SNAPSHOTs already f_ked my app three times, luckily only once in a version that reached playstore :-D
dodgex commentedon Dec 21, 2013
#822 is an updated PR for this issue with the changes for the merged refactor PR #619
DayS commentedon Mar 16, 2014
Merged. Thanks