Revert "Lazily loading capa problems (for faster grading)."

We don't want lazyness, and will save the proper fix (moving work into descriptor load for later)

common/lib/xmodule/xmodule/capa_module.py

@@ -15,7 +15,6 @@ from capa.capa_problem import LoncapaProblem
 from capa.responsetypes import StudentInputError
 from capa.util import convert_files_to_filenames
 from progress import Progress
-from xmodule.util.decorators import lazyproperty
 from xmodule.x_module import XModule
 from xmodule.raw_module import RawDescriptor
 from xmodule.exceptions import NotFoundError
@@ -122,8 +121,6 @@ class CapaModule(XModule):
 
         if instance_state is not None:
             instance_state = json.loads(instance_state)
-        self._instance_state = instance_state # We will need this later to lazily load lcp
-        
         if instance_state is not None and 'attempts' in instance_state:
             self.attempts = instance_state['attempts']
 
@@ -135,17 +132,6 @@ class CapaModule(XModule):
         else:
             self.weight = None
 
-
-
-    @lazyproperty
-    def lcp(self):
-        """
-        Loading the problem can be fairly expensive, so we do this lazily.
-        
-        This can throw errors. Always be sure to use try/catch before
-        displaying modules to a user.
-        """
-        
         if self.rerandomize == 'never':
             seed = 1
         elif self.rerandomize == "per_student" and hasattr(self.system, 'id'):
@@ -154,8 +140,10 @@ class CapaModule(XModule):
             seed = None
 
         try:
-            return LoncapaProblem(self.definition['data'], self.location.html_id(),
-                                      self._instance_state, seed=seed, system=self.system)
+            # TODO (vshnayder): move as much as possible of this work and error
+            # checking to descriptor load time
+            self.lcp = LoncapaProblem(self.definition['data'], self.location.html_id(),
+                                      instance_state, seed=seed, system=self.system)
         except Exception as err:
             msg = 'cannot create LoncapaProblem {loc}: {err}'.format(
                 loc=self.location.url(), err=err)
@@ -172,14 +160,13 @@ class CapaModule(XModule):
                 problem_text = ('<problem><text><font color="red" size="+2">'
                                 'Problem %s has an error:</font>%s</text></problem>' %
                                 (self.location.url(), msg))
-                return LoncapaProblem(
+                self.lcp = LoncapaProblem(
                     problem_text, self.location.html_id(),
-                    self._instance_state, seed=seed, system=self.system)
+                    instance_state, seed=seed, system=self.system)
             else:
                 # add extra info and raise
                 raise Exception(msg), None, sys.exc_info()[2]
 
-
     @property
     def rerandomize(self):
         """