From 8280c55992644addbf5ebfd883360ba92d5f2ba5 Mon Sep 17 00:00:00 2001 From: Your Name Date: Tue, 12 Mar 2013 17:40:26 -0400 Subject: [PATCH] pinning .1 --- .../coffee/src/discussion/content.coffee | 13 ++++++ .../static/coffee/src/discussion/utils.coffee | 2 + .../views/discussion_thread_show_view.coffee | 44 +++++++++++++++++++ .../django_comment_client/base/urls.py | 2 + .../django_comment_client/base/views.py | 15 +++++++ .../django_comment_client/permissions.py | 2 + lms/djangoapps/django_comment_client/utils.py | 2 +- lms/lib/comment_client/thread.py | 24 +++++++++- lms/static/sass/_discussion.scss | 4 +- .../discussion/_underscore_templates.html | 2 +- 10 files changed, 104 insertions(+), 6 deletions(-) diff --git a/common/static/coffee/src/discussion/content.coffee b/common/static/coffee/src/discussion/content.coffee index 4e612dfc40..00c34df686 100644 --- a/common/static/coffee/src/discussion/content.coffee +++ b/common/static/coffee/src/discussion/content.coffee @@ -79,6 +79,17 @@ if Backbone? @getContent(id).updateInfo(info) $.extend @contentInfos, infos + pinThread: -> + pinned = @get("pinned") + @set("pinned",pinned) + @trigger "change", @ + + unPinThread: -> + pinned = @get("pinned") + @set("pinned",pinned) + @trigger "change", @ + + class @Thread extends @Content urlMappers: 'retrieve' : -> DiscussionUtil.urlFor('retrieve_single_thread', @discussion.id, @id) @@ -91,6 +102,8 @@ if Backbone? 'delete' : -> DiscussionUtil.urlFor('delete_thread', @id) 'follow' : -> DiscussionUtil.urlFor('follow_thread', @id) 'unfollow' : -> DiscussionUtil.urlFor('unfollow_thread', @id) + 'pinThread' : -> DiscussionUtil.urlFor("pin_thread", @id) + 'unPinThread' : -> DiscussionUtil.urlFor("un_pin_thread", @id) initialize: -> @set('thread', @) diff --git a/common/static/coffee/src/discussion/utils.coffee b/common/static/coffee/src/discussion/utils.coffee index 6b2714dc54..41f52f1711 100644 --- a/common/static/coffee/src/discussion/utils.coffee +++ b/common/static/coffee/src/discussion/utils.coffee @@ -50,6 +50,8 @@ class @DiscussionUtil delete_thread : "/courses/#{$$course_id}/discussion/threads/#{param}/delete" upvote_thread : "/courses/#{$$course_id}/discussion/threads/#{param}/upvote" downvote_thread : "/courses/#{$$course_id}/discussion/threads/#{param}/downvote" + pin_thread : "/courses/#{$$course_id}/discussion/threads/#{param}/pin" + un_pin_thread : "/courses/#{$$course_id}/discussion/threads/#{param}/unpin" undo_vote_for_thread : "/courses/#{$$course_id}/discussion/threads/#{param}/unvote" follow_thread : "/courses/#{$$course_id}/discussion/threads/#{param}/follow" unfollow_thread : "/courses/#{$$course_id}/discussion/threads/#{param}/unfollow" diff --git a/common/static/coffee/src/discussion/views/discussion_thread_show_view.coffee b/common/static/coffee/src/discussion/views/discussion_thread_show_view.coffee index 6320c3d1e3..06a5038573 100644 --- a/common/static/coffee/src/discussion/views/discussion_thread_show_view.coffee +++ b/common/static/coffee/src/discussion/views/discussion_thread_show_view.coffee @@ -3,6 +3,7 @@ if Backbone? events: "click .discussion-vote": "toggleVote" + "click .discussion-pin": "togglePin" "click .action-follow": "toggleFollowing" "click .action-edit": "edit" "click .action-delete": "delete" @@ -24,6 +25,7 @@ if Backbone? @delegateEvents() @renderDogear() @renderVoted() + @renderPinned() @renderAttrs() @$("span.timeago").timeago() @convertMath() @@ -41,8 +43,20 @@ if Backbone? else @$("[data-role=discussion-vote]").removeClass("is-cast") + renderPinned: => + if @model.get("pinned") + @$("[data-role=thread-pin]").addClass("pinned") + @$("[data-role=thread-pin]").removeClass("notpinned") + @$(".discussion-pin .pin-label").html("Pinned") + else + @$("[data-role=thread-pin]").removeClass("pinned") + @$("[data-role=thread-pin]").addClass("notpinned") + @$(".discussion-pin .pin-label").html("Pin Thread") + + updateModelDetails: => @renderVoted() + @renderPinned() @$("[data-role=discussion-vote] .votes-count-number").html(@model.get("votes")["up_count"]) convertMath: -> @@ -99,6 +113,34 @@ if Backbone? delete: (event) -> @trigger "thread:delete", event + togglePin: (event) -> + event.preventDefault() + if @model.get('pinned') + @unPin() + else + @pin() + + pin: -> + url = @model.urlFor("pinThread") + DiscussionUtil.safeAjax + $elem: @$(".discussion-pin") + url: url + type: "POST" + success: (response, textStatus) => + if textStatus == 'success' + @model.set('pinned', true) + + unPin: -> + url = @model.urlFor("unPinThread") + DiscussionUtil.safeAjax + $elem: @$(".discussion-pin") + url: url + type: "POST" + success: (response, textStatus) => + if textStatus == 'success' + @model.set('pinned', false) + + toggleClosed: (event) -> $elem = $(event.target) url = @model.urlFor('close') @@ -137,3 +179,5 @@ if Backbone? if @model.get('username')? params = $.extend(params, user:{username: @model.username, user_url: @model.user_url}) Mustache.render(@template, params) + + \ No newline at end of file diff --git a/lms/djangoapps/django_comment_client/base/urls.py b/lms/djangoapps/django_comment_client/base/urls.py index d8fd4927fb..92826a18ae 100644 --- a/lms/djangoapps/django_comment_client/base/urls.py +++ b/lms/djangoapps/django_comment_client/base/urls.py @@ -12,6 +12,8 @@ urlpatterns = patterns('django_comment_client.base.views', url(r'threads/(?P[\w\-]+)/upvote$', 'vote_for_thread', {'value': 'up'}, name='upvote_thread'), url(r'threads/(?P[\w\-]+)/downvote$', 'vote_for_thread', {'value': 'down'}, name='downvote_thread'), url(r'threads/(?P[\w\-]+)/unvote$', 'undo_vote_for_thread', name='undo_vote_for_thread'), + url(r'threads/(?P[\w\-]+)/pin$', 'pin_thread', name='pin_thread'), + url(r'threads/(?P[\w\-]+)/unpin$', 'un_pin_thread', name='un_pin_thread'), url(r'threads/(?P[\w\-]+)/follow$', 'follow_thread', name='follow_thread'), url(r'threads/(?P[\w\-]+)/unfollow$', 'unfollow_thread', name='unfollow_thread'), url(r'threads/(?P[\w\-]+)/close$', 'openclose_thread', name='openclose_thread'), diff --git a/lms/djangoapps/django_comment_client/base/views.py b/lms/djangoapps/django_comment_client/base/views.py index d93ea19f44..f08c302bdb 100644 --- a/lms/djangoapps/django_comment_client/base/views.py +++ b/lms/djangoapps/django_comment_client/base/views.py @@ -289,6 +289,21 @@ def undo_vote_for_thread(request, course_id, thread_id): user.unvote(thread) return JsonResponse(utils.safe_content(thread.to_dict())) +@require_POST +@login_required +@permitted +def pin_thread(request, course_id, thread_id): + user = cc.User.from_django_user(request.user) + thread = cc.Thread.find(thread_id) + thread.pin(user,thread) + return JsonResponse(utils.safe_content(thread.to_dict())) + +def un_pin_thread(request, course_id, thread_id): + user = cc.User.from_django_user(request.user) + thread = cc.Thread.find(thread_id) + thread.un_pin(user,thread) + return JsonResponse(utils.safe_content(thread.to_dict())) + @require_POST @login_required diff --git a/lms/djangoapps/django_comment_client/permissions.py b/lms/djangoapps/django_comment_client/permissions.py index dfdcd3e7ba..7d21cc9783 100644 --- a/lms/djangoapps/django_comment_client/permissions.py +++ b/lms/djangoapps/django_comment_client/permissions.py @@ -90,6 +90,8 @@ VIEW_PERMISSIONS = { 'undo_vote_for_comment': [['unvote', 'is_open']], 'vote_for_thread' : [['vote', 'is_open']], 'undo_vote_for_thread': [['unvote', 'is_open']], + 'pin_thread': ['create_comment'], + 'un_pin_thread': ['create_comment'], 'follow_thread' : ['follow_thread'], 'follow_commentable': ['follow_commentable'], 'follow_user' : ['follow_user'], diff --git a/lms/djangoapps/django_comment_client/utils.py b/lms/djangoapps/django_comment_client/utils.py index dde219294c..7e31e451d1 100644 --- a/lms/djangoapps/django_comment_client/utils.py +++ b/lms/djangoapps/django_comment_client/utils.py @@ -430,7 +430,7 @@ def safe_content(content): 'updated_at', 'depth', 'type', 'commentable_id', 'comments_count', 'at_position_list', 'children', 'highlighted_title', 'highlighted_body', 'courseware_title', 'courseware_url', 'tags', 'unread_comments_count', - 'read', 'group_id', 'group_name', 'group_string' + 'read', 'group_id', 'group_name', 'group_string', 'pinned' ] if (content.get('anonymous') is False) and (content.get('anonymous_to_peers') is False): diff --git a/lms/lib/comment_client/thread.py b/lms/lib/comment_client/thread.py index ca607d3ff3..9fe1b4397f 100644 --- a/lms/lib/comment_client/thread.py +++ b/lms/lib/comment_client/thread.py @@ -11,12 +11,12 @@ class Thread(models.Model): 'closed', 'tags', 'votes', 'commentable_id', 'username', 'user_id', 'created_at', 'updated_at', 'comments_count', 'unread_comments_count', 'at_position_list', 'children', 'type', 'highlighted_title', - 'highlighted_body', 'endorsed', 'read', 'group_id', 'group_name' + 'highlighted_body', 'endorsed', 'read', 'group_id', 'group_name', 'pinned' ] updatable_fields = [ 'title', 'body', 'anonymous', 'anonymous_to_peers', 'course_id', - 'closed', 'tags', 'user_id', 'commentable_id', 'group_id', 'group_name' + 'closed', 'tags', 'user_id', 'commentable_id', 'group_id', 'group_name', 'pinned' ] initializable_fields = updatable_fields @@ -79,3 +79,23 @@ class Thread(models.Model): response = perform_request('get', url, request_params) self.update_attributes(**response) + + def pin(self, user, thread_id): + url = _url_for_pin_thread(thread_id) + params = {'user_id': user.id} + request = perform_request('put', url, params) + self.update_attributes(request) + + def un_pin(self, user, thread_id): + url = _url_for_un_pin_thread(thread_id) + params = {'user_id': user.id} + request = perform_request('put', url, params) + self.update_attributes(request) + + +def _url_for_pin_thread(thread_id): + return "{prefix}/threads/{thread_id}/pin".format(prefix=settings.PREFIX, thread_id=thread_id) + +def _url_for_un_pin_thread(thread_id): + return "{prefix}/threads/{thread_id}/unpin".format(prefix=settings.PREFIX, thread_id=thread_id) + \ No newline at end of file diff --git a/lms/static/sass/_discussion.scss b/lms/static/sass/_discussion.scss index fe4f37f7aa..2f044ca5a3 100644 --- a/lms/static/sass/_discussion.scss +++ b/lms/static/sass/_discussion.scss @@ -2444,7 +2444,7 @@ body.discussion { background-color:#fff; } -.discussion-flag-abuse { +.discussion-pin { font-size: 12px; float:right; padding-right: 5px; @@ -2457,7 +2457,7 @@ body.discussion { width: 10px; height: 14px; padding-right: 3px; - background: transparent url('../images/notpinned.png') no-repeat 0 0; + background: transparent url('../images/unpinned.png') no-repeat 0 0; } .pinned .icon diff --git a/lms/templates/discussion/_underscore_templates.html b/lms/templates/discussion/_underscore_templates.html index 7faef4195c..9bfb32b3ff 100644 --- a/lms/templates/discussion/_underscore_templates.html +++ b/lms/templates/discussion/_underscore_templates.html @@ -47,7 +47,7 @@
${'<%- body %>'}
% if has_permission(user, 'create_comment', course.id): -
+
Pin Thread
% endif ${'<% if (obj.courseware_url) { %>'}