Merge pull request #1374 from akihikodaki/log

Introduce followed log and following log

src/models/followed-log.ts

@@ -0,0 +1,11 @@
+import { ObjectID } from 'mongodb';
+import db from '../db/mongodb';
+
+const FollowedLog = db.get<IFollowedLog>('followedLogs');
+export default FollowedLog;
+
+export type IFollowedLog = {
+	_id: ObjectID;
+	userId: ObjectID;
+	count: number;
+};

src/models/following-log.ts

@@ -0,0 +1,11 @@
+import { ObjectID } from 'mongodb';
+import db from '../db/mongodb';
+
+const FollowingLog = db.get<IFollowingLog>('followingLogs');
+export default FollowingLog;
+
+export type IFollowingLog = {
+	_id: ObjectID;
+	userId: ObjectID;
+	count: number;
+};

src/models/following.ts

@@ -8,7 +8,6 @@ export default Following;
 export type IFollowing = {
 	_id: mongo.ObjectID;
 	createdAt: Date;
-	deletedAt: Date;
 	followeeId: mongo.ObjectID;
 	followerId: mongo.ObjectID;
 };

src/models/user.ts

@@ -234,8 +234,7 @@ export const pack = (
 		_user.isFollowing = (async () => {
 			const follow = await Following.findOne({
 				followerId: meId,
-				followeeId: _user.id,
+				followeeId: _user.id
-				deletedAt: { $exists: false }
 			});
 			return follow !== null;
 		})();
@@ -244,8 +243,7 @@ export const pack = (
 		_user.isFollowed = (async () => {
 			const follow2 = await Following.findOne({
 				followerId: _user.id,
-				followeeId: meId,
+				followeeId: meId
-				deletedAt: { $exists: false }
 			});
 			return follow2 !== null;
 		})();
@@ -275,15 +273,13 @@ export const pack = (
 			// Get following you know count
 			_user.followingYouKnowCount = Following.count({
 				followeeId: { $in: myFollowingIds },
-				followerId: _user.id,
+				followerId: _user.id
-				deletedAt: { $exists: false }
 			});
 
 			// Get followers you know count
 			_user.followersYouKnowCount = Following.count({
 				followeeId: _user.id,
-				followerId: { $in: myFollowingIds },
+				followerId: { $in: myFollowingIds }
-				deletedAt: { $exists: false }
 			});
 		}
 	}

src/post/distribute.ts

@@ -48,9 +48,7 @@ export default async (user, mentions, post) => {
 		// Fetch all followers
 		const followers = await Following
 			.find({
-				followeeId: user._id,
+				followeeId: user._id
-				// 削除されたドキュメントは除く
-				deletedAt: { $exists: false }
 			}, {
 				followerId: true,
 				_id: false

src/processor/http/follow.ts

@@ -3,6 +3,8 @@ import { sign } from 'http-signature';
 import { URL } from 'url';
 import User, { isLocalUser, pack as packUser } from '../../models/user';
 import Following from '../../models/following';
+import FollowingLog from '../../models/following-log';
+import FollowedLog from '../../models/followed-log';
 import event from '../../publishers/stream';
 import notify from '../../publishers/notify';
 import context from '../../remote/activitypub/renderer/context';
@@ -21,6 +23,11 @@ export default ({ data }, done) => Following.findOne({ _id: data.following }).th
 			}
 		}),
 
+		promisedFollower.then(({ followingCount }) => FollowingLog.insert({
+			userId: followerId,
+			count: followingCount + 1
+		})),
+
 		// Increment followers count
 		User.update({ _id: followeeId }, {
 			$inc: {
@@ -28,6 +35,11 @@ export default ({ data }, done) => Following.findOne({ _id: data.following }).th
 			}
 		}),
 
+		promisedFollowee.then(({ followersCount }) => FollowedLog.insert({
+			userId: followerId,
+			count: followersCount + 1
+		})),
+
 		// Notify
 		promisedFollowee.then(followee => followee.host === null ?
 			notify(followeeId, followerId, 'follow') : null),

src/server/api/common/get-friends.ts

@@ -6,9 +6,7 @@ export default async (me: mongodb.ObjectID, includeMe: boolean = true) => {
 	// SELECT followee
 	const myfollowing = await Following
 		.find({
-			followerId: me,
+			followerId: me
-			// 削除されたドキュメントは除く
-			deletedAt: { $exists: false }
 		}, {
 			fields: {
 				followeeId: true

src/server/api/endpoints/aggregation/users/followers.ts

@@ -2,8 +2,9 @@
  * Module dependencies
  */
 import $ from 'cafy';
+import { ObjectID } from 'mongodb';
 import User from '../../../../../models/user';
-import Following from '../../../../../models/following';
+import FollowedLog from '../../../../../models/followed-log';
 
 /**
  * Aggregate followers of a user
@@ -29,47 +30,36 @@ module.exports = (params) => new Promise(async (res, rej) => {
 		return rej('user not found');
 	}
 
-	const startTime = new Date(new Date().setMonth(new Date().getMonth() - 1));
+	const today = new Date();
-
-	const following = await Following
-		.find({
-			followeeId: user._id,
-			$or: [
-				{ deletedAt: { $exists: false } },
-				{ deletedAt: { $gt: startTime } }
-			]
-		}, {
-			sort: { createdAt: -1 },
-			fields: {
-				_id: false,
-				followerId: false,
-				followeeId: false
-			}
-		});
-
 	const graph = [];
 
+	today.setMinutes(0);
+	today.setSeconds(0);
+	today.setMilliseconds(0);
+
+	let cursorDate = new Date(today.getTime());
+	let cursorTime = cursorDate.setDate(new Date(today.getTime()).getDate() + 1);
+
 	for (let i = 0; i < 30; i++) {
-		let day = new Date(new Date().setDate(new Date().getDate() - i));
+		graph.push(FollowedLog.findOne({
-		day = new Date(day.setMilliseconds(999));
+			_id: { $lt: ObjectID.createFromTime(cursorTime / 1000) },
-		day = new Date(day.setSeconds(59));
+			userId: user._id
-		day = new Date(day.setMinutes(59));
+		}, {
-		day = new Date(day.setHours(23));
+			sort: { _id: -1 },
-		// day = day.getTime();
+		}).then(log => {
+			cursorDate = new Date(today.getTime());
+			cursorTime = cursorDate.setDate(today.getDate() - i);
 
-		const count = following.filter(f =>
+			return {
-			f.createdAt < day && (f.deletedAt == null || f.deletedAt > day)
-		).length;
-
-		graph.push({
 				date: {
-				year: day.getFullYear(),
+					year: cursorDate.getFullYear(),
-				month: day.getMonth() + 1, // In JavaScript, month is zero-based.
+					month: cursorDate.getMonth() + 1, // In JavaScript, month is zero-based.
-				day: day.getDate()
+					day: cursorDate.getDate()
 				},
-			count: count
+				count: log ? log.count : 0
-		});
+			};
+		}));
 	}
 
-	res(graph);
+	res(await Promise.all(graph));
 });

src/server/api/endpoints/aggregation/users/following.ts

@@ -2,8 +2,9 @@
  * Module dependencies
  */
 import $ from 'cafy';
+import { ObjectID } from 'mongodb';
 import User from '../../../../../models/user';
-import Following from '../../../../../models/following';
+import FollowingLog from '../../../../../models/following-log';
 
 /**
  * Aggregate following of a user
@@ -29,46 +30,36 @@ module.exports = (params) => new Promise(async (res, rej) => {
 		return rej('user not found');
 	}
 
-	const startTime = new Date(new Date().setMonth(new Date().getMonth() - 1));
+	const today = new Date();
-
-	const following = await Following
-		.find({
-			followerId: user._id,
-			$or: [
-				{ deletedAt: { $exists: false } },
-				{ deletedAt: { $gt: startTime } }
-			]
-		}, {
-			sort: { createdAt: -1 },
-			fields: {
-				_id: false,
-				followerId: false,
-				followeeId: false
-			}
-		});
-
 	const graph = [];
 
+	today.setMinutes(0);
+	today.setSeconds(0);
+	today.setMilliseconds(0);
+
+	let cursorDate = new Date(today.getTime());
+	let cursorTime = cursorDate.setDate(new Date(today.getTime()).getDate() + 1);
+
 	for (let i = 0; i < 30; i++) {
-		let day = new Date(new Date().setDate(new Date().getDate() - i));
+		graph.push(FollowingLog.findOne({
-		day = new Date(day.setMilliseconds(999));
+			_id: { $lt: ObjectID.createFromTime(cursorTime / 1000) },
-		day = new Date(day.setSeconds(59));
+			userId: user._id
-		day = new Date(day.setMinutes(59));
+		}, {
-		day = new Date(day.setHours(23));
+			sort: { _id: -1 },
+		}).then(log => {
+			cursorDate = new Date(today.getTime());
+			cursorTime = cursorDate.setDate(today.getDate() - i);
 
-		const count = following.filter(f =>
+			return {
-			f.createdAt < day && (f.deletedAt == null || f.deletedAt > day)
-		).length;
-
-		graph.push({
 				date: {
-				year: day.getFullYear(),
+					year: cursorDate.getFullYear(),
-				month: day.getMonth() + 1, // In JavaScript, month is zero-based.
+					month: cursorDate.getMonth() + 1, // In JavaScript, month is zero-based.
-				day: day.getDate()
+					day: cursorDate.getDate()
 				},
-			count: count
+				count: log ? log.count : 0
-		});
+			};
+		}));
 	}
 
-	res(graph);
+	res(await Promise.all(graph));
 });

src/server/api/endpoints/following/create.ts

@@ -42,8 +42,7 @@ module.exports = (params, user) => new Promise(async (res, rej) => {
 	// Check if already following
 	const exist = await Following.findOne({
 		followerId: follower._id,
-		followeeId: followee._id,
+		followeeId: followee._id
-		deletedAt: { $exists: false }
 	});
 
 	if (exist !== null) {

src/server/api/endpoints/following/delete.ts

@@ -42,8 +42,7 @@ module.exports = (params, user) => new Promise(async (res, rej) => {
 	// Check not following
 	const exist = await Following.findOne({
 		followerId: follower._id,
-		followeeId: followee._id,
+		followeeId: followee._id
-		deletedAt: { $exists: false }
 	});
 
 	if (exist === null) {
@@ -51,12 +50,8 @@ module.exports = (params, user) => new Promise(async (res, rej) => {
 	}
 
 	// Delete following
-	await Following.update({
+	await Following.findOneAndDelete({
 		_id: exist._id
-	}, {
-		$set: {
-			deletedAt: new Date()
-		}
 	});
 
 	// Send response

src/server/api/endpoints/users/followers.ts

@@ -46,8 +46,7 @@ module.exports = (params, me) => new Promise(async (res, rej) => {
 
 	// Construct query
 	const query = {
-		followeeId: user._id,
+		followeeId: user._id
-		deletedAt: { $exists: false }
 	} as any;
 
 	// ログインしていてかつ iknow フラグがあるとき

src/server/api/endpoints/users/following.ts

@@ -46,8 +46,7 @@ module.exports = (params, me) => new Promise(async (res, rej) => {
 
 	// Construct query
 	const query = {
-		followerId: user._id,
+		followerId: user._id
-		deletedAt: { $exists: false }
 	} as any;
 
 	// ログインしていてかつ iknow フラグがあるとき

test/api.js

@@ -609,20 +609,6 @@ describe('API', () => {
 			res.should.have.status(204);
 		}));
 
-		it('過去にフォロー歴があった状態でフォローできる', async(async () => {
-			const hima = await insertHimawari();
-			const me = await insertSakurako();
-			await db.get('following').insert({
-				followeeId: hima._id,
-				followerId: me._id,
-				deletedAt: new Date()
-			});
-			const res = await request('/following/create', {
-				userId: hima._id.toString()
-			}, me);
-			res.should.have.status(204);
-		}));
-
 		it('既にフォローしている場合は怒る', async(async () => {
 			const hima = await insertHimawari();
 			const me = await insertSakurako();