fix(android): Prevent crash in restoreInstanceState if bundleData is null

[hermitdemschoenenleben]

this PR fixes this (closed) issue: ionic-team/capacitor-plugins#658
closes: #5290

Without the fix, capacitor calls the camera plugin's restoreState() method even though bundleData is null

[hermitdemschoenenleben]

corresponding issue: #5290

[jcesarmobile]

Thanks for the PR.

I've made a few additions to make sure we don't save other data if we couldn't save the bundle and also to log error if the saving/restoring failed, since this will only prevent the crash, but the messages can maybe figure out why it's failing to save in the future.

[sebsoftpak]

Hi @hermitdemschoenenleben,

Thank you for making this fix. Do you know if this fix has been released?
(Haven't seen #658 in the release notes yet, that's why i'm asking.)

We are experiencing the same NullPointerException with @capacitor/camera 1.3.0.

Kind regards,
Sebastian Sørensen

[hermitdemschoenenleben]

@sebsoftpak yes, the fix has been released (at least I don't see the error in production anymore)

[sebsoftpak]

@hermitdemschoenenleben ok, good to hear! will upgrade from 1.3.0 to 1.3.1, maybe that makes the difference :)

[jcesarmobile]

The fix is in capacitor/android 3.3.3, not in the camera plugin