-
Notifications
You must be signed in to change notification settings - Fork 58
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
App Crash when opening #63
Comments
Can confirm this on Lenovo Tab 7" tablet, which also crash upon start with Android version 7.0 and latest app version from Google Play on 26th of March. Haven't checked any logs yet. |
I can also confirm this crash happening on app launch. Logs from logcat (Lenovo Tab with android 8.1.0):
|
It turns out that there was a calendar with a null name. A null check in DataProxy.getRoomNames seems to get rid of the crash:
--->
|
Can you explain to a complete newbie how one might deploy that fix? |
Rough steps for newbie to deploy the fix:
|
Thank you, saynomoo, that's exactly the direction I needed! (But you forgot the ? step. ;) ) |
So I had no luck on the code fix end, BUT I did seem to find a workaround for this on the tablet side. If you replace Google Calendar (which was my built-in calendar app) with another (I used aCalendar 2,) you can then go into aCalendar's settings and delete the "PC Sync" calendar (which I believe was the problem) and then Reservator will launch! |
The null value seems to end up to the roomName here: Line 291 in 0e9e950
which means that the key CalendarContract.Calendars.NAME is not defined in some calendars. We have replaced this with CalendarContract.Calendars.DISPLAY_NAME in our Ubikampus fork of Reservator: https://github.com/ubikampus/reservator/blob/e121e0dbc61dfbfa70b984d8d94727bc06f14dc3/app/src/main/java/com/futurice/android/reservator/model/platformcalendar/PlatformCalendarDataProxy.java#L364 but according to the issues open in our fork, this key might not be defined in all calendars either. I think the safest fix here would be to add null checking in PlatformCalendarDataProxy and to use "UNNAMED" or similar as calendar name if CalendarContract returns null. |
Expected behavior
Clicking on the app icon on phone i should be able to open and navigate within the application
Actual behavior
When i clicked to open, the app crashed
How to reproduce
Download Reservator
Launch
Device used: Tecno L9 plus
Android Version: 7.0
App version: 5.1.1
Note: I tried 3 screen recorder but could not work on recording the bug
Recording Of The Bug
https://www.youtube.com/watch?v=9YHVqzmzWUE
Posted on Utopian.io - Rewarding Open Source Contributors
The text was updated successfully, but these errors were encountered: