Opened 9 years ago

Closed 9 years ago

#8396 closed defect (fixed)

LoadProgramFromRecorded() omits some metadata

Reported by: ylee@… Owned by: danielk
Priority: minor Milestone: 0.24
Component: MythTV - General Version: Master Head
Severity: medium Keywords: year yearstars
Cc: Ticket locked: no

Description

(This bug is likely related to #8392. While that bug existed in 0.22 this is new to 0.23, though.)

After playing a recording then exiting with ESCAPE back to Watch Recordings, the frontend no longer "sees" the year data for that recording. This causes odd behavior when using the %YEAR% or %YEARSTARS% field in the theme. If other recordings are entered and exited, mythfrontend will also forget their year data. Reloading Watch Recordings by either exiting to the main menu then reentering, or by using the TV Recording Playback jumppoint during playback, fixes the issue temporarily.

Change History (7)

comment:1 Changed 9 years ago by stuartm

Component: MythTV - User Interface LibraryMythTV - General
Milestone: unknown0.23
Status: newaccepted

comment:2 Changed 9 years ago by stuartm

(In [24300]) Include airdate(year) in the metadata pulled by LoadProgramFromRecorded?(), this fixes the disappearing year and rating information when update/add events are received in the Watch Recordings screen. Refs #8396

comment:3 Changed 9 years ago by stuartm

Summary: %YEAR% and %YEARSTARS% lack year data after playing recordingLoadProgramFromRecorded() omits some metadata

Chances are that it misses additional columns so I'll leave the ticket open until I've had time to compare this database query against the others.

comment:4 Changed 9 years ago by stuartm

(In [24301]) Backport [24300] to 0.23-fixes. Include airdate(year) in the metadata pulled by LoadProgramFromRecorded?(), this fixes the disappearing year and rating information when update/add events are received in the Watch Recordings screen. Refs #8396

comment:5 Changed 9 years ago by Stuart Auchterlonie

Milestone: 0.230.23-fixes

comment:6 Changed 9 years ago by stuartm

Milestone: 0.23-fixes0.24
Owner: changed from stuartm to danielk
Status: acceptedassigned
Version: 0.23rc2Trunk Head

Daniel, assigning to you as you are refactoring this code for 0.24

comment:7 Changed 9 years ago by danielk

Resolution: fixed
Status: assignedclosed

This should be fixed in trunk, we now use the same query again when loading from the recorded table at different parts of the code. The base query is now a ProgramInfo? constant so that should prevent divergence in the future.

Note: See TracTickets for help on using tickets.