Add versioning to database; migrate if version is not current

kalite/distributed/management/commands/initialize_kalite.py

@@ -22,17 +22,19 @@ class Command(BaseCommand):
     )
 
     def setup_server_if_needed(self):
-        """Run the setup command, if necessary."""
-
-        # Ensure that the database has been synced and a Device has been created
+        """Run the setup command, if necessary.
+            It's necessary if the Settings model doesn't have a "database_version" or if that version doesn't match
+            kalite.version.VERSION, indicating the source has been changed. Then setup is run to create/migrate the db.
+        """
+
         try:
-            assert Settings.get("private_key") and Device.objects.count()
+            from kalite.version import VERSION
+            assert Settings.get("database_version") == VERSION
         except (DatabaseError, AssertionError):
-            # Otherwise, run the setup command
-            self.stdout.write("Setting up KA Lite; this may take a few minutes; please wait!\n")
+            logging.info("Setting up KA Lite; this may take a few minutes; please wait!\n")
             call_command("setup", interactive=False)
-        # Double check that the setup process successfully created a Device
-        assert Settings.get("private_key") and Device.objects.count(), "There was an error configuring the server. Please report the output of this command to Learning Equality."
+            # Double check the setup process worked ok.
+            assert Settings.get("database_version") == VERSION, "There was an error configuring the server. Please report the output of this command to Learning Equality."
 
     def reinitialize_server(self):
         """Reset the server state."""

kalite/distributed/management/commands/setup.py

@@ -365,6 +365,15 @@ def handle(self, *args, **options):
         # Should clean_pyc for (clean) reinstall purposes
         # call_command("clean_pyc", interactive=False, verbosity=options.get("verbosity"), path=os.path.join(settings.PROJECT_PATH, ".."))
 
+        # Migrate the database
+        call_command(
+            "syncdb", interactive=False, verbosity=options.get("verbosity"))
+        call_command("migrate", merge=True, verbosity=options.get("verbosity"))
+        # Create *.json and friends database
+        call_command("syncdb", interactive=False, verbosity=options.get(
+            "verbosity"), database="assessment_items")
+        Settings.set("database_version", VERSION)
+
         # download assessment items
         # This can take a long time and lead to Travis stalling. None of this
         # is required for tests, and does not apply to the central server.
@@ -373,11 +382,7 @@ def handle(self, *args, **options):
             logging.warning("Skipping assessment item downloading and configuration.")
 
         else:
-
-            # Migrate the database
-            call_command(
-                "syncdb", interactive=False, verbosity=options.get("verbosity"))
-            call_command("migrate", merge=True, verbosity=options.get("verbosity"))
+
             # Outdated location of assessment items - move assessment items from their
             # old location (CONTENT_ROOT/khan where they were mixed with other content
             # items)