Skip to content
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

Skip programs with no starttime #24

Closed
dmfrey opened this issue Apr 1, 2015 · 12 comments
Closed

Skip programs with no starttime #24

dmfrey opened this issue Apr 1, 2015 · 12 comments
Assignees
Labels

Comments

@dmfrey
Copy link
Contributor

dmfrey commented Apr 1, 2015

java.lang.RuntimeException: An error occured while executing doInBackground()
at android.os.AsyncTask$3.done(AsyncTask.java:300)
at java.util.concurrent.FutureTask.finishCompletion(FutureTask.java:355)
at java.util.concurrent.FutureTask.setException(FutureTask.java:222)
at java.util.concurrent.FutureTask.run(FutureTask.java:242)
at android.os.AsyncTask$SerialExecutor$1.run(AsyncTask.java:231)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1112)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:587)
at java.lang.Thread.run(Thread.java:818)
Caused by: java.lang.NullPointerException: Attempt to invoke virtual method 'long org.joda.time.DateTime.getMillis()' on a null object reference
at org.mythtv.android.library.persistence.service.dvr.DvrPersistenceServiceEventHandler.updateRecordedPrograms(DvrPersistenceServiceEventHandler.java:223)
at org.mythtv.android.library.core.service.v028.dvr.DvrServiceV28EventHandler.updateRecordedPrograms(DvrServiceV28EventHandler.java:92)
at org.mythtv.android.library.core.utils.RefreshRecordedProgramsTask.doInBackground(RefreshRecordedProgramsTask.java:43)
at org.mythtv.android.library.core.utils.RefreshRecordedProgramsTask.doInBackground(RefreshRecordedProgramsTask.java:12)
at android.os.AsyncTask$2.call(AsyncTask.java:288)
at java.util.concurrent.FutureTask.run(FutureTask.java:237)
... 4 more

Google Play Console error

@dmfrey dmfrey added the bug label Apr 1, 2015
@dmfrey dmfrey self-assigned this Apr 1, 2015
@dmfrey dmfrey closed this as completed in b21c407 Apr 1, 2015
@dmfrey
Copy link
Contributor Author

dmfrey commented Apr 9, 2015

Seems to be an issue with null end times as well

@dmfrey dmfrey reopened this Apr 9, 2015
@dmfrey dmfrey closed this as completed in 349c669 Apr 9, 2015
@dmfrey
Copy link
Contributor Author

dmfrey commented Apr 10, 2015

Affecting Recording Start and Ends

Reported on Google Play Console

@dmfrey dmfrey reopened this Apr 10, 2015
@dmfrey dmfrey closed this as completed in 4b6325a Apr 10, 2015
@fphillips
Copy link
Contributor

I share my recorded json dump with your gmail addy, Daniel. Look for an email for google drive.

@dmfrey
Copy link
Contributor Author

dmfrey commented Apr 10, 2015

Thank you @fpillips

@dmfrey
Copy link
Contributor Author

dmfrey commented Apr 10, 2015

@fpillips In firefox, install the JSONView extension and then open the file you sent me. It should be formatted for clearer reading. The very first Program entry has a lot of fields that are all empty. I have been going through the list, but that seems to be the only one i have found so far.

@fphillips
Copy link
Contributor

Yeah, I had seen the one for 0_00000000000000_00000000000000.nuv. It has an empty title/subtitle, notably. I was thinking from discussions way back this was some sort of default and expected or innocuous, but maybe not. I'll see if I can find any reference to where it came from.

@dmfrey
Copy link
Contributor Author

dmfrey commented Apr 10, 2015

Are you in the beta on the Google+ community? If so, I can do a quick build
of 1.1.6, which has the fix for this (missing recording start and end ts,
as well as already doing program start and end ts, etc.) and push it to the
beta channel. Then, it should come down to your phone. You can try it out
and it should just dump this one entry. Then you won't have to do anything
on your end.

Does this recording break anything on any of your frontends?

On Fri, Apr 10, 2015 at 3:50 PM Frank Phillips notifications@github.com
wrote:

Yeah, I had seen the one for 0_00000000000000_00000000000000.nuv. It has
an empty title/subtitle, notably. I was thinking from discussions way back
this was some sort of default and expected or innocuous, but maybe not.
I'll see if I can find any reference to where it came from.


Reply to this email directly or view it on GitHub
#24 (comment)
.

@fphillips
Copy link
Contributor

Not in the beta, but I can be. I haven't seen any problems on linux frontends that would make me think is related. The old android mythtv frontend stopped working a year or two ago, but I remember seeing this nuv entry much longer before that. I'm pretty sure it was inserted when I recreated my database in 2009, circa 0.21.

@dmfrey
Copy link
Contributor Author

dmfrey commented Apr 10, 2015

version 1.1.6 has been published to the beta channel. It will be available in an hour or so. Go to the G+ Community and accept the beta link. When it is available, it should just come down like any other public version.

@fphillips
Copy link
Contributor

I had been trying the beta program link and getting an error page, waiting and thinking that the build wasn't available yet. Finally saw the 'join community' button at the top - that's what I was missing. Didn't want to wait for it to auto-update, so I force-stopped the play store app and it saw the update and soon as I went back into the play store. It works now.

@dmfrey
Copy link
Contributor Author

dmfrey commented Apr 10, 2015

That's great. Let me know if you are missing any recorded programs. I will
try to diagnose the json further to try to find whatever is missing.

On Fri, Apr 10, 2015, 6:52 PM Frank Phillips notifications@github.com
wrote:

I had been trying the beta program link and getting an error page, waiting
and thinking that the build wasn't available yet. Finally saw the 'join
community' button at the top - that's what I was missing. Didn't want to
wait for it to auto-update, so I force-stopped the play store app and it
saw the update and soon as I went back into the play store. It works now.


Reply to this email directly or view it on GitHub
#24 (comment)
.

@fphillips
Copy link
Contributor

I have four recgroups: Default, Live TV, Movie, and Sitcom. When I exclude Live TV, there are 855 entries. The app shows 854, so it is only excluding the nuv, as expected.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants