Opened 14 years ago

Closed 13 years ago

#8787 closed Bug Report (Fixed)

Selection jumps to top when modifying AutoExpire List in System Status screen.

Reported by: Marc Alban <marcalban@…> Owned by: stuartm
Priority: trivial Milestone: 0.25
Component: MythTV - General Version: Master Head
Severity: low Keywords:
Cc: Ticket locked: no

Description

When disabling auto-expire, deleting, or undeleting a recording from this screen, the selection always jumps to the top. This is very frustrating when the list of recordings is large.

For instance, I have nearly 1000 recordings that are either deleted or auto-expirable and I will occasionally want to scan this list to disable auto-expire on recordings I have decided I really don't want to lose. Jumping to the top of the list each time makes this task needlessly tedious.

This is similar to ticket #8460 with the recording rule screen.

I've attached a patch to correct this behavior.

Attachments (1)

statusbox.patch (3.0 KB) - added by Marc Alban <marcalban@…> 14 years ago.
Patch to correct selection jumping on AutoExpire? List screen.

Download all attachments as: .zip

Change History (8)

Changed 14 years ago by Marc Alban <marcalban@…>

Attachment: statusbox.patch added

Patch to correct selection jumping on AutoExpire? List screen.

comment:1 Changed 14 years ago by robertm

Owner: set to stuartm
Status: newassigned

comment:2 Changed 14 years ago by stuartm

Milestone: unknown0.24
Status: assignedaccepted

comment:3 Changed 14 years ago by stuartm

(In [26203]) Don't redraw the list when deleting a list item in StatusBox?. Refs #8787

comment:4 Changed 13 years ago by stuartm

Milestone: 0.240.24.1

comment:5 Changed 13 years ago by stuartm

Type: patchBug Report

comment:6 Changed 13 years ago by Github

Fix deleting expired recordings in the status screen.

There was a bug introduced by https://github.com/MythTV/mythtv/commit/48fb2493 which was preventing recordings from being deleted or their recording group being changed.

This changes the expire button list back to reloading after something is changed, each item in the list needs to be updated to reflect the new stats anyway so there is little advantage to not reloading the list completely. To speed things up we only grab the expire list once from the BE. We also restore the position after refreshing the button list which was the original complaint being fixed by 48fb2493.

Refs #9286, #8787.

Branch: master Changeset: 9649c3ca78580a47643ba349b1038d190bedffad

comment:7 Changed 13 years ago by paulh

Milestone: 0.24.10.25
Resolution: Fixed
Status: acceptedclosed
Note: See TracTickets for help on using tickets.