generating parcelables for android aidl is not supported by plugin about android-parcelable-intellij-plugin HOT 5 OPEN

Hi,
sometimes the design of android APIs make me very sad...

So how do you imagine such feature? Checkbox option that would generate deserialization in readFromParcel method instead of constructor would be enough?

from android-parcelable-intellij-plugin.

Yea, the API is definitely weird.

You may put the checkbox, thats an option. Another option might be to generate the parcelable exactly in the same way as google showed in Rect class(link in first post). In that way it should be compatible with both aidl and standard parcelable. It is not different much from the way you do it now, right?

from android-parcelable-intellij-plugin.

The problem I can see in solution from sample is calling non-final method from constructor that is generally bad practice. Making this method final would forbid inharitance of parcelable objects.
That's why I don't want to make it a default behaviour. I guess making this optional would be better solution.

To be honest I am not sure when I'll find time to implement this. If you need this asap maybe try to implement it and pull request? I'll be happy to review it and merge. The codebase of plugin shouldn't be hard to understand.

from android-parcelable-intellij-plugin.

I would do it at start, no problem, but I am afraid I moved from the project a month ago and don't need it anymore, sorry, it is up to you.

from android-parcelable-intellij-plugin.

I have added a pull request for this: #74

I have also released an updated version in the meanwhile to be manually installed: https://github.com/koh-osug/android-parcelable-intellij-plugin/releases/tag/v0.7.1

NOTE: this must be installed over the existing plugin, otherwise some hook seems to be missing.

I'n not familiar with the meaning of the field until-build in the plugin.xml but it might not work in later versions if IntelliJ is restricting this artificially by using this.

from android-parcelable-intellij-plugin.