From 15b8b99ae27f4dde25609522f4ae09651dda2f33 Mon Sep 17 00:00:00 2001
From: Anand Chitipothu <anandology@gmail.com>
Date: Wed, 7 Apr 2021 01:19:42 +0530
Subject: [PATCH 1/3] fix: fixed the issue with dynamic routes.

- The code to install regex controller was not always executed. Forced the
  exection of that by adding it to community/__init__.py
- Renoved the that code from hooks.py
---
 community/__init__.py | 2 ++
 community/hooks.py    | 3 ---
 2 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/community/__init__.py b/community/__init__.py
index 95d1338e..eb67ffe7 100644
--- a/community/__init__.py
+++ b/community/__init__.py
@@ -3,3 +3,5 @@ from __future__ import unicode_literals
 
 __version__ = '0.0.1'
 
+from .routing import install_regex_converter
+install_regex_converter()
diff --git a/community/hooks.py b/community/hooks.py
index 7669e2fe..bdc9e026 100644
--- a/community/hooks.py
+++ b/community/hooks.py
@@ -127,9 +127,6 @@ scheduler_events = {
 #
 # auto_cancel_exempted_doctypes = ["Auto Repeat"]
 
-from .routing import install_regex_converter
-install_regex_converter()
-
 # Add all simple route rules here
 primary_rules = [
     {"from_route": "/sketches", "to_route": "sketches"},

From f1508033a3499b8329b9338748db3ad582a87e38 Mon Sep 17 00:00:00 2001
From: Anand Chitipothu <anandology@gmail.com>
Date: Wed, 7 Apr 2021 01:32:19 +0530
Subject: [PATCH 2/3] feat: implemented autosave for sections

Now the changes made to the code in each section will be autosaved and loaded
back on next page load.
---
 community/__init__.py                         |  3 ++
 community/lms/api.py                          | 23 +++++++++++++
 .../lms/doctype/lms_section/lms_section.py    | 20 ++++++++++-
 community/www/courses/topic.html              | 33 +++++++++++++++++--
 community/www/macros/livecode.html            |  3 +-
 5 files changed, 78 insertions(+), 4 deletions(-)
 create mode 100644 community/lms/api.py

diff --git a/community/__init__.py b/community/__init__.py
index eb67ffe7..65fc9069 100644
--- a/community/__init__.py
+++ b/community/__init__.py
@@ -5,3 +5,6 @@ __version__ = '0.0.1'
 
 from .routing import install_regex_converter
 install_regex_converter()
+
+# load the methods from the lms api
+from .lms import api # noqa
diff --git a/community/lms/api.py b/community/lms/api.py
new file mode 100644
index 00000000..ab69ede4
--- /dev/null
+++ b/community/lms/api.py
@@ -0,0 +1,23 @@
+"""API methods for the LMS.
+"""
+
+import frappe
+
+@frappe.whitelist()
+def autosave_section(section, code):
+    """Saves the code edited in one of the sections.
+    """
+    doc = frappe.get_doc(
+        doctype="Code Revision",
+        section=section,
+        code=code,
+        author=frappe.session.user)
+    doc.insert()
+    return {"name": doc.name}
+
+@frappe.whitelist()
+def get_section(name):
+    """Saves the code edited in one of the sections.
+    """
+    doc = frappe.get_doc("LMS Section", name)
+    return doc and doc.as_dict()
diff --git a/community/lms/doctype/lms_section/lms_section.py b/community/lms/doctype/lms_section/lms_section.py
index ad40ece6..f51fec69 100644
--- a/community/lms/doctype/lms_section/lms_section.py
+++ b/community/lms/doctype/lms_section/lms_section.py
@@ -3,9 +3,27 @@
 # For license information, please see license.txt
 
 from __future__ import unicode_literals
-# import frappe
+import frappe
 from frappe.model.document import Document
 
 class LMSSection(Document):
     def __repr__(self):
         return f"<LMSSection {self.label!r}>"
+
+    def get_latest_code_for_user(self):
+        """Returns the latest code for the logged in user.
+        """
+        if not frappe.session.user or frappe.session.user == "Guest":
+            return self.contents
+        result = frappe.get_all('Code Revision',
+            fields=["code"],
+            filters={
+                "author": frappe.session.user,
+                "section": self.name
+            },
+            order_by="creation desc",
+            page_length=1)
+        if result:
+            return result[0]['code']
+        else:
+            return self.contents
diff --git a/community/www/courses/topic.html b/community/www/courses/topic.html
index 67df39dc..bf23c6d2 100644
--- a/community/www/courses/topic.html
+++ b/community/www/courses/topic.html
@@ -44,7 +44,7 @@
   {% if s.type == "text" %}
     {{ render_section_text(s) }}
   {% elif s.type == "example" or s.type == "code" %}
-    {{ LiveCodeEditor(s.name, s.contents) }}
+    {{ LiveCodeEditor(s.name, s.get_latest_code_for_user()) }}
   {% else %}
     <div>Unknown section type: {{s.type}}</div>
   {% endif %}
@@ -63,11 +63,40 @@
   <script type="text/javascript" src="{{ livecode_url }}/static/livecode.js"></script>
   <script type="text/javascript">
     $(function() {
+      var editorLookup = {};
+
       $(".canvas-editor").each((i, e) => {
+        var data = $(e).data();
         var editor = new LiveCodeEditor(e, {
           runtime: "python-canvas",
           base_url: "{{ livecode_url }}",
-          codemirror: true
+          codemirror: true,
+          userdata: data,
+          autosave: function(editor, code) {
+            var data = editor.options.userdata;
+            var code = editor.codemirror.doc.getValue();
+            // console.log("autosaving...")
+            frappe.call("community.lms.api.autosave_section", {
+              section: data.section,
+              code: code
+            }).then((r) => {
+              // TODO: verify
+            })
+          }
+        })
+        editorLookup[data.section] = editor;
+      })
+
+      $(".canvas-editor .reset").each((i, e) => {
+        $(e).on("click", function(event) {
+          var data = $(this).parents(".canvas-editor").data();
+          var section = data.section;
+          frappe.call("community.lms.api.get_section", {
+            name: section
+          }).then(r => {
+            var editor = editorLookup[data.section];
+            editor.codemirror.doc.setValue(r.message.contents);
+          })
         })
       })
     })
diff --git a/community/www/macros/livecode.html b/community/www/macros/livecode.html
index 6336023c..763b1b54 100644
--- a/community/www/macros/livecode.html
+++ b/community/www/macros/livecode.html
@@ -21,7 +21,8 @@
 {% endmacro %}
 
 {% macro LiveCodeEditor(name, code) %}
-<div class="livecode-editor canvas-editor" id="editor-{{name}}">
+<div class="livecode-editor canvas-editor" id="editor-{{name}}"
+    data-section="{{name}}">
   <div class="row">
     <div class="col-md-9">
     <div>

From b8b767398544c9699d06d411e6db69d13dc8db98 Mon Sep 17 00:00:00 2001
From: Anand Chitipothu <anandology@gmail.com>
Date: Wed, 7 Apr 2021 12:41:17 +0530
Subject: [PATCH 3/3] fix: added a work-around to the issue of regex converter
 not loaded.

frappe app doesn't load all python modules of all the apps on startup. It
loads the hooks.py only if it is not already cached. Because of this the code
to install the regex coverter to not running, causing errors.

Fixed it by replacing the regex route with a string route. The issue is it
also matches the paths like `socket.io` and `website_script.js` etc. Handled
that by whitelisting those routes.
---
 community/__init__.py |  3 ---
 community/hooks.py    | 11 +++++++----
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/community/__init__.py b/community/__init__.py
index 65fc9069..43b477a7 100644
--- a/community/__init__.py
+++ b/community/__init__.py
@@ -3,8 +3,5 @@ from __future__ import unicode_literals
 
 __version__ = '0.0.1'
 
-from .routing import install_regex_converter
-install_regex_converter()
-
 # load the methods from the lms api
 from .lms import api # noqa
diff --git a/community/hooks.py b/community/hooks.py
index bdc9e026..5cbe5f72 100644
--- a/community/hooks.py
+++ b/community/hooks.py
@@ -129,9 +129,7 @@ scheduler_events = {
 
 # Add all simple route rules here
 primary_rules = [
-    {"from_route": "/sketches", "to_route": "sketches"},
     {"from_route": "/sketches/<sketch>", "to_route": "sketches/sketch"},
-    {"from_route": "/courses", "to_route": "courses"},
     {"from_route": "/courses/<course>", "to_route": "courses/course"},
     {"from_route": "/courses/<course>/<topic>", "to_route": "courses/topic"},
     {"from_route": "/courses/<course>/<topic>", "to_route": "courses/topic"}
@@ -142,13 +140,18 @@ whitelist = [
     "/login",
     "/update-password",
     "/update-profile",
-    "/third-party-apps"
+    "/third-party-apps",
+    "/website_script.js",
+    "/courses",
+    "/sketches",
+    "/admin",
+    "/socket.io",
 ]
 whitelist_rules = [{"from_route": p, "to_route": p[1:]} for p in whitelist]
 
 # regex rule to match all profiles
 profile_rules = [
-    {"from_route": "/<regex('[a-z0-9-]{5,}'):username>", "to_route": "profiles/profile"},
+    {"from_route": "/<string(minlength=5):username>", "to_route": "profiles/profile"},
 ]
 
 website_route_rules = primary_rules + whitelist_rules + profile_rules