fixed ajax behavior when unsigned in user vote or create wish

Author: snow

nemo/static/nemo/s/common.js

@@ -69,8 +69,17 @@
                     try {
                         resp = $.parseJSON(xhr.responseText);
                         //rcp.l(resp);
-                        alert(OUT_OF_VOTE_MSG.replace('{want}', resp.want).
+                        switch(xhr.status){
+                        case 403:
+                            alert(OUT_OF_VOTE_MSG.replace('{want}', resp.want).
                                               replace('{has}', resp.has));
+                            break;
+                        case 401:
+                            if(confirm('please signin first.\n'+
+                                        'go to signin page now?')){
+                                location = resp.signin_uri;
+                            }
+                        }
                     } catch(err) {
                         // TODO extractly what will happen here?
                     }
@@ -110,11 +119,24 @@
                 j_form.closest('.stream_li').replaceWith(data);
             },
             error: function(xhr, textstatus, err){
-                if(403 === xhr.status){
-                    var j_replace = $(xhr.responseText);
-                    rcp.l(j_replace);
-                    j_form.replaceWith(j_replace);
-                    j_replace.show();
+                try {
+                    resp = $.parseJSON(xhr.responseText);
+                    //rcp.l(resp);
+                    switch(xhr.status){
+                    case 403:
+                        var j_replace = $(xhr.responseText);
+                        // rcp.l(j_replace);
+                        j_form.replaceWith(j_replace);
+                        j_replace.show();
+                        break;
+                    case 401:
+                        if(confirm('please signin first.\n'+
+                                    'go to signin page now?')){
+                            location = resp.signin_uri;
+                        }
+                    }
+                } catch(err) {
+                    // TODO extractly what will happen here?
                 }
             }
         });
@@ -173,6 +195,21 @@
                 success: function(data){
                     j_stream.prepend(data);
                     j_content.val('').trigger('blur');
+                },
+                error: function(xhr, textstatus, err){
+                    try {
+                        resp = $.parseJSON(xhr.responseText);
+                        //rcp.l(resp);
+                        switch(xhr.status){
+                        case 401:
+                            if(confirm('please signin first.\n'+
+                                        'go to signin page now?')){
+                                location = resp.signin_uri;
+                            }
+                        }
+                    } catch(err) {
+                        // TODO extractly what will happen here?
+                    }
                 }
             });
         });
@@ -183,7 +220,7 @@
             j_votesleft = $('.sidebar .votesleft .count')
         });
 
-        j_stream.on('click', '.ay:not(.signin), .negative:not(.signin)', 
+        j_stream.on('click', '.ay, .negative', 
                 function(e){
                     e.preventDefault();
                     vote($(this));

nemo/templates/nemo/base.html

@@ -14,7 +14,7 @@
     <script type="text/javascript" src="{{ STATIC_URL }}nemo/s/jquery.cookie.js"></script>
     <script type="text/javascript" src="{{ STATIC_URL }}nemo/s/rcp.js"></script>
     <script type="text/javascript" src="{{ STATIC_URL }}nemo/s/rcp.django.js"></script>
-    <script src="{{ STATIC_URL }}nemo/s/common.js?r=0.1.5"></script>
+    <script src="{{ STATIC_URL }}nemo/s/common.js?r=0.1.6"></script>
     <script>nemo.uri_root = "{% nemo_root %}"</script>
 </head>
 

nemo/templates/nemo/list.html

@@ -3,18 +3,13 @@
 {% for e in object_list %}
 <div class="li stream_li" sid="{{ e.id }}">
     <div class="ayes">
-        {% if request.user.is_authenticated %}
         <a class="ay{% if e.count_user_ayes %} on{% endif %}" 
-           href="{% nemo_root %}ay/{{ e.id }}/">{% if e.count_user_ayes %}+{{e.count_user_ayes}}{% else %}+{% endif %}</a>
-        {% else %}
-        <a class="ay signin" href="/accounts/login/">+</a>
-        {% endif %}
+           href="{% nemo_root %}ay/{{ e.id }}/"
+           >{% if e.count_user_ayes %}+{{e.count_user_ayes}}{% else %}+{% endif %}</a>
         <div class="count">{{ e.count_votes }}</div>
-        {% if request.user.is_authenticated %}
-        <a class="negative{% if e.count_user_negatives %} on{% endif %}" href="{% nemo_root %}negative/{{ e.id }}/">{% if e.count_user_negatives %}-{{e.count_user_negatives}}{% else %}-{% endif %}</a>
-        {% else %}
-        <a class="negative signin" href="/accounts/login/">-</a>
-        {% endif %}
+        <a class="negative{% if e.count_user_negatives %} on{% endif %}" 
+            href="{% nemo_root %}negative/{{ e.id }}/"
+            >{% if e.count_user_negatives %}-{{e.count_user_negatives}}{% else %}-{% endif %}</a>        
     </div>
 
     <div class="content{% if e.author == request.user %} editable{% endif %}">{{ e.content }}</div>

nemo/urls.py

@@ -9,11 +9,11 @@
 urlpatterns = patterns('',
     #url(r'^$', nv.IndexV.as_view()),
     url(r'^(?P<stream_type>hot|top|all|done)?/?$', nv.IndexV.as_view()),
-    url(r'^create/$', login_required(nv.CreateV.as_view())),
+    url(r'^create/$', nv.CreateV.as_view()),
     url(r'^update/(?P<pk>\d+)/$', login_required(nv.UpdateV.as_view())),
     url(r'^response/(?P<pk>\d+)/$', 
         permission_required('nemo.response_wish')(nv.ResponseV.as_view())),
     url(r'^list/(?P<stream_type>hot|top|all|done)/(since)?(?P<since>\d+)?/?(till)?(?P<till>\d+)?/?$', nv.ListV.as_view()),
-    url(r'^vote/(?P<pk>\d+)/$', login_required(nv.VoteV.as_view())),
+    url(r'^vote/(?P<pk>\d+)/$', nv.VoteV.as_view()),
     url(r'^votes_left/$', login_required(nv.VotesLeftV.as_view())),
 )

nemo/views.py

@@ -30,7 +30,17 @@ def get_success_url(self):
         else:
             self.success_url = self.get_back_uri()
 
-        return self.success_url    
+        return self.success_url
+    
+    def get_unauthenticated_response(self, format='json'):
+        go_to = settings.LOGIN_URL
+        
+        if 'json' == format:
+            return HttpResponse(json.dumps(dict(signin_uri=go_to)),
+                                status=401,
+                                content_type='application/json')
+        else:
+            return HttpResponseRedirect(go_to)
 
 class IndexV(gv.TemplateView):
     '''Render index'''
@@ -89,8 +99,11 @@ def dispatch(self, request, *args, **kwargs):
         if not request.is_ajax():
             self.template_name = 'nemo/create.html'
         # else use default wish_form.html
-            
-        return super(CreateV, self).dispatch(request, *args, **kwargs)
+        
+        if request.user.is_authenticated():
+            return super(CreateV, self).dispatch(request, *args, **kwargs)
+        else:
+            return self.get_unauthenticated_response()
 
     def form_valid(self, form):
         '''override to set author'''
@@ -115,6 +128,9 @@ class VoteV(RedirectMixin, gv.View):
     '''TODO'''
     def post(self, request, pk, *args, **kwargs):
         '''TODO'''
+        if not request.user.is_authenticated():
+            return self.get_unauthenticated_response()
+        
         user = nemo.UserProfile.objects.get_by_user(request.user)
         wish = nemo.Wish.objects.get(pk=pk)
         try:
@@ -136,7 +152,7 @@ def dispatch(self, request, *args, **kwargs):
         if not request.is_ajax():
             self.template_name = 'nemo/create.html'
         # else use default wish_form.html
-            
+        
         return super(UpdateV, self).dispatch(request, *args, **kwargs)
 
     def form_invalid(self, form):