wip

2 files changed, 90 insertions, 20 deletions

diff --git a/src/api/endpoints/posts/context.ts b/src/api/endpoints/posts/context.ts
index 673da0fab8..25ac687d34 100644
--- a/src/api/endpoints/posts/context.ts
+++ b/src/api/endpoints/posts/context.ts
@@ -3,7 +3,7 @@
 /**
  * Module dependencies
  */
-import validate from '../../validator';
+import it from '../../it';
 import Post from '../../models/post';
 import serialize from '../../serializers/post';
 
@@ -18,37 +18,24 @@ module.exports = (params, user) =>
 	new Promise(async (res, rej) =>
 {
 	// Get 'post_id' parameter
-	const [postId, postIdErr] = validate(params.post_id, 'id', true);
+	const [postId, postIdErr] = it(params.post_id, 'id', true);
 	if (postIdErr) return rej('invalid post_id');
 
 	// Get 'limit' parameter
-	let [limit, limitErr] = validate(params.limit, 'number');
+	const [limit, limitErr] = it(params.limit).expect.number().range(1, 100).default(10).qed();
 	if (limitErr) return rej('invalid limit');
 
-	if (limit !== null) {
-		// From 1 to 100
-		if (!(1 <= limit && limit <= 100)) {
-			return rej('invalid limit range');
-		}
-	} else {
-		limit = 10;
-	}
-
 	// Get 'offset' parameter
-	let offset = params.offset;
-	if (offset !== undefined && offset !== null) {
-		offset = parseInt(offset, 10);
-	} else {
-		offset = 0;
-	}
+	const [offset, offsetErr] = it(params.limit).expect.number().min(0).default(0).qed();
+	if (offsetErr) return rej('invalid offset');
 
 	// Lookup post
 	const post = await Post.findOne({
-		_id: new mongo.ObjectID(postId)
+		_id: postId
 	});
 
 	if (post === null) {
-		return rej('post not found', 'POST_NOT_FOUND');
+		return rej('post not found');
 	}
 
 	const context = [];
diff --git a/src/api/it.ts b/src/api/it.ts
index 6e8aefdf2d..ec42547488 100644
--- a/src/api/it.ts
+++ b/src/api/it.ts
@@ -1,3 +1,8 @@
+/**
+ * it
+ * 楽しいバリデーション
+ */
+
 import * as mongo from 'mongodb';
 import hasDuplicates from '../common/has-duplicates';
 
@@ -11,6 +16,8 @@ interface Factory {
 
 	required: () => Factory;
 
+	default: (value: any) => Factory;
+
 	validate: (validator: Validator<any>) => Factory;
 }
 
@@ -34,6 +41,16 @@ class FactoryCore implements Factory {
 	}
 
 	/**
+	 * このインスタンスの値が設定されていないときにデフォルトで設定する値を設定します
+	 */
+	default(value: any) {
+		if (this.value === null) {
+			this.value = value;
+		}
+		return this;
+	}
+
+	/**
 	 * このインスタンスの値およびエラーを取得します
 	 */
 	qed(): [any, Error] {
@@ -80,6 +97,13 @@ class BooleanFactory extends FactoryCore {
 	}
 
 	/**
+	 * このインスタンスの値が設定されていないときにデフォルトで設定する値を設定します
+	 */
+	default(value: boolean) {
+		return super.default(value);
+	}
+
+	/**
 	 * このインスタンスの値およびエラーを取得します
 	 */
 	qed(): [boolean, Error] {
@@ -125,6 +149,30 @@ class NumberFactory extends FactoryCore {
 	}
 
 	/**
+	 * このインスタンスの値が指定された下限より下回っている場合エラーにします
+	 * @param value 下限
+	 */
+	min(value: number) {
+		if (this.error || this.value === null) return this;
+		if (this.value < value) {
+			this.error = new Error('invalid-range');
+		}
+		return this;
+	}
+
+	/**
+	 * このインスタンスの値が指定された上限より上回っている場合エラーにします
+	 * @param value 上限
+	 */
+	max(value: number) {
+		if (this.error || this.value === null) return this;
+		if (this.value > value) {
+			this.error = new Error('invalid-range');
+		}
+		return this;
+	}
+
+	/**
 	 * このインスタンスの値が undefined または　null の場合エラーにします
 	 */
 	required() {
@@ -132,6 +180,13 @@ class NumberFactory extends FactoryCore {
 	}
 
 	/**
+	 * このインスタンスの値が設定されていないときにデフォルトで設定する値を設定します
+	 */
+	default(value: number) {
+		return super.default(value);
+	}
+
+	/**
 	 * このインスタンスの値およびエラーを取得します
 	 */
 	qed(): [number, Error] {
@@ -190,6 +245,13 @@ class StringFactory extends FactoryCore {
 	}
 
 	/**
+	 * このインスタンスの値が設定されていないときにデフォルトで設定する値を設定します
+	 */
+	default(value: string) {
+		return super.default(value);
+	}
+
+	/**
 	 * このインスタンスの値およびエラーを取得します
 	 */
 	qed(): [string, Error] {
@@ -253,6 +315,13 @@ class ArrayFactory extends FactoryCore {
 	}
 
 	/**
+	 * このインスタンスの値が設定されていないときにデフォルトで設定する値を設定します
+	 */
+	default(value: any[]) {
+		return super.default(value);
+	}
+
+	/**
 	 * このインスタンスの値およびエラーを取得します
 	 */
 	qed(): [any[], Error] {
@@ -292,6 +361,13 @@ class IdFactory extends FactoryCore {
 	}
 
 	/**
+	 * このインスタンスの値が設定されていないときにデフォルトで設定する値を設定します
+	 */
+	default(value: mongo.ObjectID) {
+		return super.default(value);
+	}
+
+	/**
 	 * このインスタンスの値およびエラーを取得します
 	 */
 	qed(): [any[], Error] {
@@ -331,6 +407,13 @@ class ObjectFactory extends FactoryCore {
 	}
 
 	/**
+	 * このインスタンスの値が設定されていないときにデフォルトで設定する値を設定します
+	 */
+	default(value: any) {
+		return super.default(value);
+	}
+
+	/**
 	 * このインスタンスの値およびエラーを取得します
 	 */
 	qed(): [any, Error] {