diff --git a/erpnext/education/utils.py b/erpnext/education/utils.py
index d7b768e..bf766ad 100644
--- a/erpnext/education/utils.py
+++ b/erpnext/education/utils.py
@@ -96,6 +96,8 @@
 
 def get_course_enrollment(course_name):
 	student = get_current_student()
+	if not student:
+		return None
 	enrollment_name = frappe.get_all("Course Enrollment", filters={'student': student.name, 'course':course_name})
 	try:
 		name = enrollment_name[0].name
@@ -114,7 +116,6 @@
 		"user": frappe.session.user
 		})
 	student.save(ignore_permissions=True)
-	frappe.db.commit()
 	return student
 
 def enroll_in_course(course_name, program_name):
diff --git a/erpnext/www/lms.py b/erpnext/www/lms.py
index 69424e1..8181823 100644
--- a/erpnext/www/lms.py
+++ b/erpnext/www/lms.py
@@ -119,7 +119,7 @@
 def enroll_in_program(program_name):
 	student = utils.get_current_student()
 	if not student:
-		utils.create_student_from_current_user()
+		student = utils.create_student_from_current_user()
 	program_enrollment = student.enroll_in_program(program_name)
 	return program_name
 
@@ -215,11 +215,9 @@
 
 @frappe.whitelist()
 def get_joining_date():
-	current_student = utils.get_current_student()
-	if current_student:
+	student = utils.get_current_student()
+	if student:
 		return student.joining_date
-	else:
-		return None
 
 @frappe.whitelist()
 def get_quiz_progress(program_name):