chore: Minor code improvements

- Changed variable name in `get_content` function
- Added proper exceptions for `get_topic` and `get_content` functions in lms
This commit is contained in:
scmmishra 2019-04-22 12:28:13 +05:30
parent da2c90cd18
commit 4add826703
3 changed files with 11 additions and 9 deletions

View File

@ -32,7 +32,7 @@ export default {
methods: {
getContent() {
return lms.call('get_content', {
type: this.type,
content_type: this.type,
content: this.content
})
}

View File

@ -55,7 +55,7 @@ export default {
methods: {
getContent() {
return lms.call('get_content', {
type: this.type,
content_type: this.type,
content: this.content
})
}

View File

@ -291,13 +291,15 @@ def get_course_details(course_name):
# Functions to get program & course details
@frappe.whitelist(allow_guest=True)
def get_topics(course_name):
course = frappe.get_doc('Course', course_name)
topics = course.get_topics()
return topics
try:
course = frappe.get_doc('Course', course_name)
return course.get_topics()
except frappe.DoesNotExistError:
frappe.throw(_("Course {0} does not exist.".format(course_name)))
@frappe.whitelist()
def get_content(type, content):
def get_content(content_type, content):
try:
return frappe.get_doc(type, content)
except:
return None
return frappe.get_doc(content_type, content)
except frappe.DoesNotExistError:
frappe.throw(_("{0} {1} does not exist.".format(content_type, content)))