watchlist showing oldest and more group summary data

[nzhook]

Attached is one patch that provides two separate things (different sections in the patch so only one can be applied if required)

• When in the Watch Recordings screens and selecting group, the summary displays 'there are X recordings in this display group', however it does not say the total amount of time or the total disk space. The first part of this patch calculates these two figures and assigns them to the standard template vars (lentime, lenmins, filesize, and filesize_str)
• The watchlist currently displays the newest recordings, normally however the oldest are more important as they maybe deleted first or should be watched in order to keep in sync with what is going on. The second part of this patch adds the oldest recording to the watchlist.

Note: the second part also includes some additional debug info for doing a grep in the debug output to find the watchlist info. While its not required for the patch it will be useful when debugging any issues that occur when looking at the watchlist.

This patch applies was last updated using SVN #23010, but applies cleanly to SVN #23421 (patch from this revision)

[James Hook <mythtv.hook@…>]

Patch

[robertm]

The space portion of this is basically accomplished by the reintroduction of the delete recordings screen, if something like this is going to be considered you might just add the "time" element to that? Personally, though, I am not a huge fan of the lack of flexibility of this stuff, it doesn't give the themers any control over how the information is presented. If you modify this to add the time stuff to delete recordings, please reopen with that patch and we can review for you.

[nzhook <mythtv.hook@…>]

Sorry for taking so long to get back regarding this.

From your response I dont think you may have mis read this as something the delete recordings screen should have. Which is not really something this patch was designed to provide.

This patch provides the totals of the numbers when in the group selection, rather than the 'there are X recordings in this display group', thus giving themers more information to work with than a single line which they are unable to change. (which BTW this was a patch I created when I was creating a theme but never got the time to finish)

eg. If the user has 3 recordings in a group:

recording 1 1 hour recording 2 2 hours recording 3 30 minutes

when the user is on the group selection the lentime value available to the theme is set to 2 hours 30 minutes rather than being blank. It also does this with the other numeric values (such as disk space)

I use this patch on my frontend on a regular basis to see how much total time I have to watch still, eg. im a little behind on Lost, In fact this patch tells me im 4 hours behind.

The other part of the patch, is that it lists the oldest Lost episode before the most recent. I dont want to watch episode 13 of before I watch episode 11, currently however the Watch Recordings list is showing episode 13 and not episode 11, which is what the patch fixes.

[robertm]

No change to patch to add requested flexibility.

[robertm]

Re-opening, we'd like to work with you to make some tweaks to this so that everyone is happy.

[paulh]

Updated following the PI refactor

[paulh]

Updated the size and length part of the patch after the PI changes and adds the required text area to MythCenter?-wide.

[stuartm]

I always like exposing more data in the UI, however it would be better not to re-use existing textareas, there are more interesting things possible if group/recording info can be displayed simultaneously or with a different layout.

This would also mean changing the existing code so that group name isn't inserted into 'title' and group info isn't in 'description'.

With 'depends' and <group> we can hide those strings if desired when the recording list is selected and still allow the space to be re-used.

[Stuart Auchterlonie]

Moving unresolved tickets to next point release

[Stuart Auchterlonie]

Moving remaining open 0.28.1 tickets to 0.28.2

[Peter Bennett]

I suggest that this ticket should be superseded by #12296 which covers many improvements to the watchlist.

[Stuart Auchterlonie]

Closing any remaining tickets for 0.28, if the issue persists, feel free to reopen and align to v29 or master