ionic-team / ionic-native-google-maps

Google maps plugin for Ionic Native
Other
221 stars 125 forks source link

Xcode error "Thread 1: signal SIGABRT" when running in iOS emulator with latest (master branch) #124

Closed goldman94 closed 5 years ago

goldman94 commented 5 years ago

I'm submitting a ... (check one with "x")

If you choose 'problem or bug report', please select OS: (check one with "x")

cordova information: (run $> cordova plugin list)

cordova-plugin-device 2.0.2 "Device"
cordova-plugin-googlemaps 2.4.6 "cordova-plugin-googlemaps"
cordova-plugin-ionic-keyboard 2.1.3 "cordova-plugin-ionic-keyboard"
cordova-plugin-ionic-webview 2.2.1 "cordova-plugin-ionic-webview"
cordova-plugin-splashscreen 5.0.2 "Splashscreen"
cordova-plugin-statusbar 2.4.2 "StatusBar"
cordova-plugin-whitelist 1.3.3 "Whitelist"

If you use @ionic-native/google-maps, please tell the package.json (only @ionic-native/core and @ionic-native/google-maps are fine mostly)

Current behavior: The plugin was working this morning but I removed and re-added the cordova plugin and then I started getting this error in Xcode as soon as the app opens in the emulator. I then tried adding it to a fresh starter project and got the same error.

Expected behavior: There should be no Xcode error.

Screen capture or video record:

screen shot 2018-11-07 at 12 41 57 pm

Related code, data or error log (please format your code or data):

I can upload the mostly empty starter app if needed, but I don't believe it will be necessary in this case. It seems like the latest update was not tested on iOS.

wf9a5m75 commented 5 years ago

Please share the project files on github.

battika commented 5 years ago

I have seen this one (SIGABRT) recently every now and then on IOS and very unexpectedly. I did two things, updated wkwebview to the latest version (v2.2.x) and replaced animateCamera() with moveCamera() everywhere in my code a I believe animateCamera() can cause issues when something happens before the animation has been completed. It significantly lowered crashing and unfortunately ditched support for IOS 9 as latest version of wkwebview dropped support of it. I am not sure if you have the same issue but something that might worth trying.

wf9a5m75 commented 5 years ago

Good see you, @battika.

Do you use the multiple_maps branch? Could you make an example project files that reproduce your issue?

battika commented 5 years ago

Hello Masashi, good to see you too :) I am using the latest released version 2.4.6.

I was going to see if I can reproduce the issue with a basic project. Hope to have some time soon to create a sample project.

In the meantime, hope the creator of this issue can provide the source code.

goldman94 commented 5 years ago

Here is my repo: https://github.com/goldman94/testMaps

Thank you for the suggestion, @battika. I will try updating wkwebview.

wf9a5m75 commented 5 years ago

@goldman94 As far as I see your repo, there is no code. https://github.com/goldman94/testMaps/blob/8a09b9a5d39eddc9e331063b1cf76459ac74f294/src/pages/home/home.ts

goldman94 commented 5 years ago

Sorry! Updated now.

goldman94 commented 5 years ago

Oh, I guess I just needed to run "pod install" again. Sorry to waste your time guys. Thanks for the effort though, I appreciate it.