Opened 9 years ago

Last modified 11 months ago

#7434 assigned Bug Report - General

max episodes code doesn't respect autoexpire flag on a recording

Reported by: Stuart Auchterlonie Owned by: Stuart Auchterlonie
Priority: major Milestone: 29.2
Component: MythTV - General Version: Master Head
Severity: high Keywords:
Cc: stuartm Ticket locked: no

Description

I've a recording rule setup as follows.

  • Max episodes = 20
  • Record new / delete old
  • Allow autoexpire

I decided i wanted to keep 1 of the 20 recordings and set autoexpire=off on that recording.

However when it recorded a new episode, that episode was removed because it was the oldest, in direct violation of autoexpire=off.

Stuart

Change History (13)

comment:1 Changed 8 years ago by robertm

Status: newassigned

comment:2 Changed 8 years ago by cpinkham

Resolution: wontfix
Status: assignedclosed
Type: defectenhancement
Version: 0.22rc1Trunk Head

With reservation, I'm closing this as a feature request without a patch. This code has been this way for going on 6 years. The original author and the ticket reporter may reopen this and make changes later if they desire.

comment:3 Changed 8 years ago by Stuart Auchterlonie

Resolution: wontfix
Status: closednew

comment:4 Changed 8 years ago by Stuart Auchterlonie

Owner: changed from cpinkham to Stuart Auchterlonie
Status: newassigned

I'll take this back then. I feel it's quite a significant bug when an episode marked to not autoexpire is autoexpired.

Stuart

comment:5 Changed 8 years ago by stuartm

Cc: stuartm added
Priority: minormajor
Severity: mediumhigh

comment:6 Changed 7 years ago by stuartm

Milestone: unknown0.26
Type: enhancementBug Report - General

We need to simplify the behaviour here to match expectations, having two separate flags (preserve and autoexpire) is just too confusing. I appreciate the attempt to satisfy everyone, but I really do agree with Stuart that maybe it's better if disabling auto-expire acts the same way as preserve and preserve was eliminated.

If we do that then it would probably help to rename it from 'Disable Auto-expire' to 'Block Deletion' or similar and make it a catch-all state that doesn't allow any code to delete that recording without the 'block' being removed. That's simple enough for everyone to understand.

comment:7 Changed 6 years ago by Kenni Lund [kenni a kelu dot dk]

Milestone: 0.260.26.1

comment:8 Changed 5 years ago by paulh

Milestone: 0.26.10.28

comment:9 Changed 3 years ago by Stuart Auchterlonie

Milestone: 0.280.29

comment:10 Changed 3 years ago by Stuart Auchterlonie

Milestone: 0.2929.0

Milestone renamed

comment:11 Changed 13 months ago by Stuart Auchterlonie

Milestone: 29.029.1

comment:12 Changed 11 months ago by Stuart Auchterlonie

Milestone: 29.10.28.2

Moving remaining open tickets to 0.28.2 milestone

comment:13 Changed 11 months ago by Stuart Auchterlonie

Milestone: 0.28.229.2

Moving remaining open tickets to 29.2 milestone

Note: See TracTickets for help on using tickets.