Check empty strings in post

Author: velopert

velog-backend/src/lib/common.js

@@ -87,3 +87,9 @@ export function generalHash(text: string) {
     .digest('hex');
   return hash;
 }
+
+export function checkEmpty(text: string) {
+  const replaced = text.trim().replace(/\u3164/g, '');
+  if (replaced === '') return true;
+  return false;
+}

velog-backend/src/router/posts/post/post.ctrl.js

@@ -8,6 +8,7 @@ import {
   generateSlugId,
   escapeForUrl,
   extractKeys,
+  checkEmpty,
 } from 'lib/common';
 import { diff } from 'json-diff';
 import {
@@ -106,6 +107,17 @@ export const updatePost = async (ctx: Context): Promise<*> => {
     meta,
   }: BodySchema = (ctx.request.body: any);
 
+  const stringsToCheck = [title, body, ...tags];
+  for (let i = 0; i < stringsToCheck.length; i++) {
+    if (checkEmpty(stringsToCheck[i])) {
+      ctx.status = 400;
+      ctx.body = {
+        name: 'INVALID_TEXT',
+      };
+      return;
+    }
+  }
+
   // validate tags
   if (tags) {
     for (let i = 0; i < tags.length; i++) {

velog-backend/src/router/posts/posts.ctrl.js

@@ -10,6 +10,7 @@ import {
   isUUID,
   formatShortDescription,
   generalHash,
+  checkEmpty,
 } from 'lib/common';
 import {
   Category,
@@ -192,6 +193,17 @@ export const writePost = async (ctx: Context): Promise<*> => {
     }
   }
 
+  const stringsToCheck = [title, body, ...tags];
+  for (let i = 0; i < stringsToCheck.length; i++) {
+    if (checkEmpty(stringsToCheck[i])) {
+      ctx.status = 400;
+      ctx.body = {
+        name: 'INVALID_TEXT',
+      };
+      return;
+    }
+  }
+
   if (processedSlug === '' || processedSlug.replace(/\./g, '') === '') {
     ctx.status = 400;
     ctx.body = {
@@ -283,7 +295,7 @@ export const readPost = async (ctx: Context): Promise<*> => {
           url_slug: urlSlug,
           fk_user_id: user.id,
         },
-        order: ['created_at', 'DESC'],
+        order: [['created_at', 'DESC']],
       });
       if (!history) {
         ctx.status = 404;