eduvia/lms
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: miscellaneous

Browse Source
This commit is contained in:
pateljannat
2021-05-04 12:47:45 +05:30
parent 8933ca9ac9
commit e931ead270
12 changed files with 101 additions and 72 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
community/community/doctype/community_member/community_member.py
View File

@@ -42,7 +42,7 @@ class CommunityMember(Document):
return frappe.db.count( return frappe.db.count(
'LMS Batch Membership', { 'LMS Batch Membership', {
'member': self.name, 'member': self.name,
'member_role': 'Mentor' 'member_type': 'Mentor'
}) })
def __repr__(self): def __repr__(self):

4
community/lms/doctype/invite_request/invite_request.py
View File

@@ -11,7 +11,7 @@ from frappe.utils.password import get_decrypted_password
class InviteRequest(Document): class InviteRequest(Document):
def on_update(self): def on_update(self):
if self.has_value_changed('status') and self.status == "Approved": if self.has_value_changed("status") and self.status == "Approved":
self.send_email() self.send_email()
def create_user(self, password): def create_user(self, password):
@@ -53,7 +53,7 @@ def create_invite_request(invite_email):
}).save(ignore_permissions=True) }).save(ignore_permissions=True)
return "OK" return "OK"
except frappe.UniqueValidationError: except frappe.UniqueValidationError:
frappe.throw(_("Email {0} has already been used to request an invite").format(invite_email)) frappe.throw(_("Email {0} has already been used to request an invitation").format(invite_email))
@frappe.whitelist(allow_guest=True) @frappe.whitelist(allow_guest=True)
def update_invite(data): def update_invite(data):

8
community/lms/doctype/lms_batch_membership/lms_batch_membership.py
View File

@@ -29,7 +29,7 @@ class LMSBatchMembership(Document):
frappe.throw(_("{0} is already a {1} of {2} course through {3} batch").format(member_name, membership.member_type, course, membership.batch)) frappe.throw(_("{0} is already a {1} of {2} course through {3} batch").format(member_name, membership.member_type, course, membership.batch))
@frappe.whitelist() @frappe.whitelist()
def create_membership(batch, course, member=None, member_type="Student", role="Member"): def create_membership(batch, course=None, member=None, member_type="Student", role="Member"):
if not member: if not member:
member = frappe.db.get_value("Community Member", {"email": frappe.session.user}, "name") member = frappe.db.get_value("Community Member", {"email": frappe.session.user}, "name")
frappe.get_doc({ frappe.get_doc({
@@ -39,5 +39,7 @@ def create_membership(batch, course, member=None, member_type="Student", role="M
"member_type": member_type, "member_type": member_type,
"member": member "member": member
}).save(ignore_permissions=True) }).save(ignore_permissions=True)
course_slug = frappe.db.get_value("LMS Course", {"title": course}, ["slug"]) if course:
return course_slug course_slug = frappe.db.get_value("LMS Course", {"title": course}, ["slug"])
return course_slug
return "OK"

27
community/lms/doctype/lms_course/lms_course.py
View File

@@ -97,7 +97,7 @@ class LMSCourse(Document):
for mentor in mentors: for mentor in mentors:
member = frappe.get_doc("Community Member", mentor.mentor) member = frappe.get_doc("Community Member", mentor.mentor)
# TODO: change this to count query # TODO: change this to count query
member.batch_count = len(frappe.get_all("LMS Batch Membership", {"member": member, "member_type": "Mentor"})) member.batch_count = len(frappe.get_all("LMS Batch Membership", {"member": member.name, "member_type": "Mentor"}))
course_mentors.append(member) course_mentors.append(member)
return course_mentors return course_mentors
@@ -108,9 +108,9 @@ class LMSCourse(Document):
return False return False
member = self.get_community_member(email) member = self.get_community_member(email)
return frappe.db.exists({ return frappe.db.exists({
'doctype': 'LMS Course Mentor Mapping', "doctype": "LMS Course Mentor Mapping",
"course": self.name, "course": self.name,
"member": member "mentor": member
}) })
def get_instructor(self): def get_instructor(self):
@@ -131,15 +131,30 @@ class LMSCourse(Document):
memberships = frappe.db.get_all( memberships = frappe.db.get_all(
"LMS Batch Membership", "LMS Batch Membership",
{"member": member}, {"member": member},
["name"], as_dict=1) ["batch"])
batch_names = {m.batch for m in memberships} batch_names = {m.batch for m in memberships}
return [b for b in batches if b.name in batch_names] mentor_batches = [b for b in batches if b.name in batch_names]
return self.get_batch_mentors(mentor_batches)
def get_upcoming_batches(self): def get_upcoming_batches(self):
now = frappe.utils.nowdate() now = frappe.utils.nowdate()
return find_all("LMS Batch", batches = find_all("LMS Batch",
course=self.name, course=self.name,
start_date=[">", now]) start_date=[">", now])
batches = self.get_batch_mentors(batches)
return batches
def get_batch_mentors(self, batches):
for batch in batches:
batch.mentors = []
mentors = frappe.get_all(
"LMS Batch Membership",
{"batch": batch.name, "member_type": "Mentor"},
["member"])
for mentor in mentors:
member = frappe.db.get_value("Community Member", mentor.member, ["full_name", "photo", "abbr"], as_dict=1)
batch.mentors.append(member)
return batches
def find_all(doctype, order_by=None, **filters): def find_all(doctype, order_by=None, **filters):
"""Queries the database for documents of a doctype matching given filters. """Queries the database for documents of a doctype matching given filters.

3
community/public/css/style.css
View File

@@ -287,6 +287,7 @@ img.profile-photo {
} }
.message-section { .message-section {
margin-left: 5%; margin-left: 3%;
display: inline-block; display: inline-block;
width: 95%;
} }

2
community/www/courses/about/index.html
View File

@@ -16,7 +16,7 @@
{% if batch.description %} {% if batch.description %}
{{ BatchDetails(batch.description) }} {{ BatchDetails(batch.description) }}
{% endif %} {% endif %}
{{ MentorsSection(mentors, True) }} {{ MentorsSection(mentors, True, course.name) }}
</div> </div>
{% endblock %} {% endblock %}

4
community/www/courses/about/index.py
View File

@@ -11,13 +11,11 @@ def get_context(context):
context.instructor = get_instructor(context.course.owner) context.instructor = get_instructor(context.course.owner)
context.batch = get_batch(context.batch_code) context.batch = get_batch(context.batch_code)
context.mentors = get_mentors(context.batch.name) context.mentors = get_mentors(context.batch.name)
print(context.mentors)
def get_mentors(batch): def get_mentors(batch):
mentors = [] mentors = []
memberships = frappe.get_all("LMS Batch Membership", {"batch": batch, "member_type": "Mentor"}, ["member"]) memberships = frappe.get_all("LMS Batch Membership", {"batch": batch, "member_type": "Mentor"}, ["member"])
for membership in memberships: for membership in memberships:
member = frappe.db.get_value("Community Member", membership.member, ["name","full_name"], as_dict=True) member = frappe.get_doc("Community Member", membership.member)
member.batch_count = len(frappe.get_all("LMS Batch Membership", {"member": member.name, "member_type": "Mentor"}))
mentors.append(member) mentors.append(member)
return mentors return mentors

18
community/www/courses/course.html
View File

@@ -10,7 +10,7 @@
<div class="container"> <div class="container">
<div class="course-header"> <div class="course-header">
<div class="course-type">course</div> <div class="course-type">course</div>
<h1>{{course.title}}</h1> <h1 id="course-title" data-course="{{course.name}}">{{course.title}}</h1>
</div> </div>
<div class="row"> <div class="row">
@@ -28,7 +28,7 @@
</div> </div>
<div class="sidebar"> <div class="sidebar">
{{ MentorsSection(course.get_mentors(), course.is_mentor(frappe.session.user)) }} {{ MentorsSection(course.get_mentors(), course.is_mentor(frappe.session.user), course.name) }}
</div> </div>
</div> </div>
</div> </div>
@@ -86,9 +86,9 @@
<div class="cta"> <div class="cta">
<div class=""> <div class="">
{% if can_manage %} {% if can_manage %}
<button type="button">Manage</button> <button >Manage</button>
{% else %} {% else %}
<button type="button">Join this Batch</button> <button class="join-batch" data-batch="{{ batch.name | urlencode }}">Join this Batch</button>
{% endif %} {% endif %}
</div> </div>
</div> </div>
@@ -123,11 +123,13 @@
{% macro BatchSectionForStudents(course, upcoming_batches) %} {% macro BatchSectionForStudents(course, upcoming_batches) %}
<h2>Upcoming Batches</h2> <h2>Upcoming Batches</h2>
{% for batch in upcoming_batches %} <div class="row">
<div class="col-lg-4 col-md-6"> {% for batch in upcoming_batches %}
{{ RenderBatch(batch, can_manage=False) }} <div class="col-lg-4 col-md-6">
{{ RenderBatch(batch, can_manage=False) }}
</div>
{% endfor %}
</div> </div>
{% endfor %}
{% endmacro %} {% endmacro %}
{% macro CourseOutline(course) %} {% macro CourseOutline(course) %}

22
community/www/courses/course.js
View File

@@ -3,18 +3,19 @@ frappe.ready(() => {
frappe.call({ frappe.call({
'method': 'community.lms.doctype.lms_mentor_request.lms_mentor_request.has_requested', 'method': 'community.lms.doctype.lms_mentor_request.lms_mentor_request.has_requested',
'args': { 'args': {
course: decodeURIComponent($(".course-title").attr("data-course")), course: decodeURIComponent($("#course-title").attr("data-course")),
}, },
'callback': (data) => { 'callback': (data) => {
if (data.message) { if (data.message) {
$(".mentor-request").addClass("hide"); $("#mentor-request").addClass("hide");
$(".already-applied").removeClass("hide") $("#already-applied").removeClass("hide")
} }
} }
}) })
} }
$(".apply-now").click((e) => { $("#apply-now").click((e) => {
e.preventDefault();
if (frappe.session.user == "Guest") { if (frappe.session.user == "Guest") {
window.location.href = "/login"; window.location.href = "/login";
return; return;
@@ -26,14 +27,15 @@ frappe.ready(() => {
}, },
"callback": (data) => { "callback": (data) => {
if (data.message == "OK") { if (data.message == "OK") {
$(".mentor-request").addClass("hide"); $("#mentor-request").addClass("hide");
$(".already-applied").removeClass("hide") $("#already-applied").removeClass("hide")
} }
} }
}) })
}) })
$(".cancel-request").click((e) => { $("#cancel-request").click((e) => {
e.preventDefault()
frappe.call({ frappe.call({
"method": "community.lms.doctype.lms_mentor_request.lms_mentor_request.cancel_request", "method": "community.lms.doctype.lms_mentor_request.lms_mentor_request.cancel_request",
"args": { "args": {
@@ -41,14 +43,15 @@ frappe.ready(() => {
}, },
"callback": (data) => { "callback": (data) => {
if (data.message == "OK") { if (data.message == "OK") {
$(".mentor-request").removeClass("hide"); $("#mentor-request").removeClass("hide");
$(".already-applied").addClass("hide") $("#already-applied").addClass("hide")
} }
} }
}) })
}) })
$(".join-batch").click((e) => { $(".join-batch").click((e) => {
e.preventDefault()
if (frappe.session.user == "Guest") { if (frappe.session.user == "Guest") {
window.location.href = "/login"; window.location.href = "/login";
return; return;
@@ -62,7 +65,6 @@ frappe.ready(() => {
"callback": (data) => { "callback": (data) => {
if (data.message == "OK") { if (data.message == "OK") {
frappe.msgprint(__("You are now a student of this course.")) frappe.msgprint(__("You are now a student of this course."))
$(".upcoming-batches").addClass("hide")
} }
} }
}) })

62
community/www/dashboard/index.py
View File

@@ -1,38 +1,42 @@
import frappe import frappe
from ...lms.doctype.lms_sketch.lms_sketch import get_recent_sketches from community.lms.models import Sketch
def get_context(context): def get_context(context):
context.no_cache = 1 context.no_cache = 1
context.member = frappe.get_all("Community Member", {"email": frappe.session.user}, ["name", "email", "photo", "full_name", "abbr"])[0] if frappe.session.user == "Guest":
context.memberships = get_memberships(context.member.name) frappe.local.flags.redirect_location = "/login"
context.courses = get_courses(context.memberships) raise frappe.Redirect
context.activity = get_activity(context.memberships) context.member = frappe.get_all("Community Member", {"email": frappe.session.user}, ["name", "email", "photo", "full_name", "abbr"])[0]
context.sketches = list(filter(lambda x: x.owner == frappe.session.user, get_recent_sketches())) context.memberships = get_memberships(context.member.name)
context.courses = get_courses(context.memberships)
context.activity = get_activity(context.memberships)
context.sketches = list(filter(lambda x: x.owner == frappe.session.user, Sketch.get_recent_sketches(owner=context.member.email)))
def get_memberships(member): def get_memberships(member):
return frappe.get_all("LMS Batch Membership", {"member": member}, ["batch", "member_type", "creation"]) return frappe.get_all("LMS Batch Membership", {"member": member}, ["batch", "member_type", "creation"])
def get_courses(memberships): def get_courses(memberships):
courses = [] courses = []
for membership in memberships: for membership in memberships:
course = frappe.db.get_value("LMS Batch", membership.batch, "course") course = frappe.db.get_value("LMS Batch", membership.batch, "course")
course_details = frappe.get_doc("LMS Course", course) course_details = frappe.get_doc("LMS Course", course)
course_in_list = list(filter(lambda x: x.name == course_details.name, courses)) course_in_list = list(filter(lambda x: x.name == course_details.name, courses))
if not len(course_in_list): if not len(course_in_list):
course_details.description = course_details.description[0:100] + "..." course_details.description = course_details.description[0:100] + "..."
course_details.joining = membership.creation course_details.joining = membership.creation
if membership.member_type != "Student": if membership.member_type != "Student":
course_details.member_type = membership.member_type course_details.member_type = membership.member_type
courses.append(course_details) courses.append(course_details)
return courses return courses
def get_activity(memberships): def get_activity(memberships):
messages, courses = [], {} messages, courses = [], {}
batches = [x.batch for x in memberships] batches = [x.batch for x in memberships]
for batch in batches: for batch in batches:
courses[batch] = frappe.db.get_value("LMS Batch", batch, "course") courses[batch] = frappe.db.get_value("LMS Batch", batch, "course")
messages = frappe.get_all("LMS Message", {"batch": ["in", ",".join(batches)]}, ["message", "author", "creation", "batch"], order_by='creation desc') messages = frappe.get_all("LMS Message", {"batch": ["in", ",".join(batches)]}, ["message", "author", "creation", "batch"], order_by='creation desc')
for message in messages: for message in messages:
message.course = courses[message.batch] message.course = courses[message.batch]
message.profile, message.full_name, message.abbr = frappe.db.get_value("Community Member", message.author, ["photo", "full_name", "abbr"]) message.profile, message.full_name, message.abbr = frappe.db.get_value("Community Member", message.author, ["photo", "full_name", "abbr"])
return messages return messages

15
community/www/macros/common_macro.html
View File

@@ -6,7 +6,7 @@
</div> </div>
{% endmacro %} {% endmacro %}
{% macro MentorsSection(mentors, is_mentor) %} {% macro MentorsSection(mentors, is_mentor, course_name) %}
<h3>Mentors</h3> <h3>Mentors</h3>
{% for m in mentors %} {% for m in mentors %}
<div class="instructor"> <div class="instructor">
@@ -15,10 +15,15 @@
</div> </div>
{% endfor %} {% endfor %}
{% if not is_mentor %} {% if not is_mentor %}
<div class="notice"> <div id="mentor-request" class="notice">
Interested to become a mentor? Interested to become a mentor?
<div><a href="#">Apply Now!</a></div> <div><a id="apply-now" data-course="{{course_name | urlencode}}" href="">Apply Now!</a></div>
</div>
<div id="already-applied" class="notice hide">
You've applied to become a mentor for this course. Your request is currently under review.
If you are not any more interested to mentor this course, you can <a id="cancel-request" data-course="{{course_name | urlencode}}" href="">cancel your application</a>.
</div> </div>
{% endif %} {% endif %}
{% endmacro %} {% endmacro %}
@@ -26,7 +31,7 @@
{% macro BatchHearder(course_name, member_count) %} {% macro BatchHearder(course_name, member_count) %}
<div class="border p-3"> <div class="border p-3">
<h3>{{course_name}}</h3> <h3>{{course_name}}</h3>
<div class="text-muted">{{member_count}} members</div> <div class="text-muted">{{member_count}} members</div>
</div> </div>
{% endmacro %} {% endmacro %}

2
community/www/profiles/profile.py
View File

@@ -8,7 +8,7 @@ def get_context(context):
if not context.member: if not context.member:
context.template = "www/404.html" context.template = "www/404.html"
else: else:
context.sketches = Sketch.get_recent_sketches(context.member.email) context.sketches = Sketch.get_recent_sketches(owner=context.member.email)
def get_member(username): def get_member(username):
try: try: