modify pictorial sort

trans2es/models/pictorial.py

@@ -108,29 +108,21 @@ class Pictorial(models.Model):
 
     def get_tag_by_id(self):
         try:
-            tag_id_list = []
-            tags = PictorialTag.objects.filter(pictorial_id=self.id, is_online=True).values_list("tag_id", flat=True)
-            for i in tags:
-                tag_id_list.append(i)
+            tag_id_list = PictorialTag.objects.filter(pictorial_id=self.id, is_online=True).values_list("tag_id", flat=True)
 
             return tag_id_list
-
         except:
             logging.error("catch exception,err_msg:%s" % traceback.format_exc())
-            return False
+            return []
 
     def get_tag_by_name(self, tag_id):
         try:
-            tag_name_list = []
-            tags = Tag.objects.filter(id__in=tag_id, is_online=True).values_list("name", flat=True)
-            for i in tags:
-                tag_name_list.append(i)
+            tag_name_list = Tag.objects.filter(id__in=tag_id, is_online=True).values_list("name", flat=True)
 
             return tag_name_list
-
         except:
             logging.error("catch exception,err_msg:%s" % traceback.format_exc())
-            return False
+            return []
 
 
 class PictorialTag(models.Model):

trans2es/utils/pictorial_transfer.py

@@ -59,9 +59,9 @@ class PictorialTransfer(object):
             tzlc_udpate_time = tzlc(update_time)
             res["update_time"] = tzlc_udpate_time
             res["high_quality_topic_num"] = instance.get_high_quality_topic_num()
-            tag_id = instance.get_tag_by_id()
-            res["tag_id"] = tag_id
-            res["tag_name"] = instance.get_tag_by_name(tag_id)
+            tag_id_list = instance.get_tag_by_id()
+            res["tag_id"] = tag_id_list
+            res["tag_name"] = instance.get_tag_by_name(tag_id_list)
             res["topic_id_list"] =instance.get_topic_id()
             res["effective"] = instance.get_effective(res["topic_id_list"])
             res["offline_score"] = cls.get_offline_score(instance,res["topic_id_list"])