fix: 问答话题不能收藏的问题

src/api/question/topics.js

@@ -1,4 +1,4 @@
-import request from "../../http";
+import api from "../api.js";
 
 /**
  * The topics API query function.
@@ -12,7 +12,7 @@ import request from "../../http";
  * @returns
  */
 export function query(params = {}) {
-  return request.get("/question-topics", {
+  return api.get("/question-topics", {
     params,
     validateStatus: status => status === 200
   });
@@ -45,32 +45,26 @@ export function search(name = null, offset = 0, limit = 15, follow = true) {
   return query({ name, offset, limit, follow });
 }
 
-export function userQuery(query = {}) {
-  return request.get("/user/question-topics", {
-    params: query,
-    validateStatus: status => status === 200
+export function userQuery(params = {}) {
+  return api.get("/user/question-topics", {
+    params,
+    validateStatus: s => s === 200
   });
 }
 
-export function followTopics(after = 0, limit = 15) {
+export function getFollowTopics(after = 0, limit = 15) {
   return userQuery({ after, limit, type: "follow" });
 }
 
 /**
  * Follow a topic.
- *
- * @param {number} id
- * @return {Promise}
  * @author Seven Du <[email protected]>
+ * @param {number} topicId
+ * @returns
  */
-export function follow(id) {
-  return request.put(
-    `/user/question-topics/${id}`,
-    {},
-    {
-      validateStatus: status => status === 201
-    }
-  );
+export function followTopic(topicId) {
+  const url = `/user/question-topics/${topicId}`;
+  return api.put(url, {}, { validateStatus: s => s === 201 });
 }
 
 /**
@@ -80,8 +74,8 @@ export function follow(id) {
  * @return {Promise}
  * @author Seven Du <[email protected]>
  */
-export function unfollow(id) {
-  return request.delete(`/user/question-topics/${id}`, {
+export function unfollowTopic(id) {
+  return api.delete(`/user/question-topics/${id}`, {
     validateStatus: status => status === 204
   });
 }
@@ -94,7 +88,7 @@ export function unfollow(id) {
  * @author Seven Du <[email protected]>
  */
 export function show(topic) {
-  return request.get(`/question-topics/${topic}`, {
+  return api.get(`/question-topics/${topic}`, {
     validateStatus: status => status === 200
   });
 }
@@ -110,7 +104,7 @@ export function show(topic) {
  * @author Seven Du <[email protected]>
  */
 export function questions(topic, type, offset = 0, limit = 15) {
-  return request.get(`/question-topics/${topic}/questions`, {
+  return api.get(`/question-topics/${topic}/questions`, {
     params: { type, offset, limit },
     validateStatus: status => status === 200
   });

src/page/question/TopicList.vue

@@ -32,7 +32,7 @@
 
 <script>
 import TopicCard from "./components/TopicCard.vue";
-import { all, followTopics, follow, unfollow } from "@/api/question/topics";
+import * as api from "@/api/question/topics.js";
 
 export default {
   name: "TopicList",
@@ -77,7 +77,8 @@ export default {
     handleRefreshByAll() {
       const offset = 0;
       const limit = 15;
-      all(offset, limit)
+      api
+        .all(offset, limit)
         .then(({ data }) => {
           this.topics = data;
           this.loadContainer.topEnd(false);
@@ -92,7 +93,8 @@ export default {
     handleRefreshByFollow() {
       const after = 0;
       const limit = 15;
-      followTopics(after, limit)
+      api
+        .getFollowTopics(after, limit)
         .then(({ data }) => {
           this.topics = data;
           this.loadContainer.topEnd(false);
@@ -116,7 +118,8 @@ export default {
     handleLoadMoreByAll() {
       const offset = this.topics.length;
       const limit = 15;
-      all(offset, limit)
+      api
+        .all(offset, limit)
         .then(({ data }) => {
           this.topics = [...this.topics, ...data];
           this.loadContainer.bottomEnd(data.length < limit);
@@ -129,7 +132,8 @@ export default {
     handleLoadMoreByFollow() {
       const { id: after } = this.topisc[this.topics.length];
       const limit = 15;
-      followTopics(after, limit)
+      api
+        .getFollowTopics(after, limit)
         .then(({ data }) => {
           this.topics = [...this.topics, ...data];
           this.loadContainer.bottomEnd(data.length < limit);
@@ -140,17 +144,14 @@ export default {
         });
     },
     handleFollow(topic) {
-      follow(topic.id)
-        .then(() => {
-          topic.has_follow = true;
-          this.follows_count += 1;
-        })
-        .catch(({ response: { data } = {} }) => {
-          this.$Message.error(data);
-        });
+      api.followTopic(topic.id).then(() => {
+        topic.has_follow = true;
+        this.follows_count += 1;
+      });
     },
     handleUnfollow(topic) {
-      unfollow(topic.id)
+      api
+        .unfollowTopic(topic.id)
         .then(() => {
           topic.has_follow = false;
           topic.follows_count -= 1;

src/page/question/components/TopicCard.vue

@@ -35,10 +35,10 @@ export default {
   },
   methods: {
     handleUnfollow(topic) {
-      this.$emit("unfollow", { topic });
+      this.$emit("unfollow", topic);
     },
     handleFollow(topic) {
-      this.$emit("follow", { topic });
+      this.$emit("follow", topic);
     }
   }
 };