From f319d20d6ddb07131d15034698197e0d284ae0e3 Mon Sep 17 00:00:00 2001 From: Calen Pennington Date: Mon, 5 Aug 2019 14:29:16 -0400 Subject: [PATCH] Fix publishing for access control experiment --- lms/djangoapps/courseware/rules.py | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/lms/djangoapps/courseware/rules.py b/lms/djangoapps/courseware/rules.py index ed77878697..00326ec44a 100644 --- a/lms/djangoapps/courseware/rules.py +++ b/lms/djangoapps/courseware/rules.py @@ -4,6 +4,7 @@ django-rules and Bridgekeeper rules for courseware related features from __future__ import absolute_import import logging +import traceback import laboratory import rules @@ -63,15 +64,16 @@ class HasAccessRule(Rule): class StaffAccessExperiment(laboratory.Experiment): def compare(self, control, candidate): - return bool(control) == candidate + return bool(control.value) == candidate.value def publish(self, result): if not result.match: + LOG.warning( - u"StaffAccessExperiment: control=%r, candidate=%r", + u"StaffAccessExperiment: control=%r, candidate=%r\n%s", result.control, result.candidates[0], - exc_info=True + "".join(traceback.format_stack(limit=10)) )