New launchers seem to run into init-issues #91
Labels
No Label
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
No Milestone
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: Starfish/TinyWeatherForecastGermany#91
Loading…
Reference in New Issue
There is no content yet.
Delete Branch "%!s(<nil>)"
Deleting a branch is permanent. Although the deleted branch may exist for a short time before cleaning up, in most cases it CANNOT be undone. Continue?
0.56.4 introduced new launchers for
Warnings
andTexts
, which is a pretty handy feature.However, if Always update is checked, and use either launcher, I see spurious crashes. The app only opens for a blink and is gone again. From a gut feeling some init is missing. If I update data from the app frist everything seems fine as is the case if the app has recent data. (Looks like it is triggering the upate, but dies before everything is there.)
Thus spoke the log:
I had the crash once when changing the language, but was not able to reporoduce it. Unfortunately, I had the device not connected via adb when it occured. Will watch it. However, I currently regard this a minor issue because it does not really break the app.
Here it happens quite regularly.
Does this help:
(My first attempt with
adb logcat
This one is form theTexts
launcher failing. Right now the warnings launcher is fine. Maybe, as there are no warnings.)This is fixed now, the reason was that the warning activity tried to build some views before the theme was properly set.
You can test with the theme preview userbuild.
Just installed
TinyWeatherForecastGermany-build027-version_0.56.4-userdebug-20210815-new_theme_support
and indeed this seems to work. :) Thanks for the fix!May I ask: the log above I created using
adb
was this helpful? Is this the sort of log you can use easily? Or should I have done something differently?Yes, it was very helpful. However, it needs some experience that Error inflating class unknown results from an attribute where the value has not been set yet. :-)
There is a similar issue if one uses the starter for text forecasts:
(Userdebug build 027 20210819-widget_themes)
Fixed:
23a1330005
Explanation: the BaseAdapter responsible for the list displayed had a wrong context, and because of the new theme support the adapter was not able to find the proper attributes for the layout.
This error was both the same in the warnings and the texts activity.
Again, thanks a lot for reporting!
By the way: the log helped a lot!
Thanks for looking into it. Just installed the 20210828-RC1 userdebug and if I find something else, I'll report back.
I already created arwagner/TinyWeatherForecastGermany#2 to write up some how to create this for mere mortals. Many thanks to @dicer for his helpful comment in #74 (comment) that enabled me to pull this off :)
This would be great, you may notice that a lot of details changed ragarding the themes.
There is also a simplified forecast view available, for all the folks that want to keep it simple :-)
Would you be so kind to uninstall and re-install it, too?
Just to be sure we do not run into any "database preparing crash" again :-)
Yes. It looks cool.
I was wondering why it asked me quite a bit of my setup again, so I feared that I had to set up everything a new, which wasn't the case.
Ok, I'll give it a try later.
Yes, the setup respects any changes made before.
Reason for this screen is to customize the app to the user's needs as streightforward as possible.
I am curious about the feedback. :-)
I've moved all my findings to #96.