From c390066541a3e5d769fdbec37be9d4fa8918b6bf Mon Sep 17 00:00:00 2001 From: Johan Castiblanco Date: Thu, 5 Sep 2024 15:46:51 -0500 Subject: [PATCH] feat: remove staff access for catalog views This is controversial, but simple. To do this in a monkey patch is a bigger task and also with more work. Is different due the change is in a nested function. Anyway the alternative is to monkeyPatch the _dispatch method and handle there. But its risky and also that method manage other permission with different ways. --- lms/djangoapps/courseware/access.py | 1 - 1 file changed, 1 deletion(-) diff --git a/lms/djangoapps/courseware/access.py b/lms/djangoapps/courseware/access.py index f71f2daec5bb..e8bff5d1b1da 100644 --- a/lms/djangoapps/courseware/access.py +++ b/lms/djangoapps/courseware/access.py @@ -394,7 +394,6 @@ def can_see_in_catalog(): """ return ( _has_catalog_visibility(courselike, CATALOG_VISIBILITY_CATALOG_AND_ABOUT) - or _has_staff_access_to_descriptor(user, courselike, courselike.id) ) @function_trace('can_see_about_page')