Opened 7 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

Description

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 7 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 7 years ago by Marc Randolph <mrand@…>

comment:3 Changed 7 years ago by robertm

Owner: changed from robertm to stuartm
Status: newassigned

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 Lund [kenni a kelu dot dk]

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

Attachment: parentalLevel_fix.diff added

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: unknown0.25.1
Resolution: Fixed
Status: assignedclosed
Version: Unspecified0.25-fixes

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

Note: See TracTickets for help on using tickets.