e2net/iceshrimp-legacy
2
0
Fork 0
Code Issues Pull requests Packages Projects Releases Activity

Drop stalk feature

Browse source 
Resolve #3935
Closes #2226
Closes #1946
This commit is contained in:
syuilo 2019-01-20 12:25:00 +09:00
parent d53d059480
commit 6909add1ec
No known key found for this signature in database
GPG key ID: BDC4C49D06AB9D69
12 changed files with 11 additions and 157 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
CHANGELOG.md
View file

@ -10,6 +10,7 @@ unreleased
* ログイン時に二段階認証が分かりにくいのを改善
* 投稿のツールチップを出すのは時間の上だけに変更
* ハッシュタグ判定の強化
* ストーク機能の廃止
* クライアントのAPIリクエストをストリーム経由で行うオプションを廃止
* 一部箇所でカスタム絵文字が適用されていないのを修正

3
locales/ja-JP.yml
View file

@ -1402,9 +1402,6 @@ desktop/views/pages/user/user.photos.vue:
desktop/views/pages/user/user.profile.vue:
follows-you: "フォローされています"
stalk: "ストークする"
stalking: "ストーキングしています"
unstalk: "ストーク解除"
menu: "メニュー"
desktop/views/pages/user/user.header.vue:

27
src/client/app/desktop/views/pages/user/user.profile.vue
View file

@ -3,10 +3,6 @@
<div class="friend-form" v-if="$store.state.i.id != user.id">
<mk-follow-button :user="user" block/>
<p class="followed" v-if="user.isFollowed">{{ $t('follows-you') }}</p>
<p class="stalk" v-if="user.isFollowing">
<span v-if="user.isStalking">{{ $t('stalking') }} <a @click="unstalk"><fa icon="meh"/> {{ $t('unstalk') }}</a></span>
<span v-if="!user.isStalking"><a @click="stalk"><fa icon="user-secret"/> {{ $t('stalk') }}</a></span>
</p>
</div>
<div class="action-form">
<ui-button @click="menu" ref="menu">{{ $t('menu') }}</ui-button>
@ -24,26 +20,6 @@ export default Vue.extend({
props: ['user'],
methods: {
stalk() {
this.$root.api('following/stalk', {
userId: this.user.id
}).then(() => {
this.user.isStalking = true;
}, () => {
alert('error');
});
},
unstalk() {
this.$root.api('following/unstalk', {
userId: this.user.id
}).then(() => {
this.user.isStalking = false;
}, () => {
alert('error');
});
},
menu() {
this.$root.new(XUserMenu, {
source: this.$refs.menu.$el,
@ -78,9 +54,6 @@ export default Vue.extend({
background #eefaff
border-radius 4px
> .stalk
margin 12px 0 0 0
> .action-form
padding 16px
text-align center

5
src/docs/follow.ja-JP.md
View file

@ -1,8 +1,3 @@
# フォロー
ユーザーをフォローすると、タイムラインにそのユーザーの投稿が表示されるようになります。ただし、他のユーザーに対する返信は含まれません。
ユーザーをフォローするには、ユーザーページの「フォロー」ボタンをクリックします。フォローを解除するには、もう一度クリックします。
## ストーキング
ユーザーをフォローしている状態では、さらに「ストーキング」モードをオンにすることができます。ストーキングを行うと、タイムラインにそのユーザーの全ての投稿が表示されるようになります。つまり、他のユーザーに対する返信も含まれることになります。
ストーキングするには、ユーザーページの「ストークする」をクリックします。ストーキングをやめるには、もう一度クリックします。
ストーキングしていることは相手に通知されません。

1
src/models/following.ts
View file

@ -12,7 +12,6 @@ export type IFollowing = {
createdAt: Date;
followeeId: mongo.ObjectID;
followerId: mongo.ObjectID;
stalk: boolean;
// 非正規化
_followee: {

2
src/models/user.ts
View file

@ -219,7 +219,6 @@ export async function getRelation(me: mongo.ObjectId, target: mongo.ObjectId) {
return {
id: target,
isFollowing: following1 !== null,
isStalking: following1 ? following1.stalk : false,
hasPendingFollowRequestFromYou: followReq1 !== null,
hasPendingFollowRequestToYou: followReq2 !== null,
isFollowed: following2 !== null,
@ -352,7 +351,6 @@ export const pack = (
_user.isFollowing = relation.isFollowing;
_user.isFollowed = relation.isFollowed;
_user.isStalking = relation.isStalking;
_user.hasPendingFollowRequestFromYou = relation.hasPendingFollowRequestFromYou;
_user.hasPendingFollowRequestToYou = relation.hasPendingFollowRequestToYou;
_user.isBlocking = relation.isBlocking;

6
src/server/api/common/get-friends.ts
View file

@ -36,14 +36,12 @@ export const getFriends = async (me: mongodb.ObjectID, includeMe = true, remoteO
// ID list of other users who the I follows
const myfollowings = followings.map(following => ({
id: following.followeeId,
stalk: following.stalk
id: following.followeeId
}));
if (includeMe) {
myfollowings.push({
id: me,
stalk: true
id: me
});
}

50
src/server/api/endpoints/following/stalk.ts
View file

@ -1,50 +0,0 @@
import $ from 'cafy'; import ID, { transform } from '../../../../misc/cafy-id';
import Following from '../../../../models/following';
import define from '../../define';
export const meta = {
desc: {
'ja-JP': '指定したユーザーをストーキングします。',
'en-US': 'Stalk a user.'
},
requireCredential: true,
kind: 'following-write',
params: {
userId: {
validator: $.type(ID),
transform: transform,
desc: {
'ja-JP': '対象のユーザーのID',
'en-US': 'Target user ID'
}
}
}
};
export default define(meta, (ps, user) => new Promise(async (res, rej) => {
const follower = user;
// Fetch following
const following = await Following.findOne({
followerId: follower._id,
followeeId: ps.userId
});
if (following === null) {
return rej('following not found');
}
// Stalk
await Following.update({ _id: following._id }, {
$set: {
stalk: true
}
});
res();
// TODO: イベント
}));

50
src/server/api/endpoints/following/unstalk.ts
View file

@ -1,50 +0,0 @@
import $ from 'cafy'; import ID, { transform } from '../../../../misc/cafy-id';
import Following from '../../../../models/following';
import define from '../../define';
export const meta = {
desc: {
'ja-JP': '指定したユーザーのストーキングをやめます。',
'en-US': 'Unstalk a user.'
},
requireCredential: true,
kind: 'following-write',
params: {
userId: {
validator: $.type(ID),
transform: transform,
desc: {
'ja-JP': '対象のユーザーのID',
'en-US': 'Target user ID'
}
}
}
};
export default define(meta, (ps, user) => new Promise(async (res, rej) => {
const follower = user;
// Fetch following
const following = await Following.findOne({
followerId: follower._id,
followeeId: ps.userId
});
if (following === null) {
return rej('following not found');
}
// Stalk
await Following.update({ _id: following._id }, {
$set: {
stalk: false
}
});
res();
// TODO: イベント
}));

8
src/server/api/endpoints/notes/hybrid-timeline.ts
View file

@ -119,12 +119,10 @@ export default define(meta, (ps, user) => new Promise(async (res, rej) => {
_id: -1
};
const followQuery = followings.map(f => f.stalk ? {
userId: f.id
} : {
const followQuery = followings.map(f => ({
userId: f.id,
// ストーキングしてないならリプライは含めない(ただし投稿者自身の投稿へのリプライ、自分の投稿へのリプライ、自分のリプライは含める)
// リプライは含めない(ただし投稿者自身の投稿へのリプライ、自分の投稿へのリプライ、自分のリプライは含める)
$or: [{
// リプライでない
replyId: null
@ -140,7 +138,7 @@ export default define(meta, (ps, user) => new Promise(async (res, rej) => {
// 自分(フォロワー)が送信したリプライ
userId: user._id
}]
});
}));
const visibleQuery = user == null ? [{
visibility: { $in: [ 'public', 'home' ] }

6
src/server/api/endpoints/notes/timeline.ts
View file

@ -117,9 +117,7 @@ export default define(meta, (ps, user) => new Promise(async (res, rej) => {
_id: -1
};
const followQuery = followings.map(f => f.stalk ? {
userId: f.id
} : {
const followQuery = followings.map(f => ({
userId: f.id,
// ストーキングしてないならリプライは含めない(ただし投稿者自身の投稿へのリプライ、自分の投稿へのリプライ、自分のリプライは含める)
@ -138,7 +136,7 @@ export default define(meta, (ps, user) => new Promise(async (res, rej) => {
// 自分(フォロワー)が送信したリプライ
userId: user._id
}]
});
}));
const visibleQuery = user == null ? [{
visibility: { $in: [ 'public', 'home' ] }

9
src/services/note/create.ts
View file

@ -540,12 +540,9 @@ async function publishToFollowers(note: INote, user: IUser, noteActivity: any) {
const follower = following._follower;
if (isLocalUser(follower)) {
// ストーキングしていない場合
if (!following.stalk) {
// この投稿が返信ならスキップ
if (note.replyId && !note._reply.userId.equals(following.followerId) && !note._reply.userId.equals(note.userId))
continue;
}
// この投稿が返信ならスキップ
if (note.replyId && !note._reply.userId.equals(following.followerId) && !note._reply.userId.equals(note.userId))
continue;
// Publish event to followers stream
publishHomeTimelineStream(following.followerId, detailPackedNote);