Opened 6 years ago

Closed 6 years ago

#10388 closed Bug Report - General (Fixed)

Parental level don't work

Reported by: raptorjr08@… Owned by: stuartm
Priority: minor Milestone: 0.25.1
Component: MythTV - Video Library Version: 0.25-fixes
Severity: medium Keywords:
Cc: Ticket locked: no


Using mythfrontend version: master [v0.25pre-4709-g7c06592]. I have several files with level 3 and 4. Setting is to start at level 1. When i select Watch Videos i see all files, even level 3 and 4. If i change to level 2, level 3 and 4 files disappear. And trying to go to levels 3 or 4 asks for password as it should be.

Attachments (1)

parentalLevel_fix.diff (1019 bytes) - added by stuartm 6 years ago.

Download all attachments as: .zip

Change History (10)

comment:1 Changed 6 years ago by bhuffman@…

I see the same problem. Some more info: When entering MythVideo?, it shows the effective parental level as "0" for the gallery instead of the Level 1 that I've set in the settings.

comment:2 Changed 6 years ago by Marc Randolph <mrand@…>

comment:3 Changed 6 years ago by robertm

  • Owner changed from robertm to stuartm
  • Status changed from new to assigned

Assigning to the responsible party

comment:4 Changed 6 years ago by gmaltby@…

This issue is effecting me too. It makes 0.25 unacceptible for us because of the inability to restrict children to only suitable movies.

Is there any consideration for Marc Randolph's suggested solution? Working parental levels is far preferable to a one time offer to scan for movies.

comment:5 Changed 6 years ago by kenni

  • Ticket locked set

Please don't post me-too messages in tickets, as they don't add anything useful to the ticket. Use the mythtv-users mailing list for questions/discussions. See the TicketHowTo.

comment:6 Changed 6 years ago by stuartm

Marc's solution won't fix this, we have two different places where level is considered and I'm not even sure the filter code is connected to the default parental level. The code is an absolute car crash, what was once so very simple has become so unnecessarily complicated that I'm very tempted just to delete it all. Or maybe just revert the whole thing back to how it was immediately following the mythui port when it was so much simpler and just worked.

Changed 6 years ago by stuartm

comment:7 Changed 6 years ago by stuartm

Can you please test the new patch I've attached? This moves the setting of the initial parent level to before the load of the video list.

comment:8 Changed 6 years ago by stuartm

  • Ticket locked unset

comment:9 Changed 6 years ago by stuartm

  • Milestone changed from unknown to 0.25.1
  • Resolution set to Fixed
  • Status changed from assigned to closed
  • Version changed from Unspecified to 0.25-fixes

Fixed in [6fe9ad06] and backported to 0.25-fixes in [5b946512]

Add Comment

Modify Ticket

as closed The owner will remain stuartm.
The resolution will be deleted. Next status will be 'new'.

E-mail address and user name can be saved in the Preferences.

Note: See TracTickets for help on using tickets.