部分代码结构调整

liangliangyy · liangliangyy · commit db72d9bfa6a8 · 2018-09-21T23:16:05.000+08:00
diff --git a/DjangoBlog/blog_signals.py b/DjangoBlog/blog_signals.py
@@ -16,17 +16,29 @@
 import django.dispatch
 from django.dispatch import receiver
 from django.conf import settings
-from DjangoBlog.utils import cache, send_email, expire_view_cache
+from DjangoBlog.utils import cache, send_email, expire_view_cache, get_blog_setting
 from DjangoBlog.spider_notify import SpiderNotify
 from django.contrib.sites.models import Site
-
+from oauth.models import OAuthUser
 import logging
 
 logger = logging.getLogger(__name__)
 
 comment_save_signal = django.dispatch.Signal(providing_args=["comment_id", "username", "serverport"])
 article_save_signal = django.dispatch.Signal(providing_args=['id', 'is_update_views'])
 user_login_logout_signal = django.dispatch.Signal(providing_args=['id', 'type'])
+oauth_user_login_signal = django.dispatch.Signal(providing_args=['id'])
+
+
+@receiver(oauth_user_login_signal)
+def oauth_user_login_callback(sender, **kwargs):
+    id = kwargs['id']
+    oauthuser = OAuthUser.objects.get(id=id)
+    setting = get_blog_setting()
+    if oauthuser.picture and not oauthuser.picture.startswith(setting.resource_path):
+        from DjangoBlog.utils import save_user_avatar
+        oauthuser.picture = save_user_avatar(oauthuser.picture)
+        oauthuser.save()
 
 
 @receiver(article_save_signal)
@@ -49,22 +61,22 @@ def article_save_callback(sender, **kwargs):
                 SpiderNotify.baidu_notify([notify_url])
             except Exception as ex:
                 logger.error("notify sipder", ex)
-                print(ex)
+
+    from DjangoBlog.utils import cache
+    cache.clear()
 
 
 @receiver(comment_save_signal)
 def comment_save_callback(sender, **kwargs):
     from comments.models import Comment
-    if settings.DEBUG:
-        return
 
     serverport = kwargs['serverport']
     username = kwargs['username']
     comment = Comment.objects.get(id=kwargs['comment_id'])
     site = Site.objects.get_current().domain
     article = comment.article
-    # if not settings.DEBUG:
-    if True:
+    if not settings.DEBUG:
+
         subject = '感谢您发表的评论'
         article_url = "https://{site}{path}".format(site=site, path=comment.article.get_absolute_url())
         html_content = """
@@ -103,4 +115,5 @@ def comment_save_callback(sender, **kwargs):
     from django.core.cache.utils import make_template_fragment_key
 
     key = make_template_fragment_key('sidebar', [username])
+    logger.info('delete sidebar key:' + key)
     cache.delete(key)
diff --git a/DjangoBlog/spider_notify.py b/DjangoBlog/spider_notify.py
@@ -16,6 +16,9 @@
 from django.contrib.sitemaps import ping_google
 import requests
 from django.conf import settings
+import logging
+
+logger = logging.getLogger(__name__)
 
 
 class SpiderNotify():
@@ -26,14 +29,14 @@ def baidu_notify(urls):
             result = requests.post(settings.BAIDU_NOTIFY_URL, data=data)
             print(result.text)
         except Exception as e:
-            print(e)
+            logger.error(e)
 
     @staticmethod
     def __google_notify():
         try:
             ping_google('/sitemap.xml')
         except Exception as e:
-            print(e)
+            logger.error(e)
 
     @staticmethod
     def notify(url):
diff --git a/blog/admin.py b/blog/admin.py
@@ -58,7 +58,7 @@ def open_article_commentstatus(modeladmin, request, queryset):
 
 class ArticlelAdmin(admin.ModelAdmin):
     list_per_page = 20
-    search_fields = ('body','title')
+    search_fields = ('body', 'title')
     form = ArticleForm
     list_display = (
         'id', 'title', 'author', 'link_to_category', 'created_time', 'views', 'status', 'type', 'article_order')
@@ -83,8 +83,6 @@ def get_form(self, request, obj=None, **kwargs):
 
     def save_model(self, request, obj, form, change):
         super(ArticlelAdmin, self).save_model(request, obj, form, change)
-        from DjangoBlog.utils import cache
-        cache.clear()
 
     def get_view_on_site_url(self, obj=None):
         if obj:
diff --git a/oauth/views.py b/oauth/views.py
@@ -16,6 +16,8 @@
 from django.core.exceptions import ObjectDoesNotExist
 from django.http import HttpResponseForbidden
 from .oauthmanager import get_manager_by_type
+from DjangoBlog.blog_signals import oauth_user_login_signal
+
 import logging
 
 logger = logging.getLogger(__name__)
@@ -52,8 +54,6 @@ def authorize(request):
         return HttpResponseRedirect(manager.get_authorization_url(nexturl))
     user = manager.get_oauth_userinfo()
     if user:
-        if user.picture:
-            user.picture = save_user_avatar(user.picture)
         if not user.nikename:
             import datetime
             user.nikename = "djangoblog" + datetime.datetime.now().strftime('%y%m%d%I%M%S')
@@ -80,7 +80,10 @@ def authorize(request):
 
             user.author = author
             user.save()
+
+            oauth_user_login_signal.send(sender=authorize.__class__, id=user.id)
             login(request, author)
+
             return HttpResponseRedirect(nexturl)
         if not email:
             user.save()
diff --git a/templates/blog/article_index.html b/templates/blog/article_index.html
@@ -31,20 +31,6 @@
             {% endfor %}
             {% if is_paginated %}
                 {% load_pagination_info page_obj page_type tag_name %}
-                {% comment %} <nav id="nav-below" class="navigation" role="navigation">
-                    <h3 class="assistive-text">文章导航</h3>
-                    {% if page_obj.has_next %}
-                        <div class="nav-previous"><a
-                                href="{{ SITE_BASE_URL }}page/{{ page_obj.next_page_number }}"><span
-                                class="meta-nav">&larr;</span> 早期文章</a></div>
-                    {% endif %}
-                    {% if page_obj.has_previous %}
-                        <div class="nav-next"><a href="{{ SITE_BASE_URL }}page/{{ page_obj.previous_page_number }}">较新文章
-                            <span
-                                    class="meta-nav">→</span></a>
-                        </div>
-                    {% endif %}
-                </nav><!-- .navigation -->{% endcomment %}
 
             {% endif %}
         </div><!-- #content -->
