Merge pull request #21924 from edx/BOM-813

Updated the comparison operation to fix None comparison with an integer - BOM-813
2019-10-09 12:07:13 +05:00
parent bbadf48b8e af20d0f7d2
commit 4c9aa43c2f
1 changed files with 1 additions and 1 deletions
--- a/common/lib/xmodule/xmodule/randomize_module.py
+++ b/common/lib/xmodule/xmodule/randomize_module.py
@@ -44,7 +44,7 @@ class RandomizeModule(RandomizeFields, XModule):
        # NOTE: calling self.get_children() doesn't work until we've picked a choice
        num_choices = len(self.descriptor.get_children())

-        if self.choice > num_choices:
+        if self.choice is not None and self.choice > num_choices:
            # Oops.  Children changed. Reset.
            self.choice = None